On Wed, 2024-02-28 at 15:43 -0800, Andrii Nakryiko wrote:
[...]
> > +static void can_load_partial_object(void)
> > +{
> > + DECLARE_LIBBPF_OPTS(bpf_object_open_opts, opts);
>
> nit: prefer LIBBPF_OPTS() over DECLARE_LIBBPF_OPTS()
Ok
[...]
> > +void serial_test_struct_ops_autocreate(void)
>
> same as in the previous patch, why serial?
Because of the print callback hijacking.
Also, what would happen when two tests would set struct_ops for same
attachment point?
[...]
> > +SEC(".struct_ops.link")
>
> can you please also have a test where we use SEC("?.struct_ops.link")
> which set autoload to false by default?
As far as I understand, that would be a new behavior, currently '?'
works only for programs. I'll add a separate patch to support this.
[...]
