On Fri, 14 Feb 2014 12:59:49 +0100 Andreas Färber <afaer...@suse.de> wrote:
> Hi, > > [...] > > It might've helped if Rusty had actually used our > scripts/get_maintainer.pl script to CC people. While Anthony seems to > have reviewed some patches (usually Reviewed-by should be before the > final Signed-off-by fwiw), neither Stefan (virtio-net) nor Kevin > (virtio-blk) nor Paolo (virtio-scsi) were CC'ed, and recently Michael > stepped up as virtio maintainer, so maybe he can take them once ready. > > 1/7 looks okay to me; 3-7 are rather mechanical - people will need to > review that those changes are sufficient for the current code base. > We've since converted virtio devices to QOM realize, so a rebase is > likely needed for such an "old" series. > > Regards, > Andreas Hi, Thank you Andreas for your support. :) Here is a rebased patchset. Only the two first patches ("virtio_get_byteswap:" and "virtio:") had to be updated actually. FWIW, this the very same patchset I use, along with some ppc64 specific enablement code, to have functionnal ppc64 LE guests. --- Greg Kurz (1): hw/9pfs/virtio_9p_device: use virtio wrappers to access headers. Rusty Russell (7): virtio_get_byteswap: function for endian-ambivalent targets using virtio. virtio: allow byte swapping for vring and config access hw/net/virtio-net: use virtio wrappers to access headers. hw/net/virtio-balloon: use virtio wrappers to access page frame numbers. hw/block/virtio-blk: use virtio wrappers to access headers. hw/scsi/virtio-scsi: use virtio wrappers to access headers. hw/char/virtio-serial-bus: use virtio wrappers to access headers. hw/9pfs/virtio-9p-device.c | 3 + hw/block/virtio-blk.c | 35 +++++----- hw/char/virtio-serial-bus.c | 34 +++++----- hw/net/virtio-net.c | 15 ++-- hw/scsi/virtio-scsi.c | 33 +++++---- hw/virtio/virtio-balloon.c | 3 + hw/virtio/virtio.c | 38 ++++++----- include/hw/virtio/virtio-access.h | 132 +++++++++++++++++++++++++++++++++++++ include/hw/virtio/virtio.h | 2 + stubs/Makefile.objs | 1 stubs/virtio_get_byteswap.c | 6 ++ 11 files changed, 228 insertions(+), 74 deletions(-) create mode 100644 include/hw/virtio/virtio-access.h create mode 100644 stubs/virtio_get_byteswap.c Best Regards. -- Greg