On Fri, Dec 8, 2023 at 8:15 PM Yonghong Song <yonghong.song@xxxxxxxxx> wrote: > > > On 12/8/23 8:05 PM, Alexei Starovoitov wrote: > > On Fri, Dec 8, 2023 at 2:04 PM Andrii Nakryiko > > <andrii.nakryiko@xxxxxxxxx> wrote: > >> > >> I feel like embedding some sort of ID inside the instruction is very.. > >> unusual, shall we say? > > yeah. no magic numbers inside insns pls. > > > > I don't like JA_CFG name, since I read CFG as control flow graph, > > while you probably meant CFG as configurable. > > How about BPF_JA_OR_NOP ? > > Then in combination with BPF_JMP or BPF_JMP32 modifier > > the insn->off|imm will be used. > > 1st bit in src_reg can indicate the default action: nop or jmp. > > In asm it may look like asm("goto_or_nop +5") > > How does the C source code looks like in order to generate > BPF_JA_OR_NOP insn? Any source examples? It will be in inline asm only. The address of that insn will be taken either via && or via asm (".long %l[label]"). >From llvm pov both should go through the same relo creation logic. I hope :)