From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:59827) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bYS5Q-0000tT-KJ for qemu-devel@nongnu.org; Sat, 13 Aug 2016 02:06:25 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bYS5O-0008CL-KA for qemu-devel@nongnu.org; Sat, 13 Aug 2016 02:06:23 -0400 References: <1469723896-28049-1-git-send-email-wei@redhat.com> <20160729065453.qq44y2hxohizk3yw@hawk.localdomain> <1470053099.3971.11.camel@redhat.com> <20160801130808.2igpsx52opi7ogvk@kamzik.localdomain> <1470058019.3971.13.camel@redhat.com> From: Wei Huang Message-ID: Date: Sat, 13 Aug 2016 01:06:13 -0500 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH RFC 1/1] arm64: add an option to turn on/off vpmu support List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Maydell , Andrea Bolognani Cc: Andrew Jones , qemu-arm , QEMU Developers , Shannon Zhao On 08/01/2016 08:32 AM, Peter Maydell wrote: > On 1 August 2016 at 14:26, Andrea Bolognani wrote: >> On Mon, 2016-08-01 at 15:08 +0200, Andrew Jones wrote: >>>> I'm not sure a warning is enough: if I start a guest and >>>> explicitly ask for a PMU, I expect it to be there, or for >>>> the guest not to start at all. How does x86 behave in this >>>> regard? >>> >>> Peter had a good suggestion for this. We need to wrap the property >>> addition in an arm_feature check like the has_el3 property. That will >>> remove it from all cpu types that don't support it. >> >> Wouldn't that mean that you'd be unable to use >> >> -cpu foo,pmu=off >> >> if CPU model 'foo' doesn't support a PMU? I'd expect that >> to work. > > The current precedent (has_el3) doesn't work like that: if > foo isn't a CPU which can support EL3 then the property doesn't > exist, and it's an error to try to set it. V1 sent. I tried to follow everyone's advice. See the following: * set default pmu=off * like el3, add a new feature ARM_FEATURE_HOST_PMU * "pmu" property becomes CPU dependent. Only cortex-a53/cortex-a57/host under certain mode support this option * change struct ARMCPU field name "has_pmu" ==> "has_host_pmu" because IMO "has_pmu" is misleading BTW answering Andrea's question above: "-cpu foo,pmu=off" won't be allowed in this patch if CPU "foo" doesn't support host-backed PMU. QEMU will fail to run in this case. Maybe this is what we want? Thanks, -Wei > > thanks > -- PMM >