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 <j...@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;


Reply via email to