From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1423095AbcBQQ2Q (ORCPT ); Wed, 17 Feb 2016 11:28:16 -0500 Received: from mail-wm0-f43.google.com ([74.125.82.43]:35938 "EHLO mail-wm0-f43.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S934041AbcBQQ2M (ORCPT ); Wed, 17 Feb 2016 11:28:12 -0500 Date: Wed, 17 Feb 2016 16:28:10 +0000 From: Matt Fleming To: Thomas Gleixner Cc: LKML , Peter Zijlstra , x86@kernel.org, Stephane Eranian , Vikas Shivappa Subject: Re: [PATCH] x86/perf/intel/cqm: Get rid of the silly for_each_cpu lookups Message-ID: <20160217162810.GA2694@codeblueprint.co.uk> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.24+41 (02bc14ed1569) (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, 17 Feb, at 05:08:57PM, Thomas Gleixner wrote: > On Wed, 17 Feb 2016, Thomas Gleixner wrote: > > > CQM is a strict per package facility. Use the proper cpumasks to lookup the > > readers. > > Sorry for the noise. PEBKAC: quilt refresh missing. Correct version below. > > Thanks, > > tglx > > 8<---------- > > Subject: x86/perf/cqm: Get rid of the silly for_each_cpu lookups > From: Thomas Gleixner > Date: Sun, 14 Feb 2016 23:09:06 +0100 > > CQM is a strict per package facility. Use the proper cpumasks to lookup the > readers. > > Signed-off-by: Thomas Gleixner > --- > arch/x86/kernel/cpu/perf_event_intel_cqm.c | 34 ++++++++++------------------- > 1 file changed, 12 insertions(+), 22 deletions(-) Right, I just could not make the connection between topology_core_cpumask() and topology_physical_package_id() when I wrote this code. Reviewed-by: Matt Fleming