On Thu, Apr 03, 2014 at 05:12:34PM -0600, Keith Busch wrote: > +int nvme_io_timeout = 5; > +module_param(nvme_io_timeout, int, 0); > +MODULE_PARM_DESC(nvme_io_timeout, "timeout in seconds for io submitted to queue"); Any reason not to make this module_param 0644? That would allow it to be written by root and read by anybody.