Hi,
> -----Original Message-----
> From: Burakov, Anatoly
> Sent: Monday, July 16, 2018 4:16 PM
> To: Pattan, Reshma <reshma.pat...@intel.com>; tho...@monjalon.net;
> dev@dpdk.org
> Cc: Parthasarathy, JananeeX M <jananeex.m.parthasara...@intel.com>;
> sta...@dpdk.org
> Subject: Re: [PATCH v3 8/9] autotest: update autotest test case list
>
>
> > + {
> > + "Name": "Set rxtx mode",
> > + "Command": "set_rxtx_mode",
> > + "Func": default_autotest,
> > + "Report": None,
> > + },
> > + {
> > + "Name": "Set rxtx anchor",
> > + "Command": "set_rxtx_anchor",
> > + "Func": default_autotest,
> > + "Report": None,
> > + },
> > + {
> > + "Name": "Set rxtx sc",
> > + "Command": "set_rxtx_sc",
> > + "Func": default_autotest,
> > + "Report": None,
> > + },
>
> The above three tests don't look like autotests to me. I have no idea what
> they are for, but either they need a special function, or they need to be
> taken
> out.
>
These commands needs to be run manually from test cmd prompt to various set
rxtx mode, rxtx rate and rxtx direction .
These can be used to verify pmd perf test with vaiours set of above values.
So this can be removed from autotest.
> > + "Name": "User delay",
> > + "Command": "user_delay_us",
> > + "Func": default_autotest,
> > + "Report": None,
> > + },
>
> This doesn't look like autotests to me. I have no idea what it is for, but
> either
> it needs a special function, or it needs to be taken out.
>
This is autotest but the name does'nt have the autotest name in it. So I will
retain this.
Thanks,
Reshma
