From: Luca Tettamanti <kronos.it@xxxxxxxxx> i8k uses lahf to read the flag register in 64-bit code; early x86-64 CPUs, however, lack this instruction and we get an invalid opcode exception at runtime. Use pushf to load the flag register into the stack instead. Signed-off-by: Luca Tettamanti <kronos.it@xxxxxxxxx> Reported-by: Jeff Rickman <jrickman@xxxxxxxxxxx> Tested-by: Jeff Rickman <jrickman@xxxxxxxxxxx> Tested-by: Harry G McGavran Jr <w5pny@xxxxxxxx> Cc: stable@xxxxxxxxxx Cc: Massimo Dal Zotto <dz@xxxxxxxxxx> Signed-off-by: Jean Delvare <khali@xxxxxxxxxxxx> --- If nobody picks this fix quickly, I'll send it to Linus myself. I haven't heard from Massimo for weeks if not months :( drivers/char/i8k.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/char/i8k.c b/drivers/char/i8k.c index d72433f..ee01716 100644 --- a/drivers/char/i8k.c +++ b/drivers/char/i8k.c @@ -139,8 +139,8 @@ static int i8k_smm(struct smm_regs *regs) "movl %%edi,20(%%rax)\n\t" "popq %%rdx\n\t" "movl %%edx,0(%%rax)\n\t" - "lahf\n\t" - "shrl $8,%%eax\n\t" + "pushfq\n\t" + "popq %%rax\n\t" "andl $1,%%eax\n" :"=a"(rc) : "a"(regs) -- Jean Delvare -- To unsubscribe from this list: send the line "unsubscribe platform-driver-x86" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
