Re: [PATCH bpf-next v5 2/9] bpf: add struct_ops_tab to btf.

[Kui-Feng Lee <sinquersw@xxxxxxxxx>]

On 10/18/23 17:00, Martin KaFai Lau wrote:
> On 10/17/23 9:22 AM, thinker.li@xxxxxxxxx wrote:
> >   static const struct bpf_struct_ops *
> > -bpf_struct_ops_find_value(u32 value_id)
> > +bpf_struct_ops_find_value(struct btf *btf, u32 value_id)
> >   {
> >       unsigned int i;
>
> [ ... ]
>
> > @@ -671,7 +672,7 @@ static struct bpf_map 
> > *bpf_struct_ops_map_alloc(union bpf_attr *attr)
> >       struct bpf_map *map;
> >       int ret;
> > -    st_ops = bpf_struct_ops_find_value(attr->btf_vmlinux_value_type_id);
> > +    st_ops = 
> > bpf_struct_ops_find_value(attr->btf_vmlinux_value_type_id, btf_vmlinux);
>
> This patch does not compile because of the argument ordering.
Looks like I split the patch wrongly! Will fix it!