HomeFreeBSD

[ath] [ar9300] ensure the software scheduler is called to form more aggregates…

Description

[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

Details

Provenance
adrianAuthored on
Parents
rS312659: Avoid using non-zero argument for __builtin_frame_address().
Branches
Unknown
Tags
Unknown