Grant> Why is this an enum?

    >> +static int pcix_max_rbc = PCIX_MAX_RBC_INVALID;

    Grant> It's declared an int and is "user visible".  I think the
    Grant> user interface would be better served if the user could
    Grant> just specify "pcix_max_rbc=2048" instead of some magic
    Grant> value.

Yes, makes sense, and any invalid value (including a default value of
say 0) would mean for the driver to ignore the module parameter.

 - R.
_______________________________________________
openib-general mailing list
[email protected]
http://openib.org/mailman/listinfo/openib-general

To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general

Reply via email to