From: | "Heikki Linnakangas" <heikki(at)enterprisedb(dot)com> |
---|---|
To: | "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | "Thomas Lee" <tom(at)vector-seven(dot)com>, <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: [UPDATED] A GUC variable to replace PGBE_ACTIVITY_SIZE |
Date: | 2008-07-01 12:21:23 |
Message-ID: | 486A2143.4020604@enterprisedb.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-patches |
Tom Lane wrote:
> "Heikki Linnakangas" <heikki(at)enterprisedb(dot)com> writes:
>> Another simple optimization occurred to me while looking at this: we
>> should skip the memcpy/strcpy altogether if the BackendActivity slot is
>> not in use. That seems like a good bet, you usually don't try to max out
>> max_connections.
>
> Huh? How could we be assigning to a slot that is not in use?
Before the patch, we loop through the shared PgBackendStatus slots
(there is MaxBackends of them), and issue a memcpy for each to copy it
to our local slot. After that, we check if it was actually in use.
After the patch, we still loop through the shared slots, but only issue
the memcpy for slots that are in use.
Hope this answers your question, I'm not sure what you meant...
--
Heikki Linnakangas
EnterpriseDB http://www.enterprisedb.com
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2008-07-01 13:56:13 | Re: [UPDATED] A GUC variable to replace PGBE_ACTIVITY_SIZE |
Previous Message | Simon Riggs | 2008-07-01 11:49:47 | Re: [HACKERS] odd output in restore mode |