[PATCH] powerpc/perf: Quiet PMU registration message

[PATCH] powerpc/perf: Quiet PMU registration message

[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 <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);
 
 	power_pmu.attr_groups = ppmu->attr_groups;
 
-- 
2.17.1

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

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