On 08/20/2012 08:39 AM, Borislav Petkov wrote: > On Sat, Aug 18, 2012 at 01:15:18AM -0700, Fenghua Yu wrote: >> Documentation/x86/earlyucode.txt | 43 +++ >> arch/x86/Kconfig | 22 ++ >> arch/x86/include/asm/cpio.h | 10 + >> arch/x86/include/asm/microcode.h | 23 ++ >> arch/x86/include/asm/microcode_intel.h | 103 +++++++ >> arch/x86/kernel/Makefile | 3 + >> arch/x86/kernel/head64.c | 6 + >> arch/x86/kernel/head_32.S | 6 + >> arch/x86/kernel/microcode_core.c | 7 +- >> arch/x86/kernel/microcode_core_early.c | 74 +++++ >> arch/x86/kernel/microcode_intel.c | 185 +------------ >> arch/x86/kernel/microcode_intel_early.c | 482 >> +++++++++++++++++++++++++++++++ >> arch/x86/kernel/microcode_intel_lib.c | 163 +++++++++++ > > One more thing: > > This screams for adding a directory called "microcode" in the kernel > sources and moving all the microcode-related files there. IOW: > > arch/x86/kernel/cpu/microcode/core.c > arch/x86/kernel/cpu/microcode/core_early.c > arch/x86/kernel/cpu/microcode/intel.c > arch/x86/kernel/cpu/microcode/amd.c > ... > > And yes, they should be under "cpu" too. >
Not really, but that isn't even Fenghua's fault. The cpu directory was originally for cpu detection. The whole kernel/ directory, of course, is a morass that might as well be called "misc". Some things are legitimately there -- mainly the syscall table and things that mirror the global kernel/ -- but a ton of stuff doesn't make any sense there. Reorganizing the kernel/ directory and promoting cpu up one level would make that make more sense. The pci and irq code are examples of things that could use their own directories. -hpa -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/