On Fri, 2013-02-22 at 14:33 -0500, Hal Rosenstock wrote: > Hi Al, > > On 2/21/2013 6:39 PM, Albert Chu wrote: > > Usually coinciding with use of the force_link_speed configuration option, > > some link speeds can be observed to be incorrect. For example, a link > > may be active at QDR even if SDR & DDR speeds are the only ones enabled. > > Are you referring to the fact that after setting the enabled speed, the > IB spec does not mandate a link renegotiation and hence one may not have > occurred so the enabled speed can be out of whack with the active speed ?
Now that I re-read my patch comment, it wasn't the clearest. What you describe above very well could have occurred or maybe it was just bad hardware. The key was that ports were running at speeds (e.g. QDR) that were not enabled (e.g. only SDR & DDR) and the current opensm console was reporting invalid/incorrect error messages. > Also, isn't there a similar issue with extended speeds (and > force_link_speed_ext) ? If so, should that also be part of this patch ? > > Same issue also is there for link width as it works the same way as link > speed. Maybe that too is yet another patch. > > What do you think ? Yeah, I should have done those as well. I was only solving the one particular bug found. I'll rework the patch. Al > -- Hal > > > Under these scenarios, the portstatus may incorrectly output port status, > > confusing users. > > > > A new port output status of "incorrect speed" differentiates ports in this > > state from other states, so as to not confuse users. > > > > Signed-off-by: Albert Chu <[email protected]> > > <snip...> -- Albert Chu [email protected] Computer Scientist High Performance Systems Division Lawrence Livermore National Laboratory -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
