On Fri, Jun 14, 2019 at 2:19 PM Josh Poimboeuf <jpoimboe@xxxxxxxxxx> wrote: > > > > > > > > > > +#define JUMP_TABLE_SYM_PREFIX "jump_table." > > > > > > > > since external tool will be looking at it should it be named > > > > "bpf_jump_table." to avoid potential name conflicts? > > > > Or even more unique name? > > > > Like "bpf_interpreter_jump_table." ? > > > > > > No, the point is that it's a generic feature which can also be used any > > > non-BPF code which might also have a jump table. > > > > and you're proposing to name all such jump tables in the kernel > > as static foo jump_table[] ? > > That's the idea. Then it needs much wider discussion. I suggest to rename it to "bpf_interpreter_jump_table." so it can be resolved now for this specific issue. While bigger kernel-wide naming convention get resolved. Later we can rename it to whatever the "standard name for jump table" will be.
