Re: git: 6a79e48076bc - main - Fix two latent bugs in hpts. One where a static is put on a local variable, the other an initialization bug where we should be setting tv.tv_sec to 0.

[tuexen]

> On Nov 28, 2023, at 14:44, Ronald Klop <ronald-li...@klop.ws> wrote:
>
>
> The PR you mention gives me the friendly message of
>
> "
> You must enter a valid bug number!
> "
>
> https://bugs.freebsd.org/bugzilla/show_bug.cgi?id'5482
>
> Typo somewhere?
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id'5382
>
> A quick search gives me 
> https://bugs.freebsd.org/bugzilla/show_bug.cgi?id'5382 which looks a lot like 
> what you describe in the commit.
Yes that is it.

Best regards
Michael
>
> Regards,
> Ronald.
>
> Van: Randall Stewart <r...@freebsd.org>
> Datum: maandag, 27 november 2023 20:39
> Aan: src-committ...@freebsd.org, dev-commits-src-all@FreeBSD.org, 
> dev-commits-src-m...@freebsd.org
> Onderwerp: git: 6a79e48076bc - main - Fix two latent bugs in hpts. One where 
> a static is put on a local variable, the other an initialization bug where we 
> should be setting tv.tv_sec to 0.
> The branch main has been updated by rrs:
>
> URL: 
> https://cgit.FreeBSD.org/src/commit/?idj79e48076bcce3e902323e62689eacb98faa180
>
> commit 6a79e48076bcce3e902323e62689eacb98faa180
> Author:     Randall Stewart <r...@freebsd.org>
> AuthorDate: 2023-11-27 19:38:06 +0000
> Commit:     Randall Stewart <r...@freebsd.org>
> CommitDate: 2023-11-27 19:38:06 +0000
>
>     Fix two latent bugs in hpts. One where a static is put on
>     a local variable, the other an initialization bug where
>     we should be setting tv.tv_sec to 0.
>
>     PR:     275482
> ---
>  sys/netinet/tcp_hpts.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/sys/netinet/tcp_hpts.c b/sys/netinet/tcp_hpts.c
> index 92b2f69c9e58..d673ccbe4a73 100644
> --- a/sys/netinet/tcp_hpts.c
> +++ b/sys/netinet/tcp_hpts.c
> @@ -1519,6 +1519,7 @@ __tcp_run_hpts(struct tcp_hpts_entry *hpts)
>                 hpts->p_mysleep.tv_usec = dynamic_min_sleep;
>             /* Reschedule with new to value */
>             tcp_hpts_set_max_sleep(hpts, 0);
> +           tv.tv_sec = 0;
>             tv.tv_usec = hpts->p_hpts_sleep_time * HPTS_TICKS_PER_SLOT;
>             /* Validate its in the right ranges */
>             if (tv.tv_usec < hpts->p_mysleep.tv_usec) {
> @@ -1600,7 +1601,7 @@ tcp_choose_hpts_to_run(void)
>  void
>  tcp_run_hpts(void)
>  {
> -   static struct tcp_hpts_entry *hpts;
> +   struct tcp_hpts_entry *hpts;
>     struct epoch_tracker et;
>
>     NET_EPOCH_ENTER(et);
>
>

smime.p7s
Description: S/MIME cryptographic signature