* [PATCH] Fix date output in x86 microcode driver.
@ 2008-08-12 12:25 David Woodhouse
2008-08-13 16:56 ` Ingo Molnar
0 siblings, 1 reply; 2+ messages in thread
From: David Woodhouse @ 2008-08-12 12:25 UTC (permalink / raw)
To: Shaohua Li; +Cc: linux-kernel
The microcode stores its date in a uint32_t in some weird order
approximating pdp-endian. Rather than printing it like that, print it
properly in ISO standard form.
Signed-off-by: David Woodhouse <David.Woodhouse@intel.com>
diff --git a/arch/x86/kernel/microcode.c b/arch/x86/kernel/microcode.c
index 652fa5c..394fbdd 100644
--- a/arch/x86/kernel/microcode.c
+++ b/arch/x86/kernel/microcode.c
@@ -343,8 +343,9 @@ static void apply_microcode(int cpu)
return;
}
printk(KERN_INFO "microcode: CPU%d updated from revision "
- "0x%x to 0x%x, date = %08x \n",
- cpu_num, uci->rev, val[1], uci->mc->hdr.date);
+ "0x%x to 0x%x, date = %04x-%02x-%02x \n",
+ cpu_num, uci->rev, val[1], uci->mc->hdr.date & 0xffff,
+ uci->mc->hdr.date >> 24, (uci->mc->hdr.date >> 16) & 0xff);
uci->rev = val[1];
}
--
David Woodhouse Open Source Technology Centre
David.Woodhouse@intel.com Intel Corporation
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] Fix date output in x86 microcode driver.
2008-08-12 12:25 [PATCH] Fix date output in x86 microcode driver David Woodhouse
@ 2008-08-13 16:56 ` Ingo Molnar
0 siblings, 0 replies; 2+ messages in thread
From: Ingo Molnar @ 2008-08-13 16:56 UTC (permalink / raw)
To: David Woodhouse; +Cc: Shaohua Li, linux-kernel, Peter Oruba, Dmitry Adamushko
* David Woodhouse <dwmw2@infradead.org> wrote:
> The microcode stores its date in a uint32_t in some weird order
> approximating pdp-endian. Rather than printing it like that, print it
> properly in ISO standard form.
applied to tip/x86/microcode - thanks David.
Note that i did some manual merging because microcode support changed a
few details meanwhile, due to the addition of the AMD microcode driver.
Find the commit below.
Ingo
--------------->
>From 34a1b9fc4995102ecbb3a980b348aebf168d8196 Mon Sep 17 00:00:00 2001
From: David Woodhouse <dwmw2@infradead.org>
Date: Tue, 12 Aug 2008 13:25:44 +0100
Subject: [PATCH] Fix date output in x86 microcode driver.
The microcode stores its date in a uint32_t in some weird order
approximating pdp-endian. Rather than printing it like that, print it
properly in ISO standard form.
Signed-off-by: David Woodhouse <David.Woodhouse@intel.com>
Cc: Shaohua Li <shaohua.li@intel.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
---
arch/x86/kernel/microcode_intel.c | 7 +++++--
1 files changed, 5 insertions(+), 2 deletions(-)
diff --git a/arch/x86/kernel/microcode_intel.c b/arch/x86/kernel/microcode_intel.c
index a61986e..d2d9d74 100644
--- a/arch/x86/kernel/microcode_intel.c
+++ b/arch/x86/kernel/microcode_intel.c
@@ -339,8 +339,11 @@ static void apply_microcode(int cpu)
return;
}
printk(KERN_INFO "microcode: CPU%d updated from revision "
- "0x%x to 0x%x, date = %08x \n",
- cpu_num, uci->rev, val[1], uci->mc.mc_intel->hdr.date);
+ "0x%x to 0x%x, date = %04x-%02x-%02x \n",
+ cpu_num, uci->rev, val[1],
+ uci->mc.mc_intel->hdr.date & 0xffff,
+ uci->mc.mc_intel->hdr.date >> 24,
+ (uci->mc.mc_intel->hdr.date >> 16) & 0xff);
uci->rev = val[1];
}
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2008-08-13 16:57 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2008-08-12 12:25 [PATCH] Fix date output in x86 microcode driver David Woodhouse
2008-08-13 16:56 ` Ingo Molnar
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®