Re: [PATCH v2] vhost: add runtime locking check in unsafe APIs

2022-05-17 Thread Maxime Coquelin
On 5/11/22 08:58, Maxime Coquelin wrote: This patch adds runtime checks in unsafe Vhost async APIs, to ensure the access lock is taken. The detection won't work every time, as another thread could take the lock, but it would help to detect misuse of these unsafe API. Signed-off-by: Maxime Co

RE: [PATCH v2] vhost: add runtime locking check in unsafe APIs

2022-05-11 Thread Xia, Chenbo
> -Original Message- > From: Maxime Coquelin > Sent: Wednesday, May 11, 2022 2:59 PM > To: dev@dpdk.org; Xia, Chenbo ; Ding, Xuan > ; Hu, Jiayu ; Jiang, Cheng1 > ; Pai G, Sunil ; > david.march...@redhat.com > Cc: Maxime Coquelin > Subject: [PATCH v2] vhost: add runtime locking check in un

RE: [PATCH v2] vhost: add runtime locking check in unsafe APIs

2022-05-11 Thread Hu, Jiayu
> -Original Message- > From: David Marchand > Sent: Wednesday, May 11, 2022 3:35 PM > To: Maxime Coquelin > Cc: dev ; Xia, Chenbo ; Ding, Xuan > ; Hu, Jiayu ; Jiang, Cheng1 > ; Pai G, Sunil > Subject: Re: [PATCH v2] vhost: add runtime locking check in unsafe

Re: [PATCH v2] vhost: add runtime locking check in unsafe APIs

2022-05-11 Thread David Marchand
On Wed, May 11, 2022 at 8:58 AM Maxime Coquelin wrote: > > This patch adds runtime checks in unsafe Vhost async APIs, > to ensure the access lock is taken. > > The detection won't work every time, as another thread > could take the lock, but it would help to detect misuse > of these unsafe API. >