Christoph Hellwig
2021-Apr-20 07:01 UTC
[PATCH v2] virtio_blk: Add support for lifetime feature
Just to despit my 2 cents again: I think the way this is specified in the virtio spec is actively harmful and we should not suport it in Linux. If others override me we at least need to require a detailed documentation of these fields as the virto spec does not provide it. Please also do not add pointless over 80 character lines, and follow the one value per sysfs file rule.
Michael S. Tsirkin
2021-Apr-20 10:08 UTC
[PATCH v2] virtio_blk: Add support for lifetime feature
On Tue, Apr 20, 2021 at 08:01:29AM +0100, Christoph Hellwig wrote:> Just to despit my 2 cents again: I think the way this is specified > in the virtio spec is actively harmful and we should not suport it in > Linux. > > If others override me we at least need to require a detailed > documentation of these fields as the virto spec does not provide it. > > Please also do not add pointless over 80 character lines, and follow > the one value per sysfs file rule.Enrico would you like to raise the issues with the virtio TC for resolution? -- MST