> + > +static int (*const ctl_table[])(struct tracker *tracker, > + __u8 __user *buf, __u32 *plen) = { > + ctl_cbtinfo, > + ctl_cbtmap, > + ctl_cbtdirty, > + ctl_snapshotadd, > + ctl_snapshotinfo, > +}; Please use a switch statement like the rest of the kernel. Besides being simple to follow it also avoid having a speculation gadget on user controlled input.