* Re: Change dev->needed_headroom of virtio_net to make Patch"virtio-net: put virtio net header inline with data"work better
[not found] <60A73B7A161A82449A7DB7B382BFD15537026EF3@SZXEMA504-MBS.china.huawei.com>
@ 2014-04-24 11:44 ` Michael S. Tsirkin
2014-04-24 11:50 ` Zhangjie (HZ)
0 siblings, 1 reply; 2+ messages in thread
From: Michael S. Tsirkin @ 2014-04-24 11:44 UTC (permalink / raw)
To: Zhangjie (HZ)
Cc: jasowang@redhat.com, Qinchuanyu, Liuyongan, kvm@vger.kernel.org,
netdev
On Thu, Apr 24, 2014 at 10:19:58AM +0000, Zhangjie (HZ) wrote:
> Hi!
>
> Patch “virtio-net: put virtio net header inline with data” , has a notable
> improvement for TCP packages.
>
> But UDP packages from virtio_net nic, do not have enough head room. I wonder if
> we can set dev->needed_headroom
>
> to the size of virtio net header, so as to put the header in. By doing this,
> udp get about 5% improvement in bandwidth.
Sounds like a reasonable thing to do.
Want to post the patch so people can try it out?
>
>
> --
>
> Thanks,
>
> //Zhang Jie
>
>
>
^ permalink raw reply [flat|nested] 2+ messages in thread
* RE: Change dev->needed_headroom of virtio_net to make Patch"virtio-net: put virtio net header inline with data"work better
2014-04-24 11:44 ` Change dev->needed_headroom of virtio_net to make Patch"virtio-net: put virtio net header inline with data"work better Michael S. Tsirkin
@ 2014-04-24 11:50 ` Zhangjie (HZ)
0 siblings, 0 replies; 2+ messages in thread
From: Zhangjie (HZ) @ 2014-04-24 11:50 UTC (permalink / raw)
To: Michael S. Tsirkin
Cc: jasowang@redhat.com, Qinchuanyu, Liuyongan, kvm@vger.kernel.org,
netdev@vger.kernel.org
Thank you very much!
I will post a patch tomorrow.
--Best wishes!
//Zhang Jie
-----Original Message-----
From: Michael S. Tsirkin [mailto:mst@redhat.com]
Sent: Thursday, April 24, 2014 7:45 PM
To: Zhangjie (HZ)
Cc: jasowang@redhat.com; Qinchuanyu; Liuyongan; kvm@vger.kernel.org; netdev@vger.kernel.org
Subject: Re: Change dev->needed_headroom of virtio_net to make Patch"virtio-net: put virtio net header inline with data"work better
On Thu, Apr 24, 2014 at 10:19:58AM +0000, Zhangjie (HZ) wrote:
> Hi!
>
> Patch “virtio-net: put virtio net header inline with data” , has a
> notable improvement for TCP packages.
>
> But UDP packages from virtio_net nic, do not have enough head room. I
> wonder if we can set dev->needed_headroom
>
> to the size of virtio net header, so as to put the header in. By doing
> this, udp get about 5% improvement in bandwidth.
Sounds like a reasonable thing to do.
Want to post the patch so people can try it out?
>
>
> --
>
> Thanks,
>
> //Zhang Jie
>
>
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2014-04-24 11:50 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <60A73B7A161A82449A7DB7B382BFD15537026EF3@SZXEMA504-MBS.china.huawei.com>
2014-04-24 11:44 ` Change dev->needed_headroom of virtio_net to make Patch"virtio-net: put virtio net header inline with data"work better Michael S. Tsirkin
2014-04-24 11:50 ` Zhangjie (HZ)
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).