Commit 8cf36d2b authored by Paolo Ciarrocchi's avatar Paolo Ciarrocchi Committed by Ingo Molnar

x86: coding style fixes to arch/x86/lib/string_32.c

The patch kills 45 errors and a few warnings.

The file is now error/warning free:
total: 0 errors, 0 warnings, 237 lines checked
arch/x86/lib/string_32.c has no obvious style problems and is ready for submission.

no code changed:

arch/x86/lib/string_32.o:
   text    data     bss     dec     hex filename
    639       0       0     639     27f string_32.o.before
    639       0       0     639     27f string_32.o.after
md5:
2db1c48187cf5113bb595153ee1fc73d  string_32.o.before.asm
2db1c48187cf5113bb595153ee1fc73d  string_32.o.after.asm
Signed-off-by: default avatarPaolo Ciarrocchi <paolo.ciarrocchi@gmail.com>
Signed-off-by: default avatarIngo Molnar <mingo@elte.hu>
parent 325f86ec
...@@ -14,25 +14,25 @@ ...@@ -14,25 +14,25 @@
#include <linux/module.h> #include <linux/module.h>
#ifdef __HAVE_ARCH_STRCPY #ifdef __HAVE_ARCH_STRCPY
char *strcpy(char * dest,const char *src) char *strcpy(char *dest, const char *src)
{ {
int d0, d1, d2; int d0, d1, d2;
asm volatile( "1:\tlodsb\n\t" asm volatile("1:\tlodsb\n\t"
"stosb\n\t" "stosb\n\t"
"testb %%al,%%al\n\t" "testb %%al,%%al\n\t"
"jne 1b" "jne 1b"
: "=&S" (d0), "=&D" (d1), "=&a" (d2) : "=&S" (d0), "=&D" (d1), "=&a" (d2)
:"0" (src),"1" (dest) : "memory"); :"0" (src), "1" (dest) : "memory");
return dest; return dest;
} }
EXPORT_SYMBOL(strcpy); EXPORT_SYMBOL(strcpy);
#endif #endif
#ifdef __HAVE_ARCH_STRNCPY #ifdef __HAVE_ARCH_STRNCPY
char *strncpy(char * dest,const char *src,size_t count) char *strncpy(char *dest, const char *src, size_t count)
{ {
int d0, d1, d2, d3; int d0, d1, d2, d3;
asm volatile( "1:\tdecl %2\n\t" asm volatile("1:\tdecl %2\n\t"
"js 2f\n\t" "js 2f\n\t"
"lodsb\n\t" "lodsb\n\t"
"stosb\n\t" "stosb\n\t"
...@@ -42,17 +42,17 @@ char *strncpy(char * dest,const char *src,size_t count) ...@@ -42,17 +42,17 @@ char *strncpy(char * dest,const char *src,size_t count)
"stosb\n" "stosb\n"
"2:" "2:"
: "=&S" (d0), "=&D" (d1), "=&c" (d2), "=&a" (d3) : "=&S" (d0), "=&D" (d1), "=&c" (d2), "=&a" (d3)
:"0" (src),"1" (dest),"2" (count) : "memory"); :"0" (src), "1" (dest), "2" (count) : "memory");
return dest; return dest;
} }
EXPORT_SYMBOL(strncpy); EXPORT_SYMBOL(strncpy);
#endif #endif
#ifdef __HAVE_ARCH_STRCAT #ifdef __HAVE_ARCH_STRCAT
char *strcat(char * dest,const char * src) char *strcat(char *dest, const char *src)
{ {
int d0, d1, d2, d3; int d0, d1, d2, d3;
asm volatile( "repne\n\t" asm volatile("repne\n\t"
"scasb\n\t" "scasb\n\t"
"decl %1\n" "decl %1\n"
"1:\tlodsb\n\t" "1:\tlodsb\n\t"
...@@ -67,10 +67,10 @@ EXPORT_SYMBOL(strcat); ...@@ -67,10 +67,10 @@ EXPORT_SYMBOL(strcat);
#endif #endif
#ifdef __HAVE_ARCH_STRNCAT #ifdef __HAVE_ARCH_STRNCAT
char *strncat(char * dest,const char * src,size_t count) char *strncat(char *dest, const char *src, size_t count)
{ {
int d0, d1, d2, d3; int d0, d1, d2, d3;
asm volatile( "repne\n\t" asm volatile("repne\n\t"
"scasb\n\t" "scasb\n\t"
"decl %1\n\t" "decl %1\n\t"
"movl %8,%3\n" "movl %8,%3\n"
...@@ -83,7 +83,7 @@ char *strncat(char * dest,const char * src,size_t count) ...@@ -83,7 +83,7 @@ char *strncat(char * dest,const char * src,size_t count)
"2:\txorl %2,%2\n\t" "2:\txorl %2,%2\n\t"
"stosb" "stosb"
: "=&S" (d0), "=&D" (d1), "=&a" (d2), "=&c" (d3) : "=&S" (d0), "=&D" (d1), "=&a" (d2), "=&c" (d3)
: "0" (src),"1" (dest),"2" (0),"3" (0xffffffffu), "g" (count) : "0" (src), "1" (dest), "2" (0), "3" (0xffffffffu), "g" (count)
: "memory"); : "memory");
return dest; return dest;
} }
...@@ -91,11 +91,11 @@ EXPORT_SYMBOL(strncat); ...@@ -91,11 +91,11 @@ EXPORT_SYMBOL(strncat);
#endif #endif
#ifdef __HAVE_ARCH_STRCMP #ifdef __HAVE_ARCH_STRCMP
int strcmp(const char * cs,const char * ct) int strcmp(const char *cs, const char *ct)
{ {
int d0, d1; int d0, d1;
int res; int res;
asm volatile( "1:\tlodsb\n\t" asm volatile("1:\tlodsb\n\t"
"scasb\n\t" "scasb\n\t"
"jne 2f\n\t" "jne 2f\n\t"
"testb %%al,%%al\n\t" "testb %%al,%%al\n\t"
...@@ -106,7 +106,7 @@ int strcmp(const char * cs,const char * ct) ...@@ -106,7 +106,7 @@ int strcmp(const char * cs,const char * ct)
"orb $1,%%al\n" "orb $1,%%al\n"
"3:" "3:"
:"=a" (res), "=&S" (d0), "=&D" (d1) :"=a" (res), "=&S" (d0), "=&D" (d1)
:"1" (cs),"2" (ct) :"1" (cs), "2" (ct)
:"memory"); :"memory");
return res; return res;
} }
...@@ -114,11 +114,11 @@ EXPORT_SYMBOL(strcmp); ...@@ -114,11 +114,11 @@ EXPORT_SYMBOL(strcmp);
#endif #endif
#ifdef __HAVE_ARCH_STRNCMP #ifdef __HAVE_ARCH_STRNCMP
int strncmp(const char * cs,const char * ct,size_t count) int strncmp(const char *cs, const char *ct, size_t count)
{ {
int res; int res;
int d0, d1, d2; int d0, d1, d2;
asm volatile( "1:\tdecl %3\n\t" asm volatile("1:\tdecl %3\n\t"
"js 2f\n\t" "js 2f\n\t"
"lodsb\n\t" "lodsb\n\t"
"scasb\n\t" "scasb\n\t"
...@@ -131,7 +131,7 @@ int strncmp(const char * cs,const char * ct,size_t count) ...@@ -131,7 +131,7 @@ int strncmp(const char * cs,const char * ct,size_t count)
"orb $1,%%al\n" "orb $1,%%al\n"
"4:" "4:"
:"=a" (res), "=&S" (d0), "=&D" (d1), "=&c" (d2) :"=a" (res), "=&S" (d0), "=&D" (d1), "=&c" (d2)
:"1" (cs),"2" (ct),"3" (count) :"1" (cs), "2" (ct), "3" (count)
:"memory"); :"memory");
return res; return res;
} }
...@@ -139,11 +139,11 @@ EXPORT_SYMBOL(strncmp); ...@@ -139,11 +139,11 @@ EXPORT_SYMBOL(strncmp);
#endif #endif
#ifdef __HAVE_ARCH_STRCHR #ifdef __HAVE_ARCH_STRCHR
char *strchr(const char * s, int c) char *strchr(const char *s, int c)
{ {
int d0; int d0;
char * res; char *res;
asm volatile( "movb %%al,%%ah\n" asm volatile("movb %%al,%%ah\n"
"1:\tlodsb\n\t" "1:\tlodsb\n\t"
"cmpb %%ah,%%al\n\t" "cmpb %%ah,%%al\n\t"
"je 2f\n\t" "je 2f\n\t"
...@@ -153,7 +153,7 @@ char *strchr(const char * s, int c) ...@@ -153,7 +153,7 @@ char *strchr(const char * s, int c)
"2:\tmovl %1,%0\n\t" "2:\tmovl %1,%0\n\t"
"decl %0" "decl %0"
:"=a" (res), "=&S" (d0) :"=a" (res), "=&S" (d0)
:"1" (s),"0" (c) :"1" (s), "0" (c)
:"memory"); :"memory");
return res; return res;
} }
...@@ -161,16 +161,16 @@ EXPORT_SYMBOL(strchr); ...@@ -161,16 +161,16 @@ EXPORT_SYMBOL(strchr);
#endif #endif
#ifdef __HAVE_ARCH_STRLEN #ifdef __HAVE_ARCH_STRLEN
size_t strlen(const char * s) size_t strlen(const char *s)
{ {
int d0; int d0;
int res; int res;
asm volatile( "repne\n\t" asm volatile("repne\n\t"
"scasb\n\t" "scasb\n\t"
"notl %0\n\t" "notl %0\n\t"
"decl %0" "decl %0"
:"=c" (res), "=&D" (d0) :"=c" (res), "=&D" (d0)
:"1" (s),"a" (0), "0" (0xffffffffu) :"1" (s), "a" (0), "0" (0xffffffffu)
:"memory"); :"memory");
return res; return res;
} }
...@@ -178,19 +178,19 @@ EXPORT_SYMBOL(strlen); ...@@ -178,19 +178,19 @@ EXPORT_SYMBOL(strlen);
#endif #endif
#ifdef __HAVE_ARCH_MEMCHR #ifdef __HAVE_ARCH_MEMCHR
void *memchr(const void *cs,int c,size_t count) void *memchr(const void *cs, int c, size_t count)
{ {
int d0; int d0;
void *res; void *res;
if (!count) if (!count)
return NULL; return NULL;
asm volatile( "repne\n\t" asm volatile("repne\n\t"
"scasb\n\t" "scasb\n\t"
"je 1f\n\t" "je 1f\n\t"
"movl $1,%0\n" "movl $1,%0\n"
"1:\tdecl %0" "1:\tdecl %0"
:"=D" (res), "=&c" (d0) :"=D" (res), "=&c" (d0)
:"a" (c),"0" (cs),"1" (count) :"a" (c), "0" (cs), "1" (count)
:"memory"); :"memory");
return res; return res;
} }
...@@ -198,7 +198,7 @@ EXPORT_SYMBOL(memchr); ...@@ -198,7 +198,7 @@ EXPORT_SYMBOL(memchr);
#endif #endif
#ifdef __HAVE_ARCH_MEMSCAN #ifdef __HAVE_ARCH_MEMSCAN
void *memscan(void * addr, int c, size_t size) void *memscan(void *addr, int c, size_t size)
{ {
if (!size) if (!size)
return addr; return addr;
...@@ -219,7 +219,7 @@ size_t strnlen(const char *s, size_t count) ...@@ -219,7 +219,7 @@ size_t strnlen(const char *s, size_t count)
{ {
int d0; int d0;
int res; int res;
asm volatile( "movl %2,%0\n\t" asm volatile("movl %2,%0\n\t"
"jmp 2f\n" "jmp 2f\n"
"1:\tcmpb $0,(%0)\n\t" "1:\tcmpb $0,(%0)\n\t"
"je 3f\n\t" "je 3f\n\t"
...@@ -229,7 +229,7 @@ size_t strnlen(const char *s, size_t count) ...@@ -229,7 +229,7 @@ size_t strnlen(const char *s, size_t count)
"jne 1b\n" "jne 1b\n"
"3:\tsubl %2,%0" "3:\tsubl %2,%0"
:"=a" (res), "=&d" (d0) :"=a" (res), "=&d" (d0)
:"c" (s),"1" (count) :"c" (s), "1" (count)
:"memory"); :"memory");
return res; return res;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment