Linux Virtualization - May 2015 - [PATCH v6 0/8] vhost: support for cross endian guests

On Fri, 24 Apr 2015 15:31:54 +0200
"Michael S. Tsirkin" <mst at redhat.com> wrote:
> On Fri, Apr 24, 2015 at 02:24:15PM +0200, Greg Kurz wrote:
> > Only cosmetic and documentation changes since v5.
> > 
> > ---
> 
> Looks sane to me. I plan to review and apply next week.
> 
Hi Michael,

I realize you just got back and have tons of things to do... Do
you still plan to apply shortly ? Would you also have time to
comment the QEMU part ?

Thanks.

--
Greg
> > Greg Kurz (8):
> >       virtio: introduce virtio_is_little_endian() helper
> >       tun: add tun_is_little_endian() helper
> >       macvtap: introduce macvtap_is_little_endian() helper
> >       vringh: introduce vringh_is_little_endian() helper
> >       vhost: introduce vhost_is_little_endian() helper
> >       virtio: add explicit big-endian support to memory accessors
> >       vhost: cross-endian support for legacy devices
> >       macvtap/tun: cross-endian support for little-endian hosts
> > 
> > 
> >  drivers/net/Kconfig              |   14 ++++++
> >  drivers/net/macvtap.c            |   66
+++++++++++++++++++++++++++++-
> >  drivers/net/tun.c                |   68
++++++++++++++++++++++++++++++
> >  drivers/vhost/Kconfig            |   15 +++++++
> >  drivers/vhost/vhost.c            |   85
++++++++++++++++++++++++++++++++++++++
> >  drivers/vhost/vhost.h            |   25 ++++++++---
> >  include/linux/virtio_byteorder.h |   24 ++++++-----
> >  include/linux/virtio_config.h    |   18 +++++---
> >  include/linux/vringh.h           |   18 +++++---
> >  include/uapi/linux/if_tun.h      |    6 +++
> >  include/uapi/linux/vhost.h       |   14 ++++++
> >  11 files changed, 320 insertions(+), 33 deletions(-)
> > 
> > --
> > Greg
>

On Tue, May 12, 2015 at 12:44:26PM +0200, Greg Kurz
wrote:
> On Fri, 24 Apr 2015 15:31:54 +0200
> "Michael S. Tsirkin" <mst at redhat.com> wrote:
> 
> > On Fri, Apr 24, 2015 at 02:24:15PM +0200, Greg Kurz wrote:
> > > Only cosmetic and documentation changes since v5.
> > > 
> > > ---
> > 
> > Looks sane to me. I plan to review and apply next week.
> > 
> 
> Hi Michael,
> 
> I realize you just got back and have tons of things to do... Do
> you still plan to apply shortly ? Would you also have time to
> comment the QEMU part ?
> 
> Thanks.
> 
> --
> Greg
Yes, sorry about the delay - I also got virtio upstream landed on my lap.
I'll do my best to prioritize.
> > > Greg Kurz (8):
> > >       virtio: introduce virtio_is_little_endian() helper
> > >       tun: add tun_is_little_endian() helper
> > >       macvtap: introduce macvtap_is_little_endian() helper
> > >       vringh: introduce vringh_is_little_endian() helper
> > >       vhost: introduce vhost_is_little_endian() helper
> > >       virtio: add explicit big-endian support to memory accessors
> > >       vhost: cross-endian support for legacy devices
> > >       macvtap/tun: cross-endian support for little-endian hosts
> > > 
> > > 
> > >  drivers/net/Kconfig              |   14 ++++++
> > >  drivers/net/macvtap.c            |   66
+++++++++++++++++++++++++++++-
> > >  drivers/net/tun.c                |   68
++++++++++++++++++++++++++++++
> > >  drivers/vhost/Kconfig            |   15 +++++++
> > >  drivers/vhost/vhost.c            |   85
++++++++++++++++++++++++++++++++++++++
> > >  drivers/vhost/vhost.h            |   25 ++++++++---
> > >  include/linux/virtio_byteorder.h |   24 ++++++-----
> > >  include/linux/virtio_config.h    |   18 +++++---
> > >  include/linux/vringh.h           |   18 +++++---
> > >  include/uapi/linux/if_tun.h      |    6 +++
> > >  include/uapi/linux/vhost.h       |   14 ++++++
> > >  11 files changed, 320 insertions(+), 33 deletions(-)
> > > 
> > > --
> > > Greg
> >

On Tue, 12 May 2015 12:52:55 +0200
"Michael S. Tsirkin" <mst at redhat.com> wrote:
> On Tue, May 12, 2015 at 12:44:26PM +0200, Greg Kurz wrote:
> > On Fri, 24 Apr 2015 15:31:54 +0200
> > "Michael S. Tsirkin" <mst at redhat.com> wrote:
> > 
> > > On Fri, Apr 24, 2015 at 02:24:15PM +0200, Greg Kurz wrote:
> > > > Only cosmetic and documentation changes since v5.
> > > > 
> > > > ---
> > > 
> > > Looks sane to me. I plan to review and apply next week.
> > > 
> > 
> > Hi Michael,
> > 
> > I realize you just got back and have tons of things to do... Do
> > you still plan to apply shortly ? Would you also have time to
> > comment the QEMU part ?
> > 
> > Thanks.
> > 
> > --
> > Greg
> 
> Yes, sorry about the delay - I also got virtio upstream landed on my lap.
> I'll do my best to prioritize.
> 
Hi Michael,

This is just a friendly reminder. Are there chances you pick up this
kernel series before virtio 1.0 is in upstream QEMU ?

Cheers.

--
Greg
> > > > Greg Kurz (8):
> > > >       virtio: introduce virtio_is_little_endian() helper
> > > >       tun: add tun_is_little_endian() helper
> > > >       macvtap: introduce macvtap_is_little_endian() helper
> > > >       vringh: introduce vringh_is_little_endian() helper
> > > >       vhost: introduce vhost_is_little_endian() helper
> > > >       virtio: add explicit big-endian support to memory
accessors
> > > >       vhost: cross-endian support for legacy devices
> > > >       macvtap/tun: cross-endian support for little-endian
hosts
> > > > 
> > > > 
> > > >  drivers/net/Kconfig              |   14 ++++++
> > > >  drivers/net/macvtap.c            |   66
+++++++++++++++++++++++++++++-
> > > >  drivers/net/tun.c                |   68
++++++++++++++++++++++++++++++
> > > >  drivers/vhost/Kconfig            |   15 +++++++
> > > >  drivers/vhost/vhost.c            |   85
++++++++++++++++++++++++++++++++++++++
> > > >  drivers/vhost/vhost.h            |   25 ++++++++---
> > > >  include/linux/virtio_byteorder.h |   24 ++++++-----
> > > >  include/linux/virtio_config.h    |   18 +++++---
> > > >  include/linux/vringh.h           |   18 +++++---
> > > >  include/uapi/linux/if_tun.h      |    6 +++
> > > >  include/uapi/linux/vhost.h       |   14 ++++++
> > > >  11 files changed, 320 insertions(+), 33 deletions(-)
> > > > 
> > > > --
> > > > Greg
> > > 
>