> > >> what's actually the role of the module param? why it belongs here? is > >> that really unavoidable to have it? > > > It's nice to provide the capability for the user to adjust the netlink timeout > based his/her fabric setup. > > NO, adding module params should be really your last resort when everything > else wouldn't work, definitely not the case here, remove it. Are you suggesting that we use the timeout the caller provides? Or if there is no timeout (0) provided, use a default one? Looking through the IB drivers, I do find many module parameters. NrybXǧv^)޺{.n+{ٚ{ayʇڙ,jfh/oScڳ9u&jw(階ݢj"mzޖfh~m