[openeuler:OLK-6.6 3493/3493] mm/gmem.c:383:13: warning: unused variable 'nid'

[openeuler:OLK-6.6 3493/3493] mm/gmem.c:383:13: warning: unused variable 'nid'

[kernel test robot]

tree:   https://gitee.com/openeuler/kernel.git OLK-6.6
head:   36141432a37f7dedddde8825e724b07ce3a440fb
commit: 18b5064b5892580f89d7eca015f884ce10190bac [3493/3493] gmem: hmemcpy should use dma for phy addr
config: x86_64-randconfig-161-20251208 (https://download.01.org/0day-ci/archive/20251209/202512090002.kbElJ2I7-lkp@intel.com/config)
compiler: gcc-14 (Debian 14.2.0-19) 14.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20251209/202512090002.kbElJ2I7-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/202512090002.kbElJ2I7-lkp@intel.com/

All warnings (new ones prefixed by >>):

   mm/gmem.c: In function 'gm_as_attach':
>> mm/gmem.c:383:13: warning: unused variable 'nid' [-Wunused-variable]
     383 |         int nid;
         |             ^~~


vim +/nid +383 mm/gmem.c

380709c322aaa3 Cunshu Ni 2025-11-01  378  
380709c322aaa3 Cunshu Ni 2025-11-01  379  int gm_as_attach(struct gm_as *as, struct gm_dev *dev,
380709c322aaa3 Cunshu Ni 2025-11-01  380  			bool activate, struct gm_context **out_ctx)
380709c322aaa3 Cunshu Ni 2025-11-01  381  {
380709c322aaa3 Cunshu Ni 2025-11-01  382  	struct gm_context *ctx;
380709c322aaa3 Cunshu Ni 2025-11-01 @383  	int nid;
380709c322aaa3 Cunshu Ni 2025-11-01  384  
380709c322aaa3 Cunshu Ni 2025-11-01  385  	ctx = kmem_cache_alloc(gm_ctx_cache, GFP_KERNEL);
380709c322aaa3 Cunshu Ni 2025-11-01  386  	if (!ctx)
380709c322aaa3 Cunshu Ni 2025-11-01  387  		return -ENOMEM;
380709c322aaa3 Cunshu Ni 2025-11-01  388  
380709c322aaa3 Cunshu Ni 2025-11-01  389  	ctx->as = as;
380709c322aaa3 Cunshu Ni 2025-11-01  390  	ctx->dev = dev;
380709c322aaa3 Cunshu Ni 2025-11-01  391  
380709c322aaa3 Cunshu Ni 2025-11-01  392  	INIT_LIST_HEAD(&ctx->gm_dev_link);
380709c322aaa3 Cunshu Ni 2025-11-01  393  	INIT_LIST_HEAD(&ctx->gm_as_link);
380709c322aaa3 Cunshu Ni 2025-11-01  394  
380709c322aaa3 Cunshu Ni 2025-11-01  395  	if (!list_empty(&as->gm_ctx_list)) {
380709c322aaa3 Cunshu Ni 2025-11-01  396  		struct list_head *old_node;
380709c322aaa3 Cunshu Ni 2025-11-01  397  		struct gm_context *old_ctx;
380709c322aaa3 Cunshu Ni 2025-11-01  398  
380709c322aaa3 Cunshu Ni 2025-11-01  399  		old_node = as->gm_ctx_list.prev;
380709c322aaa3 Cunshu Ni 2025-11-01  400  		list_del_init(old_node);
380709c322aaa3 Cunshu Ni 2025-11-01  401  		old_ctx = list_entry(old_node, struct gm_context, gm_as_link);
380709c322aaa3 Cunshu Ni 2025-11-01  402  		kfree(old_ctx);
380709c322aaa3 Cunshu Ni 2025-11-01  403  	}
380709c322aaa3 Cunshu Ni 2025-11-01  404  
380709c322aaa3 Cunshu Ni 2025-11-01  405  	list_add_tail(&dev->gm_ctx_list, &ctx->gm_dev_link);
380709c322aaa3 Cunshu Ni 2025-11-01  406  	list_add_tail(&ctx->gm_as_link, &as->gm_ctx_list);
380709c322aaa3 Cunshu Ni 2025-11-01  407  
380709c322aaa3 Cunshu Ni 2025-11-01  408  	if (activate) {
380709c322aaa3 Cunshu Ni 2025-11-01  409  		/*
380709c322aaa3 Cunshu Ni 2025-11-01  410  		 * Here we should really have a callback function to perform the context switch
380709c322aaa3 Cunshu Ni 2025-11-01  411  		 * for the hardware. E.g. in x86 this function is effectively
380709c322aaa3 Cunshu Ni 2025-11-01  412  		 * flushing the CR3 value. Currently we do not care time-sliced context switch,
380709c322aaa3 Cunshu Ni 2025-11-01  413  		 * unless someone wants to support it.
380709c322aaa3 Cunshu Ni 2025-11-01  414  		 */
380709c322aaa3 Cunshu Ni 2025-11-01  415  		dev->current_ctx = ctx;
380709c322aaa3 Cunshu Ni 2025-11-01  416  	}
380709c322aaa3 Cunshu Ni 2025-11-01  417  	*out_ctx = ctx;
380709c322aaa3 Cunshu Ni 2025-11-01  418  
380709c322aaa3 Cunshu Ni 2025-11-01  419  	/*
380709c322aaa3 Cunshu Ni 2025-11-01  420  	 * gm_as_attach will be used to attach device to process address space.
380709c322aaa3 Cunshu Ni 2025-11-01  421  	 * Handle this case and add hnodes registered by device to process mems_allowed.
380709c322aaa3 Cunshu Ni 2025-11-01  422  	 */
18b5064b589258 Ni Cunshu 2025-12-08  423  #ifdef CONFIG_CPUSETS
380709c322aaa3 Cunshu Ni 2025-11-01  424  	for_each_node_mask(nid, dev->registered_hnodes)
380709c322aaa3 Cunshu Ni 2025-11-01  425  		node_set(nid, current->mems_allowed);
18b5064b589258 Ni Cunshu 2025-12-08  426  #endif
380709c322aaa3 Cunshu Ni 2025-11-01  427  	return 0;
380709c322aaa3 Cunshu Ni 2025-11-01  428  }
380709c322aaa3 Cunshu Ni 2025-11-01  429  EXPORT_SYMBOL_GPL(gm_as_attach);
2ce9e864fa288b Cunshu Ni 2025-11-02  430  

:::::: The code at line 383 was first introduced by commit
:::::: 380709c322aaa306b5ac3c278a0700043f7d33e2 gmem: prepare config, boot parameter, vma flag and basic GMEM related structs

:::::: TO: Cunshu Ni <nicunshu@huawei.com>
:::::: CC: zhangjian <zhangjian496@huawei.com>

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