Date: Fri, 04 Aug 2023 15:03:50 +0200 From: Kristof Provost <kp@FreeBSD.org> To: Michael Tuexen <tuexen@FreeBSD.org> Cc: src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org Subject: Re: git: b279e84a47dd - main - sctp: improve consistency Message-ID: <B3E69994-14C4-4339-BEEF-8BB89F218020@FreeBSD.org> In-Reply-To: <202307282203.36SM3C98056354@gitrepo.freebsd.org> References: <202307282203.36SM3C98056354@gitrepo.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--=_MailMate_77BDF089-8206-4FA3-8412-9857890C3CAE_= Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: quoted-printable On 29 Jul 2023, at 0:03, Michael Tuexen wrote: > The branch main has been updated by tuexen: > > URL: = > https://cgit.FreeBSD.org/src/commit/?id=3Db279e84a47ddb59e55b5a3cec31c5= 1cd41bf0dc3 > > commit b279e84a47ddb59e55b5a3cec31c51cd41bf0dc3 > Author: Michael Tuexen <tuexen@FreeBSD.org> > AuthorDate: 2023-07-28 12:36:11 +0000 > Commit: Michael Tuexen <tuexen@FreeBSD.org> > CommitDate: 2023-07-28 12:36:11 +0000 > > sctp: improve consistency > > This is simplifying a patch to address PR 260116. > > PR: 260116 > MFC after: 1 week It looks like this commit (or maybe the next one, = c620788150d274c09a070ab486602c98407d73b0) causes a panic in the SCTP = code during the `sys/netpfil/pf/sctp:basic_v4` test: panic: Counter goes negative cpuid =3D 7 time =3D 1691145655 KDB: stack backtrace: db_trace_self_wrapper() at db_trace_self_wrapper+0x2b/frame = 0xfffffe026ea9a3a0 vpanic() at vpanic+0x132/frame 0xfffffe026ea9a4d0 panic() at panic+0x43/frame 0xfffffe026ea9a530 sctp_abort_notification() at sctp_abort_notification/frame = 0xfffffe026ea9a540 sctp_express_handle_sack() at sctp_express_handle_sack+0x647/frame = 0xfffffe026ea9a640 sctp_process_control() at sctp_process_control+0xf62/frame = 0xfffffe026ea9a990 sctp_common_input_processing() at = sctp_common_input_processing+0x561/frame 0xfffffe026ea9ab10 sctp_input_with_port() at sctp_input_with_port+0x1fa/frame = 0xfffffe026ea9abe0 sctp_input() at sctp_input+0x10/frame 0xfffffe026ea9abf0 ip_input() at ip_input+0x2ab/frame 0xfffffe026ea9ac50 netisr_dispatch_src() at netisr_dispatch_src+0xad/frame = 0xfffffe026ea9acb0 ether_demux() at ether_demux+0x17a/frame 0xfffffe026ea9ace0 ether_nh_input() at ether_nh_input+0x39f/frame 0xfffffe026ea9ad30 netisr_dispatch_src() at netisr_dispatch_src+0xad/frame = 0xfffffe026ea9ad90 ether_input() at ether_input+0xd9/frame 0xfffffe026ea9adf0 epair_tx_start_deferred() at epair_tx_start_deferred+0xd7/frame = 0xfffffe026ea9ae40 taskqueue_run_locked() at taskqueue_run_locked+0xab/frame = 0xfffffe026ea9aec0 taskqueue_thread_loop() at taskqueue_thread_loop+0xd3/frame = 0xfffffe026ea9aef0 fork_exit() at fork_exit+0x82/frame 0xfffffe026ea9af30 fork_trampoline() at fork_trampoline+0xe/frame 0xfffffe026ea9af30 --- trap 0, rip =3D 0x35c39ec8bda0, rsp =3D 0, rbp =3D 0x35c39ec8ec90 --= - KDB: enter: panic That seems to be panicking during SCTP packet handling, so I do not = believe this to be a pf bug. Reverting both commits avoids the problem. It=E2=80=99s also failing during the CI tests: = https://ci.freebsd.org/view/Test/job/FreeBSD-main-amd64-test/23957/consol= e (To reproduce, kldload pf sctp ; cd /usr/tests/sys/netpfil/pf ; sudo = kyua test sctp:basic_v4). Best regards, Kristof --=_MailMate_77BDF089-8206-4FA3-8412-9857890C3CAE_= Content-Type: text/html; charset=UTF-8 Content-Transfer-Encoding: quoted-printable <!DOCTYPE html> <html> <head> <meta http-equiv=3D"Content-Type" content=3D"text/xhtml; charset=3Dutf-8"= > </head> <body><div style=3D"font-family: sans-serif;"><div class=3D"markdown" sty= le=3D"white-space: normal;"> <p dir=3D"auto">On 29 Jul 2023, at 0:03, Michael Tuexen wrote:</p> </div><div class=3D"plaintext" style=3D"white-space: normal;"><blockquote= style=3D"margin: 0 0 5px; padding-left: 5px; border-left: 2px solid #136= BCE; color: #136BCE;"><p dir=3D"auto">The branch main has been updated by= tuexen:</p> <p dir=3D"auto">URL: <a href=3D"https://cgit.FreeBSD.org/src/commit/?id=3D= b279e84a47ddb59e55b5a3cec31c51cd41bf0dc3">https://cgit.FreeBSD.org/src/co= mmit/?id=3Db279e84a47ddb59e55b5a3cec31c51cd41bf0dc3</a></p> <p dir=3D"auto">commit b279e84a47ddb59e55b5a3cec31c51cd41bf0dc3 <br> Author: Michael Tuexen <tuexen@FreeBSD.org> <br> AuthorDate: 2023-07-28 12:36:11 +0000 <br> Commit: Michael Tuexen <tuexen@FreeBSD.org> <br> CommitDate: 2023-07-28 12:36:11 +0000</p> <p dir=3D"auto"> sctp: improve consistency</p> <p dir=3D"auto"> This is simplifying a patch to address PR 260116.</p>= <p dir=3D"auto"> PR: 260116 <br> MFC after: 1 week</p> </blockquote></div> <div class=3D"markdown" style=3D"white-space: normal;"> <p dir=3D"auto">It looks like this commit (or maybe the next one, c620788= 150d274c09a070ab486602c98407d73b0) causes a panic in the SCTP code during= the <code style=3D"padding: 0 0.25em; background-color: #E4E4E4;">sys/ne= tpfil/pf/sctp:basic_v4</code> test:</p> <pre style=3D"margin-left: 15px; margin-right: 15px; padding: 5px; border= : thin solid gray; overflow-x: auto; max-width: 90vw; background-color: #= E4E4E4;"><code style=3D"padding: 0 0.25em; background-color: #E4E4E4;">pa= nic: Counter goes negative cpuid =3D 7 time =3D 1691145655 KDB: stack backtrace: db_trace_self_wrapper() at db_trace_self_wrapper+0x2b/frame 0xfffffe026ea= 9a3a0 vpanic() at vpanic+0x132/frame 0xfffffe026ea9a4d0 panic() at panic+0x43/frame 0xfffffe026ea9a530 sctp_abort_notification() at sctp_abort_notification/frame 0xfffffe026ea9= a540 sctp_express_handle_sack() at sctp_express_handle_sack+0x647/frame 0xffff= fe026ea9a640 sctp_process_control() at sctp_process_control+0xf62/frame 0xfffffe026ea9= a990 sctp_common_input_processing() at sctp_common_input_processing+0x561/fram= e 0xfffffe026ea9ab10 sctp_input_with_port() at sctp_input_with_port+0x1fa/frame 0xfffffe026ea9= abe0 sctp_input() at sctp_input+0x10/frame 0xfffffe026ea9abf0 ip_input() at ip_input+0x2ab/frame 0xfffffe026ea9ac50 netisr_dispatch_src() at netisr_dispatch_src+0xad/frame 0xfffffe026ea9acb= 0 ether_demux() at ether_demux+0x17a/frame 0xfffffe026ea9ace0 ether_nh_input() at ether_nh_input+0x39f/frame 0xfffffe026ea9ad30 netisr_dispatch_src() at netisr_dispatch_src+0xad/frame 0xfffffe026ea9ad9= 0 ether_input() at ether_input+0xd9/frame 0xfffffe026ea9adf0 epair_tx_start_deferred() at epair_tx_start_deferred+0xd7/frame 0xfffffe0= 26ea9ae40 taskqueue_run_locked() at taskqueue_run_locked+0xab/frame 0xfffffe026ea9a= ec0 taskqueue_thread_loop() at taskqueue_thread_loop+0xd3/frame 0xfffffe026ea= 9aef0 fork_exit() at fork_exit+0x82/frame 0xfffffe026ea9af30 fork_trampoline() at fork_trampoline+0xe/frame 0xfffffe026ea9af30 --- trap 0, rip =3D 0x35c39ec8bda0, rsp =3D 0, rbp =3D 0x35c39ec8ec90 ---= KDB: enter: panic </code></pre> <p dir=3D"auto">That seems to be panicking during SCTP packet handling, s= o I do not believe this to be a pf bug.</p> <p dir=3D"auto">Reverting both commits avoids the problem.</p> <p dir=3D"auto">It=E2=80=99s also failing during the CI tests: <a href=3D= "https://ci.freebsd.org/view/Test/job/FreeBSD-main-amd64-test/23957/conso= le">https://ci.freebsd.org/view/Test/job/FreeBSD-main-amd64-test/23957/co= nsole</a></p> <p dir=3D"auto">(To reproduce, kldload pf sctp ; cd /usr/tests/sys/netpfi= l/pf ; sudo kyua test sctp:basic_v4).</p> <p dir=3D"auto">Best regards,<br> Kristof</p> </div> </div> </body> </html> --=_MailMate_77BDF089-8206-4FA3-8412-9857890C3CAE_=--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?B3E69994-14C4-4339-BEEF-8BB89F218020>