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 PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Fri, 30 Oct 2009 12:40:21 -0700 From: Mike Travis <travis@....com> To: Dmitry Adamushko <dmitry.adamushko@...il.com> CC: Tigran Aivazian <tigran@...azian.fsnet.co.uk>, Ingo Molnar <mingo@...e.hu>, Thomas Gleixner <tglx@...utronix.de>, Andrew Morton <akpm@...ux-foundation.org>, Jack Steiner <steiner@....com>, "H. Peter Anvin" <hpa@...or.com>, x86@...nel.org, Andreas Mohr <andi@...as.de>, Hugh Dickins <hugh@...itas.com>, Hannes Eder <hannes@...neseder.net>, linux-kernel@...r.kernel.org, Roland Dreier <rdreier@...co.com>, Hidetoshi Seto <seto.hidetoshi@...fujitsu.com> Subject: [PATCH] x86_64: Limit the number of microcode messages x86_64: Limit the number of microcode messages Presented as an example of summarizing startup information, but as others have pointed out this particular message can be removed completely from the console log. Summarize microcode messages to the form: [ 8.961953] microcode: CPU0-23: sig=0x206e5, pf=0x4, revision=0xffff0016 [ 8.969494] Microcode Update Driver: v2.00 <tigran@...azian.fsnet.co.uk>, Peter Oruba (Note: I need a better method for triggering the summary message.) Cc: Tigran Aivazian <tigran@...azian.fsnet.co.uk> Cc: H. Peter Anvin <hpa@...or.com> Cc: x86@...nel.org Cc: Dmitry Adamushko <dmitry.adamushko@...il.com> Cc: Andreas Mohr <andi@...as.de> Cc: Hannes Eder <hannes@...neseder.net> Cc: linux-kernel@...r.kernel.org Signed-off-by: Mike Travis <travis@....com> --- arch/x86/kernel/microcode_intel.c | 75 +++++++++++++++++++++++++++++++++++++- 1 file changed, 74 insertions(+), 1 deletion(-) --- linux.orig/arch/x86/kernel/microcode_intel.c +++ linux/arch/x86/kernel/microcode_intel.c @@ -137,6 +137,52 @@ #define exttable_size(et) ((et)->count * EXT_SIGNATURE_SIZE + EXT_HEADER_SIZE) +static struct cpu_signature *cpusigs; +static cpumask_var_t cpusigslist; +static int cpusigs_error; + +static void summarize_cpu_info(void) +{ + char buf[128]; + int cpu; + cpumask_var_t cpulist; + + if (cpusigs_error || !alloc_cpumask_var(&cpulist, GFP_KERNEL)) { + printk(KERN_INFO "Can't print microcode summary\n"); + return; + } + + while ((cpu = cpumask_first(cpusigslist)) < nr_cpu_ids) { + struct cpu_signature *csig = &cpusigs[cpu]; + int ncpu = cpu; + + cpumask_clear(cpulist); + cpumask_set_cpu(cpu, cpulist); + + /* gather all cpu info with same data */ + while ((ncpu = cpumask_next(ncpu, cpusigslist)) < nr_cpu_ids) + if (csig->sig == cpusigs[ncpu].sig && + csig->pf == cpusigs[ncpu].pf && + csig->rev == cpusigs[ncpu].rev) + cpumask_set_cpu(ncpu, cpulist); + + cpulist_scnprintf(buf, sizeof(buf), cpulist); + + printk(KERN_INFO + "microcode: CPU%s: sig=0x%x, pf=0x%x, revision=0x%x\n", + buf, csig->sig, csig->pf, csig->rev); + + /* clear bits we just processed */ + cpumask_xor(cpusigslist, cpusigslist, cpulist); + } + + /* cleanup */ + free_cpumask_var(cpulist); + free_cpumask_var(cpusigslist); + kfree(cpusigs); + cpusigs_error = 0; +} + static int collect_cpu_info(int cpu_num, struct cpu_signature *csig) { struct cpuinfo_x86 *c = &cpu_data(cpu_num); @@ -165,9 +211,36 @@ /* get the current revision from MSR 0x8B */ rdmsr(MSR_IA32_UCODE_REV, val[0], csig->rev); - printk(KERN_INFO "microcode: CPU%d sig=0x%x, pf=0x%x, revision=0x%x\n", + if (!cpusigs && !cpusigs_error) { + if (!alloc_cpumask_var(&cpusigslist, GFP_KERNEL)) + cpusigs_error = 1; + else { + int size = sizeof(*cpusigs) * nr_cpu_ids; + cpusigs = kmalloc(size, GFP_KERNEL); + if (!cpusigs) { + free_cpumask_var(cpusigslist); + cpusigs_error = 1; + } + } + } + + /* will only print microcode revision of first cpu if cannot do all */ + if (cpusigs_error && cpu_num == 0) + printk(KERN_INFO + "microcode: CPU%d sig=0x%x, pf=0x%x, revision=0x%x\n", cpu_num, csig->sig, csig->pf, csig->rev); + else if (!cpusigs_error) { + cpusigs[cpu_num].sig = csig->sig; + cpusigs[cpu_num].pf = csig->pf; + cpusigs[cpu_num].rev = csig->rev; + cpumask_set_cpu(cpu_num, cpusigslist); + + /* (XXX Need better method for when to print summary) */ + if (cpu_num == (num_present_cpus() - 1)) + summarize_cpu_info(); + } + return 0; } -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists