test suite reviews and discussions
 help / color / mirror / Atom feed
* [dts] [PATCH V1]vhost_dequeue_zero_copy: only two cases must run on fvl40g not all suite
@ 2020-03-30  8:53 Xiao Qimai
  2020-03-30 11:39 ` Wang, Yinan
  2020-03-31  3:06 ` Tu, Lijuan
  0 siblings, 2 replies; 4+ messages in thread
From: Xiao Qimai @ 2020-03-30  8:53 UTC (permalink / raw)
  To: dts; +Cc: Xiao Qimai

*. two cases only run on flv40g have been added in checklist

Signed-off-by: Xiao Qimai <qimaix.xiao@intel.com>
---
 tests/TestSuite_vhost_dequeue_zero_copy.py | 1 -
 1 file changed, 1 deletion(-)

diff --git a/tests/TestSuite_vhost_dequeue_zero_copy.py b/tests/TestSuite_vhost_dequeue_zero_copy.py
index 9e65543..f426fe3 100644
--- a/tests/TestSuite_vhost_dequeue_zero_copy.py
+++ b/tests/TestSuite_vhost_dequeue_zero_copy.py
@@ -56,7 +56,6 @@ class TestVhostDequeueZeroCopy(TestCase):
         """
         Run at the start of each test suite.
         """
-        self.verify(self.nic in ("fortville_spirit"), "the port can not run this suite")
         self.tester.extend_external_packet_generator(TestVhostDequeueZeroCopy, self)
         self.frame_sizes = [64, 128, 256, 512, 1024, 1518]
         self.dut_ports = self.dut.get_ports()
-- 
1.8.3.1


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [dts] [PATCH V1]vhost_dequeue_zero_copy: only two cases must run on fvl40g not all suite
  2020-03-30  8:53 [dts] [PATCH V1]vhost_dequeue_zero_copy: only two cases must run on fvl40g not all suite Xiao Qimai
@ 2020-03-30 11:39 ` Wang, Yinan
  2020-03-31  2:58   ` Xiao, QimaiX
  2020-03-31  3:06 ` Tu, Lijuan
  1 sibling, 1 reply; 4+ messages in thread
From: Wang, Yinan @ 2020-03-30 11:39 UTC (permalink / raw)
  To: Xiao, QimaiX, dts; +Cc: Xiao, QimaiX

Acked-by: Wang, Yinan <yinan.wang@intel.com>

> -----Original Message-----
> From: dts <dts-bounces@dpdk.org> On Behalf Of Xiao Qimai
> Sent: 2020年3月30日 16:53
> To: dts@dpdk.org
> Cc: Xiao, QimaiX <qimaix.xiao@intel.com>
> Subject: [dts] [PATCH V1]vhost_dequeue_zero_copy: only two cases must
> run on fvl40g not all suite
> 
> *. two cases only run on flv40g have been added in checklist
> 
> Signed-off-by: Xiao Qimai <qimaix.xiao@intel.com>
> ---
>  tests/TestSuite_vhost_dequeue_zero_copy.py | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/tests/TestSuite_vhost_dequeue_zero_copy.py
> b/tests/TestSuite_vhost_dequeue_zero_copy.py
> index 9e65543..f426fe3 100644
> --- a/tests/TestSuite_vhost_dequeue_zero_copy.py
> +++ b/tests/TestSuite_vhost_dequeue_zero_copy.py
> @@ -56,7 +56,6 @@ class TestVhostDequeueZeroCopy(TestCase):
>          """
>          Run at the start of each test suite.
>          """
> -        self.verify(self.nic in ("fortville_spirit"), "the port can not run this suite")
> 
> self.tester.extend_external_packet_generator(TestVhostDequeueZeroCopy,
> self)
>          self.frame_sizes = [64, 128, 256, 512, 1024, 1518]
>          self.dut_ports = self.dut.get_ports()
> --
> 1.8.3.1


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [dts] [PATCH V1]vhost_dequeue_zero_copy: only two cases must run on fvl40g not all suite
  2020-03-30 11:39 ` Wang, Yinan
@ 2020-03-31  2:58   ` Xiao, QimaiX
  0 siblings, 0 replies; 4+ messages in thread
From: Xiao, QimaiX @ 2020-03-31  2:58 UTC (permalink / raw)
  To: Wang, Yinan, dts

Tested-by: Xiao, QimaiX <qimaix.xiao@intel.com>

Regards,
Xiao Qimai

> -----Original Message-----
> From: Wang, Yinan
> Sent: Monday, March 30, 2020 7:40 PM
> To: Xiao, QimaiX <qimaix.xiao@intel.com>; dts@dpdk.org
> Cc: Xiao, QimaiX <qimaix.xiao@intel.com>
> Subject: RE: [dts] [PATCH V1]vhost_dequeue_zero_copy: only two cases
> must run on fvl40g not all suite
> 
> Acked-by: Wang, Yinan <yinan.wang@intel.com>
> 
> > -----Original Message-----
> > From: dts <dts-bounces@dpdk.org> On Behalf Of Xiao Qimai
> > Sent: 2020年3月30日 16:53
> > To: dts@dpdk.org
> > Cc: Xiao, QimaiX <qimaix.xiao@intel.com>
> > Subject: [dts] [PATCH V1]vhost_dequeue_zero_copy: only two cases must
> > run on fvl40g not all suite
> >
> > *. two cases only run on flv40g have been added in checklist
> >
> > Signed-off-by: Xiao Qimai <qimaix.xiao@intel.com>
> > ---
> >  tests/TestSuite_vhost_dequeue_zero_copy.py | 1 -
> >  1 file changed, 1 deletion(-)
> >
> > diff --git a/tests/TestSuite_vhost_dequeue_zero_copy.py
> > b/tests/TestSuite_vhost_dequeue_zero_copy.py
> > index 9e65543..f426fe3 100644
> > --- a/tests/TestSuite_vhost_dequeue_zero_copy.py
> > +++ b/tests/TestSuite_vhost_dequeue_zero_copy.py
> > @@ -56,7 +56,6 @@ class TestVhostDequeueZeroCopy(TestCase):
> >          """
> >          Run at the start of each test suite.
> >          """
> > -        self.verify(self.nic in ("fortville_spirit"), "the port can not run this
> suite")
> >
> >
> self.tester.extend_external_packet_generator(TestVhostDequeueZeroCop
> y,
> > self)
> >          self.frame_sizes = [64, 128, 256, 512, 1024, 1518]
> >          self.dut_ports = self.dut.get_ports()
> > --
> > 1.8.3.1


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [dts] [PATCH V1]vhost_dequeue_zero_copy: only two cases must run on fvl40g not all suite
  2020-03-30  8:53 [dts] [PATCH V1]vhost_dequeue_zero_copy: only two cases must run on fvl40g not all suite Xiao Qimai
  2020-03-30 11:39 ` Wang, Yinan
@ 2020-03-31  3:06 ` Tu, Lijuan
  1 sibling, 0 replies; 4+ messages in thread
From: Tu, Lijuan @ 2020-03-31  3:06 UTC (permalink / raw)
  To: Xiao, QimaiX, dts; +Cc: Xiao, QimaiX

Applied, thanks

> -----Original Message-----
> From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of Xiao Qimai
> Sent: Monday, March 30, 2020 4:53 PM
> To: dts@dpdk.org
> Cc: Xiao, QimaiX <qimaix.xiao@intel.com>
> Subject: [dts] [PATCH V1]vhost_dequeue_zero_copy: only two cases must
> run on fvl40g not all suite
> 
> *. two cases only run on flv40g have been added in checklist
> 
> Signed-off-by: Xiao Qimai <qimaix.xiao@intel.com>
> ---
>  tests/TestSuite_vhost_dequeue_zero_copy.py | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/tests/TestSuite_vhost_dequeue_zero_copy.py
> b/tests/TestSuite_vhost_dequeue_zero_copy.py
> index 9e65543..f426fe3 100644
> --- a/tests/TestSuite_vhost_dequeue_zero_copy.py
> +++ b/tests/TestSuite_vhost_dequeue_zero_copy.py
> @@ -56,7 +56,6 @@ class TestVhostDequeueZeroCopy(TestCase):
>          """
>          Run at the start of each test suite.
>          """
> -        self.verify(self.nic in ("fortville_spirit"), "the port can not run this suite")
> 
> self.tester.extend_external_packet_generator(TestVhostDequeueZeroCopy,
> self)
>          self.frame_sizes = [64, 128, 256, 512, 1024, 1518]
>          self.dut_ports = self.dut.get_ports()
> --
> 1.8.3.1


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2020-03-31  3:06 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-03-30  8:53 [dts] [PATCH V1]vhost_dequeue_zero_copy: only two cases must run on fvl40g not all suite Xiao Qimai
2020-03-30 11:39 ` Wang, Yinan
2020-03-31  2:58   ` Xiao, QimaiX
2020-03-31  3:06 ` Tu, Lijuan

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).