[Qemu-devel] Re: [PATCH 0/5] virtio: cleanup layout assumptions

[Juan Quintela]

"Michael S. Tsirkin" <address@hidden> wrote:
> This patchset helps remove the assumption that vdev must be the 1st
> member in any virtio device implementation. Other assumptions (made by
> qdev) can be removed in a similar way separately, this patchset does not
> do this yet.
> Juan, this is an alternative to your 'tell virtio about DO_UPCAST'
> patch, pls take a look.

Answered in the other series.  I don't see what removing that assumption
brings us.  qdev "requires" that 1st element is a qdev.  And that
requeriment is not going out anytime soon.

I think that this series are "over" engineering, at least until somebody
cames with a good use for not being the 1st element.

Why do you want this?  Notice that rest of qemu has this same assumption
for rest of devices.  Principle of Least Surprise means use my
"cleanup".

Furthermore, your VIRTIO_COMMON_INIT() macro is ugly as hell, again to
gain us nothing.

Without a good reason why we don't want VirtIODevice to be the 1st
element, I think that my series are more coherent (but on the other
hand, what would you expect from me :)

Later, Juan.