From: | Magnus Hagander <magnus(at)hagander(dot)net> |
---|---|
To: | tgl(at)sss(dot)pgh(dot)pa(dot)us |
Cc: | pgsql-patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: Win32 shmem |
Date: | 2007-03-20 15:10:02 |
Message-ID: | 20070320151002.GA25258@svr2.hagander.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-patches |
On Tue, Mar 20, 2007 at 07:41:32AM +0100, Magnus Hagander wrote:
> > > Does it seem like I've overlooked anything obvious in this? I do get the
> > > feeling that this is too simple, but I don't know exactly where the
> > > problem is :-)
> >
> > I think you do still need the on_shmem_exit detach callback. Consider
> > the situation where the postmaster is trying to reinitialize after a
> > child crash. The Unix code is designed to detach and destroy the old
> > segment then create a new one. If that's not what you want to do then
> > this code still seems not right.
>
> Ok, will look into that. Haven't tested that scenario.
That was indeed so. Added in new version, attached.
> > There seem to be a lot of system calls not checked for failure here.
> > Do they really not have any failure possibilities?
I looked it over, and didn't find "a lot". I found one or two (which are
now fixed). Are you referring to anything in particular?
//Magnus
Attachment | Content-Type | Size |
---|---|---|
win32_shmem.c | text/x-csrc | 8.2 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2007-03-20 15:23:11 | Re: Stats processor not restarting |
Previous Message | Magnus Hagander | 2007-03-20 14:50:33 | Re: Stats processor not restarting |