Re: [PATCH v2 for 4.5] xen/arm: Add support for GICv3 for domU

[Ian Campbell <Ian.Campbell@citrix.com>]

On Tue, 2014-11-04 at 17:26 +0000, Julien Grall wrote:
> Hi Konrad,
> 
> On 11/03/2014 04:39 PM, Konrad Rzeszutek Wilk wrote:
> > On Mon, Nov 03, 2014 at 06:07:33PM +0530, Vijay Kilari wrote:
> >> On Mon, Nov 3, 2014 at 4:31 PM, Stefano Stabellini
> >> <stefano.stabellini@eu.citrix.com> wrote:
> >>> On Sat, 1 Nov 2014, Julien Grall wrote:
> >>>> The vGIC will emulate the same version as the hardware. The toolstack has
> >>>> to retrieve the version of the vGIC in order to be able to create the
> >>>> corresponding device tree node.
> >>>>
> >>>> A new DOMCTL has been introduced for ARM to configure the domain. For now
> >>>> it only allow the toolstack to retrieve the version of vGIC.
> >>>> This DOMCTL will be extend later to let the user choose the version of the
> >>>> emulated GIC.
> >>>>
> >>>> Signed-off-by: Vijaya Kumar K <Vijaya.Kumar@caviumnetworks.com>
> >>>> Signed-off-by: Julien Grall <julien.grall@linaro.org>
> >>>> Cc: Ian Jackson <ian.jackson@eu.citrix.com>
> >>>> Cc: Jan Beulich <jbeulich@suse.com>
> >>>> Cc: Daniel De Graaf <dgdegra@tycho.nsa.gov>
> >>>> Cc: Wei Liu <wei.liu2@citrix.com>
> >>>
> >>> Reviewed-by: Stefano Stabellini <stefano.stabellini@eu.citrix.com>
> >>
> >> Tested with GICv3 platform.
> > 
> > Thank you for doing that.
> > 
> > It also needs Acks from Daniel and Jan.
> 
> This patch doesn't modify the x86 part. So I'm not sure if Jan ack is
> required. Would Ian C. ack be enough?

Jan had comments on a previous iteration and adding a new domctl (or
hypercalls generally) could do with eyes from all arches even if they
are currently arch specific, so it would be good to know that he at
least doesn't object.

> Anyway, Jan do you have any objection on this patch?
> 
> Regards,
>