win32 signals, part 4

Lists: pgsql-hackers-win32pgsql-patches
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
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: Jan Wieck <JanWieck(at)Yahoo(dot)com>
To: Magnus Hagander <mha(at)sollentuna(dot)net>
Cc: pgsql-patches(at)postgresql(dot)org, pgsql-hackers-win32 <pgsql-hackers-win32(at)postgresql(dot)org>, Claudio Natoli <claudio(dot)natoli(at)memetrics(dot)com>
Subject: Re: [PATCHES] win32 signals, part 4
Date: 2004-02-05 01:31:08
Message-ID: 40219CDC.4040005@Yahoo.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers-win32 pgsql-patches

Magnus Hagander wrote:
> 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.

IIRC a separate "babysitter thread" just to handle message passing is
exactly what Katie Ward did for UltraSQL ... the Win32 port done at
NuSphere. Glad to see she was right about that.

Jan

--
#======================================================================#
# It's easier to get forgiveness for being wrong than for being right. #
# Let's break this rule - forgive me. #
#================================================== JanWieck(at)Yahoo(dot)com #


From: Neil Conway <neilc(at)samurai(dot)com>
To: "Magnus Hagander" <mha(at)sollentuna(dot)net>
Cc: <pgsql-patches(at)postgresql(dot)org>, "pgsql-hackers-win32" <pgsql-hackers-win32(at)postgresql(dot)org>, "Claudio Natoli" <claudio(dot)natoli(at)memetrics(dot)com>
Subject: Re: [PATCHES] win32 signals, part 4
Date: 2004-02-06 21:23:50
Message-ID: 87llnfnbq1.fsf@mailbox.samurai.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers-win32 pgsql-patches

"Magnus Hagander" <mha(at)sollentuna(dot)net> writes:
> 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.

I'll apply this tonight barring any objections.

-Neil