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)
> {
[...]