Re: [PATCH bpf-next 1/4] libbpf: Add bpf_program__add_flags() and bpf_program__clear_flags()

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Low] The newly added multi-line comment in `libbpf.h` violates the required BPF subsystem comment style.
--

commit 0677df91f2b2dedb24874db25663854a4619dfdd
Author: Toke Høiland-Jørgensen <toke@redhat.com>

libbpf: Add bpf_program__add_flags() and bpf_program__clear_flags()

This commit introduces two convenience helpers, bpf_program__add_flags() and
bpf_program__clear_flags(), to non-destructively add or remove BPF program
flags. These wrap the existing bpf_program__set_flags() using bitwise
operations to avoid clobbering existing flags.

> diff --git a/tools/lib/bpf/libbpf.h b/tools/lib/bpf/libbpf.h
> index b965ad5715403..3932bf9cb4903 100644
> --- a/tools/lib/bpf/libbpf.h
> +++ b/tools/lib/bpf/libbpf.h
[ ... ]
> @@ -1011,6 +1011,12 @@ bpf_program__set_expected_attach_type(struct bpf_program *prog,
>  LIBBPF_API __u32 bpf_program__flags(const struct bpf_program *prog);
>  LIBBPF_API int bpf_program__set_flags(struct bpf_program *prog, __u32 flags);
>  
> +/* Convenience helpers to non-destructively add or clear the specified flags
> + * instead of clobbering them as bpf_program__set_flags() does.
> + */

[Severity: Low]
This isn't a bug, but does this newly added multi-line comment violate the BPF
subsystem comment style? In the BPF subsystem, multi-line comments should have
the opening /* on its own line, with the comment text starting on the next
line.

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260903092416.223294-1-toke@redhat.com?part=1