Re: kern/58301: Recent sys/net/nd.c change panics my -current aarch64 system (nanopi m4v2)

[Markus Kilbinger <mk%kilbi.de@localhost>]

The following reply was made to PR kern/58301; it has been noted by GNATS.

From: Markus Kilbinger <mk%kilbi.de@localhost>
To: gnats-bugs%netbsd.org@localhost
Cc: kern-bug-people%netbsd.org@localhost, gnats-admin%netbsd.org@localhost, netbsd-bugs%netbsd.org@localhost
Subject: Re: kern/58301: Recent sys/net/nd.c change panics my -current aarch64
 system (nanopi m4v2)
Date: Thu, 30 May 2024 15:18:44 +0200

 Thanks for your quick response!
 
 Recompiled my kernel with your test / patch.
 Looks good so far, stress test is running ...
 
 Am Do., 30. Mai 2024 um 14:50 Uhr schrieb Michael van Elst <mlelstv%serpens.de@localhost>:
 >
 > The following reply was made to PR kern/58301; it has been noted by GNATS.
 >
 > From: mlelstv%serpens.de@localhost (Michael van Elst)
 > To: gnats-bugs%netbsd.org@localhost
 > Cc:
 > Subject: Re: kern/58301: Recent sys/net/nd.c change panics my -current aarch64 system (nanopi m4v2)
 > Date: Thu, 30 May 2024 12:46:55 -0000 (UTC)
 >
 >  mk%kilbi.de@localhost writes:
 >
 >  >              /netbsd: [ 3659.7543704] panic: kernel diagnostic assertion "lle->la_numheld == 0" failed: file "../../../../net/if_llatbl.c", line 333 la_numheld 1 > 0, pkts_dropped 0
 >
 >
 >  nd_timer doesn't seem to maintain the la_numheld counter.
 >
 >  Can you test this ?
 >
 >  --- nd.c.orig   2024-05-30 14:46:19.395480448 +0200
 >  +++ nd.c        2024-05-30 14:40:37.612760004 +0200
 >  @@ -105,8 +105,12 @@ nd_timer(void *arg)
 >
 >                          m->m_nextpkt = NULL;
 >                          ln->ln_hold = NULL;
 >  +                       ln->la_numheld = 0;
 >                  }
 >
 >  +               KASSERTMSG(ln->la_numheld == 0, "la_numheld=%d",
 >  +                   ln->la_numheld);
 >  +
 >                  missed = ND_LLINFO_INCOMPLETE;
 >                  ln->ln_state = ND_LLINFO_WAITDELETE;
 >                  break;
 >