Re: [uml-devel] [PATCH 1/1] system call notification with self_ptrace

2008-09-10 Thread Dave Hansen
On Mon, 2008-09-08 at 14:02 +0200, Pierre Morel wrote: > > + if (is_self_ptracing(regs->gprs[2])) { > + if (!entryexit) { > + struct siginfo info; > + > + memset(&info, 0, sizeof(struct siginfo)); > + info.si_sig

Re: [uml-devel] [PATCH 1/1] system call notification with self_ptrace

2008-09-10 Thread Dave Hansen
On Wed, 2008-09-10 at 20:20 +0400, Oleg Nesterov wrote: > But... well, I think we need Roland's opinion. I must admit, I am a bit > sceptical about this patch ;) I mean, I don't really understand why it > is useful Yeah, I don't think they're completely coming clean on it, yet. -- Dave --

Re: [uml-devel] [PATCH 1/1] system call notification with self_ptrace

2008-09-10 Thread Oleg Nesterov
On 09/10, Pierre Morel wrote: > > Oleg Nesterov wrote: > > > >I still think this patch shouldn't change handle_signal(). > > > >Once again. The signal handler for SIGSYS can first do > >sys_ptrace(PTRACE_SELF_OFF) (which is filtered out), and then use any > >other syscall, so this change is not nee

Re: [uml-devel] [PATCH 1/1] system call notification with self_ptrace

2008-09-10 Thread Pierre Morel
Hello, Oleg Nesterov wrote: > On 09/08, Pierre Morel wrote: > >> --- linux-2.6.26.orig/arch/s390/kernel/signal.c >> +++ linux-2.6.26/arch/s390/kernel/signal.c >> @@ -409,6 +409,11 @@ handle_signal(unsigned long sig, struct >> spin_unlock_irq(¤t->sighand->siglock); >> } >> >>

Re: [uml-devel] [PATCH 1/1] system call notification with self_ptrace

2008-09-10 Thread Pierre Morel
Hi, Andrew Morton wrote: > On Mon, 08 Sep 2008 14:02:01 +0200 > Pierre Morel <[EMAIL PROTECTED]> wrote: > > >> Subject: [PATCH] system call notification with self_ptrace >> >> From: Pierre Morel <[EMAIL PROTECTED]> >> >> >> PTRACE SELF >> >> This patch adds a new functionality to ptrace: system