Paths

Table of Contentst

Differential D56609

tests: Add a test for races between between if_detach() and vnet_if_return()
ClosedPublic
Actions

Authored by zlei on Apr 23 2026, 8:58 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, Jun 8, 7:08 AM

	Unknown Object (File)
	Thu, Jun 4, 1:59 PM

	Unknown Object (File)
	Thu, Jun 4, 11:31 AM

	Unknown Object (File)
	Fri, May 29, 8:16 PM

	Unknown Object (File)
	Tue, May 26, 4:23 AM

	Unknown Object (File)
	Tue, May 26, 4:13 AM

	Unknown Object (File)
	Mon, May 25, 3:32 PM

	Unknown Object (File)
	Mon, May 25, 3:24 PM

View All 31 Files

Subscribers

Details

Reviewers

Group Reviewers

Commits

rG33da0a256bfc: tests/netgraph: Add a test for races between if_detach() and vnet_if_return()
rG3a4c4d61ac2c: tests/netgraph: Add a test for races between if_detach() and vnet_if_return()
rG0988abd52352: tests/netgraph: Add a test for races between if_detach() and vnet_if_return()

Summary

ng_eiface(4) or physical interfaces does not involve cloner hence the
detaching is a bit different with epair(4). Add more tests to cover
that.

PR: 292993
MFC after: 5 days

Test Plan

Run against stable/15 and stable/14, with / without D56374 .

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

zlei created this revision.Apr 23 2026, 8:58 PM

Herald added subscribers: glebius, melifaro, donner, imp. · View Herald TranscriptApr 23 2026, 8:58 PM

zlei requested review of this revision.Apr 23 2026, 8:58 PM

zlei mentioned this in D56374: ifnet: if_detach(): Fix races with if_vmove().Apr 23 2026, 9:04 PM

zlei added inline comments.Apr 23 2026, 9:13 PM

tests/sys/netgraph/eiface_test.sh
33	`tests/ci/tools/ci.conf` exists on CURRENT and stable/15 to load the required modules, but not on stable/14. Shall load required modules in the test case ?

This revision was not accepted when it landed; it landed in state Needs Review.Apr 27 2026, 7:21 PM

Closed by commit rG0988abd52352: tests/netgraph: Add a test for races between if_detach() and vnet_if_return() (authored by zlei). · Explain Why

This revision was automatically updated to reflect the committed changes.

zlei added a commit: rG0988abd52352: tests/netgraph: Add a test for races between if_detach() and vnet_if_return().

zlei added inline comments.Apr 27 2026, 7:26 PM

tests/sys/netgraph/eiface_test.sh
33	`tests/ci/tools/ci.conf` exists on CURRENT and stable/15 to load the required modules, but not on stable/14. Shall load required modules in the test case ? Indeed `ngctl(8)` and `ng_socket(4)` will load required modules on demand. No need to bother `tests/ci/tools/ci.conf` for this testcase.

zlei added a commit: rG3a4c4d61ac2c: tests/netgraph: Add a test for races between if_detach() and vnet_if_return().Apr 30 2026, 9:54 AM

zlei added a commit: rG33da0a256bfc: tests/netgraph: Add a test for races between if_detach() and vnet_if_return().May 4 2026, 4:51 PM

Revision Contents
Changeset List

Path

Size

tests/

sys/

netgraph/

2 lines

67 lines

Diff 176665

tests/sys/netgraph/Makefile

Loading...

tests/sys/netgraph/eiface_test.sh

Loading...