diff mbox

[kvm-unit-test] x86/emulator: Fix inline assembler warning

Message ID 54FDE39B.9040504@siemens.com (mailing list archive)
State New, archived
Headers show

Commit Message

Jan Kiszka March 9, 2015, 6:16 p.m. UTC
Code compiles to the same binary, but now with one warning less.

Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
---
 x86/emulator.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Marcelo Tosatti March 13, 2015, 1:13 a.m. UTC | #1
On Mon, Mar 09, 2015 at 07:16:59PM +0100, Jan Kiszka wrote:
> Code compiles to the same binary, but now with one warning less.
> 
> Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
> ---
>  x86/emulator.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/x86/emulator.c b/x86/emulator.c
> index 0964e6a..e5c1c6b 100644
> --- a/x86/emulator.c
> +++ b/x86/emulator.c
> @@ -838,7 +838,7 @@ static void test_jmp_noncanonical(uint64_t *mem)
>  
>  	exceptions = 0;
>  	handle_exception(GP_VECTOR, advance_rip_by_3_and_note_exception);
> -	asm volatile ("jmp %0" : : "m"(*mem));
> +	asm volatile ("jmp *%0" : : "m"(*mem));
>  	report("jump to non-canonical address", exceptions == 1);
>  	handle_exception(GP_VECTOR, 0);
>  }
> -- 
> 2.1.4

Applied, thanks.

--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/x86/emulator.c b/x86/emulator.c
index 0964e6a..e5c1c6b 100644
--- a/x86/emulator.c
+++ b/x86/emulator.c
@@ -838,7 +838,7 @@  static void test_jmp_noncanonical(uint64_t *mem)
 
 	exceptions = 0;
 	handle_exception(GP_VECTOR, advance_rip_by_3_and_note_exception);
-	asm volatile ("jmp %0" : : "m"(*mem));
+	asm volatile ("jmp *%0" : : "m"(*mem));
 	report("jump to non-canonical address", exceptions == 1);
 	handle_exception(GP_VECTOR, 0);
 }