From: Lawrence MacIntyre <macintyrelp@ornl.gov>
To: Saurabh Mishra <saurabh.globe@gmail.com>
Cc: dev@dpdk.org
Subject: Re: [dpdk-dev] rte_mbuf size for jumbo frame
Date: Tue, 26 Jan 2016 11:50:16 -0500 [thread overview]
Message-ID: <56A7A3C8.7040200@ornl.gov> (raw)
In-Reply-To: <CAMnwyJ1M2u7wJM+5OiMd8EQocfJOVM8DFHgeDYiaLr4LZXmbYQ@mail.gmail.com>
Saurabh:
It sounds like you benchmarked Apache using Jumbo Packets, but not the
DPDK app using large mbufs. Those are two entirely different issues.
You should be able to write your packet inspection routines to work with
the mbuf chains, rather than copying them into a larger buffer (although
if there are multiple passes through the data, it could be a bit
complicated). Copying the data into a larger buffer will definitely
cause the application to be slower.
Lawrence
This one time (01/26/2016 09:40 AM), at band camp, Saurabh Mishra wrote:
>
> Hi,
>
> Since we do full content inspection, we will end up coalescing mbuf
> chains into one before inspecting the packet which would require
> allocating another buffer of larger size.
>
> I am inclined towards larger size mbuf for this reason.
>
> I have benchmarked a bit using apache benchmark and we see 3x
> performance improvement over 1500 mtu. Memory is not an issue.
>
> My only concern is that would all the dpdk drivers work with larger
> size mbuf?
>
> Thanks,
> Saurabh
>
> On Jan 26, 2016 6:23 AM, "Lawrence MacIntyre" <macintyrelp@ornl.gov
> <mailto:macintyrelp@ornl.gov>> wrote:
>
> Saurabh:
>
> Raising the mbuf size will make the packet handling for large
> packets slightly more efficient, but it will use much more memory
> unless the great majority of the packets you are handling are of
> the jumbo size. Using more memory has its own costs. In order to
> evaluate this design choice, it is necessary to understand the
> behavior of the memory subsystem, which is VERY complicated.
>
> Before you go down this path, at least benchmark your application
> using the regular sized mbufs and the large ones and see what the
> effect is.
>
> This one time (01/26/2016 09:01 AM), at band camp, Polehn, Mike A
> wrote:
>
> Jumbo frames are generally handled by link lists (but called
> something else) of mbufs.
> Enabling jumbo frames for the device driver should enable the
> right portion of the driver which handles the linked lists.
>
> Don't make the mbufs huge.
>
> Mike
>
> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org
> <mailto:dev-bounces@dpdk.org>] On Behalf Of Masaru OKI
> Sent: Monday, January 25, 2016 2:41 PM
> To: Saurabh Mishra; users@dpdk.org <mailto:users@dpdk.org>;
> dev@dpdk.org <mailto:dev@dpdk.org>
> Subject: Re: [dpdk-dev] rte_mbuf size for jumbo frame
>
> Hi,
>
> 1. Take care of unit size of mempool for mbuf.
> 2. Call rte_eth_dev_set_mtu() for each interface.
> Note that some PMDs does not supported change MTU.
>
> On 2016/01/26 6:02, Saurabh Mishra wrote:
>
> Hi,
>
> We wanted to use 10400 bytes size of each rte_mbuf to
> enable Jumbo frames.
> Do you guys see any problem with that? Would all the
> drivers like
> ixgbe, i40e, vmxnet3, virtio and bnx2x work with larger
> rte_mbuf size?
>
> We would want to avoid detailing with chained mbufs.
>
> /Saurabh
>
>
> --
> Lawrence MacIntyre macintyrelp@ornl.gov
> <mailto:macintyrelp@ornl.gov> Oak Ridge National Laboratory
> 865.574.7401 <tel:865.574.7401> Cyber Space and Information
> Intelligence Research Group
>
--
Lawrence MacIntyre macintyrelp@ornl.gov Oak Ridge National Laboratory
865.574.7401 Cyber Space and Information Intelligence Research Group
next prev parent reply other threads:[~2016-01-26 16:50 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-01-25 21:02 Saurabh Mishra
2016-01-25 22:40 ` Masaru OKI
2016-01-26 14:01 ` Polehn, Mike A
2016-01-26 14:23 ` Lawrence MacIntyre
2016-01-26 14:40 ` Saurabh Mishra
2016-01-26 16:50 ` Lawrence MacIntyre [this message]
2016-01-26 17:14 ` Saurabh Mishra
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=56A7A3C8.7040200@ornl.gov \
--to=macintyrelp@ornl.gov \
--cc=dev@dpdk.org \
--cc=saurabh.globe@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 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).