mbox series

[v4,0/4] optimize async data path

Message ID 20201013014546.2896162-1-patrick.fu@intel.com (mailing list archive)
Headers
Series optimize async data path |

Message

Patrick Fu Oct. 13, 2020, 1:45 a.m. UTC 
  This series applies optimization and fixes to the vhost
async data path.

v4:
 - fix transfer error handling in async submit function (patch 1/4)
 - add spinlock in async register function (patch 4/4)
 - no changes in patch 2/4 & 3/4

v3:
 - fix a typo in vhost error log (checkpatch warning)
 - fix travis-robot ci build warning on aarch64

v2:
 - minor rewordings on commit message
 - minor fix on poll_enenque_completion to correct a packet
   number calculation issue
 - allocate async buffer memory on the same numa with vq
 - add some comments in data path to improve readability

Patrick Fu (4):
  vhost: simplify async copy completion
  vhost: dynamically allocate async memory
  vhost: fix async vector buffer overrun
  vhost: fix async unregister deadlock

 lib/librte_vhost/rte_vhost_async.h |  15 ++-
 lib/librte_vhost/vhost.c           |  80 +++++++++-----
 lib/librte_vhost/vhost.h           |  14 +--
 lib/librte_vhost/vhost_user.c      |  10 +-
 lib/librte_vhost/virtio_net.c      | 162 +++++++++++++----------------
 5 files changed, 148 insertions(+), 133 deletions(-)

Comments

Maxime Coquelin Oct. 15, 2020, 3:40 p.m. UTC | #1 
On 10/13/20 3:45 AM, Patrick Fu wrote:
> This series applies optimization and fixes to the vhost
> async data path.
> 
> v4:
>  - fix transfer error handling in async submit function (patch 1/4)
>  - add spinlock in async register function (patch 4/4)
>  - no changes in patch 2/4 & 3/4
> 
> v3:
>  - fix a typo in vhost error log (checkpatch warning)
>  - fix travis-robot ci build warning on aarch64
> 
> v2:
>  - minor rewordings on commit message
>  - minor fix on poll_enenque_completion to correct a packet
>    number calculation issue
>  - allocate async buffer memory on the same numa with vq
>  - add some comments in data path to improve readability
> 
> Patrick Fu (4):
>   vhost: simplify async copy completion
>   vhost: dynamically allocate async memory
>   vhost: fix async vector buffer overrun
>   vhost: fix async unregister deadlock
> 
>  lib/librte_vhost/rte_vhost_async.h |  15 ++-
>  lib/librte_vhost/vhost.c           |  80 +++++++++-----
>  lib/librte_vhost/vhost.h           |  14 +--
>  lib/librte_vhost/vhost_user.c      |  10 +-
>  lib/librte_vhost/virtio_net.c      | 162 +++++++++++++----------------
>  5 files changed, 148 insertions(+), 133 deletions(-)
> 

Applied to dpdk-next-virtio/main

Thanks,
Maxime