IB/core: Set speed string to SDR for invalid active rates
authorHonggang Li <honli@redhat.com>
Thu, 15 Mar 2018 09:02:13 +0000 (17:02 +0800)
committerJason Gunthorpe <jgg@mellanox.com>
Mon, 19 Mar 2018 17:39:47 +0000 (11:39 -0600)
Before commit f1b65df5a232 ("IB/mlx5: Add support for active_width and
active_speed in RoCE"), the mlx5_ib driver set default active_width and
active_speed to IB_WIDTH_4X and IB_SPEED_QDR.

Now, the active_width and active_speed are zeros if the RoCE port
is in DOWN state. The speed string should be set to " SDR" instead of
a blank string when active_speed is zero.

Signed-off-by: Honggang Li <honli@redhat.com>
Signed-off-by: Jason Gunthorpe <jgg@mellanox.com>
drivers/infiniband/core/sysfs.c

index 8ae1308eecc715d87131af93a97bdfc10304b55b..cf36ff1f0068e5693bc3b9984137bd39d0770ffc 100644 (file)
@@ -273,6 +273,7 @@ static ssize_t rate_show(struct ib_port *p, struct port_attribute *unused,
                break;
        case IB_SPEED_SDR:
        default:                /* default to SDR for invalid rates */
+               speed = " SDR";
                rate = 25;
                break;
        }