Re: [PATCH v3 00/16] virtio-pci: towards virtio 1.0 guest support

From: Gerd Hoffmann
Date: Thu Jan 15 2015 - 16:18:27 EST


Hi,

> QEMU side is still undergoing polishing, but is already testable.

Looked art the qemu side for now, commenting here due to qemu patches
not being posted to the list yet (guess that is at least partly already
on your todo list):

* Both legacy and modern should be switchable. Modern needs to be
switchable _anyway_ for backward compatibility with old qemu
versions. But legacy should be too, for testing purposes and for
new device types which don't need legacy in the first place.

* I'd like to see some more flexibility in the pci bar layout. Stuff
I have in mind:
- New devices which don't need a legacy bar can use bar 0 for
modern.
- One MMIO bar is enough, we can place both virtio regions and
msi-x regions there. I'd suggest to add msi-x sub-regions to
the modern bar.

* What is the reason for making the modern bar 8M in size? Looks a bit
excessive, given that only 64k or so of that are actually used ...

virtio-scsi seems to be broken, at least my usual fedora guest didn't
boot up from virtio-scsi disk when using a guest kernel with this patch
series applied.

cheers,
Gerd


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/