Re: [PATCH 0/2]tools/power turbostat: enable turbostat to support Knights Mill (KNM)

From: Luc, Piotr
Date: Mon Oct 24 2016 - 12:07:35 EST


On Tue, 2016-10-18 at 23:06 +0200, Piotr Luc wrote:
> This patchset is rebased onto
> 08328814256d888634ff15ba8fb67e2ae4340b64.Â
>
> First patch improves code style by replacing raw numbers
> of CPUID with descriptive macros of processor names.
>
> The second patch adds Knights Mill (KNM) to the list of CPUIDs
> supported
> by turbostat.
>
> Both patches come in new versions where trailing comments, whichÂ
> described CPUs, are removed because self-descriptive macros areÂ
> used instead raw numbers.
>
> Piotr Luc (2):
> Â tools/power turbostat: Use Intel family processor macros
> Â tools/power turbostat: enable turbostat to support Knights Mill
> (KNM)
>
> Âtools/power/x86/turbostat/MakefileÂÂÂÂ|ÂÂÂ3 +-
> Âtools/power/x86/turbostat/turbostat.c | 250 ++++++++++++++++++----
> ------------
> Â2 files changed, 131 insertions(+), 122 deletions(-)
>

Hi

I wonder if you could share with me what I need to do more to make the
patchset be accepted?

Regards,
Piotr