[virtio-dev] Re: [virtio] Re: [virtio-dev] Re: [virtio] [PATCH RFC v7 6/8] ccw: disallow ADMIN_VQ

2022-09-01 Thread Halil Pasic
On Wed, 31 Aug 2022 10:50:21 -0400 "Michael S. Tsirkin" wrote: > > You seem to say that all the normative statements may be classified > > into two categories: > > C1) the other side does not rely on this requirement > > C2) the other side does rely on this requirement > > > > The requirements f

[virtio-dev] Re: [virtio] Re: [virtio-dev] Re: [virtio] [PATCH RFC v7 6/8] ccw: disallow ADMIN_VQ

2022-08-31 Thread Michael S. Tsirkin
On Wed, Aug 31, 2022 at 04:33:49PM +0200, Halil Pasic wrote: > On Sun, 28 Aug 2022 05:35:53 -0400 > "Michael S. Tsirkin" wrote: > > > > > Like here: > > > > +Driver MUST NOT set bit VIRTIO_F_ADMIN_VQ (bit 41) in > > > > +DriverFeatures even if offered by the device. > > > > > >

[virtio-dev] Re: [virtio] Re: [virtio-dev] Re: [virtio] [PATCH RFC v7 6/8] ccw: disallow ADMIN_VQ

2022-08-31 Thread Halil Pasic
On Sun, 28 Aug 2022 05:35:53 -0400 "Michael S. Tsirkin" wrote: > > > Like here: > > >+Driver MUST NOT set bit VIRTIO_F_ADMIN_VQ (bit 41) in > > >+DriverFeatures even if offered by the device. > > > > > > This makes sure that drivers do not make an assumption that > > > devices do not set

[virtio-dev] Re: [virtio] [PATCH RFC v7 6/8] ccw: disallow ADMIN_VQ

2022-08-30 Thread Cornelia Huck
[finally got around to looking at this thread] On Tue, Aug 30 2022, Halil Pasic wrote: > On Mon, 29 Aug 2022 14:28:05 -0400 > "Michael S. Tsirkin" wrote: > >> On Thu, Aug 18, 2022 at 03:39:58PM +0200, Halil Pasic wrote: > [..] >> > Fair point! >> > >> > I would prefer a driver normative which

[virtio-dev] Re: [virtio] [PATCH RFC v7 6/8] ccw: disallow ADMIN_VQ

2022-08-30 Thread Halil Pasic
On Mon, 29 Aug 2022 14:28:05 -0400 "Michael S. Tsirkin" wrote: > On Thu, Aug 18, 2022 at 03:39:58PM +0200, Halil Pasic wrote: [..] > > Fair point! > > > > I would prefer a driver normative which goes like this: > > > > """ > > A driver SHOULD NOT accept features (i.e. have code that would do so

[virtio-dev] Re: [virtio] [PATCH RFC v7 6/8] ccw: disallow ADMIN_VQ

2022-08-29 Thread Michael S. Tsirkin
On Thu, Aug 18, 2022 at 03:39:58PM +0200, Halil Pasic wrote: > On Tue, 16 Aug 2022 11:48:43 -0400 > "Michael S. Tsirkin" wrote: > > > On Tue, Aug 16, 2022 at 04:48:11PM +0200, Halil Pasic wrote: > > > On Fri, 12 Aug 2022 13:19:20 -0400 > > > "Michael S. Tsirkin" wrote: > > > > > > > Signed-of

Re: [virtio-dev] Re: [virtio] [PATCH RFC v7 6/8] ccw: disallow ADMIN_VQ

2022-08-28 Thread Michael S. Tsirkin
On Wed, Aug 24, 2022 at 01:45:19AM +0200, Halil Pasic wrote: > On Thu, 18 Aug 2022 23:57:39 -0400 > "Michael S. Tsirkin" wrote: > > > > > > I'm not sure I understand the intention here. I believe what we try to > > > > > accomplish here is the following. The Channel I/O transport > > > > > *curr

Re: [virtio-dev] Re: [virtio] [PATCH RFC v7 6/8] ccw: disallow ADMIN_VQ

2022-08-23 Thread Halil Pasic
On Thu, 18 Aug 2022 23:57:39 -0400 "Michael S. Tsirkin" wrote: > > > > I'm not sure I understand the intention here. I believe what we try to > > > > accomplish here is the following. The Channel I/O transport *currently* > > > > does not support the VIRTIO_F_ADMIN_VQ feature. It is not like we w

[virtio-dev] Re: [virtio] [PATCH RFC v7 6/8] ccw: disallow ADMIN_VQ

2022-08-18 Thread Michael S. Tsirkin
On Thu, Aug 18, 2022 at 03:39:58PM +0200, Halil Pasic wrote: > On Tue, 16 Aug 2022 11:48:43 -0400 > "Michael S. Tsirkin" wrote: > > > On Tue, Aug 16, 2022 at 04:48:11PM +0200, Halil Pasic wrote: > > > On Fri, 12 Aug 2022 13:19:20 -0400 > > > "Michael S. Tsirkin" wrote: > > > > > > > Signed-of

[virtio-dev] Re: [virtio] [PATCH RFC v7 6/8] ccw: disallow ADMIN_VQ

2022-08-18 Thread Halil Pasic
On Tue, 16 Aug 2022 11:48:43 -0400 "Michael S. Tsirkin" wrote: > On Tue, Aug 16, 2022 at 04:48:11PM +0200, Halil Pasic wrote: > > On Fri, 12 Aug 2022 13:19:20 -0400 > > "Michael S. Tsirkin" wrote: > > > > > Signed-off-by: Michael S. Tsirkin > > > --- > > > content.tex | 10 ++ > > >

[virtio-dev] Re: [virtio] [PATCH RFC v7 6/8] ccw: disallow ADMIN_VQ

2022-08-16 Thread Michael S. Tsirkin
On Tue, Aug 16, 2022 at 11:48:51AM -0400, Michael S. Tsirkin wrote: > On Tue, Aug 16, 2022 at 04:48:11PM +0200, Halil Pasic wrote: > > On Fri, 12 Aug 2022 13:19:20 -0400 > > "Michael S. Tsirkin" wrote: > > > > > Signed-off-by: Michael S. Tsirkin > > > --- > > > content.tex | 10 ++ > > >

[virtio-dev] Re: [virtio] [PATCH RFC v7 6/8] ccw: disallow ADMIN_VQ

2022-08-16 Thread Michael S. Tsirkin
On Tue, Aug 16, 2022 at 04:48:11PM +0200, Halil Pasic wrote: > On Fri, 12 Aug 2022 13:19:20 -0400 > "Michael S. Tsirkin" wrote: > > > Signed-off-by: Michael S. Tsirkin > > --- > > content.tex | 10 ++ > > 1 file changed, 10 insertions(+) > > > > diff --git a/content.tex b/content.tex >

[virtio-dev] Re: [virtio] [PATCH RFC v7 6/8] ccw: disallow ADMIN_VQ

2022-08-16 Thread Halil Pasic
On Fri, 12 Aug 2022 13:19:20 -0400 "Michael S. Tsirkin" wrote: > Signed-off-by: Michael S. Tsirkin > --- > content.tex | 10 ++ > 1 file changed, 10 insertions(+) > > diff --git a/content.tex b/content.tex > index 76b5a28..53be680 100644 > --- a/content.tex > +++ b/content.tex > @@ -26