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

Re: [Xen-devel] [RFC PATCH v1 02/10] xen/arm: register mmio handler at runtime

On 03/26/2014 12:29 PM, Vijay Kilari wrote:
> Hi Julien,
> On Fri, Mar 21, 2014 at 10:53 PM, Julien Grall <julien.grall@xxxxxxxxxx> 
> wrote:
>> On 03/21/2014 05:17 PM, Ian Campbell wrote:
>>> On Wed, 2014-03-19 at 19:47 +0530, vijay.kilari@xxxxxxxxx wrote:
>>>> From: Vijaya Kumar K <Vijaya.Kumar@xxxxxxxxxxxxxxxxxx>
>>>> mmio handlers are registers at compile time
>>>> for drivers like vuart and vgic.
>>>> Make mmio handler registered at runtime by
>>>> creating linked list of mmio handlers
>>> I'm not convinced of the need for this, certainly the vgic side can just
>>> demux into v2 or v3 as necessary.
>> Demux the code just add an indirection. We could have a list of mmio
>> handler per domain and rely on it to call the right handler. A bit like x86.
>  Until Andrii adds IOMMU  handling should keep this patch? and adopt
> to it later?

I'm not sure to understand. IHMO, it doesn't sound right to upstream a
patch that we know it will be superseded in a couple of months. Why
can't you rework your patch to have it in good shape now?

>   Also vgic & vuart are not per domain specific

The vuart is per domain specific. Only dom0 is using the callback.


Julien Grall

Xen-devel mailing list



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