Paths

Table of Contentst

Differential D47175

pf tests: Lower chances of false positives for mbuf:inet6_in_mbuf_len
ClosedPublic
Actions

Authored by igoro on Oct 18 2024, 2:02 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, Jun 25, 6:18 PM

	Unknown Object (File)
	Mon, Jun 15, 1:34 AM

	Unknown Object (File)
	Wed, May 27, 10:38 PM

	Unknown Object (File)
	May 8 2026, 1:29 AM

	Unknown Object (File)
	May 6 2026, 7:47 PM

	Unknown Object (File)
	May 6 2026, 11:08 AM

	Unknown Object (File)
	May 3 2026, 11:50 PM

	Unknown Object (File)
	Apr 30 2026, 3:01 AM

View All 81 Files

Subscribers

vegeta_tuxpowered.net

Details

Reviewers

kp
markj

Commits

rG8829d4cb09d3: pf tests: Lower chances of false positives for mbuf:inet6_in_mbuf_len

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 60065
Build 56949: arc lint + arc unit

Event Timeline

igoro created this revision.Oct 18 2024, 2:02 PM

Herald added subscribers: vegeta_tuxpowered.net, glebius, melifaro and 2 others. · View Herald TranscriptOct 18 2024, 2:02 PM

igoro requested review of this revision.Oct 18 2024, 2:02 PM

Harbormaster completed remote builds in B60065: Diff 145107.Oct 18 2024, 2:02 PM

This is the outcome of my analysis of the following CI test failure: https://ci.freebsd.org/view/Test/job/FreeBSD-main-amd64-test/25598/testReport/junit/sys.netpfil.pf/mbuf/inet6_in_mbuf_len/.
An extra 2001:db8::1 → 2001:db8::2 ICMPv6 Neighbor Advertisement 2001:db8::1 (sol) packet can appear. If it's sent after the net.dummymbuf.hits value is compared then it does not impact the results, but it may happen before.

I would have guessed it'd be MLD packets, but neighbour unreachability detection could also be causing this.

Approved.

This revision is now accepted and ready to land.Oct 18 2024, 2:18 PM

Closed by commit rG8829d4cb09d3: pf tests: Lower chances of false positives for mbuf:inet6_in_mbuf_len (authored by igoro). · Explain WhyOct 18 2024, 4:26 PM

This revision was automatically updated to reflect the committed changes.

igoro added a commit: rG8829d4cb09d3: pf tests: Lower chances of false positives for mbuf:inet6_in_mbuf_len.

Revision Contents
Changeset List

Path

Size

tests/

sys/

netpfil/

pf/

5 lines

Diff 145107

tests/sys/netpfil/pf/mbuf.sh

Loading...