Re: [PATCHES] win32 signals, part 5

From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Magnus Hagander <mha(at)sollentuna(dot)net>
Cc: Neil Conway <neilc(at)samurai(dot)com>, 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 5
Date: 2004-02-09 21:43:28
Message-ID: 200402092143.i19LhSJ01703@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers-win32 pgsql-patches


This patch has been applied by Neil.

---------------------------------------------------------------------------

Magnus Hagander wrote:
> >>> 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.
> >
> >Actually, please hold just a second :-)
> >
> >I have an updated version of this patch that fixes the remaining small
> >issue (FATAL on failure to set the control handler). Will post later
> >tonight or early tomorrow. Only veyr small changes, so if there are no
> >objections to this one, there shouldn't be to that one either.
>
> Here's the new one. Turns out I had already fixed the one part I thought
> I still had, so it was already ready.
>
> Changes since last patch:
>
> 1) Error messages in pgwin32_signal_initialize() are now all using
> ereport(). (Previously used fprintf to stderr, but I realised that this
> function executes on the main thread and can thus use ereport). The
> error for SetConsoleCtrlHandler() is now FATAL, per suggestion from
> Claudio.
>
> 2) Redefines CHECK_FOR_INTERRUPTS() on Win32 only, to include a call to
> WaitForSingleObjectEx(). This allows our signals to be delivered just
> before we check for their result. WIth this change, query cancel now
> works fine on win32. It's also possible to do an "administrator
> shutdown" on a backend that is running a query (previously only possible
> when the backend was idle)
>
>
> The rest is identical to patch _4.
>
> //Magnus
>
> ---------------------------(end of broadcast)---------------------------
> TIP 1: subscribe and unsubscribe commands go to majordomo(at)postgresql(dot)org
>

--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073

In response to

Browse pgsql-hackers-win32 by date

  From Date Subject
Next Message Tom Lane 2004-02-10 01:27:05 Re: [pgsql-hackers-win32] Sync vs. fsync during checkpoint
Previous Message Jan Wieck 2004-02-09 20:41:07 Re: [pgsql-hackers-win32] Sync vs. fsync during checkpoint

Browse pgsql-patches by date

  From Date Subject
Next Message Gavin Sherry 2004-02-10 03:45:46 psql documentation one liner
Previous Message Andrew Dunstan 2004-02-09 21:15:04 Re: win32 inode fix