Re: [dts] [PATCH V1] tests/unit_tests_lpm: fix set hugepage failed

["Liu, Yong" <yong.liu@intel.com>]

Peipei,
As we discussed before, please just check whether host hugepage fulfill the requirement.
If host setup can't run this case, just raise one warning and skip it.

Thanks,
Marvin

> -----Original Message-----
> From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of Lu, PeipeiX
> Sent: Tuesday, September 05, 2017 10:42 AM
> To: dts@dpdk.org
> Cc: Lu, PeipeiX <peipeix.lu@intel.com>
> Subject: Re: [dts] [PATCH V1] tests/unit_tests_lpm: fix set hugepage
> failed
> 
> Hi yong ,
> 
> Have you reviewed this patch, any comments? Thanks
> 
> Peipei
> 
> -----Original Message-----
> From: Lu, PeipeiX
> Sent: Friday, September 01, 2017 1:42 PM
> To: dts@dpdk.org
> Cc: Lu, PeipeiX
> Subject: [dts][PATCH V1] tests/unit_tests_lpm: fix set hugepage failed
> 
> jubge hugepage size and the number of hugepages,if allocate enough
> hugepage was pass, Otherwise need to adjust.
> 
> Signed-off-by: lu,peipei <peipeix.lu@intel.com>
> ---
>  tests/TestSuite_unit_tests_lpm.py | 13 ++++++++-----
>  1 file changed, 8 insertions(+), 5 deletions(-)
> 
> diff --git a/tests/TestSuite_unit_tests_lpm.py
> b/tests/TestSuite_unit_tests_lpm.py
> index e8149ff..53e65d9 100644
> --- a/tests/TestSuite_unit_tests_lpm.py
> +++ b/tests/TestSuite_unit_tests_lpm.py
> @@ -81,14 +81,17 @@ class TestUnitTestsLpmIpv6(TestCase):
>          """
>          [arch, machine, env, toolchain] = self.target.split('-')
>          self.verify(arch in ["x86_64", "arm64"], "lpm6 request huge
> memory")
> +	hugepages_size = int(self.dut.send_expect("awk '/Hugepagesize/
> {print $2}' /proc/meminfo", "# "))
> +	hugepage_ori = self.dut.get_total_huge_pages()
>          if env == 'bsdapp':
>              pass
>          else:
> -            hugepage_ori = self.dut.get_total_huge_pages()
> -            self.dut.set_huge_pages(4096)
> -            hugepage_num = self.dut.get_total_huge_pages()
> -            self.verify(hugepage_num >= 4096, "failed to request huge
> memory")
> -
> +	   1G_hugepage = 1024 * 1024
> +           8G_hugepage = 8 * 1G_hugepage
> +	   if (hugepages_size * hugepage_ori) < 8G_hugepage and
> hugepage_size != 1G_hugepage:
> +	      self.dut.set_huge_pages(8G_hugepage / hugepages_size)
> +	   else:
> +	      pass
>          self.dut.send_expect("./%s/app/test -n 1 -c f" % self.target,
> "R.*T.*E.*>.*>", 60)
>          out = self.dut.send_expect("lpm6_autotest", "RTE>>", 3600)
>          self.dut.send_expect("quit", "# ")
> --
> 1.9.3