Page MenuHomeFreeBSD

ath: use the new net80211 methods for AMPDU density/limit, short-GI
ClosedPublic

Authored by adrian on Nov 25 2024, 11:03 PM.
Referenced Files
Unknown Object (File)
Thu, Dec 26, 9:17 AM
Unknown Object (File)
Thu, Dec 26, 8:27 AM
Unknown Object (File)
Thu, Dec 26, 3:05 AM
Unknown Object (File)
Wed, Dec 25, 9:48 PM
Unknown Object (File)
Wed, Dec 25, 5:01 PM
Unknown Object (File)
Wed, Dec 25, 10:26 AM
Unknown Object (File)
Wed, Dec 25, 8:37 AM
Unknown Object (File)
Fri, Dec 20, 5:25 PM
Subscribers

Details

Summary

Now that net80211 has this code, just leverage it instead of rolling
our own.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

vap is used in the debug statement, so (void) it so it is "used" during
non-debug builds.

emaste added inline comments.
sys/dev/ath/if_ath_tx_ht.c
567

Or add __diagused to the variable declaration

bz requested changes to this revision.Nov 28 2024, 12:52 AM
bz added a subscriber: bz.
bz added inline comments.
sys/dev/ath/if_ath_tx_ht.c
567

@emaste Will that work with the assignment on it?
Alternatively add the needed #ifdef or simply replace vap in the DPRINTF with ni->ni_vap and not have any of these problems.

This revision now requires changes to proceed.Nov 28 2024, 12:52 AM
sys/dev/ath/if_ath_tx_ht.c
567

We do have some cases like that, e.g.

sys/net/route/route_ctl.c:      struct nhop_object __diagused *nh_src = rnd_src->rnd_nhop;
sys/opencrypto/criov.c: int iol __diagused = uio->uio_iovcnt;
sys/opencrypto/criov.c: int iol __diagused = uio->uio_iovcnt;
sys/opencrypto/criov.c: int iol __diagused = uio->uio_iovcnt;
sys/vm/vm_glue.c:       size_t kpages __diagused = kstack_pages + KSTACK_GUARD_PAGES;
sys/vm/vm_object.c:     vm_object_t object __diagused = objectv;

but indeed just using ni->ni_vap->iv_ampdu_limit would be easiest

This revision is now accepted and ready to land.Dec 2 2024, 7:20 AM