Re: [RESEND PATCH 00/11] Add Knights Mill CPU ID
On Thu, 2016-10-13 at 23:56 +0200, Rafael J. Wysocki wrote: > On Thursday, October 13, 2016 05:30:54 PM Piotr Luc wrote: > > > > Piotr Luc (11): > > EDAC, sb_edac: Use Intel family processor macros > > tools/power turbostat: Use Intel family processor macros > > x86/cpu/intel: Add Knights Mill to Intel family > > x86/intel_idle: Add Knights Mill CPUID > > EDAC, sb_edac: Add Knights Mill CPUID > > cpufreq: intel_pstate: Add Knights Mill CPUID > > perf/x86/intel: Add Knights Mill CPUID > > perf/x86/intel/rapl: Add Knights Mill CPUID > > perf/x86/intel/uncore: Add Knights Mill CPUID > > powercap / RAPL: Add Knights Mill CPUID > > tools/power turbostat: enable turbostat to support Knights Mill > > (KNM) > > > > arch/x86/events/intel/core.c | 3 +- > > arch/x86/events/intel/rapl.c | 1 + > > arch/x86/events/intel/uncore.c| 1 + > > arch/x86/include/asm/intel-family.h | 1 + > > drivers/cpufreq/intel_pstate.c| 1 + > > drivers/edac/sb_edac.c| 14 +- > > drivers/idle/intel_idle.c | 1 + > > drivers/powercap/intel_rapl.c | 1 + > > tools/power/x86/turbostat/Makefile| 3 +- > > tools/power/x86/turbostat/turbostat.c | 250 ++-- > > -- > > 10 files changed, 147 insertions(+), 129 deletions(-) > > Len, Jacob, any problems with any of the patches in this series? > > Thanks, > Rafael Hi I wonder if you could share with me what I need to do more to make the patches be accepted? Do I need to resend the ones that aren't accepted yet? Regards, Piotr
Re: [RESEND PATCH 00/11] Add Knights Mill CPU ID
On Thu, 2016-10-13 at 23:56 +0200, Rafael J. Wysocki wrote: > On Thursday, October 13, 2016 05:30:54 PM Piotr Luc wrote: > > > > Piotr Luc (11): > > EDAC, sb_edac: Use Intel family processor macros > > tools/power turbostat: Use Intel family processor macros > > x86/cpu/intel: Add Knights Mill to Intel family > > x86/intel_idle: Add Knights Mill CPUID > > EDAC, sb_edac: Add Knights Mill CPUID > > cpufreq: intel_pstate: Add Knights Mill CPUID > > perf/x86/intel: Add Knights Mill CPUID > > perf/x86/intel/rapl: Add Knights Mill CPUID > > perf/x86/intel/uncore: Add Knights Mill CPUID > > powercap / RAPL: Add Knights Mill CPUID > > tools/power turbostat: enable turbostat to support Knights Mill > > (KNM) > > > > arch/x86/events/intel/core.c | 3 +- > > arch/x86/events/intel/rapl.c | 1 + > > arch/x86/events/intel/uncore.c| 1 + > > arch/x86/include/asm/intel-family.h | 1 + > > drivers/cpufreq/intel_pstate.c| 1 + > > drivers/edac/sb_edac.c| 14 +- > > drivers/idle/intel_idle.c | 1 + > > drivers/powercap/intel_rapl.c | 1 + > > tools/power/x86/turbostat/Makefile| 3 +- > > tools/power/x86/turbostat/turbostat.c | 250 ++-- > > -- > > 10 files changed, 147 insertions(+), 129 deletions(-) > > Len, Jacob, any problems with any of the patches in this series? > > Thanks, > Rafael Hi I wonder if you could share with me what I need to do more to make the patches be accepted? Do I need to resend the ones that aren't accepted yet? Regards, Piotr
Re: [RESEND PATCH 00/11] Add Knights Mill CPU ID
On Thursday, October 13, 2016 05:30:54 PM Piotr Luc wrote: > Resending patches related to KNM CPU ID in one patchset for > more convenient pick up. > > This series of patches enables support for Knights Mill (KNM). > > First two patches improve code style by replacing raw numbers > of CPUID with descriptive macros of processor names. > > The third one introduces KNM CPU ID to Intel family. > > The other patches add KNM CPU ID to several modules and depend on > the third one. Because KNM is closely related to Knights Landing (KNL) > it reuses KNL code. > The turbostat patch (tools/power turbostat: Use Intel family processor > macros) comes in v2 with fixed compilation (missed including of > intel-family.h). > > Regards, > Piotr > > Piotr Luc (11): > EDAC, sb_edac: Use Intel family processor macros > tools/power turbostat: Use Intel family processor macros > x86/cpu/intel: Add Knights Mill to Intel family > x86/intel_idle: Add Knights Mill CPUID > EDAC, sb_edac: Add Knights Mill CPUID > cpufreq: intel_pstate: Add Knights Mill CPUID > perf/x86/intel: Add Knights Mill CPUID > perf/x86/intel/rapl: Add Knights Mill CPUID > perf/x86/intel/uncore: Add Knights Mill CPUID > powercap / RAPL: Add Knights Mill CPUID > tools/power turbostat: enable turbostat to support Knights Mill (KNM) > > arch/x86/events/intel/core.c | 3 +- > arch/x86/events/intel/rapl.c | 1 + > arch/x86/events/intel/uncore.c| 1 + > arch/x86/include/asm/intel-family.h | 1 + > drivers/cpufreq/intel_pstate.c| 1 + > drivers/edac/sb_edac.c| 14 +- > drivers/idle/intel_idle.c | 1 + > drivers/powercap/intel_rapl.c | 1 + > tools/power/x86/turbostat/Makefile| 3 +- > tools/power/x86/turbostat/turbostat.c | 250 > ++ > 10 files changed, 147 insertions(+), 129 deletions(-) Len, Jacob, any problems with any of the patches in this series? Thanks, Rafael
Re: [RESEND PATCH 00/11] Add Knights Mill CPU ID
On Thursday, October 13, 2016 05:30:54 PM Piotr Luc wrote: > Resending patches related to KNM CPU ID in one patchset for > more convenient pick up. > > This series of patches enables support for Knights Mill (KNM). > > First two patches improve code style by replacing raw numbers > of CPUID with descriptive macros of processor names. > > The third one introduces KNM CPU ID to Intel family. > > The other patches add KNM CPU ID to several modules and depend on > the third one. Because KNM is closely related to Knights Landing (KNL) > it reuses KNL code. > The turbostat patch (tools/power turbostat: Use Intel family processor > macros) comes in v2 with fixed compilation (missed including of > intel-family.h). > > Regards, > Piotr > > Piotr Luc (11): > EDAC, sb_edac: Use Intel family processor macros > tools/power turbostat: Use Intel family processor macros > x86/cpu/intel: Add Knights Mill to Intel family > x86/intel_idle: Add Knights Mill CPUID > EDAC, sb_edac: Add Knights Mill CPUID > cpufreq: intel_pstate: Add Knights Mill CPUID > perf/x86/intel: Add Knights Mill CPUID > perf/x86/intel/rapl: Add Knights Mill CPUID > perf/x86/intel/uncore: Add Knights Mill CPUID > powercap / RAPL: Add Knights Mill CPUID > tools/power turbostat: enable turbostat to support Knights Mill (KNM) > > arch/x86/events/intel/core.c | 3 +- > arch/x86/events/intel/rapl.c | 1 + > arch/x86/events/intel/uncore.c| 1 + > arch/x86/include/asm/intel-family.h | 1 + > drivers/cpufreq/intel_pstate.c| 1 + > drivers/edac/sb_edac.c| 14 +- > drivers/idle/intel_idle.c | 1 + > drivers/powercap/intel_rapl.c | 1 + > tools/power/x86/turbostat/Makefile| 3 +- > tools/power/x86/turbostat/turbostat.c | 250 > ++ > 10 files changed, 147 insertions(+), 129 deletions(-) Len, Jacob, any problems with any of the patches in this series? Thanks, Rafael
[RESEND PATCH 00/11] Add Knights Mill CPU ID
Resending patches related to KNM CPU ID in one patchset for more convenient pick up. This series of patches enables support for Knights Mill (KNM). First two patches improve code style by replacing raw numbers of CPUID with descriptive macros of processor names. The third one introduces KNM CPU ID to Intel family. The other patches add KNM CPU ID to several modules and depend on the third one. Because KNM is closely related to Knights Landing (KNL) it reuses KNL code. The turbostat patch (tools/power turbostat: Use Intel family processor macros) comes in v2 with fixed compilation (missed including of intel-family.h). Regards, Piotr Piotr Luc (11): EDAC, sb_edac: Use Intel family processor macros tools/power turbostat: Use Intel family processor macros x86/cpu/intel: Add Knights Mill to Intel family x86/intel_idle: Add Knights Mill CPUID EDAC, sb_edac: Add Knights Mill CPUID cpufreq: intel_pstate: Add Knights Mill CPUID perf/x86/intel: Add Knights Mill CPUID perf/x86/intel/rapl: Add Knights Mill CPUID perf/x86/intel/uncore: Add Knights Mill CPUID powercap / RAPL: Add Knights Mill CPUID tools/power turbostat: enable turbostat to support Knights Mill (KNM) arch/x86/events/intel/core.c | 3 +- arch/x86/events/intel/rapl.c | 1 + arch/x86/events/intel/uncore.c| 1 + arch/x86/include/asm/intel-family.h | 1 + drivers/cpufreq/intel_pstate.c| 1 + drivers/edac/sb_edac.c| 14 +- drivers/idle/intel_idle.c | 1 + drivers/powercap/intel_rapl.c | 1 + tools/power/x86/turbostat/Makefile| 3 +- tools/power/x86/turbostat/turbostat.c | 250 ++ 10 files changed, 147 insertions(+), 129 deletions(-) -- 2.10.1
[RESEND PATCH 00/11] Add Knights Mill CPU ID
Resending patches related to KNM CPU ID in one patchset for more convenient pick up. This series of patches enables support for Knights Mill (KNM). First two patches improve code style by replacing raw numbers of CPUID with descriptive macros of processor names. The third one introduces KNM CPU ID to Intel family. The other patches add KNM CPU ID to several modules and depend on the third one. Because KNM is closely related to Knights Landing (KNL) it reuses KNL code. The turbostat patch (tools/power turbostat: Use Intel family processor macros) comes in v2 with fixed compilation (missed including of intel-family.h). Regards, Piotr Piotr Luc (11): EDAC, sb_edac: Use Intel family processor macros tools/power turbostat: Use Intel family processor macros x86/cpu/intel: Add Knights Mill to Intel family x86/intel_idle: Add Knights Mill CPUID EDAC, sb_edac: Add Knights Mill CPUID cpufreq: intel_pstate: Add Knights Mill CPUID perf/x86/intel: Add Knights Mill CPUID perf/x86/intel/rapl: Add Knights Mill CPUID perf/x86/intel/uncore: Add Knights Mill CPUID powercap / RAPL: Add Knights Mill CPUID tools/power turbostat: enable turbostat to support Knights Mill (KNM) arch/x86/events/intel/core.c | 3 +- arch/x86/events/intel/rapl.c | 1 + arch/x86/events/intel/uncore.c| 1 + arch/x86/include/asm/intel-family.h | 1 + drivers/cpufreq/intel_pstate.c| 1 + drivers/edac/sb_edac.c| 14 +- drivers/idle/intel_idle.c | 1 + drivers/powercap/intel_rapl.c | 1 + tools/power/x86/turbostat/Makefile| 3 +- tools/power/x86/turbostat/turbostat.c | 250 ++ 10 files changed, 147 insertions(+), 129 deletions(-) -- 2.10.1