From: Avi Kivity <avi@redhat.com>
To: Mark McLoughlin <markmc@redhat.com>
Cc: Matthew Faulkner <matthew.faulkner@gmail.com>,
kvm@vger.kernel.org, Anthony Liguori <aliguori@us.ibm.com>
Subject: Re: [PATCH] kvm: qemu: increase the size of the tap buffer [was Re: TAP MTU >= 4055 problem?]
Date: Sun, 02 Nov 2008 16:32:45 +0200 [thread overview]
Message-ID: <490DBA0D.1050705@redhat.com> (raw)
In-Reply-To: <1225456125.3758.41.camel@blaa>
Mark McLoughlin wrote:
> It sucks that the kernel just silently truncates and doesn't give
> userspace a chance to retry with a larger buffer. We could add that as
> an extension, I guess.
>
>
What we want even more is to give the kernel a set of buffers (iovec)
and have it fill them as much as it can with a number of packets, giving
us some info as to where frames begin. Manage the whole thing using aio
reads.
> As debugged by Matthew Faulkner, with a 4k byte tap buffer if you
> increase the MTU on the tap device to greater than 4k, the packets
> read by qemu into the tap buffer will be truncated and the guest
> will discard the packet.
>
> With GSO enabled, we use a 64k tap buffer, so let's just use a 64k
> buffer in all cases.
>
> Also, remove the obtuse logic for figuring out the max GSO buffer
> size. We shouldn't receive IP packets larger than 64k, so let's just
> use 17 pages to make sure we've enough room for headers.
>
>
Applied, thanks.
--
error compiling committee.c: too many arguments to function
prev parent reply other threads:[~2008-11-02 14:32 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-10-30 9:41 TAP MTU >= 4055 problem? Matthew Faulkner
2008-10-30 14:29 ` Javier Guerra
2008-10-30 15:56 ` Matthew Faulkner
2008-10-30 17:47 ` Matthew Faulkner
2008-10-30 19:43 ` Matthew Faulkner
2008-10-31 12:28 ` [PATCH] kvm: qemu: increase the size of the tap buffer [was Re: TAP MTU >= 4055 problem?] Mark McLoughlin
2008-11-02 14:32 ` Avi Kivity [this message]
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=490DBA0D.1050705@redhat.com \
--to=avi@redhat.com \
--cc=aliguori@us.ibm.com \
--cc=kvm@vger.kernel.org \
--cc=markmc@redhat.com \
--cc=matthew.faulkner@gmail.com \
/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 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.