public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] perf/x86/intel: Keep anythread_deprecated capability
@ 2026-04-13 21:29 Namhyung Kim
  2026-04-14  2:09 ` Mi, Dapeng
  0 siblings, 1 reply; 3+ messages in thread
From: Namhyung Kim @ 2026-04-13 21:29 UTC (permalink / raw)
  To: Peter Zijlstra, Ingo Molnar
  Cc: Mark Rutland, Alexander Shishkin, Arnaldo Carvalho de Melo, LKML,
	Stephane Eranian, Dapeng Mi

In update_pmu_cap(), it reads MSR_IA32_PERF_CAPABILITIES on non-MTL CPUs
to update the PMU capabilities.  But it resets the anythread_deprecated
bit on GNR so I can see /sys/bus/event_source/devices/cpu/format/any
even if dmesg says AnyThread is deprecated.

  # dmesg | grep -A10 AnyThread
  [   10.662423] Performance Events: XSAVE Architectural LBR, PEBS fmt5+-baseline,  AnyThread deprecated, Granite Rapids events, 32-deep LBR, full-width counters, Intel PMU driver.
  [   10.663172] ... version:                   5
  [   10.663173] ... bit width:                 48
  [   10.663174] ... generic counters:          8
  [   10.663174] ... generic bitmap:            00000000000000ff
  [   10.663175] ... fixed-purpose counters:    4
  [   10.663176] ... fixed-purpose bitmap:      000000000000000f
  [   10.663176] ... value mask:                0000ffffffffffff
  [   10.663177] ... max period:                00007fffffffffff
  [   10.663178] ... global_ctrl mask:          0001000f000000ff
  [   10.668979] signal: max sigframe size: 11952

I guess it's not intentional and we want to keep deprecating anythread
on these machines.

Fixes: 25c623f41438fafc ("perf/x86/intel: Parse CPUID archPerfmonExt leaves for non-hybrid CPUs")
Cc: Dapeng Mi <dapeng1.mi@linux.intel.com>
Signed-off-by: Namhyung Kim <namhyung@kernel.org>
---
 arch/x86/events/intel/core.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/arch/x86/events/intel/core.c b/arch/x86/events/intel/core.c
index 4768236c054bbcf8..f1fae640cc8e5991 100644
--- a/arch/x86/events/intel/core.c
+++ b/arch/x86/events/intel/core.c
@@ -5945,8 +5945,12 @@ static void update_pmu_cap(struct pmu *pmu)
 	}
 
 	if (!intel_pmu_broken_perf_cap()) {
+		bool anythread = hybrid(pmu, intel_cap).anythread_deprecated;
+
 		/* Perf Metric (Bit 15) and PEBS via PT (Bit 16) are hybrid enumeration */
 		rdmsrq(MSR_IA32_PERF_CAPABILITIES, hybrid(pmu, intel_cap).capabilities);
+		/* It seems anythread_deprecated is deleted unintentionally */
+		hybrid(pmu, intel_cap).anythread_deprecated = anythread;
 	}
 }
 
-- 
2.54.0.rc0.605.g598a273b03-goog


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] perf/x86/intel: Keep anythread_deprecated capability
  2026-04-13 21:29 [PATCH] perf/x86/intel: Keep anythread_deprecated capability Namhyung Kim
@ 2026-04-14  2:09 ` Mi, Dapeng
  2026-04-14  6:56   ` Namhyung Kim
  0 siblings, 1 reply; 3+ messages in thread
From: Mi, Dapeng @ 2026-04-14  2:09 UTC (permalink / raw)
  To: Namhyung Kim, Peter Zijlstra, Ingo Molnar
  Cc: Mark Rutland, Alexander Shishkin, Arnaldo Carvalho de Melo, LKML,
	Stephane Eranian


On 4/14/2026 5:29 AM, Namhyung Kim wrote:
> In update_pmu_cap(), it reads MSR_IA32_PERF_CAPABILITIES on non-MTL CPUs
> to update the PMU capabilities.  But it resets the anythread_deprecated
> bit on GNR so I can see /sys/bus/event_source/devices/cpu/format/any
> even if dmesg says AnyThread is deprecated.
>
>   # dmesg | grep -A10 AnyThread
>   [   10.662423] Performance Events: XSAVE Architectural LBR, PEBS fmt5+-baseline,  AnyThread deprecated, Granite Rapids events, 32-deep LBR, full-width counters, Intel PMU driver.
>   [   10.663172] ... version:                   5
>   [   10.663173] ... bit width:                 48
>   [   10.663174] ... generic counters:          8
>   [   10.663174] ... generic bitmap:            00000000000000ff
>   [   10.663175] ... fixed-purpose counters:    4
>   [   10.663176] ... fixed-purpose bitmap:      000000000000000f
>   [   10.663176] ... value mask:                0000ffffffffffff
>   [   10.663177] ... max period:                00007fffffffffff
>   [   10.663178] ... global_ctrl mask:          0001000f000000ff
>   [   10.668979] signal: max sigframe size: 11952
>
> I guess it's not intentional and we want to keep deprecating anythread
> on these machines.
>
> Fixes: 25c623f41438fafc ("perf/x86/intel: Parse CPUID archPerfmonExt leaves for non-hybrid CPUs")
> Cc: Dapeng Mi <dapeng1.mi@linux.intel.com>
> Signed-off-by: Namhyung Kim <namhyung@kernel.org>
> ---
>  arch/x86/events/intel/core.c | 4 ++++
>  1 file changed, 4 insertions(+)
>
> diff --git a/arch/x86/events/intel/core.c b/arch/x86/events/intel/core.c
> index 4768236c054bbcf8..f1fae640cc8e5991 100644
> --- a/arch/x86/events/intel/core.c
> +++ b/arch/x86/events/intel/core.c
> @@ -5945,8 +5945,12 @@ static void update_pmu_cap(struct pmu *pmu)
>  	}
>  
>  	if (!intel_pmu_broken_perf_cap()) {
> +		bool anythread = hybrid(pmu, intel_cap).anythread_deprecated;
> +
>  		/* Perf Metric (Bit 15) and PEBS via PT (Bit 16) are hybrid enumeration */
>  		rdmsrq(MSR_IA32_PERF_CAPABILITIES, hybrid(pmu, intel_cap).capabilities);
> +		/* It seems anythread_deprecated is deleted unintentionally */
> +		hybrid(pmu, intel_cap).anythread_deprecated = anythread;
>  	}
>  }
>  

Namhyung, thanks for reporting this issue. But it may be not the best way
to fix the issue by restoring the anythread_deprecated bit in intel_cap. 

anythread_deprecated is enumerated in CPUID.0AH:EDX[15] instead of the
PERF_CAPABILITIES MSR. There is no anythread_deprecated bit in
PERF_CAPABILITIES MSR, It's not a good practice to
define anythread_deprecated bit in perf_capabilities .

Maybe we can do this, remove the anythread_deprecated from
perf_capabilities and directly depends on the CPUID.0AH:EDX[15] to check
if anythread is deprecated.

diff --git a/arch/x86/events/intel/core.c b/arch/x86/events/intel/core.c
index 793335c3ce78..450c63165a22 100644
--- a/arch/x86/events/intel/core.c
+++ b/arch/x86/events/intel/core.c
@@ -7612,11 +7612,8 @@ __init int intel_pmu_init(void)

        x86_add_quirk(intel_arch_events_quirk); /* Install first, so it
runs last */

-       if (version >= 5) {
-               x86_pmu.intel_cap.anythread_deprecated =
edx.split.anythread_deprecated;
-               if (x86_pmu.intel_cap.anythread_deprecated)
-                       pr_cont(" AnyThread deprecated, ");
-       }
+       if (version >= 5 && edx.split.anythread_deprecated)
+               pr_cont(" AnyThread deprecated, ");

        /* The perf side of core PMU is ready to support the mediated vPMU. */
        x86_get_pmu(smp_processor_id())->capabilities |=
PERF_PMU_CAP_MEDIATED_VPMU;
@@ -8467,7 +8464,7 @@ __init int intel_pmu_init(void)
                                      &x86_pmu.intel_ctrl);

        /* AnyThread may be deprecated on arch perfmon v5 or later */
-       if (x86_pmu.intel_cap.anythread_deprecated)
+       if (edx.split.anythread_deprecated)
                x86_pmu.format_attrs = intel_arch_formats_attr;

        intel_pmu_check_event_constraints_all(NULL);
diff --git a/arch/x86/events/perf_event.h b/arch/x86/events/perf_event.h
index fad87d3c8b2c..01217c663dff 100644
--- a/arch/x86/events/perf_event.h
+++ b/arch/x86/events/perf_event.h
@@ -660,7 +660,7 @@ union perf_capabilities {
                u64     perf_metrics:1;
                u64     pebs_output_pt_available:1;
                u64     pebs_timing_info:1;
-               u64     anythread_deprecated:1;
+               u64     __reserved:1;
                u64     rdpmc_metrics_clear:1;
        };
        u64     capabilities;

Not fully tested, only test on SPR and it looks good. 

Thanks.



^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] perf/x86/intel: Keep anythread_deprecated capability
  2026-04-14  2:09 ` Mi, Dapeng
@ 2026-04-14  6:56   ` Namhyung Kim
  0 siblings, 0 replies; 3+ messages in thread
From: Namhyung Kim @ 2026-04-14  6:56 UTC (permalink / raw)
  To: Mi, Dapeng
  Cc: Peter Zijlstra, Ingo Molnar, Mark Rutland, Alexander Shishkin,
	Arnaldo Carvalho de Melo, LKML, Stephane Eranian

Hello,

On Tue, Apr 14, 2026 at 10:09:58AM +0800, Mi, Dapeng wrote:
> 
> On 4/14/2026 5:29 AM, Namhyung Kim wrote:
> > In update_pmu_cap(), it reads MSR_IA32_PERF_CAPABILITIES on non-MTL CPUs
> > to update the PMU capabilities.  But it resets the anythread_deprecated
> > bit on GNR so I can see /sys/bus/event_source/devices/cpu/format/any
> > even if dmesg says AnyThread is deprecated.
> >
> >   # dmesg | grep -A10 AnyThread
> >   [   10.662423] Performance Events: XSAVE Architectural LBR, PEBS fmt5+-baseline,  AnyThread deprecated, Granite Rapids events, 32-deep LBR, full-width counters, Intel PMU driver.
> >   [   10.663172] ... version:                   5
> >   [   10.663173] ... bit width:                 48
> >   [   10.663174] ... generic counters:          8
> >   [   10.663174] ... generic bitmap:            00000000000000ff
> >   [   10.663175] ... fixed-purpose counters:    4
> >   [   10.663176] ... fixed-purpose bitmap:      000000000000000f
> >   [   10.663176] ... value mask:                0000ffffffffffff
> >   [   10.663177] ... max period:                00007fffffffffff
> >   [   10.663178] ... global_ctrl mask:          0001000f000000ff
> >   [   10.668979] signal: max sigframe size: 11952
> >
> > I guess it's not intentional and we want to keep deprecating anythread
> > on these machines.
> >
> > Fixes: 25c623f41438fafc ("perf/x86/intel: Parse CPUID archPerfmonExt leaves for non-hybrid CPUs")
> > Cc: Dapeng Mi <dapeng1.mi@linux.intel.com>
> > Signed-off-by: Namhyung Kim <namhyung@kernel.org>
> > ---
> >  arch/x86/events/intel/core.c | 4 ++++
> >  1 file changed, 4 insertions(+)
> >
> > diff --git a/arch/x86/events/intel/core.c b/arch/x86/events/intel/core.c
> > index 4768236c054bbcf8..f1fae640cc8e5991 100644
> > --- a/arch/x86/events/intel/core.c
> > +++ b/arch/x86/events/intel/core.c
> > @@ -5945,8 +5945,12 @@ static void update_pmu_cap(struct pmu *pmu)
> >  	}
> >  
> >  	if (!intel_pmu_broken_perf_cap()) {
> > +		bool anythread = hybrid(pmu, intel_cap).anythread_deprecated;
> > +
> >  		/* Perf Metric (Bit 15) and PEBS via PT (Bit 16) are hybrid enumeration */
> >  		rdmsrq(MSR_IA32_PERF_CAPABILITIES, hybrid(pmu, intel_cap).capabilities);
> > +		/* It seems anythread_deprecated is deleted unintentionally */
> > +		hybrid(pmu, intel_cap).anythread_deprecated = anythread;
> >  	}
> >  }
> >  
> 
> Namhyung, thanks for reporting this issue. But it may be not the best way
> to fix the issue by restoring the anythread_deprecated bit in intel_cap. 
> 
> anythread_deprecated is enumerated in CPUID.0AH:EDX[15] instead of the
> PERF_CAPABILITIES MSR. There is no anythread_deprecated bit in
> PERF_CAPABILITIES MSR, It's not a good practice to
> define anythread_deprecated bit in perf_capabilities .
> 
> Maybe we can do this, remove the anythread_deprecated from
> perf_capabilities and directly depends on the CPUID.0AH:EDX[15] to check
> if anythread is deprecated.

Looks good to me.  You can add

Acked-by: Namhyung Kim <namhyung@kernel.org>

Thanks,
Namhyung

> 
> diff --git a/arch/x86/events/intel/core.c b/arch/x86/events/intel/core.c
> index 793335c3ce78..450c63165a22 100644
> --- a/arch/x86/events/intel/core.c
> +++ b/arch/x86/events/intel/core.c
> @@ -7612,11 +7612,8 @@ __init int intel_pmu_init(void)
> 
>         x86_add_quirk(intel_arch_events_quirk); /* Install first, so it
> runs last */
> 
> -       if (version >= 5) {
> -               x86_pmu.intel_cap.anythread_deprecated =
> edx.split.anythread_deprecated;
> -               if (x86_pmu.intel_cap.anythread_deprecated)
> -                       pr_cont(" AnyThread deprecated, ");
> -       }
> +       if (version >= 5 && edx.split.anythread_deprecated)
> +               pr_cont(" AnyThread deprecated, ");
> 
>         /* The perf side of core PMU is ready to support the mediated vPMU. */
>         x86_get_pmu(smp_processor_id())->capabilities |=
> PERF_PMU_CAP_MEDIATED_VPMU;
> @@ -8467,7 +8464,7 @@ __init int intel_pmu_init(void)
>                                       &x86_pmu.intel_ctrl);
> 
>         /* AnyThread may be deprecated on arch perfmon v5 or later */
> -       if (x86_pmu.intel_cap.anythread_deprecated)
> +       if (edx.split.anythread_deprecated)
>                 x86_pmu.format_attrs = intel_arch_formats_attr;
> 
>         intel_pmu_check_event_constraints_all(NULL);
> diff --git a/arch/x86/events/perf_event.h b/arch/x86/events/perf_event.h
> index fad87d3c8b2c..01217c663dff 100644
> --- a/arch/x86/events/perf_event.h
> +++ b/arch/x86/events/perf_event.h
> @@ -660,7 +660,7 @@ union perf_capabilities {
>                 u64     perf_metrics:1;
>                 u64     pebs_output_pt_available:1;
>                 u64     pebs_timing_info:1;
> -               u64     anythread_deprecated:1;
> +               u64     __reserved:1;
>                 u64     rdpmc_metrics_clear:1;
>         };
>         u64     capabilities;
> 
> Not fully tested, only test on SPR and it looks good. 
> 
> Thanks.
> 
> 

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2026-04-14  6:56 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-04-13 21:29 [PATCH] perf/x86/intel: Keep anythread_deprecated capability Namhyung Kim
2026-04-14  2:09 ` Mi, Dapeng
2026-04-14  6:56   ` Namhyung Kim

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox