Thread: shared_preload_libraries support on Win32?

shared_preload_libraries support on Win32?

From
Date:
(working on the PL debugger...) <br /><br /> It appears that the libraries listed in shared_preload_libraries will
*not*be inherited by spawned backends on Win32 platforms.<br /><br /> Do we have to do something special to make that
work?<br/><br /> Using ProcessExplorer (from sysinternals.com), I can see that my plugins are loaded into the
postmaster,but not into the individual backends.<br /><br /> If I set local_preload_libraries equal to
shared_preload_libraries,the plugins are loaded into the backends as expected (so it seems unlikely that I have a
pathnameor permissions problem).<br /><br /> Should we just call process_shared_preload_libraries() after calling
read_nondefault_variables()(in SubPostmasterMain())?<br /><br />             -- Korry<br /><table cellpadding="0"
cellspacing="0"width="100%"><tr><td><br /><br /> --<br />   Korry Douglas    <a
href="mailto:korryd@enterprisedb.com">korryd@enterprisedb.com</a><br/>   EnterpriseDB      <a
href="http://www.enterprisedb.com">http://www.enterprisedb.com</a></td></tr></table>

Re: shared_preload_libraries support on Win32?

From
Tom Lane
Date:
<korryd@enterprisedb.com> writes:
> It appears that the libraries listed in shared_preload_libraries will
> *not* be inherited by spawned backends on Win32 platforms.

Well, yeah, because it's a fork/exec on that platform.

> Should we just call process_shared_preload_libraries() after calling
> read_nondefault_variables() (in SubPostmasterMain())?

I don't entirely see the point.  The value of shared_preload_libraries
is to avoid paying per-process overhead to load the libraries, and that
benefit is already lost in a fork/exec world.  Might as well just let
the libraries be loaded when and if needed.

I think we've failed to document that shared_preload_libraries doesn't
work on Windows, which is an oversight.
        regards, tom lane


Re: shared_preload_libraries support on Win32?

From
Andrew Dunstan
Date:

Tom Lane wrote:
> <korryd@enterprisedb.com> writes:
>   
>> It appears that the libraries listed in shared_preload_libraries will
>> *not* be inherited by spawned backends on Win32 platforms.
>>     
>
> Well, yeah, because it's a fork/exec on that platform.
>
>   
>> Should we just call process_shared_preload_libraries() after calling
>> read_nondefault_variables() (in SubPostmasterMain())?
>>     
>
> I don't entirely see the point.  The value of shared_preload_libraries
> is to avoid paying per-process overhead to load the libraries, and that
> benefit is already lost in a fork/exec world.  Might as well just let
> the libraries be loaded when and if needed.
>
>   

The only benefit I can see is that it would assist in having a common 
configuration.

> I think we've failed to document that shared_preload_libraries doesn't
> work on Windows, which is an oversight.
>
>   

Maybe postmaster should also log a warning if it detects this.

cheers

andrew



Re: shared_preload_libraries support on Win32?

From
Tom Lane
Date:
Andrew Dunstan <andrew@dunslane.net> writes:
> Tom Lane wrote:
>> I don't entirely see the point.  The value of shared_preload_libraries
>> is to avoid paying per-process overhead to load the libraries, and that
>> benefit is already lost in a fork/exec world.  Might as well just let
>> the libraries be loaded when and if needed.

> The only benefit I can see is that it would assist in having a common 
> configuration.

Actually ... I take that back.  I was thinking of the original purpose
of preload_libraries, which was strictly performance optimization.
But in the new world of plugins there may be functional reasons for
wanting libraries to be loaded into backends --- and
shared_preload_libraries is not isomorphic to local_preload_libraries.
The permissions situation is different.

Korry's right, we should force re-loading of shared_preload_libraries
in the EXEC_BACKEND case.  The needed documentation change is to point
out that on Windows this is not a performance win, but it might still
be wanted for instrumentation or debugging plugins.
        regards, tom lane


Re: shared_preload_libraries support on Win32?

From
Date:
<blockquote type="CITE"><pre>
<font color="#000000">Actually ... I take that back.  I was thinking of the original purpose</font>
<font color="#000000">of preload_libraries, which was strictly performance optimization.</font>
<font color="#000000">But in the new world of plugins there may be functional reasons for</font>
<font color="#000000">wanting libraries to be loaded into backends --- and</font>
<font color="#000000">shared_preload_libraries is not isomorphic to local_preload_libraries.</font>
<font color="#000000">The permissions situation is different.</font>
</pre></blockquote><br /> And, shared_preload_libraries is processed (in the postmaster) before the shared-memory
segmentis created, so a shared_preload_library can call RequestAddinShmemSpace() and RequestAddinLWLocks(), but a
local_preload_librarycannot.<br /><br /><br />             -- Korry  

Re: shared_preload_libraries support on Win32?

From
Tom Lane
Date:
<korryd@enterprisedb.com> writes:
>> But in the new world of plugins there may be functional reasons for
>> wanting libraries to be loaded into backends --- and
>> shared_preload_libraries is not isomorphic to local_preload_libraries.
>> The permissions situation is different.

> And, shared_preload_libraries is processed (in the postmaster) before
> the shared-memory segment is created, so a shared_preload_library can
> call RequestAddinShmemSpace() and RequestAddinLWLocks(), but a
> local_preload_library cannot.

That doesn't seem like an issue though, since the copy in the postmaster
will have done that anyway.
        regards, tom lane


Re: shared_preload_libraries support on Win32?

From
Date:
<blockquote type="CITE"><pre>
<font color="#000000">> And, shared_preload_libraries is processed (in the postmaster) before</font>
<font color="#000000">> the shared-memory segment is created, so a shared_preload_library can</font>
<font color="#000000">> call RequestAddinShmemSpace() and RequestAddinLWLocks(), but a</font>
<font color="#000000">> local_preload_library cannot.</font>

<font color="#000000">That doesn't seem like an issue though, since the copy in the postmaster</font>
<font color="#000000">will have done that anyway.</font>
</pre></blockquote><br /> You're right - we need the copy in the postmaster (to setup shared memory and LW locks), and
weneed them in the backends too.  I just want to avoid having to set both shared_preload_libraries and
local_preload_libraries(to the same thing).  Adding a call to process_shared_preload_libraries() in SubPostmasterMain()
seemsto fix the problem for me.<br /><br /> Thanks for your input.  I'll submit a patch.<br /><br /><br />            
--Korry  

Re: shared_preload_libraries support on Win32?

From
Alvaro Herrera
Date:
korryd@enterprisedb.com wrote:
> > > And, shared_preload_libraries is processed (in the postmaster) before
> > > the shared-memory segment is created, so a shared_preload_library can
> > > call RequestAddinShmemSpace() and RequestAddinLWLocks(), but a
> > > local_preload_library cannot.
> > 
> > That doesn't seem like an issue though, since the copy in the postmaster
> > will have done that anyway.
> 
> 
> You're right - we need the copy in the postmaster (to setup shared
> memory and LW locks), and we need them in the backends too.  I just want
> to avoid having to set both shared_preload_libraries and
> local_preload_libraries (to the same thing).  Adding a call to
> process_shared_preload_libraries() in SubPostmasterMain() seems to fix
> the problem for me.

Just make sure you don't load the libraries in bgwriter et al ...

-- 
Alvaro Herrera                                http://www.CommandPrompt.com/
The PostgreSQL Company - Command Prompt, Inc.


Re: shared_preload_libraries support on Win32?

From
Tom Lane
Date:
Alvaro Herrera <alvherre@commandprompt.com> writes:
> korryd@enterprisedb.com wrote:
>> You're right - we need the copy in the postmaster (to setup shared
>> memory and LW locks), and we need them in the backends too.

> Just make sure you don't load the libraries in bgwriter et al ...

I see that Korry's patch doesn't do that, but I'm wondering why exactly.
In a Unix environment such libraries *would* be propagated into bgwriter
and every other postmaster child; is there a reason for the setup on
Windows to be different?  In particular, what about autovacuum, which
ISTM should be as close to a standard backend as possible?

Either way we do it, authors of plugins used this way will have to test
both cases (I'm glad I insisted on EXEC_BACKEND mode being testable under
Unix ...)
        regards, tom lane


Re: shared_preload_libraries support on Win32?

From
Date:
<blockquote type="CITE"><pre>
<font color="#000000">>> You're right - we need the copy in the postmaster (to setup shared</font>
<font color="#000000">>> memory and LW locks), and we need them in the backends too.</font>

<font color="#000000">> Just make sure you don't load the libraries in bgwriter et al ...</font>

<font color="#000000">I see that Korry's patch doesn't do that, but I'm wondering why exactly.</font>
<font color="#000000">In a Unix environment such libraries *would* be propagated into bgwriter</font>
<font color="#000000">and every other postmaster child; is there a reason for the setup on</font>
<font color="#000000">Windows to be different?  In particular, what about autovacuum, which</font>
<font color="#000000">ISTM should be as close to a standard backend as possible?</font>
</pre></blockquote> I thought about that too...  does autovacuum ever need to re-index?  If so, it seems that it would
needaccess to any index functions (for function-based indexes) that might reside in a shared_preload_library.<br /><br
/><blockquotetype="CITE"><pre>
 
<font color="#000000">Either way we do it, authors of plugins used this way will have to test</font>
<font color="#000000">both cases (I'm glad I insisted on EXEC_BACKEND mode being testable under</font>
<font color="#000000">Unix ...)</font>
</pre></blockquote><br /> And I'm glad that RequestAddinShmemSpace() and RequestAddinLWLocks() don't complain if called
afterpostmaster start :-)<br /><br />             -- Korry<br /><table cellpadding="0" cellspacing="0"
width="100%"><tr><td><br/><br /> --<br />   Korry Douglas    <a
href="mailto:korryd@enterprisedb.com">korryd@enterprisedb.com</a><br/>   EnterpriseDB      <a
href="http://www.enterprisedb.com">http://www.enterprisedb.com</a></td></tr></table>

Re: shared_preload_libraries support on Win32?

From
Tom Lane
Date:
<korryd@enterprisedb.com> writes:
>> I see that Korry's patch doesn't do that, but I'm wondering why exactly.
>> In a Unix environment such libraries *would* be propagated into bgwriter
>> and every other postmaster child; is there a reason for the setup on
>> Windows to be different?  In particular, what about autovacuum, which
>> ISTM should be as close to a standard backend as possible?

> I thought about that too...  does autovacuum ever need to re-index?  If
> so, it seems that it would need access to any index functions (for
> function-based indexes) that might reside in a shared_preload_library.

Any ordinary C-language function is not an issue, because its library
will get autoloaded upon use.  AFAICS what we have to think about here
is instrumentation or debugging plugins that someone might wish to have
running in the postmaster's special children.  Maybe there's no such
animal; I'm not sure.  But in the Unix environment they'd be active in
those processes.
        regards, tom lane