[prev in list] [next in list] [prev in thread] [next in thread] 

List:       git-commits-head
Subject:    x86: use limited register constraint for setnz
From:       Linux Kernel Mailing List <linux-kernel () vger ! kernel ! org>
Date:       2008-11-30 22:00:00
Message-ID: 200811302200.mAUM00QE001558 () hera ! kernel ! org
[Download RAW message or body]

Gitweb:     http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=5cf02b7bafddb6c3c16ddfb23d3ce187f70528ba
Commit:     5cf02b7bafddb6c3c16ddfb23d3ce187f70528ba
Parent:     86bbc2c235e500957b213e7e64ce2e0ccb8bc131
Author:     Steven Rostedt <rostedt@goodmis.org>
AuthorDate: Tue Nov 25 00:42:37 2008 -0500
Committer:  Ingo Molnar <mingo@elte.hu>
CommitDate: Tue Nov 25 15:38:03 2008 +0100

    x86: use limited register constraint for setnz
    
    Impact: build fix with certain compilers
    
    GCC can decide to use %dil when "r" is used, which is not valid for
    setnz.
    
    This bug was brought out by Stephen Rothwell's merging of the
    branch tracer into linux-next.
    
    [ Thanks to Uros Bizjak for recommending 'q' over 'Q' ]
    
    Signed-off-by: Steven Rostedt <srostedt@redhat.com>
    Signed-off-by: Ingo Molnar <mingo@elte.hu>
---
 arch/x86/boot/tty.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/arch/x86/boot/tty.c b/arch/x86/boot/tty.c
index 0be77b3..7e8e8b2 100644
--- a/arch/x86/boot/tty.c
+++ b/arch/x86/boot/tty.c
@@ -74,7 +74,7 @@ static int kbd_pending(void)
 {
 	u8 pending;
 	asm volatile("int $0x16; setnz %0"
-		     : "=rm" (pending)
+		     : "=qm" (pending)
 		     : "a" (0x0100));
 	return pending;
 }
--
To unsubscribe from this list: send the line "unsubscribe git-commits-head" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
[prev in list] [next in list] [prev in thread] [next in thread]