-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Roland Dreier wrote: > 1) What makes ipath special so that we want this warning for ipath > devices but not other IB hardware?
There's nothing special about our hardware that requires this. We just wanted that in there so we could direct customers to look at dmesg to see if the warning popped up if they call with a problem. It is useful to have for this purpose. > If this warning is actually > useful, then I think it would make more sense to start a timer when > any IB device is added, and warn if ports with a physical link don't > become active after the timeout time. I'd be OK with doing that, too. > But I'm having a hard time > seeing why we want this message in the kernel log. It's useful when you're trying to track down problems. > 2) You do cancel_delayed_work() but not flush_scheduled_work(), so > it's possible for your timeout function to be running after the module > text is gone. OK - I'll fix this up. Thanks for spotting it. Regards, Robert. -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.5 (GNU/Linux) Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org iQEVAwUBRO9gU/zvnpzTd9fxAQJBQwgAkbgrEA4/UpbcD0gsGC+39r5ZAAz+4d3I /QAIKn239juMf8TfrlekAzD9MCj5Rna1bk3yu1gu/Z0Jg5vHvQNmBxDtveQ4bDyu 1DAUbvmclNknzM00LtMHI6AZfYbRgsbCIKXJw0reXctAkbJAvMU0U6Ff1imvO0Tw 38C24ktDalaaKpz4DHO261UHlmtD4wlJojKLYI5yH39JSHK449zjJznrP9W8SPIU RbxGktSsD69gQXmpqgY5KEmbcukZ9AIF4VHTG2uEz1aO7eOQ+1BsUg140EcWXC// R1Jg56WhCYsMDVik7+u994VgQi34beos9pwbLIUkq+315VHN3QFbQg== =XhNA -----END PGP SIGNATURE----- _______________________________________________ openib-general mailing list [email protected] http://openib.org/mailman/listinfo/openib-general To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general
