mbox series

[v2,0/4] vhost: remove dequeue zero-copy support

Message ID 20200928091712.7946-1-maxime.coquelin@redhat.com (mailing list archive)
Headers show
Series vhost: remove dequeue zero-copy support | expand

Message

Maxime Coquelin Sept. 28, 2020, 9:17 a.m. UTC
As announced in DPDK v20.08, this series removes 
dequeue zero-copy support to Vhost library.

Support for the feature is also being removed in OVS.

Changes in v2:
--------------
* Fix build issue due to unused symbol (Chenbo)
* Remove remaining zmbuf related struct definition (Chenbo)
* Fix typo in commit message (Chenbo)
* Remove helper string in Vhost PMD (Chenbo)

Maxime Coquelin (4):
  net/vhost: remove dequeue zero-copy support
  examples/vhost_crypto: use vhost async-copy flag
  examples/vhost: remove dequeue zero-copy support
  vhost: remove dequeue zero-copy support

 doc/guides/prog_guide/vhost_lib.rst |  52 +----
 drivers/net/vhost/rte_eth_vhost.c   |  14 --
 examples/vhost/main.c               |  18 +-
 examples/vhost_crypto/main.c        |   2 +-
 lib/librte_vhost/rte_vhost.h        |   2 +-
 lib/librte_vhost/socket.c           |  47 ----
 lib/librte_vhost/vhost.c            |  14 --
 lib/librte_vhost/vhost.h            |  28 ---
 lib/librte_vhost/vhost_user.c       |  80 +------
 lib/librte_vhost/virtio_net.c       | 326 +++-------------------------
 10 files changed, 35 insertions(+), 548 deletions(-)

Comments

Maxime Coquelin Sept. 30, 2020, 4:21 p.m. UTC | #1
On 9/28/20 11:17 AM, Maxime Coquelin wrote:
> As announced in DPDK v20.08, this series removes 
> dequeue zero-copy support to Vhost library.
> 
> Support for the feature is also being removed in OVS.
> 
> Changes in v2:
> --------------
> * Fix build issue due to unused symbol (Chenbo)
> * Remove remaining zmbuf related struct definition (Chenbo)
> * Fix typo in commit message (Chenbo)
> * Remove helper string in Vhost PMD (Chenbo)
> 
> Maxime Coquelin (4):
>   net/vhost: remove dequeue zero-copy support
>   examples/vhost_crypto: use vhost async-copy flag
>   examples/vhost: remove dequeue zero-copy support
>   vhost: remove dequeue zero-copy support
> 
>  doc/guides/prog_guide/vhost_lib.rst |  52 +----
>  drivers/net/vhost/rte_eth_vhost.c   |  14 --
>  examples/vhost/main.c               |  18 +-
>  examples/vhost_crypto/main.c        |   2 +-
>  lib/librte_vhost/rte_vhost.h        |   2 +-
>  lib/librte_vhost/socket.c           |  47 ----
>  lib/librte_vhost/vhost.c            |  14 --
>  lib/librte_vhost/vhost.h            |  28 ---
>  lib/librte_vhost/vhost_user.c       |  80 +------
>  lib/librte_vhost/virtio_net.c       | 326 +++-------------------------
>  10 files changed, 35 insertions(+), 548 deletions(-)
> 


Applied to dpdk-next-virtio/main.

Thanks,
Maxime
Ferruh Yigit Sept. 30, 2020, 8:56 p.m. UTC | #2
On 9/28/2020 10:17 AM, Maxime Coquelin wrote:
> As announced in DPDK v20.08, this series removes
> dequeue zero-copy support to Vhost library.
> 
> Support for the feature is also being removed in OVS.
> 
> Changes in v2:
> --------------
> * Fix build issue due to unused symbol (Chenbo)
> * Remove remaining zmbuf related struct definition (Chenbo)
> * Fix typo in commit message (Chenbo)
> * Remove helper string in Vhost PMD (Chenbo)
> 
> Maxime Coquelin (4):
>    net/vhost: remove dequeue zero-copy support
>    examples/vhost_crypto: use vhost async-copy flag
>    examples/vhost: remove dequeue zero-copy support
>    vhost: remove dequeue zero-copy support
> 
>   doc/guides/prog_guide/vhost_lib.rst |  52 +----
>   drivers/net/vhost/rte_eth_vhost.c   |  14 --
>   examples/vhost/main.c               |  18 +-
>   examples/vhost_crypto/main.c        |   2 +-
>   lib/librte_vhost/rte_vhost.h        |   2 +-
>   lib/librte_vhost/socket.c           |  47 ----
>   lib/librte_vhost/vhost.c            |  14 --
>   lib/librte_vhost/vhost.h            |  28 ---
>   lib/librte_vhost/vhost_user.c       |  80 +------
>   lib/librte_vhost/virtio_net.c       | 326 +++-------------------------
>   10 files changed, 35 insertions(+), 548 deletions(-)
> 

Should release notes updated for this removal?