[PATCH] tfork

[Ralph Böhme]

On Thu, Apr 20, 2017 at 03:23:13PM -0700, Jeremy Allison wrote:
> On Thu, Apr 20, 2017 at 05:05:45PM +0200, Ralph Böhme wrote:
> > 
> > SA_RESTART is just screwed. Imho the code must deal with EINTR anyway in most if
> > not all places, so we should just remove it where applicable. The tevent signal
> > interface allows passing it, but we should not use it in our code.
> > 
> > I only see two callers: tdb_robust_mutex_handler() who doesn't need it and
> > heimdal.
> 
> So do you want to remove the automatic adding of
> SA_RESTART inside of CatchSignal() ?

yes.

> > It's a nice utility function where the caller mustn't prepate the sigaction structure.
> > 
> > Maybe change it to take and additional sa_flags and two addtional out args, ie
> > 
> > int CatchSignal(int signum,
> >                 int sa_flags,
> > 		void (*handler)(int),
> >                 int *old_sa_flags,
> > 		void (**old_handler)(int));
> > 
> > Most existing callers would just call CatchSignal(..., NULL, NULL);
> 
> Yep, I like that. Might prepare a patch for review that
> fixes this !

:)

-slow