Re: [Xen-devel] [PATCH v2 20/21] libxl: Add support for non-PCI passthrough

[Julien Grall <julien.grall@xxxxxxxxxx>]

Hi Stefano,

On 08/06/2014 05:44 PM, Stefano Stabellini wrote:
> On Thu, 31 Jul 2014, Julien Grall wrote:
>> On ARM, every non-PCI device are described in the device tree. Each of them
>> can be found via a path.
>>
>> This path will be used to retrieved the different informations about the
>> device (compatible string, interrupts, MMIOs). Libxl will take care of:
>>     - Allocate the MMIOs regions for the device in the guest
>>     - Create the device node in the guest device tree
>>     - Map the IRQs and MMIOs range in the guest P2M
>>
>> Note, that the device node won't contains specific properties for the node.
>> Only generic one (compatible, interrupts, regs) will be created by libxl.
>>
>> In the future, per-device properties will be added. Maybe via a configuration
>> file listing what is needed.
>>
>> Signed-off-by: Julien Grall <julien.grall@xxxxxxxxxx>
>> Cc: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>
>> Cc: Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>
>> Cc: Ian Campbell <ian.campbell@xxxxxxxxxx>
>>
>> ---
>>     Changes in v2:
>>         - Get DT infos earlier
>>         - Allocate/map IRQ in libxl__arch_domain_create rather than in
>>         libxl__device_dt_add
> 
> I understand why you moved it earlier but if it is not too hard I would
> suggest to try to keep mmio and irq mapping all together, because it
> would make the code much easier to read and modify in the future, being
> all in one place.

I could fixup the device tree later. So we don't have to "map/retrieve"
the IRQ earlier.

But I don't much like this solution.

Regards,

-- 
Julien Grall

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