<div dir="ltr">Hello<br><br><div>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. <br><br>Thanks for your help</div><div>Owen</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Sep 1, 2020 at 10:02 PM Ma, LihongX <<a href="mailto:lihongx.ma@intel.com">lihongx.ma@intel.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi, Owen<br>
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,<br>
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.<br>
<br>
<br>
Regards,<br>
Ma,lihong<br>
<br>
> -----Original Message-----<br>
> From: dts <<a href="mailto:dts-bounces@dpdk.org" target="_blank">dts-bounces@dpdk.org</a>> On Behalf Of Owen Hilyard<br>
> Sent: Wednesday, August 26, 2020 11:04 PM<br>
> To: <a href="mailto:dts@dpdk.org" target="_blank">dts@dpdk.org</a><br>
> Cc: Zhang, Yuwei1 <<a href="mailto:yuwei1.zhang@intel.com" target="_blank">yuwei1.zhang@intel.com</a>>; <a href="mailto:changqingx.wu@intel.com" target="_blank">changqingx.wu@intel.com</a>; Xiao,<br>
> QimaiX <<a href="mailto:qimaix.xiao@intel.com" target="_blank">qimaix.xiao@intel.com</a>>; Hunt, David <<a href="mailto:david.hunt@intel.com" target="_blank">david.hunt@intel.com</a>>;<br>
> <a href="mailto:lylavoie@iol.unh.edu" target="_blank">lylavoie@iol.unh.edu</a>; Owen Hilyard <<a href="mailto:ohilyard@iol.unh.edu" target="_blank">ohilyard@iol.unh.edu</a>><br>
> Subject: [dts] [PATCH] rx interrupt: Fixed test case<br>
> <br>
> fixed test case issues with eal params<br>
> removed extra instances of l3fwd-power<br>
> <br>
> Signed-off-by: Owen Hilyard <<a href="mailto:ohilyard@iol.unh.edu" target="_blank">ohilyard@iol.unh.edu</a>><br>
> ---<br>
>  test_plans/interrupt_pmd_test_plan.rst | 58 +++++++++-----------<br>
>  tests/TestSuite_interrupt_pmd.py       | 73 ++++++++++++++------------<br>
>  2 files changed, 64 insertions(+), 67 deletions(-)<br>
> <br>
> diff --git a/test_plans/interrupt_pmd_test_plan.rst<br>
> b/test_plans/interrupt_pmd_test_plan.rst<br>
> index cb8b2f1..1f8816d 100644<br>
> --- a/test_plans/interrupt_pmd_test_plan.rst<br>
> +++ b/test_plans/interrupt_pmd_test_plan.rst<br>
<br>
</blockquote></div>