DPDK usage discussions
 help / color / mirror / Atom feed
From: Cliff Burdick <shaklee3@gmail.com>
To: Thomas Monjalon <thomas@monjalon.net>
Cc: David Marchand <david.marchand@redhat.com>,
	users <users@dpdk.org>,  Ori Kam <orika@nvidia.com>
Subject: Re: Generic flow string parser
Date: Thu, 27 Apr 2023 06:19:21 -0700	[thread overview]
Message-ID: <CA+Gp1nZZmf-U_J0Y=N9OHNK4+1yk=XBcWm7eM3BKW0XBEYEvhA@mail.gmail.com> (raw)
In-Reply-To: <1814734.QZUTf85G27@thomas>

[-- Attachment #1: Type: text/plain, Size: 1672 bytes --]

Hi Thomas, testpmd has a 12,000 line parser just for taking in strings and
converting it to flow rules. This is obviously useful for testing flows,
but it also is an interface for any type of flow imaginable since this is
where they're tested.

Now imagine you're developing an application that allows the user to
specify custom flows in a config. Your only option is to make your own flow
string input (json, etc) and convert that to to the flow spec. This is
reinventing almost all of what testpmd already does, and it's extremely
error-prone. I think it would be very useful to have this as an API call
rather than a user constructing each flow by hand so that all these other
applications can benefit and not be worries about bugs during conversions.



On Thu, Apr 27, 2023, 01:37 Thomas Monjalon <thomas@monjalon.net> wrote:

> 26/04/2023 07:47, David Marchand:
> > On Wed, Apr 26, 2023 at 6:47 AM Cliff Burdick <shaklee3@gmail.com>
> wrote:
> > >
> > > Does anyone know if a generic parser for flow strings exists anywhere?
> The one inside of testpmd is ideal, but unfortunately it's self-contained
> and not distributed as part of a normal DPDK install. This seems like
> something that is likely reinvented over and over and it would be useful if
> there was a single API to take in strings and generate flows.
> >
> > I heard this same question in the past, but I don't remember the answer.
> > Copying Thomas and Ori who might know.
>
> I'm not sure how the testpmd code could help another application.
> And in general, if your application has a CLI,
> you need to integrate the flow commands in a broader context.
>
>
>

[-- Attachment #2: Type: text/html, Size: 2151 bytes --]

  reply	other threads:[~2023-04-27 13:19 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-04-26  4:46 Cliff Burdick
2023-04-26  5:47 ` David Marchand
2023-04-27  8:37   ` Thomas Monjalon
2023-04-27 13:19     ` Cliff Burdick [this message]
2023-04-28 17:36       ` Tom Barbette
2023-04-28 18:09         ` Stephen Hemminger
2023-04-28 19:13           ` Cliff Burdick
2023-04-29  0:04             ` Stephen Hemminger
2023-04-29  0:08               ` Stephen Hemminger
2023-04-29 14:23               ` Cliff Burdick
2023-04-29 21:39                 ` Thomas Monjalon
2023-04-29 21:49                   ` Cliff Burdick
2023-05-26 22:35                     ` Cliff Burdick
2023-06-05 16:36                   ` kumaraparameshwaran rathinavel

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='CA+Gp1nZZmf-U_J0Y=N9OHNK4+1yk=XBcWm7eM3BKW0XBEYEvhA@mail.gmail.com' \
    --to=shaklee3@gmail.com \
    --cc=david.marchand@redhat.com \
    --cc=orika@nvidia.com \
    --cc=thomas@monjalon.net \
    --cc=users@dpdk.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).