Re: [PATCH] IB/mlx5: Fix passing casted pointer in mlx5_query_port_roce

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

 



On 01/09/2016 06:01 AM, Leon Romanovsky wrote:
> Fix static checker warning:
>         drivers/infiniband/hw/mlx5/main.c:149 mlx5_query_port_roce()
>         warn: passing casted pointer '&props->qkey_viol_cntr' to
> 	'mlx5_query_nic_vport_qkey_viol_cntr()' 32 vs 16.
> 
> Fixes: 3f89a643eb29 ("IB/mlx5: Extend query_device/port to support RoCE")
> Reported-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
> Signed-off-by: Leon Romanovsky <leon@xxxxxxxxxxxx>
> ---
>  drivers/infiniband/hw/mlx5/main.c | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/infiniband/hw/mlx5/main.c b/drivers/infiniband/hw/mlx5/main.c
> index 5e3d3ae..890cf20 100644
> --- a/drivers/infiniband/hw/mlx5/main.c
> +++ b/drivers/infiniband/hw/mlx5/main.c
> @@ -132,6 +132,7 @@ static int mlx5_query_port_roce(struct ib_device *device, u8 port_num,
>  	struct mlx5_ib_dev *dev = to_mdev(device);
>  	struct net_device *ndev;
>  	enum ib_mtu ndev_ib_mtu;
> +	u16 qkey_viol_cntr;
>  
>  	memset(props, 0, sizeof(*props));
>  
> @@ -146,8 +147,8 @@ static int mlx5_query_port_roce(struct ib_device *device, u8 port_num,
>  	props->state            = IB_PORT_DOWN;
>  	props->phys_state       = 3;
>  
> -	mlx5_query_nic_vport_qkey_viol_cntr(dev->mdev,
> -					    (u16 *)&props->qkey_viol_cntr);
> +	mlx5_query_nic_vport_qkey_viol_cntr(dev->mdev, &qkey_viol_cntr);
> +	props->qkey_viol_cntr = qkey_viol_cntr;
>  
>  	ndev = mlx5_ib_get_netdev(device, port_num);
>  	if (!ndev)
> 

Thanks, applied.

-- 
Doug Ledford <dledford@xxxxxxxxxx>
              GPG KeyID: 0E572FDD


Attachment: signature.asc
Description: OpenPGP digital signature


[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