This is the mail archive of the cygwin-patches@cygwin.com mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [Patch] Signal mask handling


On Wed, Mar 10, 2004 at 11:26:19PM -0500, Pierre A. Humblet wrote:
>2004-02-11  Pierre Humblet <pierre.humblet@ieee.org>
>	
>	* gendef (_sigdelayed): Replace the call to 
>	set_process_mask by a call to set_process_mask_delta.
>	* exceptions.cc (_cygtls::interrupt_setup): Set oldmask
>	to the delta and don't set newmask.
>	(set_process_mask_delta): New function.
>	(_cygtls::call_signal_handler): Replace the first call to 
>	set_process_mask by a call to set_process_mask_delta.

I tried applying this patch and saw a difference in behavior with
the attached program.  It wasn't setting the signal mask in the handler
correctly.  I have changes in my sandbox which conflicted with your
patch, so I probably misapplied something, though since your patch
looks correct to me otherwise.

Can you confirm the same behavior on the below program before and
after your change?  If so, I'd say it's ok to check in but I'd like to
check my changes in first.  I hope to have them completed soon.

Btw, I think that if you check this in, set_process_mask becomes
obsolete, right?

cgf

Attachment: sigmask.cc
Description: Text document


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]