Re: [PATCH v8 01/14] Consolidate IO memcpy/memset into iomap_copy.c

[kernel test robot <lkp@intel.com>]

Hi Julian,

kernel test robot noticed the following build warnings:

[auto build test WARNING on powerpc/next]
[also build test WARNING on powerpc/fixes arm64/for-next/core deller-parisc/for-next s390/features mtd/mtd/next mtd/mtd/fixes linus/master arm/for-next arm/fixes v6.12-rc2 next-20241008]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]

url:    https://github.com/intel-lab-lkp/linux/commits/Julian-Vetter/Consolidate-IO-memcpy-memset-into-iomap_copy-c/20241008-163134
base:   https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git next
patch link:    https://lore.kernel.org/r/20241008075023.3052370-2-jvetter%40kalrayinc.com
patch subject: [PATCH v8 01/14] Consolidate IO memcpy/memset into iomap_copy.c
config: x86_64-randconfig-121-20241009 (https://download.01.org/0day-ci/archive/20241009/202410091406.PM7WcKcj-lkp@intel.com/config)
compiler: gcc-12 (Debian 12.2.0-14) 12.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20241009/202410091406.PM7WcKcj-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/202410091406.PM7WcKcj-lkp@intel.com/

sparse warnings: (new ones prefixed by >>)
>> drivers/misc/ibmasm/lowlevel.c:35:22: sparse: sparse: incorrect type in argument 1 (different address spaces) @@     expected void volatile [noderef] __iomem *to @@     got struct i2o_header * @@
   drivers/misc/ibmasm/lowlevel.c:35:22: sparse:     expected void volatile [noderef] __iomem *to
   drivers/misc/ibmasm/lowlevel.c:35:22: sparse:     got struct i2o_header *
>> drivers/misc/ibmasm/lowlevel.c:36:22: sparse: sparse: incorrect type in argument 1 (different address spaces) @@     expected void volatile [noderef] __iomem *to @@     got void ** @@
   drivers/misc/ibmasm/lowlevel.c:36:22: sparse:     expected void volatile [noderef] __iomem *to
   drivers/misc/ibmasm/lowlevel.c:36:22: sparse:     got void **
   drivers/misc/ibmasm/lowlevel.c: note: in included file:
   drivers/misc/ibmasm/lowlevel.h:120:17: sparse: sparse: cast removes address space '__iomem' of expression
   drivers/misc/ibmasm/lowlevel.h:120:17: sparse: sparse: cast removes address space '__iomem' of expression
--
>> drivers/misc/ibmasm/heartbeat.c:85:36: sparse: sparse: incorrect type in argument 2 (different address spaces) @@     expected void const volatile [noderef] __iomem *from @@     got void *message @@
   drivers/misc/ibmasm/heartbeat.c:85:36: sparse:     expected void const volatile [noderef] __iomem *from
   drivers/misc/ibmasm/heartbeat.c:85:36: sparse:     got void *message
--
>> drivers/misc/ibmasm/event.c:54:36: sparse: sparse: incorrect type in argument 2 (different address spaces) @@     expected void const volatile [noderef] __iomem *from @@     got void *data @@
   drivers/misc/ibmasm/event.c:54:36: sparse:     expected void const volatile [noderef] __iomem *from
   drivers/misc/ibmasm/event.c:54:36: sparse:     got void *data
--
   drivers/misc/ibmasm/remote.c:180:39: sparse: sparse: cast removes address space '__iomem' of expression
>> drivers/misc/ibmasm/remote.c:180:39: sparse: sparse: incorrect type in argument 2 (different address spaces) @@     expected void const volatile [noderef] __iomem *from @@     got void * @@
   drivers/misc/ibmasm/remote.c:180:39: sparse:     expected void const volatile [noderef] __iomem *from
   drivers/misc/ibmasm/remote.c:180:39: sparse:     got void *
--
>> drivers/misc/ibmasm/command.c:168:36: sparse: sparse: incorrect type in argument 2 (different address spaces) @@     expected void const volatile [noderef] __iomem *from @@     got void *response @@
   drivers/misc/ibmasm/command.c:168:36: sparse:     expected void const volatile [noderef] __iomem *from
   drivers/misc/ibmasm/command.c:168:36: sparse:     got void *response

vim +35 drivers/misc/ibmasm/lowlevel.c

^1da177e4c3f41 Linus Torvalds 2005-04-16  17  
^1da177e4c3f41 Linus Torvalds 2005-04-16  18  
^1da177e4c3f41 Linus Torvalds 2005-04-16  19  int ibmasm_send_i2o_message(struct service_processor *sp)
^1da177e4c3f41 Linus Torvalds 2005-04-16  20  {
^1da177e4c3f41 Linus Torvalds 2005-04-16  21  	u32 mfa;
^1da177e4c3f41 Linus Torvalds 2005-04-16  22  	unsigned int command_size;
^1da177e4c3f41 Linus Torvalds 2005-04-16  23  	struct i2o_message *message;
^1da177e4c3f41 Linus Torvalds 2005-04-16  24  	struct command *command = sp->current_command;
^1da177e4c3f41 Linus Torvalds 2005-04-16  25  
^1da177e4c3f41 Linus Torvalds 2005-04-16  26  	mfa = get_mfa_inbound(sp->base_address);
^1da177e4c3f41 Linus Torvalds 2005-04-16  27  	if (!mfa)
^1da177e4c3f41 Linus Torvalds 2005-04-16  28  		return 1;
^1da177e4c3f41 Linus Torvalds 2005-04-16  29  
^1da177e4c3f41 Linus Torvalds 2005-04-16  30  	command_size = get_dot_command_size(command->buffer);
^1da177e4c3f41 Linus Torvalds 2005-04-16  31  	header.message_size = outgoing_message_size(command_size);
^1da177e4c3f41 Linus Torvalds 2005-04-16  32  
^1da177e4c3f41 Linus Torvalds 2005-04-16  33  	message = get_i2o_message(sp->base_address, mfa);
^1da177e4c3f41 Linus Torvalds 2005-04-16  34  
278d72ae8803ff Max Asbock     2005-06-21 @35  	memcpy_toio(&message->header, &header, sizeof(struct i2o_header));
278d72ae8803ff Max Asbock     2005-06-21 @36  	memcpy_toio(&message->data, command->buffer, command_size);
^1da177e4c3f41 Linus Torvalds 2005-04-16  37  
^1da177e4c3f41 Linus Torvalds 2005-04-16  38  	set_mfa_inbound(sp->base_address, mfa);
^1da177e4c3f41 Linus Torvalds 2005-04-16  39  
^1da177e4c3f41 Linus Torvalds 2005-04-16  40  	return 0;
^1da177e4c3f41 Linus Torvalds 2005-04-16  41  }
^1da177e4c3f41 Linus Torvalds 2005-04-16  42  

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