From: | "Magnus Hagander" <mha(at)sollentuna(dot)net> |
---|---|
To: | <pgsql-patches(at)postgresql(dot)org> |
Cc: | "pgsql-hackers-win32" <pgsql-hackers-win32(at)postgresql(dot)org>, "Claudio Natoli" <claudio(dot)natoli(at)memetrics(dot)com> |
Subject: | win32 signals, part 4 |
Date: | 2004-02-04 22:09:00 |
Message-ID: | 6BCB9D8A16AC4241919521715F4D8BCE34B136@algol.sollentuna.se |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers-win32 pgsql-patches |
Ok, here we go again.
Taking into account Claudios comments on the previous patch, as well as
some more fooling around here of my own, here's a fourth (and final?)
one.
If there are no further comments from Claudio or anyone else, I feel
this is now ready to be applied.
Differences from last version:
1) Per Claudios suggestion, create a "babysitter thread" for each
process that waits on the process and signals ourselves. This reduces
the amount of code (=good) and most importantly removes all the
synchronisation issues (=even better). The only thing left to sync is
the signal delivery, and that has alreay been taken care of in previous
patches.
2) Make pg_queue_signal() no longer be static. This way we don't have to
go through named pipes when signalling ourselves.
3) Remove the redefinition of select() again. This also means that the
select.c file previously incouded should *NOT* be included any more. See
separate mail to hackers-win32 on this issue.
4) Fix Claudios wrong-assignment error in win32_removeChild. Also
improve an error message in win32_waitpid.
Again, the preivously attached select.c file should *NOT* be included.
//Magnus
From | Date | Subject | |
---|---|---|---|
Next Message | Magnus Hagander | 2004-02-04 22:15:11 | Re: win32 signals, part 4 |
Previous Message | Ronald Kuczek | 2004-02-04 21:59:57 | psql does not link with readline |
From | Date | Subject | |
---|---|---|---|
Next Message | Magnus Hagander | 2004-02-04 22:15:11 | Re: win32 signals, part 4 |
Previous Message | Heikki Linnakangas | 2004-02-04 20:22:16 | Two-phase commit |