]> CyberLeo.Net >> Repos - FreeBSD/FreeBSD.git/commit
cxgbe(4): major tx rework.
authornp <np@FreeBSD.org>
Wed, 31 Dec 2014 23:19:16 +0000 (23:19 +0000)
committernp <np@FreeBSD.org>
Wed, 31 Dec 2014 23:19:16 +0000 (23:19 +0000)
commitb2f095aaa6409529c4b5377e0d521e55e60a1d81
tree0986a3d93f9d693b0bfdcf10417d6eef4f8ce20f
parent06c1511222c2634e60b2b997582869fd01901440
cxgbe(4): major tx rework.

a) Front load as much work as possible in if_transmit, before any driver
lock or software queue has to get involved.

b) Replace buf_ring with a brand new mp_ring (multiproducer ring).  This
is specifically for the tx multiqueue model where one of the if_transmit
producer threads becomes the consumer and other producers carry on as
usual.  mp_ring is implemented as standalone code and it should be
possible to use it in any driver with tx multiqueue.  It also has:
- the ability to enqueue/dequeue multiple items.  This might become
  significant if packet batching is ever implemented.
- an abdication mechanism to allow a thread to give up writing tx
  descriptors and have another if_transmit thread take over.  A thread
  that's writing tx descriptors can end up doing so for an unbounded
  time period if a) there are other if_transmit threads continuously
  feeding the sofware queue, and b) the chip keeps up with whatever the
  thread is throwing at it.
- accurate statistics about interesting events even when the stats come
  at the expense of additional branches/conditional code.

The NIC txq lock is uncontested on the fast path at this point.  I've
left it there for synchronization with the control events (interface
up/down, modload/unload).

c) Add support for "type 1" coalescing work request in the normal NIC tx
path.  This work request is optimized for frames with a single item in
the DMA gather list.  These are very common when forwarding packets.
Note that netmap tx in cxgbe already uses these "type 1" work requests.

d) Do not request automatic cidx updates every 32 descriptors.  Instead,
request updates via bits in individual work requests (still every 32
descriptors approximately).  Also, request an automatic final update
when the queue idles after activity.  This means NIC tx reclaim is still
performed lazily but it will catch up quickly as soon as the queue
idles.  This seems to be the best middle ground and I'll probably do
something similar for netmap tx as well.

e) Implement a faster tx path for WRQs (used by TOE tx and control
queues, _not_ by the normal NIC tx).  Allow work requests to be written
directly to the hardware descriptor ring if room is available.  I will
convert t4_tom and iw_cxgbe modules to this faster style gradually.

MFC after: 2 months
sys/conf/files
sys/dev/cxgbe/adapter.h
sys/dev/cxgbe/t4_l2t.c
sys/dev/cxgbe/t4_main.c
sys/dev/cxgbe/t4_mp_ring.c [new file with mode: 0644]
sys/dev/cxgbe/t4_mp_ring.h [new file with mode: 0644]
sys/dev/cxgbe/t4_sge.c
sys/modules/cxgbe/if_cxgbe/Makefile