* [PATCH v3 0/7] vhost: support for cross endian guests
@ 2015-04-07 12:09 Greg Kurz
0 siblings, 0 replies; 9+ messages in thread
From: Greg Kurz @ 2015-04-07 12:09 UTC (permalink / raw)
To: Rusty Russell, Michael S. Tsirkin
Cc: linux-api, linux-kernel, kvm, virtualization
Hi,
This patchset allows vhost to be used with legacy virtio when guest and host
have a different endianness.
Patches 1-6 remain the same as the previous post. Patch 7 was heavily changed
according to MST's comments.
---
Greg Kurz (7):
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: feature to set the vring endianness
drivers/net/macvtap.c | 11 ++++++--
drivers/net/tun.c | 11 ++++++--
drivers/vhost/Kconfig | 10 +++++++
drivers/vhost/vhost.c | 55 ++++++++++++++++++++++++++++++++++++++
drivers/vhost/vhost.h | 34 +++++++++++++++++++----
include/linux/virtio_byteorder.h | 24 ++++++++++-------
include/linux/virtio_config.h | 19 +++++++++----
include/linux/vringh.h | 19 +++++++++----
include/uapi/linux/vhost.h | 5 +++
9 files changed, 156 insertions(+), 32 deletions(-)
--
Greg
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v3 0/7] vhost: support for cross endian guests
@ 2015-04-07 16:08 ` Greg Kurz
0 siblings, 0 replies; 9+ messages in thread
From: Greg Kurz @ 2015-04-07 16:08 UTC (permalink / raw)
To: Michael S. Tsirkin
Cc: Rusty Russell, linux-api, linux-kernel, kvm, virtualization
On Tue, 7 Apr 2015 17:55:08 +0200
"Michael S. Tsirkin" <mst@redhat.com> wrote:
> On Tue, Apr 07, 2015 at 02:09:29PM +0200, Greg Kurz wrote:
> > Hi,
> >
> > This patchset allows vhost to be used with legacy virtio when guest and host
> > have a different endianness.
> >
> > Patches 1-6 remain the same as the previous post. Patch 7 was heavily changed
> > according to MST's comments.
>
> This still doesn't actually work, right?
> tun and macvtap need new ioctls too ...
>
Yes they do. I already have a patch but I wasn't sure if I should send it
along this series... Since it looks like there will be a v4, I'll add the
tun/macvtap patch.
Thanks.
--
Greg
> > ---
> >
> > Greg Kurz (7):
> > 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: feature to set the vring endianness
> >
> >
> > drivers/net/macvtap.c | 11 ++++++--
> > drivers/net/tun.c | 11 ++++++--
> > drivers/vhost/Kconfig | 10 +++++++
> > drivers/vhost/vhost.c | 55 ++++++++++++++++++++++++++++++++++++++
> > drivers/vhost/vhost.h | 34 +++++++++++++++++++----
> > include/linux/virtio_byteorder.h | 24 ++++++++++-------
> > include/linux/virtio_config.h | 19 +++++++++----
> > include/linux/vringh.h | 19 +++++++++----
> > include/uapi/linux/vhost.h | 5 +++
> > 9 files changed, 156 insertions(+), 32 deletions(-)
> >
> > --
> > Greg
>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v3 0/7] vhost: support for cross endian guests
@ 2015-04-07 16:08 ` Greg Kurz
0 siblings, 0 replies; 9+ messages in thread
From: Greg Kurz @ 2015-04-07 16:08 UTC (permalink / raw)
To: Michael S. Tsirkin
Cc: Rusty Russell, linux-api-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA, kvm-u79uwXL29TY76Z2rM5mHXA,
virtualization-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA
On Tue, 7 Apr 2015 17:55:08 +0200
"Michael S. Tsirkin" <mst-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> wrote:
> On Tue, Apr 07, 2015 at 02:09:29PM +0200, Greg Kurz wrote:
> > Hi,
> >
> > This patchset allows vhost to be used with legacy virtio when guest and host
> > have a different endianness.
> >
> > Patches 1-6 remain the same as the previous post. Patch 7 was heavily changed
> > according to MST's comments.
>
> This still doesn't actually work, right?
> tun and macvtap need new ioctls too ...
>
Yes they do. I already have a patch but I wasn't sure if I should send it
along this series... Since it looks like there will be a v4, I'll add the
tun/macvtap patch.
Thanks.
--
Greg
> > ---
> >
> > Greg Kurz (7):
> > 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: feature to set the vring endianness
> >
> >
> > drivers/net/macvtap.c | 11 ++++++--
> > drivers/net/tun.c | 11 ++++++--
> > drivers/vhost/Kconfig | 10 +++++++
> > drivers/vhost/vhost.c | 55 ++++++++++++++++++++++++++++++++++++++
> > drivers/vhost/vhost.h | 34 +++++++++++++++++++----
> > include/linux/virtio_byteorder.h | 24 ++++++++++-------
> > include/linux/virtio_config.h | 19 +++++++++----
> > include/linux/vringh.h | 19 +++++++++----
> > include/uapi/linux/vhost.h | 5 +++
> > 9 files changed, 156 insertions(+), 32 deletions(-)
> >
> > --
> > Greg
>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v3 0/7] vhost: support for cross endian guests
2015-04-07 15:55 ` Michael S. Tsirkin
(?)
@ 2015-04-07 16:08 ` Greg Kurz
-1 siblings, 0 replies; 9+ messages in thread
From: Greg Kurz @ 2015-04-07 16:08 UTC (permalink / raw)
To: Michael S. Tsirkin; +Cc: linux-api, linux-kernel, kvm, virtualization
On Tue, 7 Apr 2015 17:55:08 +0200
"Michael S. Tsirkin" <mst@redhat.com> wrote:
> On Tue, Apr 07, 2015 at 02:09:29PM +0200, Greg Kurz wrote:
> > Hi,
> >
> > This patchset allows vhost to be used with legacy virtio when guest and host
> > have a different endianness.
> >
> > Patches 1-6 remain the same as the previous post. Patch 7 was heavily changed
> > according to MST's comments.
>
> This still doesn't actually work, right?
> tun and macvtap need new ioctls too ...
>
Yes they do. I already have a patch but I wasn't sure if I should send it
along this series... Since it looks like there will be a v4, I'll add the
tun/macvtap patch.
Thanks.
--
Greg
> > ---
> >
> > Greg Kurz (7):
> > 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: feature to set the vring endianness
> >
> >
> > drivers/net/macvtap.c | 11 ++++++--
> > drivers/net/tun.c | 11 ++++++--
> > drivers/vhost/Kconfig | 10 +++++++
> > drivers/vhost/vhost.c | 55 ++++++++++++++++++++++++++++++++++++++
> > drivers/vhost/vhost.h | 34 +++++++++++++++++++----
> > include/linux/virtio_byteorder.h | 24 ++++++++++-------
> > include/linux/virtio_config.h | 19 +++++++++----
> > include/linux/vringh.h | 19 +++++++++----
> > include/uapi/linux/vhost.h | 5 +++
> > 9 files changed, 156 insertions(+), 32 deletions(-)
> >
> > --
> > Greg
>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v3 0/7] vhost: support for cross endian guests
@ 2015-04-07 15:55 ` Michael S. Tsirkin
0 siblings, 0 replies; 9+ messages in thread
From: Michael S. Tsirkin @ 2015-04-07 15:55 UTC (permalink / raw)
To: Greg Kurz; +Cc: Rusty Russell, linux-api, linux-kernel, kvm, virtualization
On Tue, Apr 07, 2015 at 02:09:29PM +0200, Greg Kurz wrote:
> Hi,
>
> This patchset allows vhost to be used with legacy virtio when guest and host
> have a different endianness.
>
> Patches 1-6 remain the same as the previous post. Patch 7 was heavily changed
> according to MST's comments.
This still doesn't actually work, right?
tun and macvtap need new ioctls too ...
> ---
>
> Greg Kurz (7):
> 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: feature to set the vring endianness
>
>
> drivers/net/macvtap.c | 11 ++++++--
> drivers/net/tun.c | 11 ++++++--
> drivers/vhost/Kconfig | 10 +++++++
> drivers/vhost/vhost.c | 55 ++++++++++++++++++++++++++++++++++++++
> drivers/vhost/vhost.h | 34 +++++++++++++++++++----
> include/linux/virtio_byteorder.h | 24 ++++++++++-------
> include/linux/virtio_config.h | 19 +++++++++----
> include/linux/vringh.h | 19 +++++++++----
> include/uapi/linux/vhost.h | 5 +++
> 9 files changed, 156 insertions(+), 32 deletions(-)
>
> --
> Greg
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v3 0/7] vhost: support for cross endian guests
@ 2015-04-07 15:55 ` Michael S. Tsirkin
0 siblings, 0 replies; 9+ messages in thread
From: Michael S. Tsirkin @ 2015-04-07 15:55 UTC (permalink / raw)
To: Greg Kurz
Cc: Rusty Russell, linux-api-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA, kvm-u79uwXL29TY76Z2rM5mHXA,
virtualization-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA
On Tue, Apr 07, 2015 at 02:09:29PM +0200, Greg Kurz wrote:
> Hi,
>
> This patchset allows vhost to be used with legacy virtio when guest and host
> have a different endianness.
>
> Patches 1-6 remain the same as the previous post. Patch 7 was heavily changed
> according to MST's comments.
This still doesn't actually work, right?
tun and macvtap need new ioctls too ...
> ---
>
> Greg Kurz (7):
> 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: feature to set the vring endianness
>
>
> drivers/net/macvtap.c | 11 ++++++--
> drivers/net/tun.c | 11 ++++++--
> drivers/vhost/Kconfig | 10 +++++++
> drivers/vhost/vhost.c | 55 ++++++++++++++++++++++++++++++++++++++
> drivers/vhost/vhost.h | 34 +++++++++++++++++++----
> include/linux/virtio_byteorder.h | 24 ++++++++++-------
> include/linux/virtio_config.h | 19 +++++++++----
> include/linux/vringh.h | 19 +++++++++----
> include/uapi/linux/vhost.h | 5 +++
> 9 files changed, 156 insertions(+), 32 deletions(-)
>
> --
> Greg
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v3 0/7] vhost: support for cross endian guests
2015-04-07 12:09 ` Greg Kurz
(?)
@ 2015-04-07 15:55 ` Michael S. Tsirkin
-1 siblings, 0 replies; 9+ messages in thread
From: Michael S. Tsirkin @ 2015-04-07 15:55 UTC (permalink / raw)
To: Greg Kurz; +Cc: linux-api, linux-kernel, kvm, virtualization
On Tue, Apr 07, 2015 at 02:09:29PM +0200, Greg Kurz wrote:
> Hi,
>
> This patchset allows vhost to be used with legacy virtio when guest and host
> have a different endianness.
>
> Patches 1-6 remain the same as the previous post. Patch 7 was heavily changed
> according to MST's comments.
This still doesn't actually work, right?
tun and macvtap need new ioctls too ...
> ---
>
> Greg Kurz (7):
> 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: feature to set the vring endianness
>
>
> drivers/net/macvtap.c | 11 ++++++--
> drivers/net/tun.c | 11 ++++++--
> drivers/vhost/Kconfig | 10 +++++++
> drivers/vhost/vhost.c | 55 ++++++++++++++++++++++++++++++++++++++
> drivers/vhost/vhost.h | 34 +++++++++++++++++++----
> include/linux/virtio_byteorder.h | 24 ++++++++++-------
> include/linux/virtio_config.h | 19 +++++++++----
> include/linux/vringh.h | 19 +++++++++----
> include/uapi/linux/vhost.h | 5 +++
> 9 files changed, 156 insertions(+), 32 deletions(-)
>
> --
> Greg
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH v3 0/7] vhost: support for cross endian guests
@ 2015-04-07 12:09 ` Greg Kurz
0 siblings, 0 replies; 9+ messages in thread
From: Greg Kurz @ 2015-04-07 12:09 UTC (permalink / raw)
To: Rusty Russell, Michael S. Tsirkin
Cc: linux-api, linux-kernel, kvm, virtualization
Hi,
This patchset allows vhost to be used with legacy virtio when guest and host
have a different endianness.
Patches 1-6 remain the same as the previous post. Patch 7 was heavily changed
according to MST's comments.
---
Greg Kurz (7):
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: feature to set the vring endianness
drivers/net/macvtap.c | 11 ++++++--
drivers/net/tun.c | 11 ++++++--
drivers/vhost/Kconfig | 10 +++++++
drivers/vhost/vhost.c | 55 ++++++++++++++++++++++++++++++++++++++
drivers/vhost/vhost.h | 34 +++++++++++++++++++----
include/linux/virtio_byteorder.h | 24 ++++++++++-------
include/linux/virtio_config.h | 19 +++++++++----
include/linux/vringh.h | 19 +++++++++----
include/uapi/linux/vhost.h | 5 +++
9 files changed, 156 insertions(+), 32 deletions(-)
--
Greg
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH v3 0/7] vhost: support for cross endian guests
@ 2015-04-07 12:09 ` Greg Kurz
0 siblings, 0 replies; 9+ messages in thread
From: Greg Kurz @ 2015-04-07 12:09 UTC (permalink / raw)
To: Rusty Russell, Michael S. Tsirkin
Cc: linux-api-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA, kvm-u79uwXL29TY76Z2rM5mHXA,
virtualization-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA
Hi,
This patchset allows vhost to be used with legacy virtio when guest and host
have a different endianness.
Patches 1-6 remain the same as the previous post. Patch 7 was heavily changed
according to MST's comments.
---
Greg Kurz (7):
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: feature to set the vring endianness
drivers/net/macvtap.c | 11 ++++++--
drivers/net/tun.c | 11 ++++++--
drivers/vhost/Kconfig | 10 +++++++
drivers/vhost/vhost.c | 55 ++++++++++++++++++++++++++++++++++++++
drivers/vhost/vhost.h | 34 +++++++++++++++++++----
include/linux/virtio_byteorder.h | 24 ++++++++++-------
include/linux/virtio_config.h | 19 +++++++++----
include/linux/vringh.h | 19 +++++++++----
include/uapi/linux/vhost.h | 5 +++
9 files changed, 156 insertions(+), 32 deletions(-)
--
Greg
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2015-04-07 16:08 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-04-07 12:09 [PATCH v3 0/7] vhost: support for cross endian guests Greg Kurz
2015-04-07 12:09 Greg Kurz
2015-04-07 12:09 ` Greg Kurz
2015-04-07 15:55 ` Michael S. Tsirkin
2015-04-07 15:55 ` Michael S. Tsirkin
2015-04-07 15:55 ` Michael S. Tsirkin
2015-04-07 16:08 ` Greg Kurz
2015-04-07 16:08 ` Greg Kurz
2015-04-07 16:08 ` Greg Kurz
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.