[bug report] IB/cma: Add default RoCE TOS to CMA configfs

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

 



Hello Majd Dibbiny,

The patch eeec11e5c404: "IB/cma: Add default RoCE TOS to CMA
configfs" from Dec 19, 2016, leads to the following static checker
warning:

	drivers/infiniband/core/cma.c:303 cma_get_default_roce_tos()
	warn: signedness bug returning '(-22)'

drivers/infiniband/core/cma.c
   298  
   299  u8 cma_get_default_roce_tos(struct cma_device *cma_dev, unsigned int port)
   300  {
   301          if (port < rdma_start_port(cma_dev->device) ||
   302              port > rdma_end_port(cma_dev->device))
   303                  return -EINVAL;

This is a u8 function so you can't return negative error codes.

   304  
   305          return cma_dev->default_roce_tos[port - rdma_start_port(cma_dev->device)];
   306  }
   307  

regards,
dan carpenter
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[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