From mboxrd@z Thu Jan 1 00:00:00 1970 From: Shannon Zhao Subject: Re: [PATCH v11 19/21] KVM: ARM64: Add a new feature bit for PMUv3 Date: Mon, 22 Feb 2016 15:43:56 +0800 Message-ID: <56CABC3C.8000304@huawei.com> References: <1454656456-11640-1-git-send-email-zhaoshenglong@huawei.com> <1454656456-11640-20-git-send-email-zhaoshenglong@huawei.com> <20160208122957.GE620@cbox> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: kvm@vger.kernel.org, marc.zyngier@arm.com, will.deacon@arm.com, kvmarm@lists.cs.columbia.edu, linux-arm-kernel@lists.infradead.org, shannon.zhao@linaro.org To: Christoffer Dall Return-path: In-Reply-To: <20160208122957.GE620@cbox> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: kvmarm-bounces@lists.cs.columbia.edu Sender: kvmarm-bounces@lists.cs.columbia.edu List-Id: kvm.vger.kernel.org On 2016/2/8 20:29, Christoffer Dall wrote: >> +bool kvm_arm_support_pmu_v3(void) >> > +{ >> > + /* Check if HW_PERF_EVENTS are supported by checking the number of >> > + * hardware performance counters. This could ensure physical PMU and >> > + * PERF_EVENT driver existing. >> > + */ > nit: coding style, opening comment block should be on its own line > > I don't understand the last sentence. Do you mean: "This ensures the > presence of a physical PMU and that CONFIG_PERF_EVENT is selected." ? Right. -- Shannon