Re: fork/exec patch: pre-CreateProcess finalization - Mailing list pgsql-patches

From Jan Wieck
Subject Re: fork/exec patch: pre-CreateProcess finalization
Date
Msg-id 3FFE161E.3000200@Yahoo.com
Whole thread Raw
In response to Re: fork/exec patch: pre-CreateProcess finalization  (Bruce Momjian <pgman@candle.pha.pa.us>)
List pgsql-patches
Bruce Momjian wrote:

> Claudio Natoli wrote:
>>
>> Tom Lane writes:
>> > Actually, on further reflection a separate array to store PIDs and
>> cancel keys is probably a better idea.
>> [snip]
>> > I still think it's unnecessary to make a separate shmem segment for it,
>> though.
>>
>> Why is that? Don't we need the backends to have access to it to do a cancel
>> request? I think I've missed something here...
>
> I think they are saying put the cancel key inside the existing shared
> memory segment.  I don't know when we actually attach to the main shared
> memory sigment in the child, but it would have to be sooner than when we
> need the cancel key.

I said move it into the PGPROC structure. And keep the pid in both, the
PGPROC structure and postmaster local memory.

The backend attaches to the shared memory during
AttachSharedMemoryAndSemaphores() ... where else?


Jan

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


pgsql-patches by date:

Previous
From: Claudio Natoli
Date:
Subject: Re: fork/exec patch: pre-CreateProcess finalization
Next
From: Bruce Momjian
Date:
Subject: Re: fork/exec patch: pre-CreateProcess finalization