lkml.org 
[lkml]   [2004]   [Aug]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] notify_parent and ptrace cleanup
> ptrace() is frangible, and racy. And looks like few things can't improve
> without user visible change. So, I'm thinking I would like to rewrite
> it by another interface.

I don't think such vague statements are useful. Are there other races you
are implicitly referring to here, or only the ones I have just cited?
These issues are with the implementation, not the interface. Changing the
ptrace interface won't do anything about them, and fixing them need not
change the ptrace interface.

The ptrace interface could use replacing, but that is really a separate
issue. I will be first in line to replace it with something that has saner
semantics and a more convenient user interface. But that won't help a whit
with things like these race concerns. Let's keep the issue of an ugly
interface separate from the issue of potential bugs in the one we have. If
there are bugs (aside from the inherent limitations of the intended
semantics), they need to be fixed.

> > + read_lock_irq(&tasklist_lock); /* Protects child->sighand. */
> ^^^^
> _irq is unneeded?

Correct, thanks.


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

\
 
 \ /
  Last update: 2005-03-22 14:05    [W:0.049 / U:0.104 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site