Re: [PATCH] powerpc/powernv: Add kernel cmdline parameter to disable imc

From: Madhavan Srinivasan
Date: Thu Oct 12 2017 - 04:05:50 EST




On Thursday 12 October 2017 01:21 PM, Stewart Smith wrote:
Anju T Sudhakar <anju@xxxxxxxxxxxxxxxxxx> writes:
On Wednesday 11 October 2017 01:55 AM, Stewart Smith wrote:
Michael Ellerman <mpe@xxxxxxxxxxxxxx> writes:
Anju T Sudhakar <anju@xxxxxxxxxxxxxxxxxx> writes:

Add a kernel command line parameter option to disable In-Memory Collection
(IMC) counters and add documentation. This helps in debug.
I'd really rather we didn't. Do we *really* need this?

We don't have command line parameters to disable any of the other ~20
PMUs, why is this one special?
This one is really helpful in debugging, incase if we want to proceed
without nest counters OR
core counters . But if we have the facility to do the same from
petitboot, its fine.
https://patchwork.ozlabs.org/patch/824497/ (skiboot patch) should do the
same thing, right? That means we come up with them off (as we should
have from the start) and if you don't use the PMUs then the ucode won't
be started.
Stewart,

Thats right. With that skiboot patch, IMC will be paused at start.