ARM: 7410/1: Add extra clobber registers for assembly in kernel_execve
authorTim Bird <tim.bird@am.sony.com>
Wed, 2 May 2012 21:55:39 +0000 (22:55 +0100)
committer黄涛 <huangtao@rock-chips.com>
Tue, 29 May 2012 02:14:57 +0000 (10:14 +0800)
commit74cfe8b84a7fb85c7624dc451e12bf0ea7d91692
treea5be8a3d41cb412c3a4bb1fe57b6a462dd05bd67
parent62ef46fdc59b65442643f0475265588fd95a9fd9
ARM: 7410/1: Add extra clobber registers for assembly in kernel_execve

commit e787ec1376e862fcea1bfd523feb7c5fb43ecdb9 upstream.

The inline assembly in kernel_execve() uses r8 and r9.  Since this
code sequence does not return, it usually doesn't matter if the
register clobber list is accurate.  However, I saw a case where a
particular version of gcc used r8 as an intermediate for the value
eventually passed to r9.  Because r8 is used in the inline
assembly, and not mentioned in the clobber list, r9 was set
to an incorrect value.

This resulted in a kernel panic on execution of the first user-space
program in the system.  r9 is used in ret_to_user as the thread_info
pointer, and if it's wrong, bad things happen.

Signed-off-by: Tim Bird <tim.bird@am.sony.com>
Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
arch/arm/kernel/sys_arm.c