On 15/07/2015 08:34, damu wrote:
> Hi,
>
> Today I tried no-huge option with app/test, the test app crashed.
> It seems that memory can not be allocated.
> I think it could be this change causing the panic. Can someone check it?
>
>
Hi Damu,
Actually it is the following commit causing the panic:
commit b3dfffd962ecd7a1d8700193b4b3305dc85e7ae4
Author: Cyril Chemparathy <cchemparathy at ezchip.com>
Date: Thu Jul 9 16:25:15 2015 +0800
mem: allow multiple page sizes to be requested
Cause is that since allowing multiple page sizes it always expects the
hugepage_sz
field to be set to a value, but when using --no-huge, the field
hugepage_sz is 0 and
the logic does not work as expected.
The issue could be fixed but adding logic to expect hugepage_sz being 0
or by setting
hugepage_sz to a value (RTE_PGSIZE_4K) when using --no-huge.
The latter would be my preference. I'll send a patch fixing it.
Sergio
