Il 15/07/2013 18:52, Anthony Liguori ha scritto: > Cornelia Huck <cornelia.h...@de.ibm.com> writes: > >> The following changes since commit c3cb8e77804313e1be99b5f28a34a346736707a5: >> >> ioport: remove LITTLE_ENDIAN mark for portio (2013-07-12 14:37:47 -0500) >> >> are available in the git repository at: >> >> git://github.com/cohuck/qemu virtio-ccw-upstr >> >> for you to fetch changes up to bf72d89f0a8fb27a6bfde1a689690fd577227762: >> >> virtio-ccw: Enable x-data-plane for virtio-ccw-blk (2013-07-15 >> 17:39:04 +0200) > > This is v1? This was just posted on Thursday? That's a bit too quick > for a pull request. > > I've seen no review from Stefan or Paolo and I would think that given > the work that's been done already, it would be counterproductive to > enable x-data-plane for another backend. > > Paolo/Stefan, what do you guys think?
Considering what the patch looks like, I don't think it's a huge problem... In fact, perhaps x-data-plane could be even added to DEFINE_VIRTIO_BLK_PROPERTIES. This would make it clear that support for non-ioeventfd hosts (including TCG) is one of the things to do to make data plane the default. Paolo