Roland, Ultimately, I agree with you. Besides the general usability issue, there are some obvious limitations to this method. However, we need this functionality and there is nothing else that exists that we could use. As Mike points out, we need these to be available to the module at module init.
Another reason for this method is that it gave us backward compatibility with old configurations (settings will not brake for customers who only update the driver.) - Mitko > -----Original Message----- > From: linux-rdma-ow...@vger.kernel.org [mailto:linux-rdma- > ow...@vger.kernel.org] On Behalf Of Roland Dreier > Sent: Thursday, July 19, 2012 11:27 > To: Marciniszyn, Mike > Cc: linux-rdma@vger.kernel.org > Subject: Re: [PATCH 3/5] IB/qib: Add support for per-device/per-port > parameters > > [resending because I forgot to cc linux-rdma] > > On Thu, Jul 19, 2012 at 6:04 AM, Mike Marciniszyn > <mike.marcinis...@intel.com> wrote: > > Add support for per-device/per-port driver parameters allowing users > > to specify different values for different ports/devices. All converted > > parameters will behave as expected if the old parameter format is > > used. > > > > New per-device/per-port parameter format: > > <param>=[<d>,]<u>[:<p>]=<v>[,<u>[:<p>]=<v>] > > where: > > <param> - driver parameter name > > <d> - default value applicable to all devices/ports > > <u> - unit number (zero-based) > > <p> - port number (1-based) > > <v> - numeric value > > I think this is pushing the module parameter facility way past the breaking > point of usability. We really need to come up with a better way to get > complex structured config info like this into the drive > -- > 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 -- 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
