Re: [PATCH v2] vhost-user: send SET_STATUS 0 after GET_VRING_BASE

2023-05-05 Thread Eugenio Perez Martin
On Tue, May 2, 2023 at 1:42 PM Stefan Hajnoczi wrote: > > Setting the VIRTIO Device Status Field to 0 resets the device. The > device's state is lost, including the vring configuration. > > vhost-user.c currently sends SET_STATUS 0 before GET_VRING_BASE. This > risks confusion about the lifetime o

Re: [PATCH v2] vhost-user: send SET_STATUS 0 after GET_VRING_BASE

2023-05-03 Thread Yajun Wu
On 5/2/2023 7:04 AM, Stefan Hajnoczi wrote: Setting the VIRTIO Device Status Field to 0 resets the device. The device's state is lost, including the vring configuration. vhost-user.c currently sends SET_STATUS 0 before GET_VRING_BASE. This risks confusion about the lifetime of the vhost-user s

Re: [PATCH v2] vhost-user: send SET_STATUS 0 after GET_VRING_BASE

2023-05-03 Thread Maxime Coquelin
On 5/2/23 01:04, Stefan Hajnoczi wrote: Setting the VIRTIO Device Status Field to 0 resets the device. The device's state is lost, including the vring configuration. vhost-user.c currently sends SET_STATUS 0 before GET_VRING_BASE. This risks confusion about the lifetime of the vhost-user stat

[PATCH v2] vhost-user: send SET_STATUS 0 after GET_VRING_BASE

2023-05-02 Thread Stefan Hajnoczi
Setting the VIRTIO Device Status Field to 0 resets the device. The device's state is lost, including the vring configuration. vhost-user.c currently sends SET_STATUS 0 before GET_VRING_BASE. This risks confusion about the lifetime of the vhost-user state (e.g. vring last_avail_idx) across VIRTIO d