Re: Signals inheritance work - major problems - Mailing list pgsql-hackers-win32

From Magnus Hagander
Subject Re: Signals inheritance work - major problems
Date
Msg-id 6BCB9D8A16AC4241919521715F4D8BCE4760AA@algol.sollentuna.se
Whole thread Raw
In response to Signals inheritance work - major problems  ("Magnus Hagander" <mha@sollentuna.net>)
List pgsql-hackers-win32
> > But if I move the signals initialization code to after the backend
> > file read, that is a *lot* later. This is also *after* the
> backend has
> > set up it's signal handlers, which will crash if things
> aren't initialized.
>
> Hm?  read_backend_variables() happens in SubPostmasterMain(),
> which certainly hasn't yet set up any signal handlers.  (It
> has done a PG_SETMASK, but that could be moved down a few
> lines if it's a problem; but my guess is that you want to do
> that before starting the signals thread
> anyway.)

Right. It's the PG_SETMASK that causes teh crash.


> I'd be inclined to think in terms of just moving the
> pgwin32_signal_initialize() call out of main.c and into
> SubPostmasterMain; you'd also need it in PostmasterMain and
> BootstrapMain I suppose, but that's not as bad as any of the
> alternatives you mention.

Hmm. I thought I looked at that and for some reason thought it was bad.
Can't remember what it was any more. I'll try something along this line
and let you know how it works out.


//Magnus


pgsql-hackers-win32 by date:

Previous
From: Tom Lane
Date:
Subject: Re: Signals inheritance work - major problems
Next
From: Bruce Momjian
Date:
Subject: Re: Signals inheritance work - major problems