Re: [PATCH 4/6] test_bpf: add module parameters to filter the tests to run.

On 08/03/2015 06:23 PM, Nicolas Schichan wrote:
...

quoted

Btw, for the range test in prepare_bpf_tests(), you could also reject
a negative lower bound index right there.

I thought it was better to have all the sanity checks grouped in
prepare_bpf_tests() (with the checking of the test_name and test_id parameters
nearby) ? Also a negative lower bound is meaning that no range has been set so
all tests should be run.

I just got a bit confused when loading test_range=-100,1 was not rejected, but
they do indeed all run in this case.

Thanks,
Daniel