You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@httpd.apache.org by jean-frederic clere <jf...@gmail.com> on 2023/03/19 08:52:56 UTC

mod_watchdog.c looping for mutex?

Hi,

While debugging I noted something in mod_watchdog.c
https://github.com/apache/httpd/blob/trunk/modules/core/mod_watchdog.c#L115

Shouldn't we have a else { } with a apr_sleep()?
Basically the thread is looping there :-(
-- 
Cheers

Jean-Frederic

Re: mod_watchdog.c looping for mutex?

Posted by jean-frederic clere <jf...@gmail.com>.

On 3/19/23 12:21, Yann Ylavic wrote:
> On Sun, Mar 19, 2023 at 9:53 AM jean-frederic clere <jf...@gmail.com> wrote:
>>
>> Hi,
>>
>> While debugging I noted something in mod_watchdog.c
>> https://github.com/apache/httpd/blob/trunk/modules/core/mod_watchdog.c#L115
>>
>> Shouldn't we have a else { } with a apr_sleep()?
>> Basically the thread is looping there :-(
> 
> Hm, it seems that there is a sleep() already here:
> https://github.com/apache/httpd/blob/trunk/modules/core/mod_watchdog.c#L132
> ?

Oops sorry for the noise.

> 
> 
> Regards;
> Yann.

-- 
Cheers

Jean-Frederic

Re: mod_watchdog.c looping for mutex?

Posted by Yann Ylavic <yl...@gmail.com>.

On Sun, Mar 19, 2023 at 9:53 AM jean-frederic clere <jf...@gmail.com> wrote:
>
> Hi,
>
> While debugging I noted something in mod_watchdog.c
> https://github.com/apache/httpd/blob/trunk/modules/core/mod_watchdog.c#L115
>
> Shouldn't we have a else { } with a apr_sleep()?
> Basically the thread is looping there :-(

Hm, it seems that there is a sleep() already here:
https://github.com/apache/httpd/blob/trunk/modules/core/mod_watchdog.c#L132
?

Regards;
Yann.