Re: Shave a few instructions from child-process startup sequence - Mailing list pgsql-hackers

From Amit Kapila
Subject Re: Shave a few instructions from child-process startup sequence
Date
Msg-id CAA4eK1LxCkeRXK0A+8XeVj6jOGHv_+zCUKbY1+WPOMXEXSmhGg@mail.gmail.com
Whole thread Raw
In response to Shave a few instructions from child-process startup sequence  (Gurjeet Singh <gurjeet@singh.im>)
Responses Re: Shave a few instructions from child-process startup sequence  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
On Thu, Oct 31, 2013 at 2:41 AM, Gurjeet Singh <gurjeet@singh.im> wrote:
> Just a small patch; hopefully useful.

This is valid saving as we are filling array ListenSocket[] in
StreamServerPort() serially, so during ClosePostmasterPorts() once if
it encountered PGINVALID_SOCKET, it is valid to break the loop.
Although savings are small considering this doesn't occur in any
performance path, still I think this is right thing to do in code.

It is better to register this patch in CF app list, unless someone
feels this is not right.

With Regards,
Amit Kapila.
EnterpriseDB: http://www.enterprisedb.com



pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: API bug in DetermineTimeZoneOffset()
Next
From: Tom Lane
Date:
Subject: Re: Shave a few instructions from child-process startup sequence