From: Maxime Coquelin <maxime.coquelin@redhat.com>
To: "Dey, Souvik" <sodey@sonusnet.com>,
"stephen@networkplumber.org" <stephen@networkplumber.org>
Cc: "dev@dpdk.org" <dev@dpdk.org>
Subject: Re: [dpdk-dev] [PATCH v2] add mtu set in virtio
Date: Fri, 2 Sep 2016 09:05:49 +0200 [thread overview]
Message-ID: <49bce2ef-ffbd-96ff-6f9c-6c37bb7c7a90@redhat.com> (raw)
In-Reply-To: <BN6PR03MB274023874CCD5C981E176920DAE20@BN6PR03MB2740.namprd03.prod.outlook.com>
Hi Souvik,
On 09/02/2016 12:20 AM, Dey, Souvik wrote:
> Hi Maxime,
> When is patches or new implementation going to come in the release ? if it is not 16.11 then, can we keep this change till the new virtio changes come in the release. And if it is already planned for 16.11, then can I get a little more information on that.
>
I'm currently working on qemu part implementation, first RFC should be
sent next week.
Goal is to have it in 16.11, but I cannot commit, as the spec update
has not been acked yet.
For more information, you can start by having a look at the spec review:
https://lists.oasis-open.org/archives/virtio-dev/201608/msg00056.html
Regards,
Maxime
> --
> Regards,
> Souvik
>
> -----Original Message-----
> From: Maxime Coquelin [mailto:maxime.coquelin@redhat.com]
> Sent: Tuesday, August 30, 2016 3:58 AM
> To: Dey, Souvik <sodey@sonusnet.com>; stephen@networkplumber.org; huawei.xie@intel.com; yuanhan.liu@linux.intel.com
> Cc: dev@dpdk.org
> Subject: Re: [dpdk-dev] [PATCH v2] add mtu set in virtio
>
> Hi Souvik,
>
> On 08/30/2016 01:02 AM, souvikdey33 wrote:
>> Signed-off-by: Souvik Dey <sodey@sonusnet.com>
>>
>> Fixes: 1fb8e8896ca8 ("Signed-off-by: Souvik Dey <sodey@sonusnet.com>")
>> Reviewed-by: Stephen Hemminger <stephen@networkplumber.org>
>>
>> Virtio interfaces should also support setting of mtu, as in case of
>> cloud it is expected to have the consistent mtu across the
>> infrastructure that the dhcp server sends and not hardcoded to 1500(default).
>> ---
>> drivers/net/virtio/virtio_ethdev.c | 12 ++++++++++++
>> 1 file changed, 12 insertions(+)
>
> FYI, there are some on-going changes in the VIRTIO specification so that the VHOST interface exposes its MTU to its VIRTIO peer.
> It may also be used as an alternative of what you patch achieves.
>
> I am working on its implementation in Qemu/DPDK, our goal being to reduce performance drops for small packets with Rx mergeable buffers feature enabled.
>
> Regards,
> Maxime
>
next prev parent reply other threads:[~2016-09-02 7:05 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-08-29 23:02 souvikdey33
2016-08-30 7:57 ` Maxime Coquelin
2016-09-01 22:20 ` Dey, Souvik
2016-09-02 7:05 ` Maxime Coquelin [this message]
2016-09-07 2:11 ` Dey, Souvik
2016-09-07 8:28 ` Maxime Coquelin
2016-09-07 3:25 ` Yuanhan Liu
2016-09-07 9:16 ` Maxime Coquelin
2016-09-08 7:30 ` Yuanhan Liu
2016-09-08 7:50 ` Maxime Coquelin
2016-09-08 7:57 ` Yuanhan Liu
2016-09-09 3:44 ` Dey, Souvik
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=49bce2ef-ffbd-96ff-6f9c-6c37bb7c7a90@redhat.com \
--to=maxime.coquelin@redhat.com \
--cc=dev@dpdk.org \
--cc=sodey@sonusnet.com \
--cc=stephen@networkplumber.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).