From mboxrd@z Thu Jan 1 00:00:00 1970 From: Julien Grall Subject: Re: [PATCH v2 for 4.5] xen/arm: Add support for GICv3 for domU Date: Wed, 05 Nov 2014 11:28:34 +0000 Message-ID: <545A09E2.3090408@linaro.org> References: <1414872625-2961-1-git-send-email-julien.grall@linaro.org> <1415183041.11486.74.camel@citrix.com> <545A058F.5040603@linaro.org> <1415186575.15317.9.camel@citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: Received: from mail6.bemta4.messagelabs.com ([85.158.143.247]) by lists.xen.org with esmtp (Exim 4.72) (envelope-from ) id 1XlylS-0006Lf-K2 for xen-devel@lists.xenproject.org; Wed, 05 Nov 2014 11:28:38 +0000 Received: by mail-wi0-f173.google.com with SMTP id n3so12098877wiv.12 for ; Wed, 05 Nov 2014 03:28:37 -0800 (PST) In-Reply-To: <1415186575.15317.9.camel@citrix.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Ian Campbell Cc: Wei Liu , Ian Jackson , Vijaya Kumar K , tim@xen.org, stefano.stabellini@citrix.com, Jan Beulich , xen-devel@lists.xenproject.org, Daniel De Graaf List-Id: xen-devel@lists.xenproject.org On 05/11/2014 11:22, Ian Campbell wrote: >>> >>>> + * hardware GIC. Only the value XEN_DOMCTL_CONFIG_GIC_DEFAULT >>>> + * is allowed. The DOMCTL will return the actual version of the >>>> + * GIC. >>>> + */ >>>> + if ( domctl->u.configuredomain.gic_version != XEN_DOMCTL_CONFIG_GIC_DEFAULT ) >>>> + return -EOPNOTSUPP; >> >>> I'm also tempted to suggest that we should accept gic_version == the hw >>> value, i.e. by moping the switch below up and including >>> && domctl->u.configuredomain.gic_version != gic_version >>> in the condition. >> >> I consider that explicitly asking for a version of the GIC in Xen 4.5 is >> invalid. The user should only be able to use the default GIC. >> >> As the DOMCTL is not set in stone, > > I disagree with your reasoning, but given this I'm not inclined to keep > arguing about it. TBH, I should just ignore the value for Xen 4.5 and let the hypervisor return the version of the vGIC. The check was just here to explicitly show that we don't support anything else in DOM0. Regards, -- Julien Grall