Mailing List Archive: Apache: Dev

Re: svn commit: r820427 - in /httpd/httpd/trunk/modules: cluster/mod_heartmonitor.c core/mod_watchdog.c core/mod_watchdog.h

Index | Next | Previous | View Threaded

fuankg at apache

Sep 30, 2009, 6:56 PM

Post #1 of 3 (348 views)
Permalink

Re: svn commit: r820427 - in /httpd/httpd/trunk/modules: cluster/mod_heartmonitor.c core/mod_watchdog.c core/mod_watchdog.h

Hi Jeff,
trawick [at] apache schrieb:
> Author: trawick
> Date: Wed Sep 30 20:00:37 2009
> New Revision: 820427
>
> URL: http://svn.apache.org/viewvc?rev=820427&view=rev
> Log:
> change the callable functions in the mod_watchdog API
> to optional hooks to avoid module ordering or other symbol
> resolution issues; affected:
>
> ap_watchdog_get_instance
> ap_watchdog_register_callback
> ap_watchdog_set_callback_interval
>
> Modified:
> httpd/httpd/trunk/modules/cluster/mod_heartmonitor.c
> httpd/httpd/trunk/modules/core/mod_watchdog.c
> httpd/httpd/trunk/modules/core/mod_watchdog.h
this breaks NetWare export list; and I did spent hours with Rainer last
time get the mod_watchdog exported symbols in; can you perhaps help a
bit with the awk stuff - I'm no expert with this; see:
http://svn.apache.org/viewvc?rev=820503&view=rev
where I started with it, but too late to get it right ATM ...

thanks, Gün.

trawick at gmail

Oct 1, 2009, 4:26 AM

Post #2 of 3 (324 views)
Permalink

Re: svn commit: r820427 - in /httpd/httpd/trunk/modules: cluster/mod_heartmonitor.c core/mod_watchdog.c core/mod_watchdog.h [In reply to]

On Wed, Sep 30, 2009 at 9:56 PM, Guenter Knauf <fuankg [at] apache> wrote:

> Hi Jeff,
> trawick [at] apache schrieb:
> > Author: trawick
> > Date: Wed Sep 30 20:00:37 2009
> > New Revision: 820427
> >
> > URL: http://svn.apache.org/viewvc?rev=820427&view=rev
> > Log:
> > change the callable functions in the mod_watchdog API
> > to optional hooks to avoid module ordering or other symbol
> > resolution issues; affected:
> >
> > ap_watchdog_get_instance
> > ap_watchdog_register_callback
> > ap_watchdog_set_callback_interval
> >
> > Modified:
> > httpd/httpd/trunk/modules/cluster/mod_heartmonitor.c
> > httpd/httpd/trunk/modules/core/mod_watchdog.c
> > httpd/httpd/trunk/modules/core/mod_watchdog.h
> this breaks NetWare export list; and I did spent hours with Rainer last
> time get the mod_watchdog exported symbols in; can you perhaps help a
> bit with the awk stuff - I'm no expert with this; see:
> http://svn.apache.org/viewvc?rev=820503&view=rev
> where I started with it, but too late to get it right ATM ...
>

What is the symptom?

The AP_WD_DECLARE stuff is still needed for the external hooks implemented
by mod_watchdog, such as

APR_DECLARE_EXTERNAL_HOOK(ap, AP_WD, int, watchdog_init, (
server_rec *s,
const char *name,
apr_pool_t *pool))

NetWare must already support APR_DECLARE_OPTIONAL_FN one way or another,
long before this change. I'm unable to tell how mod_watchdog's use of that
is different from existing modules, such as mod_ldap or mod_ssl or core.

fuankg at apache

Oct 3, 2009, 5:43 PM

Post #3 of 3 (305 views)
Permalink

Re: svn commit: r820427 - in /httpd/httpd/trunk/modules: cluster/mod_heartmonitor.c core/mod_watchdog.c core/mod_watchdog.h [In reply to]

Jeff,
Jeff Trawick schrieb:
> On Wed, Sep 30, 2009 at 9:56 PM, Guenter Knauf <fuankg [at] apache
> What is the symptom?
that I was too tired :)

> The AP_WD_DECLARE stuff is still needed for the external hooks
> implemented by mod_watchdog, such as
>
> APR_DECLARE_EXTERNAL_HOOK(ap, AP_WD, int, watchdog_init, (
> server_rec *s,
> const char *name,
> apr_pool_t *pool))
>
> NetWare must already support APR_DECLARE_OPTIONAL_FN one way or another,
> long before this change. I'm unable to tell how mod_watchdog's use of
> that is different from existing modules, such as mod_ldap or mod_ssl or
> core.
yes, seems NW does - I did now build the test modules, and from the logs
it seems that optional functions / hooks work ...
[Sat Oct 03 11:45:46 2009] [error] Optional function test said: GET
/manual/images/feather.gif HTTP/1.1
[Sat Oct 03 11:45:46 2009] [error] Optional hook test said: GET
/manual/images/feather.gif HTTP/1.1
[Sat Oct 03 11:45:46 2009] [error] Optional function test said: GET
/manual/images/left.gif HTTP/1.1
[Sat Oct 03 11:45:46 2009] [error] Optional hook test said: GET
/manual/images/left.gif HTTP/1.1
[Sat Oct 03 11:45:46 2009] [error] Optional function test said: GET
/manual/images/up.gif HTTP/1.1
[Sat Oct 03 11:45:46 2009] [error] Optional hook test said: GET
/manual/images/up.gif HTTP/1.1

I had the impression that I need to bring them into the export list, but
seems not the case....
did spend some minutes with that, and although I got mod_watchdog now to
load it gives me an error in the log that it cant create the worker
process because 'not implemented for this platform', sniff ...
I think I have to look again deeper into the mpm stuff, and finish what
I have started there (I know there are still some outstanding
renamings), though its hard to catch up when you have not followed close
to all the changes for close to a year where I had really only
error(ENOTIME_PERMANENT) ... :(

thanks, G�n.

Index | Next | Previous | View Threaded

Interested in having your list archived? Contact Gossamer Threads