lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Mon, 6 May 2019 13:48:19 +0200 From: Ingo Molnar <mingo@...nel.org> To: Len Brown <lenb@...nel.org> Cc: x86@...nel.org, linux-kernel@...r.kernel.org, Kan Liang <kan.liang@...ux.intel.com>, Peter Zijlstra <peterz@...radead.org>, Len Brown <len.brown@...el.com>, Thomas Gleixner <tglx@...utronix.de> Subject: Re: [PATCH 17/18] perf/x86/intel/rapl: Support multi-die/package * Len Brown <lenb@...nel.org> wrote: > > static inline struct rapl_pmu *cpu_to_rapl_pmu(unsigned int cpu) > { > - unsigned int pkgid = topology_logical_package_id(cpu); > + unsigned int pkgid = topology_logical_die_id(cpu); Yeah, I think Peter pointed this out before: why is the local variable still named 'pkg' when we now have a die ID? The other patches have such problems too: when new facilities are introduced and used then function names and variable names need to be harmonized. Makes for rather confusing code otherwise, which invites bugs down the road. There's many such instances left in this series, please review the whole patchset for such problems. Thanks, Ingo
Powered by blists - more mailing lists