Re: [virtio-dev] Re: [PATCH 1/4] content: Introduce driver/device auxiliary notifications

2022-08-11 Thread Halil Pasic
On Thu, 11 Aug 2022 11:12:06 +0200 Cornelia Huck wrote: > On Wed, Aug 10 2022, "Michael S. Tsirkin" wrote: > > > On Wed, Aug 10, 2022 at 07:41:08PM +0200, Halil Pasic wrote: > >> On Wed, 10 Aug 2022 11:54:35 +0200 > >> Cornelia Huck wrote: > >> > This highlights another problem, however: W

Re: [virtio-dev] Re: [PATCH 1/4] content: Introduce driver/device auxiliary notifications

2022-08-11 Thread David Hildenbrand
On 11.08.22 16:09, Halil Pasic wrote: > On Thu, 11 Aug 2022 10:53:35 +0200 > Cornelia Huck wrote: > >> On Wed, Aug 10 2022, Halil Pasic wrote: >> >>> On Wed, 10 Aug 2022 11:54:35 +0200 >>> Cornelia Huck wrote: >>> >> These device-specific notifications are needed later when adding support

Re: [virtio-dev] Re: [PATCH 1/4] content: Introduce driver/device auxiliary notifications

2022-08-11 Thread Halil Pasic
On Thu, 11 Aug 2022 11:12:06 +0200 Cornelia Huck wrote: > > > >> I think making the people not motivated to do the design and write the > >> spec for all the platforms add to that list is a reasonable middle > >> ground. It would also make the differences very clear, and the same goes > >> for

Re: [virtio-dev] Re: [PATCH 1/4] content: Introduce driver/device auxiliary notifications

2022-08-11 Thread Halil Pasic
On Thu, 11 Aug 2022 10:53:35 +0200 Cornelia Huck wrote: > On Wed, Aug 10 2022, Halil Pasic wrote: > > > On Wed, 10 Aug 2022 11:54:35 +0200 > > Cornelia Huck wrote: > > > >> >> These device-specific notifications are needed later when adding support > >> >> for virtio-vhost-user device. > >>

Re: [virtio-dev] Re: [PATCH 1/4] content: Introduce driver/device auxiliary notifications

2022-08-11 Thread Michael S. Tsirkin
On Thu, Aug 11, 2022 at 11:12:06AM +0200, Cornelia Huck wrote: > >> OTOH when reading the spec, it my strike one as strange, that for example > >> CCW does not mention aux notifications at all. One idea: maybe we could > >> add a note, or a subsection, or something, which states the list of > >> ge

Re: [virtio-dev] Re: [PATCH 1/4] content: Introduce driver/device auxiliary notifications

2022-08-11 Thread Cornelia Huck
On Wed, Aug 10 2022, "Michael S. Tsirkin" wrote: > On Wed, Aug 10, 2022 at 07:41:08PM +0200, Halil Pasic wrote: >> On Wed, 10 Aug 2022 11:54:35 +0200 >> Cornelia Huck wrote: >> > This highlights another problem, however: When we introduce new features >> > that require a transport-specific imple

Re: [virtio-dev] Re: [PATCH 1/4] content: Introduce driver/device auxiliary notifications

2022-08-11 Thread Cornelia Huck
On Wed, Aug 10 2022, Halil Pasic wrote: > On Wed, 10 Aug 2022 11:54:35 +0200 > Cornelia Huck wrote: > >> >> These device-specific notifications are needed later when adding support >> >> for virtio-vhost-user device. >> >> >> >> Signed-off-by: Usama Arif >> >> Signed-off-by: Stefan Hajnoczi >

Re: [virtio-dev] Re: [PATCH 1/4] content: Introduce driver/device auxiliary notifications

2022-08-10 Thread Michael S. Tsirkin
On Wed, Aug 10, 2022 at 07:41:08PM +0200, Halil Pasic wrote: > On Wed, 10 Aug 2022 11:54:35 +0200 > Cornelia Huck wrote: > > > >> These device-specific notifications are needed later when adding support > > >> for virtio-vhost-user device. > > >> > > >> Signed-off-by: Usama Arif > > >> Signed-o

Re: [virtio-dev] Re: [PATCH 1/4] content: Introduce driver/device auxiliary notifications

2022-08-10 Thread Halil Pasic
On Wed, 10 Aug 2022 08:45:25 -0400 "Michael S. Tsirkin" wrote: > On Wed, Aug 10, 2022 at 11:54:35AM +0200, Cornelia Huck wrote: > > On Tue, Aug 09 2022, "Michael S. Tsirkin" wrote: > > > > > On Wed, Mar 30, 2022 at 04:21:02PM +0100, Usama Arif wrote: > > >> Driver auxiliary notifications al

Re: [virtio-dev] Re: [PATCH 1/4] content: Introduce driver/device auxiliary notifications

2022-08-10 Thread Halil Pasic
On Wed, 10 Aug 2022 11:54:35 +0200 Cornelia Huck wrote: > >> These device-specific notifications are needed later when adding support > >> for virtio-vhost-user device. > >> > >> Signed-off-by: Usama Arif > >> Signed-off-by: Stefan Hajnoczi > >> Signed-off-by: Nikos Dragazis > > > > I see c

[virtio-dev] Re: [PATCH 1/4] content: Introduce driver/device auxiliary notifications

2022-08-10 Thread Cornelia Huck
On Wed, Aug 10 2022, "Michael S. Tsirkin" wrote: > On Wed, Aug 10, 2022 at 11:54:35AM +0200, Cornelia Huck wrote: >> On Tue, Aug 09 2022, "Michael S. Tsirkin" wrote: >> >> > On Wed, Mar 30, 2022 at 04:21:02PM +0100, Usama Arif wrote: >> >> Driver auxiliary notifications allow the device to send

[virtio-dev] Re: [PATCH 1/4] content: Introduce driver/device auxiliary notifications

2022-08-10 Thread Michael S. Tsirkin
On Wed, Aug 10, 2022 at 11:54:35AM +0200, Cornelia Huck wrote: > On Tue, Aug 09 2022, "Michael S. Tsirkin" wrote: > > > On Wed, Mar 30, 2022 at 04:21:02PM +0100, Usama Arif wrote: > >> Driver auxiliary notifications allow the device to send notifications > >> other than configuration changes and

[virtio-dev] Re: [PATCH 1/4] content: Introduce driver/device auxiliary notifications

2022-08-10 Thread Cornelia Huck
On Tue, Aug 09 2022, "Michael S. Tsirkin" wrote: > On Wed, Mar 30, 2022 at 04:21:02PM +0100, Usama Arif wrote: >> Driver auxiliary notifications allow the device to send notifications >> other than configuration changes and used buffer notifications to the >> driver, these are optional and their

[virtio-dev] Re: [PATCH 1/4] content: Introduce driver/device auxiliary notifications

2022-08-09 Thread Michael S. Tsirkin
On Wed, Mar 30, 2022 at 04:21:02PM +0100, Usama Arif wrote: > Driver auxiliary notifications allow the device to send notifications > other than configuration changes and used buffer notifications to the > driver, these are optional and their meaning is device-specific. > > Device auxiliary notifc

[virtio-dev] Re: [PATCH 1/4] content: Introduce driver/device auxiliary notifications

2022-04-04 Thread Stefan Hajnoczi
On Wed, Mar 30, 2022 at 04:26:56PM +0100, Usama Arif wrote: > +Available buffer notifications and device auxiliary notifications > +are sent by the driver, the recipient is the device. Available buffer > +notifications indicate that a buffer may have been made available on the > +virtqueue designat