Differential D58282

iflib: remove the unused TX queue state machine
ClosedPublic
Actions

Authored by netchild on Jul 16 2026, 11:11 AM.

Tags

None

Referenced Files

	F170854436: D58282.diff
	Mon, Sep 7, 1:44 AM

	F170827857: D58282.id183392.diff
	Sun, Sep 6, 9:22 PM

	Unknown Object (File)
	Sun, Sep 6, 6:38 AM

	Unknown Object (File)
	Sat, Sep 5, 5:24 PM

	Unknown Object (File)
	Sat, Sep 5, 5:24 PM

	Unknown Object (File)
	Sat, Sep 5, 5:13 PM

	Unknown Object (File)
	Sat, Sep 5, 4:58 AM

	Unknown Object (File)
	Sat, Sep 5, 1:40 AM

View All 71 Files

Subscribers

Details

Reviewers

gallatin
kbowling
markj
erj
shurd

Group Reviewers

Commits

rG5da9bc88936a: iflib: remove the unused TX queue state machine

Summary

The previous commit stopped using ift_qstatus and the IFLIB_QUEUE_*
states for the TX watchdog decision, leaving only dead stores.
Remove the field, the states, and all assignments. The byte the
field frees stays behind as explicit padding. No functional change.

MFC after: 1 month
Assisted-by: Claude Code (Fable 5, Opus 5)

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

netchild created this revision.Jul 16 2026, 11:11 AM

Herald added a reviewer: shurd. · View Herald TranscriptJul 16 2026, 11:11 AM

Herald added 1 blocking reviewer(s): iflib. · View Herald Transcript

Herald added subscribers: glebius, melifaro, ae, imp. · View Herald Transcript

netchild requested review of this revision.Jul 16 2026, 11:11 AM

Harbormaster completed remote builds in B74848: Diff 182076.Jul 16 2026, 11:11 AM

netchild mentioned this in D58266: iflib: restore TX watchdog functionality.Jul 19 2026, 3:05 PM

markj accepted this revision.Jul 29 2026, 2:39 PM

Make the padding explicit.

gallatin accepted this revision.Aug 3 2026, 6:07 PM

This revision is now accepted and ready to land.Aug 3 2026, 6:07 PM

Closed by commit rG5da9bc88936a: iflib: remove the unused TX queue state machine (authored by netchild). · Explain WhyAug 4 2026, 5:15 PM

This revision was automatically updated to reflect the committed changes.

netchild added a commit: rG5da9bc88936a: iflib: remove the unused TX queue state machine.

Revision Contents
Changeset List

Path

Size

sys/

net/

21 lines

Diff 183392

sys/net/iflib.c

Loading...