Re: [PATCH bpf-next v3 4/6] libbpf: Initialize the bpf_seq_printf parameters array field by field

From: Andrii Nakryiko
Date: Tue Apr 13 2021 - 19:18:17 EST


On Mon, Apr 12, 2021 at 8:38 AM Florent Revest <revest@xxxxxxxxxxxx> wrote:
>
> When initializing the __param array with a one liner, if all args are
> const, the initial array value will be placed in the rodata section but
> because libbpf does not support relocation in the rodata section, any
> pointer in this array will stay NULL.
>
> Fixes: c09add2fbc5a ("tools/libbpf: Add bpf_iter support")
> Signed-off-by: Florent Revest <revest@xxxxxxxxxxxx>
> ---

Looks good!

Acked-by: Andrii Nakryiko <andrii@xxxxxxxxxx>

> tools/lib/bpf/bpf_tracing.h | 40 +++++++++++++++++++++++++++----------
> 1 file changed, 29 insertions(+), 11 deletions(-)
>

[...]