CC: [email protected]
CC: [email protected]
TO: Parav Pandit <[email protected]>
CC: Saeed Mahameed <[email protected]>
CC: Leon Romanovsky <[email protected]>
CC: Shay Drory <[email protected]>

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 
master
head:   1c52283265a462a100ae63ddf58b4e5884acde86
commit: 5958a6fad623ad3b67a9e4d8dbd5f1874cc7039e net/mlx5: Reorganize current 
and maximal capabilities to be per-type
date:   5 months ago
:::::: branch date: 15 hours ago
:::::: commit date: 5 months ago
config: microblaze-randconfig-m031-20220122 
(https://download.01.org/0day-ci/archive/20220123/[email protected]/config)
compiler: microblaze-linux-gcc (GCC) 11.2.0

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <[email protected]>
Reported-by: Dan Carpenter <[email protected]>

New smatch warnings:
drivers/net/ethernet/mellanox/mlx5/core/lib/dm.c:34 mlx5_dm_create() warn: 
should '(((1))) << 
(((((__builtin_constant_p(((*((dev->caps.hca[15]->cur)[(($expr_0x7f369fa16400(30))
 / 32)]))))) ?((((((*((dev->caps.hca[15]->cur)[(($expr_0x7f369fa16d10(30)) / 
32)]))) & 255) << 24) | 
((((*((dev->caps.hca[15]->cur)[(($expr_0x7f369fa076f0(30)) / 32)]))) & 65280) 
<< 8) | ((((*((dev->caps.hca[15]->cur)[(($expr_0x7f369fa080a0(30)) / 32)]))) & 
16711680) >> 8) | ((((*((dev->caps.hca[15]->cur)[(($expr_0x7f369fa08a50(30)) / 
32)]))) & 4278190080) >> 24))):__fswab32((*((dev->caps.hca[15]->cur) + 
(($expr_0x' be a 64 bit type?

Old smatch warnings:
drivers/net/ethernet/mellanox/mlx5/core/lib/dm.c:46 mlx5_dm_create() warn: 
should '(((1))) << 
(((((__builtin_constant_p(((*((dev->caps.hca[15]->cur)[(($expr_0x7f369f9fd730(30))
 / 32)]))))) ?((((((*((dev->caps.hca[15]->cur)[(($expr_0x7f369f9fe040(30)) / 
32)]))) & 255) << 24) | 
((((*((dev->caps.hca[15]->cur)[(($expr_0x7f369f9fe9f0(30)) / 32)]))) & 65280) 
<< 8) | ((((*((dev->caps.hca[15]->cur)[(($expr_0x7f369f9e73d0(30)) / 32)]))) & 
16711680) >> 8) | ((((*((dev->caps.hca[15]->cur)[(($expr_0x7f369f9e7d80(30)) / 
32)]))) & 4278190080) >> 24))):__fswab32((*((dev->caps.hca[15]->cur) + 
(($expr_0x' be a 64 bit type?

vim +34 drivers/net/ethernet/mellanox/mlx5/core/lib/dm.c

c9b9dcb430b3cd Ariel Levkovich 2019-08-29  16  
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  17  struct mlx5_dm 
*mlx5_dm_create(struct mlx5_core_dev *dev)
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  18  {
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  19   u64 header_modify_icm_blocks = 
0;
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  20   u64 steering_icm_blocks = 0;
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  21   struct mlx5_dm *dm;
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  22  
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  23   if (!(MLX5_CAP_GEN_64(dev, 
general_obj_types) & MLX5_GENERAL_OBJ_TYPES_CAP_SW_ICM))
586ee9e8a3b007 Saeed Mahameed  2020-01-22  24           return NULL;
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  25  
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  26   dm = kzalloc(sizeof(*dm), 
GFP_KERNEL);
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  27   if (!dm)
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  28           return ERR_PTR(-ENOMEM);
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  29  
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  30   spin_lock_init(&dm->lock);
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  31  
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  32   if (MLX5_CAP64_DEV_MEM(dev, 
steering_sw_icm_start_address)) {
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  33           steering_icm_blocks =
c9b9dcb430b3cd Ariel Levkovich 2019-08-29 @34                   
BIT(MLX5_CAP_DEV_MEM(dev, log_steering_sw_icm_size) -
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  35                       
MLX5_LOG_SW_ICM_BLOCK_SIZE(dev));
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  36  
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  37           
dm->steering_sw_icm_alloc_blocks =
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  38                   
kcalloc(BITS_TO_LONGS(steering_icm_blocks),
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  39                           
sizeof(unsigned long), GFP_KERNEL);
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  40           if 
(!dm->steering_sw_icm_alloc_blocks)
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  41                   goto 
err_steering;
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  42   }
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  43  
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  44   if (MLX5_CAP64_DEV_MEM(dev, 
header_modify_sw_icm_start_address)) {
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  45           
header_modify_icm_blocks =
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  46                   
BIT(MLX5_CAP_DEV_MEM(dev, log_header_modify_sw_icm_size) -
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  47                       
MLX5_LOG_SW_ICM_BLOCK_SIZE(dev));
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  48  
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  49           
dm->header_modify_sw_icm_alloc_blocks =
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  50                   
kcalloc(BITS_TO_LONGS(header_modify_icm_blocks),
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  51                           
sizeof(unsigned long), GFP_KERNEL);
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  52           if 
(!dm->header_modify_sw_icm_alloc_blocks)
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  53                   goto 
err_modify_hdr;
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  54   }
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  55  
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  56   return dm;
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  57  
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  58  err_modify_hdr:
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  59   
kfree(dm->steering_sw_icm_alloc_blocks);
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  60  
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  61  err_steering:
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  62   kfree(dm);
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  63  
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  64   return ERR_PTR(-ENOMEM);
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  65  }
c9b9dcb430b3cd Ariel Levkovich 2019-08-29  66  

:::::: The code at line 34 was first introduced by commit
:::::: c9b9dcb430b3cd0ad2b04c360c4e528d73430481 net/mlx5: Move device memory 
management to mlx5_core

:::::: TO: Ariel Levkovich <[email protected]>
:::::: CC: Saeed Mahameed <[email protected]>

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/[email protected]
_______________________________________________
kbuild mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to