mod_auth_mellon icon indicating copy to clipboard operation
mod_auth_mellon copied to clipboard

Fix am_mod_cfg_rec not being reused.(#36)

Open oss-aimoto opened this issue 5 years ago • 6 comments

Reuse am_mod_cfg_rec. mellon Sessions are not lost on httpd restart (signal: HUP, USR1). However, the cache size is not changed.

oss-aimoto avatar Sep 18 '20 06:09 oss-aimoto

My httpd internals memories are a bit rusty in places, but are you moving the mutex from allocation on a global memory pool to a per-process pool ?

simo5 avatar Sep 18 '20 14:09 simo5

The global memory pool is apr_pool_clear() at Gracefull restart. (by reset_process_pconf() in main.c) s->process->pool is not cleared, the mutex can be reused. I refer to the code of mod_ssl.

oss-aimoto avatar Sep 23 '20 02:09 oss-aimoto

I guess the question is how thi affects the usage of the mutex, does it still remain a global mutex that all processes can reach? Or does this change open up the possibility that different processes will se actually different mutexes and not be coordinated anymore ?

simo5 avatar Sep 23 '20 14:09 simo5

A global mutex that is reachable by all processes. s-> process-> pool is also a global pool.

oss-aimoto avatar Sep 24 '20 01:09 oss-aimoto

In that case I am ok with this, but I'd like another pair of eyes to take a look.

simo5 avatar Sep 24 '20 13:09 simo5

@jhrozek Can you take a look perhaps?

thijskh avatar Apr 08 '24 06:04 thijskh