From: "Michael S. Tsirkin" <mst@redhat.com>
To: Greg Kurz <gkurz@linux.vnet.ibm.com>
Cc: Cornelia Huck <cornelia.huck@de.ibm.com>, qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH v3 0/3] virtio-net/vhost-net: share cross-endian enablement
Date: Wed, 18 Nov 2015 22:48:06 +0200 [thread overview]
Message-ID: <20151118224739-mutt-send-email-mst@redhat.com> (raw)
In-Reply-To: <20151118162119.9938.45927.stgit@bahia.huguette.org>
On Wed, Nov 18, 2015 at 05:23:00PM +0100, Greg Kurz wrote:
> Since QEMU 2.4.0, vhost-net uses the cross-endian support of TAP devices to
> fix vnet headers. In fact, virtio-net can do the same instead of hackily
> patching headers in virtio_net_hdr_swap().
>
> This series moves the enablement of cross-endian support from vhost-net to
> virtio-net: both vhost and full emulation can now benefit from it. Of course
> we keep the current hack to fall back on when the backend doesn't support
> cross-endian.
Thanks!
This is an optimization rather than a bugfix, right?
As such I'd rather defer this until after 2.5.
> ---
>
> Greg Kurz (3):
> virtio-net: use the backend cross-endian capabilities
> Revert "vhost-net: tell tap backend about the vnet endianness"
> virtio: drop the virtio_needs_swap() helper
>
>
> hw/net/vhost_net.c | 33 +------------------------------
> hw/net/virtio-net.c | 40 +++++++++++++++++++++++++++++++++++--
> include/hw/virtio/virtio-access.h | 9 --------
> include/hw/virtio/virtio-net.h | 1 +
> 4 files changed, 40 insertions(+), 43 deletions(-)
next prev parent reply other threads:[~2015-11-18 20:48 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-11-18 16:23 [Qemu-devel] [PATCH v3 0/3] virtio-net/vhost-net: share cross-endian enablement Greg Kurz
2015-11-18 16:24 ` [Qemu-devel] [PATCH v3 1/3] virtio-net: use the backend cross-endian capabilities Greg Kurz
2015-11-18 16:24 ` [Qemu-devel] [PATCH v3 2/3] Revert "vhost-net: tell tap backend about the vnet endianness" Greg Kurz
2015-11-18 16:24 ` [Qemu-devel] [PATCH v3 3/3] virtio: drop the virtio_needs_swap() helper Greg Kurz
2015-11-18 20:48 ` Michael S. Tsirkin [this message]
2015-11-18 21:46 ` [Qemu-devel] [PATCH v3 0/3] virtio-net/vhost-net: share cross-endian enablement Greg Kurz
2015-12-17 8:02 ` Greg Kurz
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20151118224739-mutt-send-email-mst@redhat.com \
--to=mst@redhat.com \
--cc=cornelia.huck@de.ibm.com \
--cc=gkurz@linux.vnet.ibm.com \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).