]> CyberLeo.Net >> Repos - FreeBSD/FreeBSD.git/commit
[ath] [ar9300] ensure the software scheduler is called to form more aggregates for...
authoradrian <adrian@FreeBSD.org>
Mon, 23 Jan 2017 04:20:37 +0000 (04:20 +0000)
committeradrian <adrian@FreeBSD.org>
Mon, 23 Jan 2017 04:20:37 +0000 (04:20 +0000)
commita81c01253b93a81f32d928bedb359d29d8febef0
treeecd53596ad890dcea188f112818387a75d02f63a
parent0aa520541e7a23e3c91a4cb5cb0a84be132d0868
[ath] [ar9300] ensure the software scheduler is called to form more aggregates for EDMA chips

When investigating performance on UDP TX on the AR9380 I found that the
following sequence was occuring:

* INTR
* EINPROGRESS - nothing yet
* INTR
* TXSTATUS - process a TX completion for an aggregate
* INTR, INTR
* TXSTATUS - process a TX completion for an aggregate
* TXD, TXD ... populate frames from the hardware queue and submit

What should be happening is a completed TXSTATUS fires off more packets
that are queued on active TIDs.

What /was/ happening was after that first TXSTATUS the TX queue hardware queue
was still empty, so it didn't push anything into the FIFO.  Only after the
second TXSTATUS did any progress get made.

This is one of two commits - it ensures that the software TX queue scheduler
is called /after/ TX completion, otherwise no frames from the software staging
queues will be processed into the hardware queues.

The second commit will fix it so it populates aggregate frames correctly
when the above occurs - right now ath_txq_sched() is called, but it doesn't
populate anything because its pre-check conditions are wrong.

Whilst here, add/tweak debugging.

Tested:

* AR9380 STA (testing device) -> AR9580 hostap
sys/dev/ath/if_ath_tx_edma.c