[openeuler:OLK-6.6 1513/1513] drivers/crypto/montage/tsse/tsse_ipc_api.c:62:36: warning: variable 'device_handle' is uninitialized when used here

[openeuler:OLK-6.6 1513/1513] drivers/crypto/montage/tsse/tsse_ipc_api.c:62:36: warning: variable 'device_handle' is uninitialized when used here

[kernel test robot]

tree:   https://gitee.com/openeuler/kernel.git OLK-6.6
head:   dccd6d8473b2f20aaa76e23820a3bf4934fc8d36
commit: 914854f2adb6988ac3b6521088ec96833d6743e2 [1513/1513] driver: crypto - update support for Mont-TSSE Driver
config: x86_64-allyesconfig (https://download.01.org/0day-ci/archive/20241122/202411221252.PUnKjTWH-lkp@intel.com/config)
compiler: clang version 19.1.3 (https://github.com/llvm/llvm-project ab51eccf88f5321e7c60591c5546b254b6afab99)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20241122/202411221252.PUnKjTWH-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202411221252.PUnKjTWH-lkp@intel.com/

All warnings (new ones prefixed by >>):

   In file included from drivers/crypto/montage/tsse/tsse_ipc_api.c:10:
   In file included from drivers/crypto/montage/tsse/tsse_dev.h:13:
   In file included from include/linux/pci.h:1669:
   In file included from include/linux/dmapool.h:14:
   In file included from include/linux/scatterlist.h:8:
   In file included from include/linux/mm.h:2243:
   include/linux/vmstat.h:508:43: warning: arithmetic between different enumeration types ('enum zone_stat_item' and 'enum numa_stat_item') [-Wenum-enum-conversion]
     508 |         return vmstat_text[NR_VM_ZONE_STAT_ITEMS +
         |                            ~~~~~~~~~~~~~~~~~~~~~ ^
     509 |                            item];
         |                            ~~~~
   include/linux/vmstat.h:515:43: warning: arithmetic between different enumeration types ('enum zone_stat_item' and 'enum numa_stat_item') [-Wenum-enum-conversion]
     515 |         return vmstat_text[NR_VM_ZONE_STAT_ITEMS +
         |                            ~~~~~~~~~~~~~~~~~~~~~ ^
     516 |                            NR_VM_NUMA_EVENT_ITEMS +
         |                            ~~~~~~~~~~~~~~~~~~~~~~
   include/linux/vmstat.h:522:36: warning: arithmetic between different enumeration types ('enum node_stat_item' and 'enum lru_list') [-Wenum-enum-conversion]
     522 |         return node_stat_name(NR_LRU_BASE + lru) + 3; // skip "nr_"
         |                               ~~~~~~~~~~~ ^ ~~~
   include/linux/vmstat.h:527:43: warning: arithmetic between different enumeration types ('enum zone_stat_item' and 'enum numa_stat_item') [-Wenum-enum-conversion]
     527 |         return vmstat_text[NR_VM_ZONE_STAT_ITEMS +
         |                            ~~~~~~~~~~~~~~~~~~~~~ ^
     528 |                            NR_VM_NUMA_EVENT_ITEMS +
         |                            ~~~~~~~~~~~~~~~~~~~~~~
   include/linux/vmstat.h:536:43: warning: arithmetic between different enumeration types ('enum zone_stat_item' and 'enum numa_stat_item') [-Wenum-enum-conversion]
     536 |         return vmstat_text[NR_VM_ZONE_STAT_ITEMS +
         |                            ~~~~~~~~~~~~~~~~~~~~~ ^
     537 |                            NR_VM_NUMA_EVENT_ITEMS +
         |                            ~~~~~~~~~~~~~~~~~~~~~~
>> drivers/crypto/montage/tsse/tsse_ipc_api.c:62:36: warning: variable 'device_handle' is uninitialized when used here [-Wuninitialized]
      62 |         service_instance->device_handle = device_handle;
         |                                           ^~~~~~~~~~~~~
   drivers/crypto/montage/tsse/tsse_ipc_api.c:56:19: note: initialize the variable 'device_handle' to silence this warning
      56 |         int device_handle;
         |                          ^
         |                           = 0
   6 warnings generated.


vim +/device_handle +62 drivers/crypto/montage/tsse/tsse_ipc_api.c

    41	
    42	/**
    43	 * tsse_im_service_handle_alloc() - Allocate IPC Message service handle for specific service.
    44	 * @name: IPC Message service name
    45	 * @cb: request callback for the service
    46	 * @handle: function output for the service handle
    47	 * Return: 0 if allocated successfully, other values for failure
    48	 */
    49	int tsse_im_service_handle_alloc(
    50		const char *name,
    51		tsse_im_cb_func cb,
    52		tsse_im_service_handle *handle)
    53	{
    54		struct tsse_service_instance *service_instance;
    55		int ret;
    56		int device_handle;
    57	
    58		service_instance = kzalloc(sizeof(struct tsse_service_instance), GFP_ATOMIC);
    59		if (!service_instance)
    60			return -ENOMEM;
    61		service_instance->service_opened = 0;
  > 62		service_instance->device_handle = device_handle;
    63		service_instance->cb = cb;
    64		strscpy(service_instance->service_name, name, TSSE_IM_SERVICE_NAME_LEN);
    65	
    66		ret = tsse_schedule_device_handle(service_instance);
    67		if (ret) {
    68			kfree(service_instance);
    69			return ret;
    70		}
    71	
    72		ret = tsse_service_open(service_instance);
    73		if (ret) {
    74			pr_err("%s(): open service: %s failed: %d\n",
    75				__func__, service_instance->service_name, ret);
    76			kfree(service_instance);
    77			return ret;
    78		}
    79		*handle = service_instance;
    80		return 0;
    81	}
    82	EXPORT_SYMBOL_GPL(tsse_im_service_handle_alloc);
    83	

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki