On 4/2/2012 3:27 PM, Roland Dreier wrote: > On Mon, Apr 2, 2012 at 7:45 AM, Or Gerlitz <ogerl...@mellanox.com> wrote: >> When the IB port is down, the active_speed value returned by the MAD_IFC >> command equals seven (7) which isn't among the IB speeds defined by the >> ib_port_speed enum. This results in invalid speed value seen by higher >> layers or applications who do port query. Fix that by setting the speed >> to be SDR - the lowest possible when the port is down. > > This looks reasonable, applied.
How about validating the speed in mlx4 before overwriting it ? Would you take such a patch ? -- Hal > -- > 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