From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 73389C61DA4 for ; Fri, 3 Feb 2023 20:28:22 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233019AbjBCU2U (ORCPT ); Fri, 3 Feb 2023 15:28:20 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:33190 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233338AbjBCU2R (ORCPT ); Fri, 3 Feb 2023 15:28:17 -0500 Received: from ams.source.kernel.org (ams.source.kernel.org [145.40.68.75]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 8844F125BD for ; Fri, 3 Feb 2023 12:28:15 -0800 (PST) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ams.source.kernel.org (Postfix) with ESMTPS id 67B58B82B8E for ; Fri, 3 Feb 2023 20:28:14 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id B1449C433D2; Fri, 3 Feb 2023 20:28:12 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1675456092; bh=jH0mfNTLonbjusZsctWSDRNA+5PQjyYsFOLOO66NM4c=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=GrYDQGhoyuCAxe1nU06K7mIpFqXbfg+cgNFfTM/14TlTyi+BNUTpKLD1Vqu6dE3rb Wb95E8IR7nHYMQ687g50iwH6XeGskEepxkbZqaT67EL+yO35/di2t4iFNnx4PRu/JJ DK1y3BXYeOKtwFp1KIvvgAT6na08GJkDGT8IlXoXyX9NoflGRDkrULK/B5kNcyxT/e 1epqLbdTzv++cHAiwKKElyZLbfrrqXi6tS7FHlIrFv9LU2OA8D6ijiGFiaphmGj3Ym zISU/sG8ZyVNwPYmPFe3PYW8x5GT9cNlpTeUJtzLDTIpYbJs1zG1nZwbyRvAvd3EKg r7hTIRe+U42DA== Received: by quaco.ghostprotocols.net (Postfix, from userid 1000) id 436D6405BE; Fri, 3 Feb 2023 17:28:10 -0300 (-03) Date: Fri, 3 Feb 2023 17:28:10 -0300 From: Arnaldo Carvalho de Melo To: "Liang, Kan" Cc: peterz@infradead.org, mingo@redhat.com, linux-kernel@vger.kernel.org, ak@linux.intel.com, eranian@google.com, irogers@google.com Subject: Re: [PATCH V2 7/9] perf/x86/msr: Add Meteor Lake support Message-ID: References: <20230104201349.1451191-1-kan.liang@linux.intel.com> <20230104201349.1451191-7-kan.liang@linux.intel.com> <79807730-73ac-c8c2-fc9d-b7cd00f9336e@linux.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-Url: http://acmel.wordpress.com Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Em Fri, Feb 03, 2023 at 05:21:15PM -0300, Arnaldo Carvalho de Melo escreveu: > Em Thu, Feb 02, 2023 at 09:34:02AM -0500, Liang, Kan escreveu: > > Hi Arnaldo, > > > > On 2023-02-01 8:47 p.m., Arnaldo Carvalho de Melo wrote: > > > Em Wed, Jan 04, 2023 at 12:13:47PM -0800, kan.liang@linux.intel.com escreveu: > > >> From: Kan Liang > > >> > > >> Meteor Lake is Intel's successor to Raptor lake. PPERF and SMI_COUNT MSRs > > >> are also supported. > > >> > > >> Reviewed-by: Andi Kleen > > >> Signed-off-by: Kan Liang > > >> --- > > > > > > Did the kernel bits land upstream? > > Yes, the kernel part has been merged into the tip.git perf/core branch. > > > > Thanks for checking the status. There are two perf tool patches in this > > series, which hasn't been merged. Should I resend them? > > Lemme try cherry-picking just the tooling bits from this series. There was a clash with: commit 3fd7a168bf51497909dbfb7347af28b5c57e74a6 Author: Namhyung Kim Date: Thu Jan 26 13:36:10 2023 -0800 perf script: Add 'cgroup' field for output And a minor fuzz on the first patch, I applied manually and resolved the conflict, Thanks, - Arnaldo > > Thanks, > > Kan > > > > > > - Arnaldo > > > > > >> No change since V1 > > >> > > >> arch/x86/events/msr.c | 2 ++ > > >> 1 file changed, 2 insertions(+) > > >> > > >> diff --git a/arch/x86/events/msr.c b/arch/x86/events/msr.c > > >> index ecced3a52668..074150d28fa8 100644 > > >> --- a/arch/x86/events/msr.c > > >> +++ b/arch/x86/events/msr.c > > >> @@ -107,6 +107,8 @@ static bool test_intel(int idx, void *data) > > >> case INTEL_FAM6_RAPTORLAKE: > > >> case INTEL_FAM6_RAPTORLAKE_P: > > >> case INTEL_FAM6_RAPTORLAKE_S: > > >> + case INTEL_FAM6_METEORLAKE: > > >> + case INTEL_FAM6_METEORLAKE_L: > > >> if (idx == PERF_MSR_SMI || idx == PERF_MSR_PPERF) > > >> return true; > > >> break; > > >> -- > > >> 2.35.1 > > > > > -- > > - Arnaldo -- - Arnaldo