RE: [PATCH V3] perf/x86/intel/uncore: remove nonexistent clockticks event for client uncore

From: Liang, Kan
Date: Tue Jul 11 2017 - 11:44:31 EST


>
> > User observable change with the patch.
> > clockticks event is removed from CBOX. User may need to change their
> > script to use uncore_clock/clockticks/ instead.
>
> I don't think we can do that and break everyone's scripts. Have to keep the
> old behavior, even though it is not ideal.
>
> Or you fix the the uncore subsystem to support a separate attribute array for
> the first cbox.
>

I once had a patch to only support clockticks for first box.
https://lkml.org/lkml/2016/12/1/477
But Peter think it's better to move it to its own PMU.
https://lkml.org/lkml/2016/12/1/484

Each solution has its own advantage and drawback. I'm OK for either of them.
I'm not sure what others prefer.

Peter? Ingo? What do you think?

Thanks,
Kan