Re: [PATCH for 8.1 v2 2/6] vdpa: add vhost_vdpa_reset_status_fd

2023-04-21 Thread Michael S. Tsirkin
On Tue, Mar 28, 2023 at 09:37:16AM +0200, Eugenio Perez Martin wrote: > > > diff --git a/hw/virtio/vhost-vdpa.c b/hw/virtio/vhost-vdpa.c > > > index bbabea18f3..7a2053b8d9 100644 > > > --- a/hw/virtio/vhost-vdpa.c > > > +++ b/hw/virtio/vhost-vdpa.c > > > @@ -335,38 +335,45 @@ static const

Re: [PATCH for 8.1 v2 2/6] vdpa: add vhost_vdpa_reset_status_fd

2023-03-28 Thread Eugenio Perez Martin
On Tue, Mar 28, 2023 at 8:53 AM Jason Wang wrote: > > On Fri, Mar 24, 2023 at 3:54 AM Eugenio Pérez wrote: > > > > The title needs some tweak, I think the more appropriate one is > "introduce vhost_vdpa_call_fd()" > We can do it otherwise, but the function exported in the

Re: [PATCH for 8.1 v2 2/6] vdpa: add vhost_vdpa_reset_status_fd

2023-03-28 Thread Jason Wang
On Fri, Mar 24, 2023 at 3:54 AM Eugenio Pérez wrote: > The title needs some tweak, I think the more appropriate one is "introduce vhost_vdpa_call_fd()" > This allows to reset a vhost-vdpa device from external subsystems like > vhost-net, since it does not have any struct vhost_dev by the time

Re: [PATCH for 8.1 v2 2/6] vdpa: add vhost_vdpa_reset_status_fd

2023-03-27 Thread Stefano Garzarella
On Thu, Mar 23, 2023 at 08:54:00PM +0100, Eugenio Pérez wrote: This allows to reset a vhost-vdpa device from external subsystems like vhost-net, since it does not have any struct vhost_dev by the time we need to use it. It is used in subsequent patches to negotiate features and probe for CVQ

[PATCH for 8.1 v2 2/6] vdpa: add vhost_vdpa_reset_status_fd

2023-03-23 Thread Eugenio Pérez
This allows to reset a vhost-vdpa device from external subsystems like vhost-net, since it does not have any struct vhost_dev by the time we need to use it. It is used in subsequent patches to negotiate features and probe for CVQ ASID isolation. Signed-off-by: Eugenio Pérez ---