Re: [Xen-devel] [PATCH v6 04/11] arch, arm: make pfn range passed to map_mmio_regions() inclusive

[Ian Campbell <Ian.Campbell@xxxxxxxxxxxxx>]

On Tue, 2014-04-22 at 20:52 +0100, Julien Grall wrote:
> Hi Arianna,
> 
> On 21/04/14 14:44, Arianna Avanzini wrote:
> > Currently, the map_mmio_regions() function accepts a range of pfns
> > which is not inclusive of the end frame number. However, the
> > corresponding operation performed for the x86 architecture, which
> > will be factored out from the memory_mapping DOMCTL in one of the
> > following commits, accepts a pfn range which is inclusive of the
> > end pfn.
> > This commit attempts to make such an interface consistent, by
> > letting map_mmio_regions() consider a pfn range inclusive of the
> > end pfn.
> 
> 
> I think this patch can be merged with the previous patch (i.e #3).

I think it would be better to keep the type change and the limits
handling separate. All the +1/-1/align to something is tricky enough
already done step by step.

Acked-by: Ian Campbell <ian.campbell@xxxxxxxxxx>

Ian.



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel