[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-stable] [Qemu-devel] [PATCH] net: don't set native endianness
From: |
Jason Wang |
Subject: |
Re: [Qemu-stable] [Qemu-devel] [PATCH] net: don't set native endianness |
Date: |
Thu, 15 Oct 2015 09:45:26 +0800 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 |
On 10/14/2015 05:26 PM, Michael S. Tsirkin wrote:
> commit 5be7d9f1b1452613b95c6ba70b8d7ad3d0797991
> vhost-net: tell tap backend about the vnet endianness
> makes vhost net always try to set LE - even if that matches the
> native endian-ness.
>
> This makes it fail on older kernels on x86 without TUNSETVNETLE support.
>
> To fix, make qemu_set_vnet_le/qemu_set_vnet_be skip the
> ioctl if it matches the host endian-ness.
>
> Reported-by: Marcel Apfelbaum <address@hidden>
> Cc: Greg Kurz <address@hidden>
> Cc: address@hidden
> Signed-off-by: Michael S. Tsirkin <address@hidden>
Acked-by: Jason Wang <address@hidden>
> ---
> net/net.c | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/net/net.c b/net/net.c
> index 28a5597..8e96011 100644
> --- a/net/net.c
> +++ b/net/net.c
> @@ -517,20 +517,28 @@ void qemu_set_vnet_hdr_len(NetClientState *nc, int len)
>
> int qemu_set_vnet_le(NetClientState *nc, bool is_le)
> {
> +#ifdef HOST_WORDS_BIGENDIAN
> if (!nc || !nc->info->set_vnet_le) {
> return -ENOSYS;
> }
>
> return nc->info->set_vnet_le(nc, is_le);
> +#else
> + return 0;
> +#endif
> }
>
> int qemu_set_vnet_be(NetClientState *nc, bool is_be)
> {
> +#ifdef HOST_WORDS_BIGENDIAN
> + return 0;
> +#else
> if (!nc || !nc->info->set_vnet_be) {
> return -ENOSYS;
> }
>
> return nc->info->set_vnet_be(nc, is_be);
> +#endif
> }
>
> int qemu_can_send_packet(NetClientState *sender)