On Thu, 28 Jan 2010 11:13:27 pm Christoph Hellwig wrote: > Back iSeptember 2007 Michael made the serial number support in qemu > optional and off by default, and in October 2009 Rusty reverted the > Linux virtio-blk support for it. Given that I can't find support in > any other virtio implementation that makes the feature look essentially > dead. > > How should we proceed with adding more fields to struct virtio_blk_config? > I would suggest removing the identity field, declaring > VIRTIO_BLK_F_IDENTIFY officially deprecated and adding the new fields > directly after blk_size again, maybe with a comment that these new > features can't be advertized together with VIRTIO_BLK_F_IDENTIFY.
Yes, this is the correct approach. Agreed, Rusty.