rpms/kernel/devel linux-2.6-ia32-syscall-restart.patch, NONE, 1.1 kernel.spec, 1.471, 1.472
Roland McGrath (roland)
fedora-extras-commits at redhat.com
Tue Mar 4 06:30:59 UTC 2008
- Previous message (by thread): rpms/perl-Date-Calc/devel perl-Date-Calc.spec,1.23,1.24
- Next message (by thread): rpms/xorg-x11-server/devel .cvsignore, 1.35, 1.36 commitid, 1.8, 1.9 make-git-snapshot.sh, 1.1, 1.2 sources, 1.30, 1.31 xorg-x11-server.spec, 1.299, 1.300 xserver-1.3.0-honor-displaysize.patch, 1.2, NONE xserver-1.3.0-randr12-config-hack.patch, 1.3, NONE xserver-1.3.0-xnest-exposures.patch, 1.2, NONE xserver-1.4.99-selinux-fix.patch, 1.1, NONE
- Messages sorted by:
[ date ]
[ thread ]
[ subject ]
[ author ]
Author: roland
Update of /cvs/pkgs/rpms/kernel/devel
In directory cvs-int.fedora.redhat.com:/tmp/cvs-serv8215
Modified Files:
kernel.spec
Added Files:
linux-2.6-ia32-syscall-restart.patch
Log Message:
* Mon Mar 3 2008 Roland McGrath <roland at redhat.com>
- x86_64: fix 32-bit process syscall restart (#434995)
linux-2.6-ia32-syscall-restart.patch:
--- NEW FILE linux-2.6-ia32-syscall-restart.patch ---
>From 702b92b212afb1cb9d9590f5a9133113c2b4168c Mon Sep 17 00:00:00 2001
From: Roland McGrath <roland at redhat.com>
Date: Thu, 28 Feb 2008 19:40:17 -0800
Subject: [PATCH] x86_64 ia32 syscall restart fix
The code to restart syscalls after signals depends on checking for a
negative orig_ax, and for particular negative -ERESTART* values in ax.
These fields are 64 bits and for a 32-bit task they get zero-extended.
The syscall restart behavior is lost, a regression from a native 32-bit
kernel and from 64-bit tasks' behavior. This patch fixes the problem by
doing sign-extension where it matters. For orig_ax, the only time the
value should be -1 but winds up as 0x0ffffffff is via a 32-bit ptrace
call. So the patch changes ptrace to sign-extend the 32-bit orig_eax
value when it's stored; it doesn't change the checks on orig_ax, though
it uses the new current_syscall() inline to better document the subtle
importance of the used of signedness there. The ax value is stored a
lot of ways and it seems hard to get them all sign-extended at their
origins. So for that, we use the current_syscall_ret() to sign-extend
it only for 32-bit tasks at the time of the -ERESTART* comparisons.
Signed-off-by: Roland McGrath <roland at redhat.com>
---
arch/x86/kernel/ptrace.c | 9 ++++++++-
arch/x86/kernel/signal_64.c | 38 +++++++++++++++++++++++++++++++++-----
2 files changed, 41 insertions(+), 6 deletions(-)
diff --git a/arch/x86/kernel/ptrace.c b/arch/x86/kernel/ptrace.c
index f41fdc9..f8eed1b 100644
--- a/arch/x86/kernel/ptrace.c
+++ b/arch/x86/kernel/ptrace.c
@@ -1045,10 +1045,17 @@ static int putreg32(struct task_struct *child, unsigned regno, u32 value)
R32(esi, si);
R32(ebp, bp);
R32(eax, ax);
- R32(orig_eax, orig_ax);
R32(eip, ip);
R32(esp, sp);
+ case offsetof(struct user32, regs.orig_eax):
+ /*
+ * Sign-extend the value so that orig_eax = -1
+ * causes (long)orig_ax < 0 tests to fire correctly.
+ */
+ regs->orig_ax = (long) (s32) value;
+ break;
+
case offsetof(struct user32, regs.eflags):
return set_flags(child, value);
diff --git a/arch/x86/kernel/signal_64.c b/arch/x86/kernel/signal_64.c
index 7347bb1..ce317ee 100644
--- a/arch/x86/kernel/signal_64.c
+++ b/arch/x86/kernel/signal_64.c
@@ -311,6 +311,35 @@ give_sigsegv:
}
/*
+ * Return -1L or the syscall number that @regs is executing.
+ */
+static long current_syscall(struct pt_regs *regs)
+{
+ /*
+ * We always sign-extend a -1 value being set here,
+ * so this is always either -1L or a syscall number.
+ */
+ return regs->orig_ax;
+}
+
+/*
+ * Return a value that is -EFOO if the system call in @regs->orig_ax
+ * returned an error. This only works for @regs from @current.
+ */
+static long current_syscall_ret(struct pt_regs *regs)
+{
+#ifdef CONFIG_IA32_EMULATION
+ if (test_thread_flag(TIF_IA32))
+ /*
+ * Sign-extend the value so (int)-EFOO becomes (long)-EFOO
+ * and will match correctly in comparisons.
+ */
+ return (int) regs->ax;
+#endif
+ return regs->ax;
+}
+
+/*
* OK, we're invoking a handler
*/
@@ -327,9 +356,9 @@ handle_signal(unsigned long sig, siginfo_t *info, struct k_sigaction *ka,
#endif
/* Are we from a system call? */
- if ((long)regs->orig_ax >= 0) {
+ if (current_syscall(regs) >= 0) {
/* If so, check system call restarting.. */
- switch (regs->ax) {
+ switch (current_syscall_ret(regs)) {
case -ERESTART_RESTARTBLOCK:
case -ERESTARTNOHAND:
regs->ax = -EINTR;
@@ -426,10 +455,9 @@ static void do_signal(struct pt_regs *regs)
}
/* Did we come from a system call? */
- if ((long)regs->orig_ax >= 0) {
+ if (current_syscall(regs) >= 0) {
/* Restart the system call - no handlers present */
- long res = regs->ax;
- switch (res) {
+ switch (current_syscall_ret(regs)) {
case -ERESTARTNOHAND:
case -ERESTARTSYS:
case -ERESTARTNOINTR:
--
1.5.4.1
Index: kernel.spec
===================================================================
RCS file: /cvs/pkgs/rpms/kernel/devel/kernel.spec,v
retrieving revision 1.471
retrieving revision 1.472
diff -u -r1.471 -r1.472
--- kernel.spec 3 Mar 2008 22:02:24 -0000 1.471
+++ kernel.spec 4 Mar 2008 06:30:07 -0000 1.472
@@ -579,6 +579,8 @@
Patch42: linux-2.6-x86-tune-generic.patch
Patch75: linux-2.6-x86-debug-boot.patch
+Patch80: linux-2.6-ia32-syscall-restart.patch
+
Patch123: linux-2.6-ppc-rtc.patch
Patch140: linux-2.6-ps3-ehci-iso.patch
Patch141: linux-2.6-ps3-storage-alias.patch
@@ -1009,6 +1011,8 @@
# Compile 686 kernels tuned for Pentium4.
ApplyPatch linux-2.6-x86-tune-generic.patch
+ApplyPatch linux-2.6-ia32-syscall-restart.patch
+
#
# PowerPC
#
@@ -1744,6 +1748,9 @@
%kernel_variant_files -a /%{image_install_path}/xen*-%{KVERREL} -e /etc/ld.so.conf.d/kernelcap-%{KVERREL}.conf %{with_xen} xen
%changelog
+* Mon Mar 3 2008 Roland McGrath <roland at redhat.com>
+- x86_64: fix 32-bit process syscall restart (#434995)
+
* Mon Mar 03 2008 John W. Linville <linville at redhat.com>
- ssb: Add CHIPCO IRQ access functions
- p54: print unknown eeprom fields
- Previous message (by thread): rpms/perl-Date-Calc/devel perl-Date-Calc.spec,1.23,1.24
- Next message (by thread): rpms/xorg-x11-server/devel .cvsignore, 1.35, 1.36 commitid, 1.8, 1.9 make-git-snapshot.sh, 1.1, 1.2 sources, 1.30, 1.31 xorg-x11-server.spec, 1.299, 1.300 xserver-1.3.0-honor-displaysize.patch, 1.2, NONE xserver-1.3.0-randr12-config-hack.patch, 1.3, NONE xserver-1.3.0-xnest-exposures.patch, 1.2, NONE xserver-1.4.99-selinux-fix.patch, 1.1, NONE
- Messages sorted by:
[ date ]
[ thread ]
[ subject ]
[ author ]
More information about the fedora-extras-commits
mailing list