> +static int mod_param_num_mac = 1; > +module_param_named(num_mac, mod_param_num_mac, int, 0444); Why prefix these with "mod_param_"? Seems to make things a little harder to read.