On Mon, Nov 23, 2020 at 5:35 PM Ruediger Pluem <[email protected]> wrote:
>
> On 11/23/20 12:14 PM, [email protected] wrote:
> >
> >      /* Get access to rmm in child */
> > +    client_rmm = main_client_rmm;
>
> Why is this needed?

It's not (r1883807).

> >
> > +    client_lock = main_client_lock;
>
> Why is this needed?

This one and the following (opaque_lock) are, by the look of:

APR_DECLARE(apr_status_t) apr_global_mutex_child_init(
                              apr_global_mutex_t **mutex,
                              const char *fname,
                              apr_pool_t *pool)
{
    apr_status_t rv;

    rv = apr_proc_mutex_child_init(&((*mutex)->proc_mutex), fname, pool);
    return rv;
}

I keep forgetting which apr_*_{child_init,attach}() need a real
pointer to the parent, or not.. :)


Regards;
Yann.

Reply via email to