Re: [dts] [PATCH V1 2/4] fix TestSuite unit_tests_lpm

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

Hi gang, test suite is not the right instance to handle hugepage. We 
have planned to let unified module to handle this.
My suggestion is that just increased default FreeBSD huge page memory 
size in framework.
And warning that huge page is insufficient in linux OS.

On 09/21/2016 03:20 PM, xu,gang wrote:
> Signed-off-by: xu,gang <gangx.xu@intel.com>
> ---
>   tests/TestSuite_unit_tests_lpm.py | 15 +++++++++------
>   1 file changed, 9 insertions(+), 6 deletions(-)
>
> diff --git a/tests/TestSuite_unit_tests_lpm.py b/tests/TestSuite_unit_tests_lpm.py
> index dadb492..253cbb4 100644
> --- a/tests/TestSuite_unit_tests_lpm.py
> +++ b/tests/TestSuite_unit_tests_lpm.py
> @@ -83,16 +83,19 @@ class TestUnitTestsLpmIpv6(TestCase):
>           """
>           [arch, machine, env, toolchain] = self.target.split('-')
>           self.verify(arch == "x86_64", "lpm6 request huge memory")
> -
> -        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")
> +        if env == 'bsdapp':
> +            self.dut.send_expect("kenv hw.contigmem.num_buffers=4", "#", 60)
> +        else:
> +            hugepage_ori = self.dut.get_total_huge_pages()
> +            page_size = self.dut.send_expect("awk '/Hugepagesize/ {print $2}' /proc/meminfo", "# ")
> +            if hugepage_ori * int(page_size) <= 4096 * 2048:
> +                self.dut.set_huge_pages(4096)
> +                hugepage_num = self.dut.get_total_huge_pages()
> +                self.verify(hugepage_num >= 4096, "failed to request huge memory")
>   
>           self.dut.send_expect("./app/test/test -n 1 -c ffff", "R.*T.*E.*>.*>", 60)
>           out = self.dut.send_expect("lpm6_autotest", "RTE>>", 3600)
>           self.dut.send_expect("quit", "# ")
> -        self.dut.set_huge_pages(hugepage_ori)
>           self.verify("Test OK" in out, "Test failed")
>   
>       def tear_down(self):