<div dir="auto"><div dir="auto"><span style="font-family:sans-serif">> -        self.dut.send_expect("./%s/app</span><span style="font-family:sans-serif">/testpmd -c f -n 4 -- -i" % self.target,</span><br style="font-family:sans-serif"><span style="font-family:sans-serif">> "testpmd> ", 120)</span><br style="font-family:sans-serif"><span style="font-family:sans-serif">> +        self.dut.send_expect("./%s/app</span><span style="font-family:sans-serif">/testpmd -c %s -n 4 -- -i" %</span><br style="font-family:sans-serif"><span style="font-family:sans-serif">> + (self.target, self.coremask), "testpmd> ", 120)</span></div><div dir="auto"><br style="font-family:sans-serif"><span style="font-family:sans-serif">[Lijuan] Do you forget to change hardcode of memory channel?</span><br style="font-family:sans-serif"><span style="font-family:sans-serif">>          out = self.dut.send_expect("create bonded device 3 0", "testpmd> ", 30)</span><br style="font-family:sans-serif"><span style="font-family:sans-serif">>          bond_device_id = int(re.search("port \d+", out).group().split(" ")[-1].strip())</span><br style="font-family:sans-serif"></div><div dir="auto"><br></div><div>Hi all, </div><div dir="auto"><br></div><div dir="auto">Good point. </div><div dir="auto">Actually, the number of channels is a non mandatory EAL parameter, and in DTS it defaults to 1.</div><div dir="auto"><a href="http://git.dpdk.org/tools/dts/tree/framework/dut.py#n565">http://git.dpdk.org/tools/dts/tree/framework/dut.py#n565</a></div><div dir="auto"><br>In this module, the other invocations of testpmd were indeed with get_memory_channels().</div><div dir="auto">In this occurrence it is hardcoded to 4.</div><div dir="auto"><div dir="auto" style="font-family:sans-serif"><br>The number of memory channels is merely an optimization, as you can see in the following thread:</div><div dir="auto" style="font-family:sans-serif"><a href="https://dev.dpdk.narkive.com/78gtSXha/dpdk-dev-patch-eal-default-to-one-memory-channel-if-not-specified">https://dev.dpdk.narkive.com/78gtSXha/dpdk-dev-patch-eal-default-to-one-memory-channel-if-not-specified</a></div><div dir="auto" style="font-family:sans-serif"><br></div><div dir="auto" style="font-family:sans-serif">According to it,</div><div dir="auto" style="font-family:sans-serif">...</div></div><div dir="auto" style="font-family:sans-serif"><span style="color:rgb(34,34,34);font-family:"helvetica neue",arial,sans-serif;font-size:15px">Obtaining the correct value of memory channels, especially from a</span><br style="color:rgb(34,34,34);font-family:"helvetica neue",arial,sans-serif;font-size:15px"><span style="color:rgb(34,34,34);font-family:"helvetica neue",arial,sans-serif;font-size:15px">running system, can be anything from difficult to plain impossible.</span><br></div><div dir="auto" style="font-family:sans-serif"><span style="color:rgb(34,34,34);font-family:"helvetica neue",arial,sans-serif;font-size:15px">...</span></div><div dir="auto" style="font-family:sans-serif"><span style="color:rgb(34,34,34);font-family:"helvetica neue",arial,sans-serif;font-size:15px"><br></span></div><div dir="auto">Not sure what was the reason for hard coding it to 4 in this test case, but anyhow on this occasion maybe it is better to replace it with get_memory_channels() as with the rest of the module, for the sake of consistency.</div><div dir="auto"><br></div><div dir="auto">Regards,</div><div dir="auto">Rami Rosen</div><div dir="auto"><br></div><div dir="auto"><br></div><div dir="auto"><br><br><div class="gmail_quote" dir="auto"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br><br>
</blockquote></div></div></div>