RE: [PATCH v3 1/4] vhost: change vhost_virtqueue access lock to a read/write one

2023-05-31 Thread Xia, Chenbo
> -Original Message- > From: Eelco Chaudron > Sent: Wednesday, May 31, 2023 7:14 PM > To: Maxime Coquelin > Cc: Xia, Chenbo ; david.march...@redhat.com; > dev@dpdk.org > Subject: Re: [PATCH v3 1/4] vhost: change vhost_virtqueue access lock to a > read/write one &

Re: [PATCH v3 1/4] vhost: change vhost_virtqueue access lock to a read/write one

2023-05-31 Thread Eelco Chaudron
Chenbo ; >>> david.march...@redhat.com >>> Cc: dev@dpdk.org >>> Subject: [PATCH v3 1/4] vhost: change vhost_virtqueue access lock to a >>> read/write one >>> >>> This change will allow the vhost interrupt datapath handling to be split >

Re: [PATCH v3 1/4] vhost: change vhost_virtqueue access lock to a read/write one

2023-05-31 Thread Maxime Coquelin
On 5/31/23 08:37, Xia, Chenbo wrote: Hi Eelco, -Original Message- From: Eelco Chaudron Sent: Wednesday, May 17, 2023 5:09 PM To: maxime.coque...@redhat.com; Xia, Chenbo ; david.march...@redhat.com Cc: dev@dpdk.org Subject: [PATCH v3 1/4] vhost: change vhost_virtqueue access lock to

RE: [PATCH v3 1/4] vhost: change vhost_virtqueue access lock to a read/write one

2023-05-30 Thread Xia, Chenbo
Hi Eelco, > -Original Message- > From: Eelco Chaudron > Sent: Wednesday, May 17, 2023 5:09 PM > To: maxime.coque...@redhat.com; Xia, Chenbo ; > david.march...@redhat.com > Cc: dev@dpdk.org > Subject: [PATCH v3 1/4] vhost: change vhost_virtqueue access lock to a >

Re: [PATCH v3 1/4] vhost: change vhost_virtqueue access lock to a read/write one

2023-05-18 Thread Eelco Chaudron
On 17 May 2023, at 19:33, Maxime Coquelin wrote: > Hi Eelco, > > On 5/17/23 11:08, Eelco Chaudron wrote: >> This change will allow the vhost interrupt datapath handling to be split >> between two processed without one of them holding an explicit lock. >> > > As I had a tuned PVP benchmarking se

Re: [PATCH v3 1/4] vhost: change vhost_virtqueue access lock to a read/write one

2023-05-17 Thread Maxime Coquelin
Hi Eelco, On 5/17/23 11:08, Eelco Chaudron wrote: This change will allow the vhost interrupt datapath handling to be split between two processed without one of them holding an explicit lock. As I had a tuned PVP benchmarking setup at hand, I ran a 0.02% loss RFC2544 test with and without this

[PATCH v3 1/4] vhost: change vhost_virtqueue access lock to a read/write one

2023-05-17 Thread Eelco Chaudron
This change will allow the vhost interrupt datapath handling to be split between two processed without one of them holding an explicit lock. Signed-off-by: Eelco Chaudron --- lib/eal/include/generic/rte_rwlock.h | 17 ++ lib/vhost/vhost.c| 46 + lib/vh