libbpf: Add a check to ensure that page_cnt is non-zero
authorYuntao Wang <ytcoode@gmail.com>
Thu, 3 Mar 2022 00:59:21 +0000 (08:59 +0800)
committerDaniel Borkmann <daniel@iogearbox.net>
Thu, 3 Mar 2022 15:23:22 +0000 (16:23 +0100)
commit41332d6e3a430adc91e0af115b4261b0d2f116ec
treeb90fb49b6b8b7bf751e72dc0114e7b010c6cb0f3
parent8bbe98bdccef0bb4fe88c666c28a3d4fe51151f7
libbpf: Add a check to ensure that page_cnt is non-zero

The page_cnt parameter is used to specify the number of memory pages
allocated for each per-CPU buffer, it must be non-zero and a power of 2.

Currently, the __perf_buffer__new() function attempts to validate that
the page_cnt is a power of 2 but forgets checking for the case where
page_cnt is zero, we can fix it by replacing 'page_cnt & (page_cnt - 1)'
with 'page_cnt == 0 || (page_cnt & (page_cnt - 1))'.

If so, we also don't need to add a check in perf_buffer__new_v0_6_0() to
make sure that page_cnt is non-zero and the check for zero in
perf_buffer__new_raw_v0_6_0() can also be removed.

The code will be cleaner and more readable.

Signed-off-by: Yuntao Wang <ytcoode@gmail.com>
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
Link: https://lore.kernel.org/bpf/20220303005921.53436-1-ytcoode@gmail.com
tools/lib/bpf/libbpf.c