[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