d/freebsd-skq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Flip on fast frames support for AR5416 and AR9300 series NICs.

Browse Source 
This was off because the net80211 aggregation code was using the same
state pointers for both fast frames and ampdu tx support which led to some
pretty unfortunate panic-y behaviour.

Now that net80211 doesn't panic, let's flip this back on.

It doesn't (yet) do the horrific sounding thing of A-MPDU aggregates
of fast frames; that'll come next.  It's a pre-requisite to supporting
AMSDU + AMPDU anyway, which actually speeds things up quite considerably
(think packing lots of little ACK frames into a single AMSDU.)

Tested:

* QCA955x SoC, AP mode
* AR5416, STA mode
* AR9170, STA mode (with local fast frame patches)
This commit is contained in:
Adrian Chadd 2015-10-10 00:13:45 +00:00
parent 4006520bea
commit dc809fc15f
2 changed files with 2 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
sys/contrib/dev/ath/ath_hal/ar9300/ar9300_attach.c
View File

@ -2671,7 +2671,7 @@ ar9300_fill_capability_info(struct ath_hal *ah)
p_cap->halBurstSupport = AH_TRUE;
p_cap->halChapTuningSupport = AH_TRUE;
p_cap->halTurboPrimeSupport = AH_TRUE;
p_cap->halFastFramesSupport = AH_FALSE;
p_cap->halFastFramesSupport = AH_TRUE;
p_cap->halTurboGSupport = p_cap->halWirelessModes & HAL_MODE_108G;

8
sys/dev/ath/ath_hal/ar5416/ar5416_attach.c
View File

@ -937,13 +937,7 @@ ar5416FillCapabilityInfo(struct ath_hal *ah)
pCap->halCompressSupport = AH_FALSE;
pCap->halBurstSupport = AH_TRUE;
/*
* This is disabled for now; the net80211 layer needs to be
* taught when it is and isn't appropriate to enable FF processing
* with 802.11n NICs (it tries to enable both A-MPDU and
* fast frames, with very tragic crash-y results.)
*/
pCap->halFastFramesSupport = AH_FALSE;
pCap->halFastFramesSupport = AH_TRUE;
pCap->halChapTuningSupport = AH_TRUE;
pCap->halTurboPrimeSupport = AH_TRUE;