[DTrace-devel] [PATCH 11/14] dtprobed: fix parser child timeout

Kris Van Hees kris.van.hees at oracle.com
Fri Oct 25 02:54:24 UTC 2024


On Thu, Oct 24, 2024 at 12:37:55PM +0100, Nick Alcock wrote:
> This was always meant to be five seconds, but when the option was
> rejigged to take seconds rather than milliseconds, the default was
> not changed, leading to a ridiculously huge multi-hour default
> timeout.  We definitely don't want the parser child hanging for
> nearly two hours: go back to seconds again.
> 
> Signed-off-by: Nick Alcock <nick.alcock at oracle.com>

Reviewed-by: Kris Van Hees <kris.van.hees at oracle.com>

> ---
>  dtprobed/dtprobed.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/dtprobed/dtprobed.c b/dtprobed/dtprobed.c
> index c83f0adb56c4..f4c6be1e045d 100644
> --- a/dtprobed/dtprobed.c
> +++ b/dtprobed/dtprobed.c
> @@ -82,7 +82,7 @@ static pid_t parser_pid;
>  static int parser_in_pipe;
>  static int parser_out_pipe;
>  static int sync_fd = -1;
> -static int timeout = 5000; 			/* In seconds.  */
> +static int timeout = 5; 			/* In seconds.  */
>  static int rq_count = 0;
>  static int cleanup_interval = 128;		/* In requests.  */
>  
> -- 
> 2.46.0.278.g36e3a12567
> 



More information about the DTrace-devel mailing list