From: Jeremy Spewock <jspewock@iol.unh.edu>
To: "Juraj Linkeš" <juraj.linkes@pantheon.tech>
Cc: probb@iol.unh.edu, paul.szczepanek@arm.com,
yoan.picchi@foss.arm.com, thomas@monjalon.net,
wathsala.vithanage@arm.com, Honnappa.Nagarahalli@arm.com,
alex.chapman@arm.com, Luca.Vizzarro@arm.com,
npratte@iol.unh.edu, dev@dpdk.org
Subject: Re: [PATCH v1] dts: add send_packets to test_suite
Date: Thu, 26 Sep 2024 14:09:11 -0400 [thread overview]
Message-ID: <CAAA20UShCEHEtsbbgVNfq=6BHU_-17v-SMT+zKP3_rcyp6h2Pw@mail.gmail.com> (raw)
In-Reply-To: <856c8b97-a658-4ee9-a14b-d91a171c88a3@pantheon.tech>
On Thu, Sep 26, 2024 at 5:56 AM Juraj Linkeš <juraj.linkes@pantheon.tech> wrote:
>
>
>
> On 25. 9. 2024 20:36, jspewock@iol.unh.edu wrote:
> > From: Jeremy Spewock <jspewock@iol.unh.edu>
> >
> > Currently the only methods provided in the test suite class for sending
> > packets capture the resulting received traffic after sending. There is,
> > in some cases, a need to send multiple packets at once while not really
> > needing to capture any of said received traffic. It is favorable to
> > avoid capturing received traffic when you don't need it since not all
> > traffic generators will necessarily be capturing traffic generators.
> > The method to fulfill this need exists in the traffic generator
> > already, but this patch exposes the method to test suites.
> >
> > Depends-on: patch-144440 ("dts: rework packet addressing")
>
> This could be in the same series. I wanted this to be in a different
> patch, but both patches could (and should) be in the same series.
>
Oops, I just submitted a new version of the other series before seeing
this, I will add this back into that series and resubmit.
> >
> > Signed-off-by: Jeremy Spewock <jspewock@iol.unh.edu>
> > ---
>
> The Depends-on: tag should be here, below the three hyphens. [0]
>
> [0] https://doc.dpdk.org/guides/contributing/patches.html#patch-dependencies
>
Ack, sorry, I always forget this.
> Other than that,
> Reviewed-by: Juraj Linkeš <juraj.linkes@pantheon.tech>
>
>
next prev parent reply other threads:[~2024-09-26 18:09 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-09-25 18:36 jspewock
2024-09-26 9:56 ` Juraj Linkeš
2024-09-26 18:09 ` Jeremy Spewock [this message]
2024-09-26 12:59 ` Patrick Robb
2024-09-26 16:36 ` Patrick Robb
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='CAAA20UShCEHEtsbbgVNfq=6BHU_-17v-SMT+zKP3_rcyp6h2Pw@mail.gmail.com' \
--to=jspewock@iol.unh.edu \
--cc=Honnappa.Nagarahalli@arm.com \
--cc=Luca.Vizzarro@arm.com \
--cc=alex.chapman@arm.com \
--cc=dev@dpdk.org \
--cc=juraj.linkes@pantheon.tech \
--cc=npratte@iol.unh.edu \
--cc=paul.szczepanek@arm.com \
--cc=probb@iol.unh.edu \
--cc=thomas@monjalon.net \
--cc=wathsala.vithanage@arm.com \
--cc=yoan.picchi@foss.arm.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).