git: 89f1b82a859f - stable/13 - libpmc: fix the 'cycles' event alias on x86

From: Mitchell Horne <mhorne_at_FreeBSD.org>
Date: Fri, 15 Oct 2021 15:26:13 UTC
The branch stable/13 has been updated by mhorne:

URL: https://cgit.FreeBSD.org/src/commit/?id=89f1b82a859fe4b9d4a59e9089a634ec24f127dd

commit 89f1b82a859fe4b9d4a59e9089a634ec24f127dd
Author:     Mitchell Horne <mhorne@FreeBSD.org>
AuthorDate: 2021-09-30 14:11:36 +0000
Commit:     Mitchell Horne <mhorne@FreeBSD.org>
CommitDate: 2021-10-15 15:21:35 +0000

    libpmc: fix the 'cycles' event alias on x86
    
    Looking for "tsc-tsc" in the pmu tables will fail every time. Instead,
    make this an alias for the static TSC event defined in pmc_events.h.
    This fixes 'pmcstat -s cycles' on Intel and AMD.
    
    Reviewed by:    emaste
    MFC after:      1 week
    Sponsored by:   The FreeBSD Foundation
    Differential Revision:  https://reviews.freebsd.org/D32197
    
    (cherry picked from commit 937539e0a32ce1da46223664ca1cf3b252e02ece)
---
 lib/libpmc/libpmc.c          | 2 +-
 lib/libpmc/libpmc_pmu_util.c | 1 -
 sys/dev/hwpmc/pmc_events.h   | 3 +++
 3 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/lib/libpmc/libpmc.c b/lib/libpmc/libpmc.c
index e2422f167267..78fde0ed1f2c 100644
--- a/lib/libpmc/libpmc.c
+++ b/lib/libpmc/libpmc.c
@@ -179,7 +179,7 @@ static const struct pmc_event_descr cortex_a76_event_table[] =
 
 static const struct pmc_event_descr tsc_event_table[] =
 {
-	__PMC_EV_TSC()
+	__PMC_EV_ALIAS_TSC()
 };
 
 #undef	PMC_CLASS_TABLE_DESC
diff --git a/lib/libpmc/libpmc_pmu_util.c b/lib/libpmc/libpmc_pmu_util.c
index e6f74e6abe81..b72f45e85a72 100644
--- a/lib/libpmc/libpmc_pmu_util.c
+++ b/lib/libpmc/libpmc_pmu_util.c
@@ -74,7 +74,6 @@ static struct pmu_alias pmu_intel_alias_table[] = {
 	{"BRANCH-INSTRUCTION-RETIRED", "BR_INST_RETIRED.ALL_BRANCHES"},
 	{"BRANCH_MISSES_RETIRED", "BR_MISP_RETIRED.ALL_BRANCHES"},
 	{"BRANCH-MISSES-RETIRED", "BR_MISP_RETIRED.ALL_BRANCHES"},
-	{"cycles", "tsc-tsc"},
 	{"unhalted-cycles", "CPU_CLK_UNHALTED.THREAD_P_ANY"},
 	{"instructions", "inst_retired.any_p"},
 	{"branch-mispredicts", "br_misp_retired.all_branches"},
diff --git a/sys/dev/hwpmc/pmc_events.h b/sys/dev/hwpmc/pmc_events.h
index df56d728907a..6a727c6e7f25 100644
--- a/sys/dev/hwpmc/pmc_events.h
+++ b/sys/dev/hwpmc/pmc_events.h
@@ -54,6 +54,9 @@
 #define        PMC_EV_TSC_FIRST        PMC_EV_TSC_TSC
 #define        PMC_EV_TSC_LAST         PMC_EV_TSC_TSC
 
+#define	__PMC_EV_ALIAS_TSC()			\
+__PMC_EV_ALIAS("cycles",	TSC_TSC)
+
 /*
  * Software events are dynamically defined.
  */