[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v4 3/7] arch, arm: let map_mmio_regions() take pfn as parameters

On 03/28/2014 12:51 PM, Arianna Avanzini wrote:
> On 03/25/2014 01:54 PM, Julien Grall wrote:
> Hello,

Hi Arianna,

>> On 03/25/2014 12:22 PM, Stefano Stabellini wrote:
>>>> -    return apply_p2m_changes(d, INSERT, start_gaddr, end_gaddr,
>>>> -                             maddr, MATTR_DEV, p2m_mmio_direct);
>>>> +    return apply_p2m_changes(d, INSERT,
>>>> +                             pfn_to_paddr(start_gfn),
>>>> +                             pfn_to_paddr(end_gfn),
>>>> +                             pfn_to_paddr(mfn),
>>>> +                             MATTR_DEV, p2m_mmio_direct);
>>>>  }
>>> This doesn't make any sense. Shouldn't you modify apply_p2m_changes to
>>> take pfns as parameters too?
>> If so, I think it should be a separate patch. Her change makes sense
>> with the current interface.
> Thank you both for the feedback. Sorry if I bother you with a question, would
> you like a separate patch to modify also apply_p2m_changes() to take pfns as
> parameters, or would you prefer to keep the current interface by now?

As you want, I don't think this change is critical for now.


Julien Grall

Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.