[virtio-dev] Re: [PATCH v4] virtio: Improve queue_reset polarity to match to default reset state

2022-04-28 Thread Jason Wang
在 2022/4/29 06:16, Parav Pandit 写道: Currently when driver initiates a queue reset, device is expected to communicate reset status to the driver by changing the value of the queue_reset register twice. First to return value other than 1 when reset is ongoing, later to return 1 when queue reset

RE: [virtio-dev] [PATCH v2] virtio: Improve queue_reset polarity to match to default reset state

2022-04-28 Thread Cornelia Huck
On Thu, Apr 28 2022, Parav Pandit wrote: > All above comments are taken care now with Cornelia's wording for your input > and by correctly updating device side section now. > Sending v3. Honestly, I think it's a bit early for sending a v3 already, since it seems we've not reached complete

Re: [virtio-dev] [PATCH v2] virtio: Improve queue_reset polarity to match to default reset state

2022-04-28 Thread Jason Wang
On Thu, Apr 28, 2022 at 2:30 PM Michael S. Tsirkin wrote: > > On Thu, Apr 28, 2022 at 02:13:42PM +0800, Jason Wang wrote: > > > queue_reset removal has far more editorial changes. > > > Before I update the v4 to incorporate, please discuss/sync with Michael > > > or others what is best course of

Re: [virtio-dev] [PATCH v2] virtio: Improve queue_reset polarity to match to default reset state

2022-04-28 Thread Michael S. Tsirkin
On Thu, Apr 28, 2022 at 02:13:42PM +0800, Jason Wang wrote: > > queue_reset removal has far more editorial changes. > > Before I update the v4 to incorporate, please discuss/sync with Michael or > > others what is best course of action from spec timing perspective. > > I am ok either way to

Re: [virtio-dev] [PATCH v2] virtio: Improve queue_reset polarity to match to default reset state

2022-04-28 Thread Michael S. Tsirkin
On Thu, Apr 28, 2022 at 02:10:46PM +0800, Jason Wang wrote: > On Thu, Apr 28, 2022 at 12:56 PM Michael S. Tsirkin wrote: > > > > On Thu, Apr 28, 2022 at 11:43:16AM +0800, Jason Wang wrote: > > > > > > 在 2022/4/28 11:24, Parav Pandit 写道: > > > > > > > > > From: Jason Wang > > > > > Sent:

Re: [virtio-dev] [PATCH v2] virtio: Improve queue_reset polarity to match to default reset state

2022-04-28 Thread Jason Wang
On Thu, Apr 28, 2022 at 12:00 PM Parav Pandit wrote: > > > > From: Jason Wang > > Sent: Wednesday, April 27, 2022 11:41 PM > > > > > > Hypervisor needs to trap the access to q_reset, otherwise there will be a > > race anyhow. So it knows the device is being reset and only after the device > >

Re: [virtio-dev] [PATCH v2] virtio: Improve queue_reset polarity to match to default reset state

2022-04-28 Thread Jason Wang
On Thu, Apr 28, 2022 at 12:56 PM Michael S. Tsirkin wrote: > > On Thu, Apr 28, 2022 at 11:43:16AM +0800, Jason Wang wrote: > > > > 在 2022/4/28 11:24, Parav Pandit 写道: > > > > > > > From: Jason Wang > > > > Sent: Wednesday, April 27, 2022 11:15 PM > > > > > > > > On Wed, Apr 27, 2022 at 11:39 PM