On Fri, 2024-08-30 at 00:29 -0700, Tony Ambardar wrote:

[...]

> @@ -940,6 +942,21 @@ bpf_object__add_programs(struct bpf_object *obj, 
> Elf_Data *sec_data,
>       return 0;
>  }
>  
> +static void bpf_object_bswap_progs(struct bpf_object *obj)
> +{
> +     struct bpf_program *prog = obj->programs;
> +     struct bpf_insn *insn;
> +     int p, i;
> +
> +     for (p = 0; p < obj->nr_programs; p++, prog++) {
> +             insn = prog->insns;
> +             for (i = 0; i < prog->insns_cnt; i++, insn++)
> +                     bpf_insn_bswap(insn);
> +             pr_debug("prog '%s': converted %zu insns to native byte 
> order\n",
> +                      prog->name, prog->insns_cnt);

Nit: pr_debug already printed available programs at this point,
     maybe move this call outside of both loops?

> +     }
> +}
> +
>  static const struct btf_member *
>  find_member_by_offset(const struct btf_type *t, __u32 bit_offset)
>  {

[...]


Reply via email to