On Mon, 19 Dec 2016, H. Peter Anvin wrote: > P.S. I would love to (a) move the CPUID bits into a structure instead of > passing all those crazy pointers around, and (b) stop passing struct > cpuinfo * around when we only use it for the current processor anyway (a > lot of these functions are in fact completely invalid if we don't); we > could define this_cpu_info as (*this_cpu_ptr(&cpu_info)) -- basically > what I have above -- or directly use percpu functions to access these > variables. The whole cpu identify code wants to be rewritten completely. It's full of conditionals and places which read the same cpuid leaf over and over. The whole thing is just insane. The proper solution is to have a single function which reads all available leafs into a data structure and then do all the conditional computations and decisions based on that. Thanks, tglx -- To unsubscribe from this list: send the line "unsubscribe linux-tip-commits" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
|