Re: [PATCHSET ptrace] ptrace: implement PTRACE_SEIZE/INTERRUPT andgroup stop notification, take#2

[Tejun Heo]

Hello, Linus.

On Thu, May 19, 2011 at 08:04:28AM -0700, Linus Torvalds wrote:
> On Mon, May 16, 2011 at 11:17 AM, Tejun Heo <tj@xxxxxxxxxx> wrote:
> >
> > This is the second try at implementing PTRACE_SEIZE/INTERRUPT and
> > group stop notification.  Notable changes from the first take[1] are,
> >
> > * Prep patches moved to a separate patchset[2].
> 
> So having followed the discussion so far, quite frankly I'm not
> convinced this series is 2.6.40 material.

Definitely not.  The interface is not even fixed/complete yet.  I
think we should document the whole ptrace, job control and signal
interaction properly before committing to the interface.

> I think that conceptually the split-up of PTRACE_ATTACH into
> SEIZE/INTERRUPT might be fine, but I don't think the interface is
> necessarily cooked, and perhaps more importantly I'm not at all sure
> that the (few) current users of ptrace() would even switch over.

I think it shouldn't be too difficult for strace.  gdb would be a
challenge tho.  At any rate, given that the existing mechanism is
simply broken, I'm hoping they would switch over soonish.

> So I think Oleg's branch with cleanups is probably ready, and maybe a
> few of the preparatory patches from this branch can be merged, but I
> would _strongly_ suggest that the plan for 2.6.40 should be to not
> actually mess with interfaces to the kernel, but just cleaning up the
> actual internal implementation. I would like to keep 2.6.40 small and
> simple.

Upto the patchset which cleaned up job control notifications, Oleg's
recent sigprocmask plus some odd fix patches should be basically it
for 2.6.40.  The current ptrace branch in Oleg's tree seems to have
just that.

Thanks.

-- 
tejun
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/