[dts] [PATCH v3] Add Cryptodev Unit Tests Cases

Liu, Yong yong.liu at intel.com
Wed Mar 8 08:43:22 CET 2017


Zhaoyan,
I guess the reason why Kill this applcation is for next around. Am I right?
We used to handle this kind of action in tear_down function.
For exception handler, there's no need to dump exception for the only one reason is timeout.
In general, let framework handle those kind exceptions and keep suite clean. 

Thanks,
Marvin

> -----Original Message-----
> From: Chen, Zhaoyan
> Sent: Wednesday, March 08, 2017 3:32 PM
> To: Liu, Yong <yong.liu at intel.com>; dts at dpdk.org
> Subject: RE: [dts] [PATCH v3] Add Cryptodev Unit Tests Cases
> 
> I do not recover the application, just error handling.
> Since QAT HW unstable sometime.
> 
> Since just function test, (not stable test),
> So, for each test case will launch a separated test application.
> 
> But, there is another idea that run different unit tests by
> a defined scenario. Actually, there is 1 DPDK-AE bug find by this mode.
> 
> Anyway, for the unit test, I just want to de-couple the completed factors.
> So each test app launching  for each unit test suite.
> 
> Next step is that design some combined tests by requirements.
> 
> Thanks.
> 
> Regards,
> Zhaoyan Chen
> 
> > -----Original Message-----
> > From: Liu, Yong
> > Sent: Wednesday, March 8, 2017 3:05 PM
> > To: Chen, Zhaoyan <zhaoyan.chen at intel.com>; dts at dpdk.org
> > Subject: Re: [dts] [PATCH v3] Add Cryptodev Unit Tests Cases
> >
> > Zhaoyan,
> > Why try to recover test application when there's no response? Is that
> > application not stable after run crypto unit test?
> > If there's one need to do that, please do it in tear_down function.
> >
> > Thanks,
> > Marvin
> >
> > On 03/08/2017 01:01 PM, Chen, Zhaoyan wrote:
> > > +        try:
> > > +            out = self.dut.send_expect(testsuite, "RTE>>", timeout)
> > > +            self.dut.send_expect("quit", "# ", 30)
> > > +        except Exception, ex:
> > > +            print ex
> > > +            print "run cryptodev_unit_tests process coredump"
> > > +            self.dut.alt_session.send_expect("killall test", "# ", 30)
> > > +            dmesg = self.dut.alt_session.send_expect("dmesg", "# ",
> 30)
> > > +            print dmesg
> > > +        print out



More information about the dts mailing list