On 23/1/26 11:55, Alexei Starovoitov wrote: > On Thu, Jan 22, 2026 at 7:25 PM Leon Hwang <[email protected]> wrote: >> >> >> +static int probe_bpf_syscall_common_attrs(int token_fd) >> +{ >> + int ret; >> + >> + ret = probe_sys_bpf_ext(); >> + return ret > 0; >> +} > > When you look at the above, what thoughts come to mind? > > ... and please don't use ai for answers. My initial thought was whether probe_fd() is needed here to handle and close a returned fd, since the return value of probe_sys_bpf_ext() isn’t obvious from the call site. Thanks, Leon
- [PATCH bpf-next v7 0/9] bpf: Extend BPF syscall with co... Leon Hwang
- [PATCH bpf-next v7 1/9] bpf: Extend BPF syscall wi... Leon Hwang
- [PATCH bpf-next v7 2/9] libbpf: Add support for ex... Leon Hwang
- Re: [PATCH bpf-next v7 2/9] libbpf: Add suppor... Alexei Starovoitov
- Re: [PATCH bpf-next v7 2/9] libbpf: Add su... Leon Hwang
- Re: [PATCH bpf-next v7 2/9] libbpf: Ad... Alexei Starovoitov
- Re: [PATCH bpf-next v7 2/9] libbp... Leon Hwang
- Re: [PATCH bpf-next v7 2/9] l... Andrii Nakryiko
- Re: [PATCH bpf-next v7 2/... Leon Hwang
- [PATCH bpf-next v7 4/9] bpf: Add syscall common at... Leon Hwang
- [PATCH bpf-next v7 3/9] bpf: Refactor reporting lo... Leon Hwang
- [PATCH bpf-next v7 5/9] bpf: Refactor reporting bt... Leon Hwang
- [PATCH bpf-next v7 7/9] bpf: Add syscall common at... Leon Hwang
- [PATCH bpf-next v7 6/9] bpf: Add syscall common at... Leon Hwang
- [PATCH bpf-next v7 9/9] selftests/bpf: Add tests t... Leon Hwang

