On Fri, Jun 14, 2019 at 02:22:59PM -0700, Alexei Starovoitov wrote: > 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. Why would it need wider discussion? It only has one user. If you honestly believe that it will be controversial to require future users to call a static jump table "jump_table" then we can have that discussion when it comes up. -- Josh
