lkml.org 
[lkml]   [2009]   [Jun]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH] coredump: Retry writes where appropriate
Date
> I don't think ->real_blocked is a good choice, we have to add more checks
> to the signal sending path. Note that currenly it is only checked under
> sig_fatal() && !SIGNAL_GROUP_EXIT.

No, you're right. It's not a good idea.

> Perhaps it is easier to change dump_write() to clear TIF_SIGPENDING
> unless fatal_signal_pending(),

That is almost a separate subject, really. Having i/o calls' waits wrongly
interrupted and then clearing TIF_SIGPENDING just seems goofy to me. It
might not be just the ->write call, it might affect filp_open or dump_seek
or whatever. It makes no sense to me to take the approach of fiddling
after doing the wrong thing. Just don't do the wrong thing to begin with.

That means clear TIF_SIGPENDING and don't let it be set again by a signal
that is not meant to abort the core dump. Why do anything but that?

It's almost certain that recalc_sigpending_tsk won't be called once dumping
has started. But there is the possibility of recalc_sigpending_and_wake
via cancel_freezing, at least. Seems safer to make recalc_sigpending_tsk
robust in this case.


Thanks,
Roland


\
 
 \ /
  Last update: 2009-06-02 01:07    [W:7.920 / U:0.428 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site