On 6/18/20 6:28 PM, Matan Azrad wrote: > No need to take access lock in the vhost-user massage handler when s/massage/message/ > vDPA driver controls all the data-path of the vhost device. > > It allows the vDPA set_vring_state operation callback to configure > guest notifications. > > Signed-off-by: Matan Azrad <ma...@mellanox.com> > --- > lib/librte_vhost/vhost_user.c | 6 ++++-- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/lib/librte_vhost/vhost_user.c b/lib/librte_vhost/vhost_user.c > index cddfa4b..b0849b9 100644 > --- a/lib/librte_vhost/vhost_user.c > +++ b/lib/librte_vhost/vhost_user.c > @@ -2699,8 +2699,10 @@ typedef int (*vhost_message_handler_t)(struct > virtio_net **pdev, > case VHOST_USER_SEND_RARP: > case VHOST_USER_NET_SET_MTU: > case VHOST_USER_SET_SLAVE_REQ_FD: > - vhost_user_lock_all_queue_pairs(dev); > - unlock_required = 1; > + if (!(dev->flags & VIRTIO_DEV_VDPA_CONFIGURED)) { > + vhost_user_lock_all_queue_pairs(dev); > + unlock_required = 1; > + } > break; > default: > break; > Makes sense: Reviewed-by: Maxime Coquelin <maxime.coque...@redhat.com> Thanks, Maxime
- Re: [dpdk-dev] [PATCH v1 3/4] vh... Maxime Coquelin
- Re: [dpdk-dev] [PATCH v1 3/4] vh... Maxime Coquelin
- Re: [dpdk-dev] [PATCH v1 3/4] vh... Matan Azrad
- Re: [dpdk-dev] [PATCH v1 3/4] vh... Maxime Coquelin
- Re: [dpdk-dev] [PATCH v1 3/4] vh... Matan Azrad
- Re: [dpdk-dev] [PATCH v1 3/4] vh... Maxime Coquelin
- Re: [dpdk-dev] [PATCH v1 3/4] vh... Matan Azrad
- Re: [dpdk-dev] [PATCH v1 3/4] vh... Maxime Coquelin
- [dpdk-dev] [PATCH v1 4/4] vdpa/mlx5: support queue update Matan Azrad
- [dpdk-dev] [PATCH v1 2/4] vhost: skip access lock when vDP... Matan Azrad
- Re: [dpdk-dev] [PATCH v1 2/4] vhost: skip access lock... Maxime Coquelin
- [dpdk-dev] [PATCH v1 1/4] vhost: support host notifier que... Matan Azrad
- Re: [dpdk-dev] [PATCH v1 1/4] vhost: support host not... Maxime Coquelin
- Re: [dpdk-dev] [PATCH v1 1/4] vhost: support host... Matan Azrad
- Re: [dpdk-dev] [PATCH v1 1/4] vhost: support ... Maxime Coquelin
- Re: [dpdk-dev] [PATCH v1 1/4] vhost: supp... Matan Azrad
- Re: [dpdk-dev] [PATCH v1 1/4] vhost:... Maxime Coquelin
- [dpdk-dev] [PATCH v2 0/5] vhost: improve ready state Matan Azrad
- [dpdk-dev] [PATCH v2 2/5] vhost: improve device readi... Matan Azrad
- Re: [dpdk-dev] [PATCH v2 2/5] vhost: improve devi... Maxime Coquelin
- Re: [dpdk-dev] [PATCH v2 2/5] vhost: improve devi... Xia, Chenbo