[CFR][PATCH] Call proper signal handler from libthr for non-SA_SIGINFO case
Eygene Ryabinkin
rea at freebsd.org
Thu Nov 27 04:21:28 UTC 2014
Thu, Nov 27, 2014 at 12:17:35AM +0200, Konstantin Belousov wrote:
> On Thu, Nov 27, 2014 at 01:12:31AM +0300, Eygene Ryabinkin wrote:
> >
> > Added them too. New patch is at
> > http://codelabs.ru/fbsd/patches/libthr/11-CURRENT-fix-SIGINFO-processing-with-RESETHAND-v3.diff
> >
>
> You went silent for several days, I committed the modified patch
> earlier as r275120.
>
> Thank you.
Fine, thanks a lot!
"td->td_ru.ru_nsignals++;" perhaps doesn't belong to the new function,
but it is a matter of taste. By the way, shouldn't
"PROC_LOCK_ASSERT(td->td_proc, MA_OWNED);" be also present single
kern_sigprocmask() is called with SIGPROCMASK_PROC_LOCKED?
--
Eygene Ryabinkin ,,,^..^,,,
[ Life's unfair - but root password helps! | codelabs.ru ]
[ 82FE 06BC D497 C0DE 49EC 4FF0 16AF 9EAE 8152 ECFB | freebsd.org ]
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 358 bytes
Desc: not available
URL: <http://lists.freebsd.org/pipermail/freebsd-threads/attachments/20141127/068f194e/attachment.sig>
More information about the freebsd-threads
mailing list