RE: [PATCH rdma-core] libibumad/umad.c: In get_port, ignore sysfs rate file errors

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




> -----Original Message-----
> From: linux-rdma-owner@xxxxxxxxxxxxxxx [mailto:linux-rdma-
> owner@xxxxxxxxxxxxxxx] On Behalf Of Hal Rosenstock
> Sent: Tuesday, October 31, 2017 7:15 AM
> To: linux-rdma@xxxxxxxxxxxxxxx
> Cc: Ghazale Hosseinabadi <ghazale.hosseinabadi@xxxxxxxxxx>
> Subject: [PATCH rdma-core] libibumad/umad.c: In get_port, ignore sysfs rate file
> errors
> 
> 
> This can cause ibpanic in ibstat when width is not set properly as can occur when
> QSPF is not plugged into port.
> 
> ibpanic: [7851] main: stat of IB device 'mlx5_1' failed: Invalid argument
> 
> It's caused by kernel sysfs.c:show_rate returning -EINVAL and that error being
> treated as failure in umad.c:get_port.
> With this change, Rate is displayed as 0 with ibstat for this scenario.
> 
> Signed-off-by: Hal Rosenstock <hal@xxxxxxxxxxxx>
> ---
>  libibumad/umad.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/libibumad/umad.c b/libibumad/umad.c index 535256e..dcb2c68
> 100644
> --- a/libibumad/umad.c
> +++ b/libibumad/umad.c
> @@ -150,8 +150,7 @@ static int get_port(const char *ca_name, const char
> *dir, int portnum, umad_port
>  		goto clean;
>  	if (sys_read_uint(port_dir, SYS_PORT_PHY_STATE, &port->phys_state) <
> 0)
>  		goto clean;
> -	if (sys_read_uint(port_dir, SYS_PORT_RATE, &port->rate) < 0)
> -		goto clean;
> +	sys_read_uint(port_dir, SYS_PORT_RATE, &port->rate);
>  	if (sys_read_uint(port_dir, SYS_PORT_CAPMASK, &capmask) < 0)
>  		goto clean;
> 
Looks good to me.
��.n��������+%������w��{.n�����{���fk��ܨ}���Ơz�j:+v�����w����ޙ��&�)ߡ�a����z�ޗ���ݢj��w�f




[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]     [Yosemite Photos]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux