<div dir="ltr">Sorry for late response, I spot this one just now.<div>Liu already merge it so please take this just as note for consideration.</div><div><br></div><div>Since the NIC is the same in ThunderX and OcteonTX, it in fact apply to both.</div><div>So V1 was better in my opinion.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On 1 December 2017 at 12:08, Jogarao Nartu <span dir="ltr"><<a href="mailto:njogarao@caviumnetworks.com" target="_blank">njogarao@caviumnetworks.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Testpmd must be run with hardware VLAN filter disabled as OcteonTx driver<br>
does not support this feature.<br>
<br>
Signed-off-by: Jogarao Nartu <<a href="mailto:njogarao@caviumnetworks.com">njogarao@caviumnetworks.com</a>><br>
---<br>
<br>
 v2 changes:<br>
   Made NIC name specific to Cavium OcteonTx<br>
<br>
 tests/TestSuite_unit_tests_<wbr>dump.py | 21 ++++++++++++++++++---<br>
 1 file changed, 18 insertions(+), 3 deletions(-)<br>
<br>
diff --git a/tests/TestSuite_unit_tests_<wbr>dump.py b/tests/TestSuite_unit_tests_<wbr>dump.py<br>
index e36fa64..f7985f2 100644<br>
--- a/tests/TestSuite_unit_tests_<wbr>dump.py<br>
+++ b/tests/TestSuite_unit_tests_<wbr>dump.py<br>
@@ -85,7 +85,12 @@ class TestUnitTestsDump(TestCase):<br>
         """<br>
         Run history log dump test case.<br>
         """<br>
-        self.dut.send_expect("./%s/<wbr>app/testpmd -n 1 -c f -- -i" % (self.target), "testpmd>", self.start_test_time)<br>
+        cmd = "./%s/app/testpmd -n 1 -c f -- -i" % self.target<br>
+<br>
+        if "cavium_a034" in self.dut.nic_type:<br>
+            cmd += " --disable-hw-vlan-filter"<br>
+<br>
+        self.dut.send_expect("%s" % cmd, "testpmd>", self.start_test_time)<br>
         out = self.dut.send_expect("dump_<wbr>ring", "testpmd>", self.run_cmd_time)<br>
         self.dut.send_expect("quit", "# ")<br>
         match_regex = "ring <(.*?)>"<br>
@@ -100,7 +105,12 @@ class TestUnitTestsDump(TestCase):<br>
         """<br>
         Run mempool dump test case.<br>
         """<br>
-        self.dut.send_expect("./%s/<wbr>app/testpmd -n 1 -c f -- -i" % (self.target), "testpmd>", self.start_test_time)<br>
+        cmd = "./%s/app/testpmd -n 1 -c f -- -i" % self.target<br>
+<br>
+        if "cavium_a034" in self.dut.nic_type:<br>
+            cmd += " --disable-hw-vlan-filter"<br>
+<br>
+        self.dut.send_expect("%s" % cmd, "testpmd>", self.start_test_time)<br>
         out = self.dut.send_expect("dump_<wbr>mempool", "testpmd>", self.run_cmd_time * 2)<br>
         self.dut.send_expect("quit", "# ")<br>
         match_regex = "mempool <(.*?)>@0x(.*?)\r\n"<br>
@@ -133,7 +143,12 @@ class TestUnitTestsDump(TestCase):<br>
         """<br>
         Run memzone dump test case.<br>
         """<br>
-        self.dut.send_expect("./%s/<wbr>app/testpmd -n 1 -c f -- -i" % (self.target), "testpmd>", self.start_test_time)<br>
+        cmd = "./%s/app/testpmd -n 1 -c f -- -i" % self.target<br>
+<br>
+        if "cavium_a034" in self.dut.nic_type:<br>
+            cmd += " --disable-hw-vlan-filter"<br>
+<br>
+        self.dut.send_expect("%s" % cmd, "testpmd>", self.start_test_time)<br>
         out = self.dut.send_expect("dump_<wbr>memzone", "testpmd>", self.run_cmd_time * 2)<br>
         self.dut.send_expect("quit", "# ")<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
2.7.4<br>
<br>
</font></span></blockquote></div><br></div>