| Message ID | 20181009055444.20317-1-joel@jms.id.au (mailing list archive) |
|---|---|
| State | Rejected |
| Headers | show |
| Series | powerpc/perf: Quiet PMU registration message | expand |
On Tuesday 09 October 2018 11:24 AM, Joel Stanley wrote: > On a Power9 box we get a few screens full of these on boot. Drop > them to pr_debug. > > [ 5.993645] nest_centaur6_imc performance monitor hardware support registered > [ 5.993728] nest_centaur7_imc performance monitor hardware support registered > [ 5.996510] core_imc performance monitor hardware support registered > [ 5.996569] nest_mba0_imc performance monitor hardware support registered > [ 5.996631] nest_mba1_imc performance monitor hardware support registered > [ 5.996685] nest_mba2_imc performance monitor hardware support registered > > Signed-off-by: Joel Stanley <joel@jms.id.au> > --- > arch/powerpc/perf/core-book3s.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/arch/powerpc/perf/core-book3s.c b/arch/powerpc/perf/core-book3s.c > index 81f8a0c838ae..a01c521694e8 100644 > --- a/arch/powerpc/perf/core-book3s.c > +++ b/arch/powerpc/perf/core-book3s.c > @@ -2249,8 +2249,8 @@ int register_power_pmu(struct power_pmu *pmu) > return -EBUSY; /* something's already registered */ > > ppmu = pmu; > - pr_info("%s performance monitor hardware support registered\n", > - pmu->name); > + pr_debug("%s performance monitor hardware support registered\n", > + pmu->name); We should have this message since this is for CPU PMU. But we can imc once. Something like this diff --git a/arch/powerpc/perf/imc-pmu.c b/arch/powerpc/perf/imc-pmu.c index 1fafc32b12a0..6954636b16d1 100644 --- a/arch/powerpc/perf/imc-pmu.c +++ b/arch/powerpc/perf/imc-pmu.c @@ -1392,7 +1392,7 @@ int init_imc_pmu(struct device_node *parent, struct imc_pmu *pmu_ptr, int pmu_id if (ret) goto err_free_cpuhp_mem; - pr_info("%s performance monitor hardware support registered\n", + pr_debug("%s performance monitor hardware support registered\n", pmu_ptr->pmu.name); return 0; Maddy > power_pmu.attr_groups = ppmu->attr_groups; >
diff --git a/arch/powerpc/perf/core-book3s.c b/arch/powerpc/perf/core-book3s.c index 81f8a0c838ae..a01c521694e8 100644 --- a/arch/powerpc/perf/core-book3s.c +++ b/arch/powerpc/perf/core-book3s.c @@ -2249,8 +2249,8 @@ int register_power_pmu(struct power_pmu *pmu) return -EBUSY; /* something's already registered */ ppmu = pmu; - pr_info("%s performance monitor hardware support registered\n", - pmu->name); + pr_debug("%s performance monitor hardware support registered\n", + pmu->name); power_pmu.attr_groups = ppmu->attr_groups;
On a Power9 box we get a few screens full of these on boot. Drop them to pr_debug. [ 5.993645] nest_centaur6_imc performance monitor hardware support registered [ 5.993728] nest_centaur7_imc performance monitor hardware support registered [ 5.996510] core_imc performance monitor hardware support registered [ 5.996569] nest_mba0_imc performance monitor hardware support registered [ 5.996631] nest_mba1_imc performance monitor hardware support registered [ 5.996685] nest_mba2_imc performance monitor hardware support registered Signed-off-by: Joel Stanley <joel@jms.id.au> --- arch/powerpc/perf/core-book3s.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)