mbox

[v1,00/14] vhost: clean-up and simplify async implementation

Message ID 20211018130229.308694-1-maxime.coquelin@redhat.com (mailing list archive)
Headers

Message

Maxime Coquelin Oct. 18, 2021, 1:02 p.m. UTC 
  This series aims at cleaning and simplifying the async
enqueue path. I think it makes the code easier to
understand, and is necessary before integrating new
changes.

I may have more reworks to propose in next revisions,
but I wanted to share my current status early so that
you have time to review/test it.

It is only compile-tested, as I don't have HW with IOAT
support to test it.

Changes in v1:
==============
- Fix missing iovec index increment (Jiayu)
- Remove unneeded sanity check on IOV iter index (Jiayu)
- Fix async->pkts_info allocation check (Jiayu)
- Prevent NULL pointer dereference on async metadata cleanup (Jiayu)

Maxime Coquelin (14):
  vhost: move async data in a dedicated structure
  vhost: hide inflight async structure
  vhost: simplify async IO vectors
  vhost: simplify async IO vectors iterators
  vhost: remove async batch threshold
  vhost: introduce specific iovec structure
  vhost: remove useless fields in async iterator struct
  vhost: improve IO vector logic
  vhost: remove notion of async descriptor
  vhost: simplify async enqueue completion
  vhost: simplify getting the first in-flight index
  vhost: prepare async for mbuf to desc refactoring
  vhost: prepare sync for mbuf to desc refactoring
  vhost: merge sync and async mbuf to desc filling

 examples/vhost/ioat.c       |  30 +-
 examples/vhost/ioat.h       |   2 +-
 lib/vhost/rte_vhost_async.h |  42 +--
 lib/vhost/vhost.c           | 130 +++----
 lib/vhost/vhost.h           |  67 ++--
 lib/vhost/vhost_user.c      |   4 +-
 lib/vhost/virtio_net.c      | 693 +++++++++++++-----------------------
 7 files changed, 376 insertions(+), 592 deletions(-)