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

[Arianna Avanzini <avanzini.arianna@gmail.com>]

On 03/25/2014 01:54 PM, Julien Grall wrote:
> Hi Stefano and Arianna,
> 

Hello,

> 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?

Thank you again.


> Regards,
> 


-- 
/*
 * Arianna Avanzini
 * avanzini.arianna@gmail.com
 * 73628@studenti.unimore.it
 */