[dts] [PATCH] rx interrupt: Fixed test case

[Owen Hilyard]

Hello

I'm able to see a material difference between what I've suggested and what
the prior test case did. I was attempting to reduce the number of
invocations of a pmd during the test, since those invocations are time
consuming and, from what I measured, made up the majority of the runtime of
the test. Is there a reason why all queues and port's can't be opened at
the same time and then ignored until they are needed? The way I re-did the
configs was designed to create all possible combinations of settings in the
format that was originally there. Are all 3 invocations of the pmd needed
or is it possible to merge those and throw out my other changes? Most of my
changes were done because I was already planning on submitting a patch to
remove the extra invocations and aren't as important.

Thanks for your help
Owen

On Tue, Sep 1, 2020 at 10:02 PM Ma, LihongX <lihongx.ma at intel.com> wrote:

> Hi, Owen
> I think the change of the plan is not make sense, the case ' PF interrupt
> pmd with different queue' is want to test the interrupt on different queue,
> The original case will test the queue on min number, max number and normal
> number(between minimum and maximum), but your patch will only test one
> situation.
>
>
> Regards,
> Ma,lihong
>
> > -----Original Message-----
> > From: dts <dts-bounces at dpdk.org> On Behalf Of Owen Hilyard
> > Sent: Wednesday, August 26, 2020 11:04 PM
> > To: dts at dpdk.org
> > Cc: Zhang, Yuwei1 <yuwei1.zhang at intel.com>; changqingx.wu at intel.com;
> Xiao,
> > QimaiX <qimaix.xiao at intel.com>; Hunt, David <david.hunt at intel.com>;
> > lylavoie at iol.unh.edu; Owen Hilyard <ohilyard at iol.unh.edu>
> > Subject: [dts] [PATCH] rx interrupt: Fixed test case
> >
> > fixed test case issues with eal params
> > removed extra instances of l3fwd-power
> >
> > Signed-off-by: Owen Hilyard <ohilyard at iol.unh.edu>
> > ---
> >  test_plans/interrupt_pmd_test_plan.rst | 58 +++++++++-----------
> >  tests/TestSuite_interrupt_pmd.py       | 73 ++++++++++++++------------
> >  2 files changed, 64 insertions(+), 67 deletions(-)
> >
> > diff --git a/test_plans/interrupt_pmd_test_plan.rst
> > b/test_plans/interrupt_pmd_test_plan.rst
> > index cb8b2f1..1f8816d 100644
> > --- a/test_plans/interrupt_pmd_test_plan.rst
> > +++ b/test_plans/interrupt_pmd_test_plan.rst
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mails.dpdk.org/archives/dts/attachments/20200902/7caa4c08/attachment.html>