Differential D43336

net80211 amdpu: Simplify a few loops that drain an mbufq
ClosedPublic
Actions

Authored by jhb on Jan 5 2024, 9:34 PM.

Tags

None

Referenced Files

	F109363316: D43336.id132515.diff
	Tue, Feb 4, 2:20 AM

	Unknown Object (File)
	Nov 16 2024, 1:57 PM

	Unknown Object (File)
	Nov 16 2024, 1:51 PM

	Unknown Object (File)
	Nov 16 2024, 11:44 AM

	Unknown Object (File)
	Nov 4 2024, 6:55 AM

	Unknown Object (File)
	Oct 22 2024, 10:15 AM

	Unknown Object (File)
	Oct 22 2024, 10:14 AM

	Unknown Object (File)
	Oct 22 2024, 10:14 AM

View All 35 Files

Subscribers

Details

Reviewers

bz
markj

Commits

rG2bc7fdc3a8d0: net80211 amdpu: Simplify a few loops that drain an mbufq
rG18df65561983: net80211 amdpu: Simplify a few loops that drain an mbufq
rG6977311633c7: net80211 amdpu: Simplify a few loops that drain an mbufq

Summary

These loops already handled a NULL return from mbufq_dequeue when the
queue was empty, so remove a redundant check of mbufq_len before
dequeueing.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

jhb created this revision.Jan 5 2024, 9:34 PM

Herald added subscribers: glebius, melifaro, imp. · View Herald TranscriptJan 5 2024, 9:34 PM

jhb requested review of this revision.Jan 5 2024, 9:34 PM

jhb added a child revision: D43337: mbuf: Add mbufq_empty.

Harbormaster completed remote builds in B55264: Diff 132350.Jan 5 2024, 9:34 PM

LGTM

This revision is now accepted and ready to land.Jan 6 2024, 4:10 PM

Closed by commit rG6977311633c7: net80211 amdpu: Simplify a few loops that drain an mbufq (authored by jhb). · Explain WhyJan 9 2024, 7:04 PM

This revision was automatically updated to reflect the committed changes.

jhb added a commit: rG6977311633c7: net80211 amdpu: Simplify a few loops that drain an mbufq.

jhb added a commit: rG18df65561983: net80211 amdpu: Simplify a few loops that drain an mbufq.Jan 18 2024, 10:40 PM

jhb added a commit: rG2bc7fdc3a8d0: net80211 amdpu: Simplify a few loops that drain an mbufq.

Revision Contents
Changeset List

Path

Size

sys/

net80211/

4 lines

Diff 132515

sys/net80211/ieee80211_ht.c

Loading...