Re: [PATCH] powerpc/perf: Quiet PMU registration message

2018-10-09 Thread Madhavan Srinivasan




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 
---
  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;





[PATCH] powerpc/perf: Quiet PMU registration message

2018-10-08 Thread Joel Stanley
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 
---
 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);
 
power_pmu.attr_groups = ppmu->attr_groups;
 
-- 
2.17.1