From: "De Lara Guarch, Pablo" <pablo.de.lara.guarch@intel.com>
To: "Ananyev, Konstantin" <konstantin.ananyev@intel.com>,
"dev@dpdk.org" <dev@dpdk.org>
Subject: Re: [dpdk-dev] [PATCHv5 1/2] testpmd: add ability to split outgoing packets
Date: Mon, 9 Nov 2015 14:06:05 +0000 [thread overview]
Message-ID: <E115CCD9D858EF4F90C690B0DCB4D8973C854DC5@IRSMSX108.ger.corp.intel.com> (raw)
In-Reply-To: <2601191342CEEE43887BDE71AB97725836AC61B0@irsmsx105.ger.corp.intel.com>
> -----Original Message-----
> From: Ananyev, Konstantin
> Sent: Monday, November 09, 2015 1:45 PM
> To: De Lara Guarch, Pablo; dev@dpdk.org
> Subject: RE: [dpdk-dev] [PATCHv5 1/2] testpmd: add ability to split outgoing
> packets
>
>
>
> > -----Original Message-----
> > From: De Lara Guarch, Pablo
> > Sent: Monday, November 09, 2015 1:39 PM
> > To: Ananyev, Konstantin; dev@dpdk.org
> > Subject: RE: [dpdk-dev] [PATCHv5 1/2] testpmd: add ability to split
> outgoing packets
> >
> > Hi Konstantin,
> >
> > > -----Original Message-----
> > > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Konstantin
> > > Ananyev
> > > Sent: Monday, November 09, 2015 9:20 AM
> > > To: dev@dpdk.org
> > > Subject: [dpdk-dev] [PATCHv5 1/2] testpmd: add ability to split outgoing
> > > packets
> > >
> > > For CSUM forwarding mode add ability to copy & split outgoing packet
> > > into the new mbuf that consists of multiple segments.
> > > For TXONLY and CSUM forwarding modes add ability to make number of
> > > segments in the outgoing packet to vary on a per packet basis.
> > > Number of segments and size of each segment is controlled by
> > > 'set txpkts' command.
> > > Split policy is controlled by 'set txsplit' command.
> > > Possible values are: on | off | rand.
> > > Tha allows to increase test coverage for TX PMD codepaths.
> > >
> > > Signed-off-by: Konstantin Ananyev <konstantin.ananyev@intel.com>
> > > ---
> > > app/test-pmd/cmdline.c | 57 +++++++++-
> > > app/test-pmd/config.c | 61 +++++++++++
> > > app/test-pmd/csumonly.c | 163
> > > +++++++++++++++++++++++++++-
> > > app/test-pmd/testpmd.c | 3 +
> > > app/test-pmd/testpmd.h | 10 ++
> > > app/test-pmd/txonly.c | 13 ++-
> > > doc/guides/testpmd_app_ug/testpmd_funcs.rst | 11 +-
> > > 7 files changed, 309 insertions(+), 9 deletions(-)
> > >
> >
> > [...]
> >
> > > diff --git a/doc/guides/testpmd_app_ug/testpmd_funcs.rst
> > > b/doc/guides/testpmd_app_ug/testpmd_funcs.rst
> > > index 4fb1e0b..4608b3f 100644
> > > --- a/doc/guides/testpmd_app_ug/testpmd_funcs.rst
> > > +++ b/doc/guides/testpmd_app_ug/testpmd_funcs.rst
> > > @@ -211,7 +211,7 @@ show config
> > > Displays the configuration of the application.
> > > The configuration comes from the command-line, the runtime or the
> > > application defaults::
> > >
> > > - testpmd> show config (rxtx|cores|fwd)
> > > + testpmd> show config (rxtx|cores|fwd|txpkts)
> > >
> > > The available information categories are:
> > >
> > > @@ -221,6 +221,8 @@ The available information categories are:
> > >
> > > * ``fwd``: Packet forwarding configuration.
> > >
> > > +* ``txpkts``: Packets to TX configuration.
> > > +
> > > For example:
> > >
> > > .. code-block:: console
> > > @@ -396,6 +398,13 @@ Set the length of each segment of the TX-ONLY
> > > packets::
> > >
> > > Where x[,y]* represents a CSV list of values, without white space.
> > >
> > > +set txsplit
> > > +~~~~~~~~~~~
> > > +
> > > +Set the length of each segment of the TX packets, applicable for TX-
> ONLY
> > > and CSUM forwarding modes::
> > > +
> > > + testpmd> set txsplit (off|on|rand)
> > > +
> > > set corelist
> > > ~~~~~~~~~~~~
> > >
> > > --
> > > 1.8.5.3
> >
> > Description of "set txsplit" does not look correct. Could you fix it?
>
> Ups, sure will fix it.
>
> > It would be nice to describe the three different options (off, on, rand) as
> well.
>
> Ok, will do.
> Any other comments before I'll re-spin it?
Just a typo ("packts") in config.c
+ tx_pkt_split tx_pkt_split = TX_PKT_SPLIT_OFF;
+ /**< Split policy for packts to TX. */
> Thanks
> Konstantin
>
>
> >
> > Thanks,
> > Pablo
next prev parent reply other threads:[~2015-11-09 14:06 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-11-09 9:20 [dpdk-dev] [PATCHv5 0/2] ixgbe: fix TX hang when RS distance exceeds HW limit Konstantin Ananyev
2015-11-09 9:20 ` [dpdk-dev] [PATCHv5 1/2] testpmd: add ability to split outgoing packets Konstantin Ananyev
2015-11-09 13:38 ` De Lara Guarch, Pablo
2015-11-09 13:45 ` Ananyev, Konstantin
2015-11-09 14:06 ` De Lara Guarch, Pablo [this message]
2015-11-09 9:20 ` [dpdk-dev] [PATCHv5 2/2] ixgbe: fix TX hang when RS distance exceeds HW limit Konstantin Ananyev
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=E115CCD9D858EF4F90C690B0DCB4D8973C854DC5@IRSMSX108.ger.corp.intel.com \
--to=pablo.de.lara.guarch@intel.com \
--cc=dev@dpdk.org \
--cc=konstantin.ananyev@intel.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).