Re: [dts] [PATCH V1] tests/unit_tests_lpm: Add the size of the judgment hugepage

[Jianbo Liu <jianbo.liu@linaro.org>]

Hi Peipei,

Is there any requirement that the total hugepages should be larger than 8G?
Besides, self.verify will exit this test, not raise one warning as you
declared. Could it be wrong?

Thanks!
Jianbo

On 6 September 2017 at 16:05, lu,peipei <peipeix.lu@intel.com> wrote:
> Check whether host hugepage fulfill the requirement.If host setup can't run this case, just raise one warning
>
> Signed-off-by: lu,peipei <peipeix.lu@intel.com>
> ---
>  tests/TestSuite_unit_tests_lpm.py | 15 ++++++---------
>  1 file changed, 6 insertions(+), 9 deletions(-)
>
> diff --git a/tests/TestSuite_unit_tests_lpm.py b/tests/TestSuite_unit_tests_lpm.py
> index e8149ff..f9a2bf7 100644
> --- a/tests/TestSuite_unit_tests_lpm.py
> +++ b/tests/TestSuite_unit_tests_lpm.py
> @@ -81,21 +81,18 @@ class TestUnitTestsLpmIpv6(TestCase):
>          """
>          [arch, machine, env, toolchain] = self.target.split('-')
>          self.verify(arch in ["x86_64", "arm64"], "lpm6 request huge memory")
> -        if env == 'bsdapp':
> +        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")
> +           hugepage_8G = 8 * 1024 * 1024
> +            total_hugepage = hugepages_size * hugepage_ori
> +            self.verify(total_hugepage >= hugepage_8G,"have no enough hugepage")
>
>          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", "# ")
> -        if env == 'bsdapp':
> -            pass
> -        else:
> -            self.dut.set_huge_pages(hugepage_ori)
>          self.verify("Test OK" in out, "Test failed")
>
>      def tear_down(self):
> --
> 1.9.3
>