"Randy.Dunlap" writes: > On Wed, 20 Apr 2005 13:02:12 -0400 Ed L Cashin wrote: > > Just a nit/typo: > > | + modprobe aoe_iflist="eth1 eth3" > > | static char aoe_iflist[IFLISTSZ]; > | +module_param_string(aoe_iflist, aoe_iflist, IFLISTSZ, 0600); > | +MODULE_PARM_DESC(aoe_iflist, " aoe_iflist=\"dev1 [dev2 ...]\n"); > > No leading space (" aoe_iflist=") and put a trailing \" in it: > > +MODULE_PARM_DESC(aoe_iflist, "aoe_iflist=\"dev1 [dev2 ...]\"\n"); Thanks for catching that. improve allowed interfaces configuration Signed-off-by: Ed L. Cashin