]> Pileus Git - ~andy/linux/commitdiff
x86: Be consistent with data size in getuser.S
authorH. Peter Anvin <hpa@zytor.com>
Tue, 12 Feb 2013 07:14:48 +0000 (23:14 -0800)
committerH. Peter Anvin <hpa@zytor.com>
Tue, 12 Feb 2013 07:14:48 +0000 (23:14 -0800)
Consistently use the data register by name and use a sized assembly
instruction in getuser.S.  There is never any reason to macroize it,
and being inconsistent in the same file is just annoying.

No actual code change.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>
arch/x86/lib/getuser.S

index d3bf9f99ca770fb779a547713ce98a3304a1d34a..a4512359656aea8fcda1d2c19d45f3bb4936bee3 100644 (file)
@@ -41,7 +41,7 @@ ENTRY(__get_user_1)
        cmp TI_addr_limit(%_ASM_DX),%_ASM_AX
        jae bad_get_user
        ASM_STAC
-1:     movzb (%_ASM_AX),%edx
+1:     movzbl (%_ASM_AX),%edx
        xor %eax,%eax
        ASM_CLAC
        ret
@@ -71,7 +71,7 @@ ENTRY(__get_user_4)
        cmp TI_addr_limit(%_ASM_DX),%_ASM_AX
        jae bad_get_user
        ASM_STAC
-3:     mov -3(%_ASM_AX),%edx
+3:     movl -3(%_ASM_AX),%edx
        xor %eax,%eax
        ASM_CLAC
        ret
@@ -87,7 +87,7 @@ ENTRY(__get_user_8)
        cmp TI_addr_limit(%_ASM_DX),%_ASM_AX
        jae bad_get_user
        ASM_STAC
-4:     movq -7(%_ASM_AX),%_ASM_DX
+4:     movq -7(%_ASM_AX),%rdx
        xor %eax,%eax
        ASM_CLAC
        ret
@@ -98,8 +98,8 @@ ENTRY(__get_user_8)
        cmp TI_addr_limit(%_ASM_DX),%_ASM_AX
        jae bad_get_user_8
        ASM_STAC
-4:     mov -7(%_ASM_AX),%edx
-5:     mov -3(%_ASM_AX),%ecx
+4:     movl -7(%_ASM_AX),%edx
+5:     movl -3(%_ASM_AX),%ecx
        xor %eax,%eax
        ASM_CLAC
        ret