Re: [RFC PATCH 1/1] virtio: fix feature negotiation for ACCESS_PLATFORM

2022-02-07 Thread Halil Pasic
On Mon, 7 Feb 2022 16:46:04 -0300 Daniel Henrique Barboza wrote: > On 2/7/22 11:46, Halil Pasic wrote: > > On Mon, 7 Feb 2022 08:46:34 -0300 > > Daniel Henrique Barboza wrote: > > > > I have considered this and decided against it. The reason why is > > if that approach is taken, we can't real

Re: [RFC PATCH 1/1] virtio: fix feature negotiation for ACCESS_PLATFORM

2022-02-07 Thread Daniel Henrique Barboza
On 2/7/22 11:46, Halil Pasic wrote: On Mon, 7 Feb 2022 08:46:34 -0300 Daniel Henrique Barboza wrote: On 2/3/22 13:45, Halil Pasic wrote: Unlike most virtio features ACCESS_PATFORM is considered mandatory, i.e. the driver must accept it if offered by the device. The virtio specification say

Re: [RFC PATCH 1/1] virtio: fix feature negotiation for ACCESS_PLATFORM

2022-02-07 Thread Michael S. Tsirkin
On Mon, Feb 07, 2022 at 04:05:16PM +0100, Halil Pasic wrote: > On Mon, 07 Feb 2022 14:41:58 +0100 > Cornelia Huck wrote: > > > On Mon, Feb 07 2022, Daniel Henrique Barboza wrote: > > > > > On 2/3/22 13:45, Halil Pasic wrote: > > >> Unlike most virtio features ACCESS_PATFORM is considered mand

Re: [RFC PATCH 1/1] virtio: fix feature negotiation for ACCESS_PLATFORM

2022-02-07 Thread Halil Pasic
On Mon, 07 Feb 2022 16:21:31 +0100 Cornelia Huck wrote: > On Mon, Feb 07 2022, Halil Pasic wrote: > > > On Mon, 07 Feb 2022 14:41:58 +0100 > > Cornelia Huck wrote: > > >> OTOH, the decision to make it mandatory is certainly sound, and covered > >> by the spec. As the driver must be prepared

Re: [RFC PATCH 1/1] virtio: fix feature negotiation for ACCESS_PLATFORM

2022-02-07 Thread Cornelia Huck
On Mon, Feb 07 2022, Halil Pasic wrote: > On Mon, 07 Feb 2022 14:41:58 +0100 > Cornelia Huck wrote: >> OTOH, the decision to make it mandatory is certainly sound, and covered >> by the spec. As the driver must be prepared for the device failing to >> accept FEATURES_OK, we can make it mandatory

Re: [RFC PATCH 1/1] virtio: fix feature negotiation for ACCESS_PLATFORM

2022-02-07 Thread Halil Pasic
On Mon, 07 Feb 2022 14:41:58 +0100 Cornelia Huck wrote: > On Mon, Feb 07 2022, Daniel Henrique Barboza wrote: > > > On 2/3/22 13:45, Halil Pasic wrote: > >> Unlike most virtio features ACCESS_PATFORM is considered mandatory, i.e. > > s/ACCESS_PATFORM/ACCESS_PLATFORM/ Will fix. > > >> th

Re: [RFC PATCH 1/1] virtio: fix feature negotiation for ACCESS_PLATFORM

2022-02-07 Thread Halil Pasic
On Mon, 7 Feb 2022 08:46:34 -0300 Daniel Henrique Barboza wrote: > On 2/3/22 13:45, Halil Pasic wrote: > > Unlike most virtio features ACCESS_PATFORM is considered mandatory, i.e. > > the driver must accept it if offered by the device. The virtio > > specification says that the driver SHOULD acce

Re: [RFC PATCH 1/1] virtio: fix feature negotiation for ACCESS_PLATFORM

2022-02-07 Thread Daniel Henrique Barboza
On 2/7/22 10:41, Cornelia Huck wrote: On Mon, Feb 07 2022, Daniel Henrique Barboza wrote: On 2/3/22 13:45, Halil Pasic wrote: Unlike most virtio features ACCESS_PATFORM is considered mandatory, i.e. s/ACCESS_PATFORM/ACCESS_PLATFORM/ the driver must accept it if offered by the device. T

Re: [RFC PATCH 1/1] virtio: fix feature negotiation for ACCESS_PLATFORM

2022-02-07 Thread Cornelia Huck
On Mon, Feb 07 2022, Daniel Henrique Barboza wrote: > On 2/3/22 13:45, Halil Pasic wrote: >> Unlike most virtio features ACCESS_PATFORM is considered mandatory, i.e. s/ACCESS_PATFORM/ACCESS_PLATFORM/ >> the driver must accept it if offered by the device. The virtio >> specification says that th

Re: [RFC PATCH 1/1] virtio: fix feature negotiation for ACCESS_PLATFORM

2022-02-07 Thread Daniel Henrique Barboza
On 2/3/22 13:45, Halil Pasic wrote: Unlike most virtio features ACCESS_PATFORM is considered mandatory, i.e. the driver must accept it if offered by the device. The virtio specification says that the driver SHOULD accept the ACCESS_PLATFORM feature if offered, and that the device MAY fail to o

[RFC PATCH 1/1] virtio: fix feature negotiation for ACCESS_PLATFORM

2022-02-03 Thread Halil Pasic
Unlike most virtio features ACCESS_PATFORM is considered mandatory, i.e. the driver must accept it if offered by the device. The virtio specification says that the driver SHOULD accept the ACCESS_PLATFORM feature if offered, and that the device MAY fail to operate if ACCESS_PLATFORM was offered but