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

Reply via email to