lkml.org 
[lkml]   [2010]   [Nov]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 04/14] signal: don't notify parent if not stopping after tracehook_notify_jctl() in do_signal_stop()
Hello, Oleg.

On 11/26/2010 03:46 PM, Oleg Nesterov wrote:
>> This doesn't cause any behavior difference as the condition never
>> triggers in the current code.
>
> I don't understand the motivation then (probably I should read
> the next patches).

The thing is that with the further changes in the series
tracehook_notify_jctl() becomes rather pointless as do_signal_stop()
becomes more involved with the ptrace logic and bypasses
tracehook_notify_jctl(), so I'm basically getting it out of the way.

tracehook might not be such a bad idea for parts which are further
away but for parts which are this close to ptrace implementation, it
seems more of obfuscation than helpful layering. Especially,
restrictions and capabilities of tracehooks without actual (even
proposed) users aren't very healthy to keep. Other people have to
keep guessing what the intentions behind the unused features are,
which is very frustrating and silly.

Anyways, we can deal with tracehooks later. Let's just ignore
tracehook related changes for now.

Thanks.

--
tejun


\
 
 \ /
  Last update: 2010-11-26 16:09    [W:0.065 / U:0.064 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site