On Mon, Sep 12, 2016 at 07:17:35PM +0000, Yuval Mintz wrote: > >> +uint debug; > >> +module_param(debug, uint, 0); > >> +MODULE_PARM_DESC(debug, "Default debug msglevel"); > > >Why are you adding this as a module parameter? > > I believe this is mostly to follow same line as qede which also defines > 'debug' module parameter for allowing easy user control of debug > prints [& specifically for probe prints, which can't be controlled > otherwise].
Can you give us an example where dynamic debug and tracing infrastructures are not enough? AFAIK, most of these debug module parameters are legacy copy/paste code which is useless in real life scenarios. Thanks > -- > To unsubscribe from this list: send the line "unsubscribe linux-rdma" in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html
signature.asc
Description: PGP signature
