Re: [PATCH v2] selftests/bpf/test_maps: exit child process without error in ENOMEM case

From: Daniel Borkmann
Date: Thu Feb 22 2018 - 09:25:20 EST


On 02/22/2018 03:34 AM, Li Zhijian wrote:
> From: Li Zhijian <zhijianx.li@xxxxxxxxx>
>
> test_maps contains a series of stress tests, and previously it will break the
> rest tests when it failed to alloc memory.
> -----------------------
> Failed to create hashmap key=8 value=262144 'Cannot allocate memory'
> Failed to create hashmap key=16 value=262144 'Cannot allocate memory'
> Failed to create hashmap key=8 value=262144 'Cannot allocate memory'
> Failed to create hashmap key=8 value=262144 'Cannot allocate memory'
> test_maps: test_maps.c:955: run_parallel: Assertion `status == 0' failed.
> Aborted
> not ok 1..3 selftests: test_maps [FAIL]
> -----------------------
> after this patch, the rest tests will be continue when it occurs an ENOMEM failure
>
> CC: Alexei Starovoitov <alexei.starovoitov@xxxxxxxxx>
> CC: Philip Li <philip.li@xxxxxxxxx>
> Suggested-by: Daniel Borkmann <daniel@xxxxxxxxxxxxx>
> Signed-off-by: Li Zhijian <zhijianx.li@xxxxxxxxx>

Applied to bpf tree, thanks Li!