diff options
author | John Baldwin <jhb@FreeBSD.org> | 2024-01-09 18:59:48 +0000 |
---|---|---|
committer | John Baldwin <jhb@FreeBSD.org> | 2024-01-09 18:59:48 +0000 |
commit | 6977311633c73f594ff3dae150d6d1fe06105a8f (patch) | |
tree | 945fb83aa68ed3013e84de3ba7959c1d04716c55 | |
parent | 6631e2f9b49e08f53c7beb560ee8509c343b3927 (diff) | |
download | src-6977311633c73f594ff3dae150d6d1fe06105a8f.tar.gz src-6977311633c73f594ff3dae150d6d1fe06105a8f.zip |
net80211 amdpu: Simplify a few loops that drain an mbufq
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.
Reviewed by: bz
Sponsored by: Chelsio Communications
Differential Revision: https://reviews.freebsd.org/D43336
-rw-r--r-- | sys/net80211/ieee80211_ht.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sys/net80211/ieee80211_ht.c b/sys/net80211/ieee80211_ht.c index ea59bb622dd8..5297b741d9b9 100644 --- a/sys/net80211/ieee80211_ht.c +++ b/sys/net80211/ieee80211_ht.c @@ -517,7 +517,7 @@ ampdu_rx_purge_slot(struct ieee80211_rx_ampdu *rap, int i) struct mbuf *m; /* Walk the queue, removing frames as appropriate */ - while (mbufq_len(&rap->rxa_mq[i]) != 0) { + for (;;) { m = mbufq_dequeue(&rap->rxa_mq[i]); if (m == NULL) break; @@ -812,7 +812,7 @@ ampdu_dispatch_slot(struct ieee80211_rx_ampdu *rap, struct ieee80211_node *ni, struct mbuf *m; int n = 0; - while (mbufq_len(&rap->rxa_mq[i]) != 0) { + for (;;) { m = mbufq_dequeue(&rap->rxa_mq[i]); if (m == NULL) break; |