diff options
| author | Denys Dmytriyenko <denis@denix.org> | 2009-03-17 14:32:59 -0400 |
|---|---|---|
| committer | Denys Dmytriyenko <denis@denix.org> | 2009-03-17 14:32:59 -0400 |
| commit | 709c4d66e0b107ca606941b988bad717c0b45d9b (patch) | |
| tree | 37ee08b1eb308f3b2b6426d5793545c38396b838 /recipes/uclibc/uclibc-0.9.28/thumb-swi-r7.patch | |
| parent | fa6cd5a3b993f16c27de4ff82b42684516d433ba (diff) | |
rename packages/ to recipes/ per earlier agreement
See links below for more details:
http://thread.gmane.org/gmane.comp.handhelds.openembedded/21326
http://thread.gmane.org/gmane.comp.handhelds.openembedded/21816
Signed-off-by: Denys Dmytriyenko <denis@denix.org>
Acked-by: Mike Westerhof <mwester@dls.net>
Acked-by: Philip Balister <philip@balister.org>
Acked-by: Khem Raj <raj.khem@gmail.com>
Acked-by: Marcin Juszkiewicz <hrw@openembedded.org>
Acked-by: Koen Kooi <koen@openembedded.org>
Acked-by: Frans Meulenbroeks <fransmeulenbroeks@gmail.com>
Diffstat (limited to 'recipes/uclibc/uclibc-0.9.28/thumb-swi-r7.patch')
| -rw-r--r-- | recipes/uclibc/uclibc-0.9.28/thumb-swi-r7.patch | 48 |
1 files changed, 48 insertions, 0 deletions
diff --git a/recipes/uclibc/uclibc-0.9.28/thumb-swi-r7.patch b/recipes/uclibc/uclibc-0.9.28/thumb-swi-r7.patch new file mode 100644 index 0000000000..330ceb6869 --- /dev/null +++ b/recipes/uclibc/uclibc-0.9.28/thumb-swi-r7.patch @@ -0,0 +1,48 @@ +# This is a work round for a fairly serious GCC compiler bug - when +# the syscall assembler overwrites r7 (required on thumb) the +# compiler fails to protect the register when it is using it as a +# frame pointer. +# +--- uClibc-0.9.28/libc/sysdeps/linux/arm/bits/syscalls.h 2005-09-12 17:51:26.062205918 -0700 ++++ uClibc-0.9.28/libc/sysdeps/linux/arm/bits/syscalls.h 2005-09-12 18:16:15.507930828 -0700 +@@ -111,6 +111,11 @@ + } \ + (int) _sys_result; }) + #else ++#if 0 ++/* This doesn't work because GCC uses r7 as a frame pointer in ++ * some cases and doesn't notice that the _r7 value changes ++ * it, resulting in mysterious crashes after the SWI. ++ */ + #define INTERNAL_SYSCALL(name, err, nr, args...) \ + ({ unsigned int _sys_result; \ + { \ +@@ -124,6 +129,28 @@ + _sys_result = _a1; \ + } \ + (int) _sys_result; }) ++#else ++/* So hide the use of r7 from the compiler, this would be a lot ++ * easier but for the fact that the syscalls can exceed 255. ++ * For the moment the LOAD_ARG_7 is sacrificed. ++ */ ++#define INTERNAL_SYSCALL(name, err, nr, args...) \ ++ ({ unsigned int _sys_result; \ ++ { \ ++ register int _a1 asm ("a1"); \ ++ LOAD_ARGS_##nr (args) \ ++ register int _v3 asm ("v3") = (int) (SYS_ify(name)); \ ++ asm volatile ("push {r7}\n" \ ++ "\tmov r7, v3\n" \ ++ "\tswi 0 @ syscall " #name "\n" \ ++ "\tpop {r7}" \ ++ : "=r" (_a1) \ ++ : "r" (_v3) ASM_ARGS_##nr \ ++ : "memory"); \ ++ _sys_result = _a1; \ ++ } \ ++ (int) _sys_result; }) ++#endif + #endif + + #undef INTERNAL_SYSCALL_ERROR_P |