From: Paolo 'Blaisorblade' Giarrusso If enable is moved by GCC in a register its value may not be preserved after coming back there with longjmp(). So, mark it as volatile to prevent this; this is suggested (it seems) in info gcc, when it talks about -Wuninitialized. I re-read this and it seems to say something different, but I still believe this may be needed. Signed-off-by: Paolo 'Blaisorblade' Giarrusso Cc: Jeff Dike Signed-off-by: Andrew Morton --- arch/um/include/longjmp.h | 3 ++- 1 files changed, 2 insertions(+), 1 deletion(-) diff -puN arch/um/include/longjmp.h~uml-make-uml_setjmp-always-safe arch/um/include/longjmp.h --- a/arch/um/include/longjmp.h~uml-make-uml_setjmp-always-safe +++ a/arch/um/include/longjmp.h @@ -12,7 +12,8 @@ extern void longjmp(jmp_buf, int); } while(0) #define UML_SETJMP(buf) ({ \ - int n, enable; \ + int n; \ + volatile int enable; \ enable = get_signals(); \ n = setjmp(*buf); \ if(n != 0) \ _