| Message ID | 20220610044006.2095806-1-joel@jms.id.au (mailing list archive) |
|---|---|
| State | Accepted |
| Headers | show |
| Series | [v2] powerpc/perf: Give generic PMU a nice name | expand |
| Context | Check | Description |
|---|---|---|
| snowpatch_ozlabs/github-powerpc_selftests | success | Successfully ran 10 jobs. |
| snowpatch_ozlabs/github-powerpc_ppctests | success | Successfully ran 10 jobs. |
| snowpatch_ozlabs/github-powerpc_sparse | success | Successfully ran 4 jobs. |
| snowpatch_ozlabs/github-powerpc_clang | success | Successfully ran 7 jobs. |
| snowpatch_ozlabs/github-powerpc_kernel_qemu | success | Successfully ran 23 jobs. |
> On 10-Jun-2022, at 10:10 AM, Joel Stanley <joel@jms.id.au> wrote: > > When booting on a machine that uses the compat pmu driver we see this: > > [ 0.071192] GENERIC_COMPAT performance monitor hardware support registered > > Which is a bit shouty. Give it a nicer name. > > Signed-off-by: Joel Stanley <joel@jms.id.au> > --- > v2: Go with ISAv3 > > arch/powerpc/perf/generic-compat-pmu.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/powerpc/perf/generic-compat-pmu.c b/arch/powerpc/perf/generic-compat-pmu.c > index f3db88aee4dd..16392962c511 100644 > --- a/arch/powerpc/perf/generic-compat-pmu.c > +++ b/arch/powerpc/perf/generic-compat-pmu.c > @@ -292,7 +292,7 @@ static int generic_compute_mmcr(u64 event[], int n_ev, > } > > static struct power_pmu generic_compat_pmu = { > - .name = "GENERIC_COMPAT", > + .name = "ISAv3", Looks good. Reviewed-by: Athira Rajeev <atrajeev@linux.vnet.ibm.com> > .n_counter = MAX_PMU_COUNTERS, > .add_fields = ISA207_ADD_FIELDS, > .test_adder = ISA207_TEST_ADDER, > -- > 2.35.1 >
On Fri, 10 Jun 2022 14:10:06 +0930, Joel Stanley wrote: > When booting on a machine that uses the compat pmu driver we see this: > > [ 0.071192] GENERIC_COMPAT performance monitor hardware support registered > > Which is a bit shouty. Give it a nicer name. > > > [...] Applied to powerpc/next. [1/1] powerpc/perf: Give generic PMU a nice name https://git.kernel.org/powerpc/c/78988b273d592ce74c8aecdd5d748906c38a9e9d cheers
diff --git a/arch/powerpc/perf/generic-compat-pmu.c b/arch/powerpc/perf/generic-compat-pmu.c index f3db88aee4dd..16392962c511 100644 --- a/arch/powerpc/perf/generic-compat-pmu.c +++ b/arch/powerpc/perf/generic-compat-pmu.c @@ -292,7 +292,7 @@ static int generic_compute_mmcr(u64 event[], int n_ev, } static struct power_pmu generic_compat_pmu = { - .name = "GENERIC_COMPAT", + .name = "ISAv3", .n_counter = MAX_PMU_COUNTERS, .add_fields = ISA207_ADD_FIELDS, .test_adder = ISA207_TEST_ADDER,
When booting on a machine that uses the compat pmu driver we see this: [ 0.071192] GENERIC_COMPAT performance monitor hardware support registered Which is a bit shouty. Give it a nicer name. Signed-off-by: Joel Stanley <joel@jms.id.au> --- v2: Go with ISAv3 arch/powerpc/perf/generic-compat-pmu.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)