Re: [PATCH v4] examples/vhost: fix retry logic on eth rx path

2022-07-01 Thread Maxime Coquelin
On 6/22/22 11:25, Yuan Wang wrote: drain_eth_rx() uses rte_vhost_avail_entries() to calculate the available entries to determine if a retry is required. However, this function only works with split rings, and calculating packed rings will return the wrong value and cause unnecessary retries re

RE: [PATCH v4] examples/vhost: fix retry logic on eth rx path

2022-06-23 Thread Ling, WeiX
[PATCH v4] examples/vhost: fix retry logic on eth rx path > > drain_eth_rx() uses rte_vhost_avail_entries() to calculate the available > entries to determine if a retry is required. > However, this function only works with split rings, and calculating packed > rings will return

RE: [PATCH v4] examples/vhost: fix retry logic on eth rx path

2022-06-22 Thread Xia, Chenbo
> -Original Message- > From: Wang, YuanX > Sent: Wednesday, June 22, 2022 5:26 PM > To: maxime.coque...@redhat.com; Xia, Chenbo ; > dev@dpdk.org > Cc: Hu, Jiayu ; He, Xingguang ; > Jiang, Cheng1 ; Wang, YuanX ; > sta...@dpdk.org; Ling, WeiX > Subject: [PATCH v4

[PATCH v4] examples/vhost: fix retry logic on eth rx path

2022-06-22 Thread Yuan Wang
drain_eth_rx() uses rte_vhost_avail_entries() to calculate the available entries to determine if a retry is required. However, this function only works with split rings, and calculating packed rings will return the wrong value and cause unnecessary retries resulting in a significant performance pen