Re: [PATCH v3 08/16] kmsan: convert kmsan_handle_dma to use physical addresses

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

Hi Leon,

kernel test robot noticed the following build errors:

[auto build test ERROR on axboe-block/for-next]
[also build test ERROR on akpm-mm/mm-everything trace/for-next linus/master v6.17-rc1 next-20250815]
[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/Leon-Romanovsky/dma-mapping-introduce-new-DMA-attribute-to-indicate-MMIO-memory/20250815-015647
base:   https://git.kernel.org/pub/scm/linux/kernel/git/axboe/linux-block.git for-next
patch link:    https://lore.kernel.org/r/38de1c5ffb567c5705826f14742fcaf54522c083.1755193625.git.leon%40kernel.org
patch subject: [PATCH v3 08/16] kmsan: convert kmsan_handle_dma to use physical addresses
config: x86_64-buildonly-randconfig-004-20250816 (https://download.01.org/0day-ci/archive/20250816/202508161823.vfpArxa2-lkp@intel.com/config)
compiler: clang version 20.1.8 (https://github.com/llvm/llvm-project 87f0227cb60147a26a1eeb4fb06e3b505e9c7261)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250816/202508161823.vfpArxa2-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/202508161823.vfpArxa2-lkp@intel.com/

All errors (new ones prefixed by >>):

>> mm/kmsan/hooks.c:339:6: error: conflicting types for 'kmsan_handle_dma'
     339 | void kmsan_handle_dma(phys_addr_t phys, size_t size,
         |      ^
   include/linux/kmsan.h:194:6: note: previous declaration is here
     194 | void kmsan_handle_dma(phys_addr_t phys, size_t size,
         |      ^
>> mm/kmsan/hooks.c:371:6: error: too many arguments to function call, expected 3, have 4
     370 |                 kmsan_handle_dma(sg_page(item), item->offset, item->length,
         |                 ~~~~~~~~~~~~~~~~
     371 |                                  dir);
         |                                  ^~~
   mm/kmsan/hooks.c:361:19: note: 'kmsan_handle_dma' declared here
     361 | EXPORT_SYMBOL_GPL(kmsan_handle_dma);
         |                   ^~~~~~~~~~~~~~~~
   include/linux/export.h:90:48: note: expanded from macro 'EXPORT_SYMBOL_GPL'
      90 | #define EXPORT_SYMBOL_GPL(sym)          _EXPORT_SYMBOL(sym, "GPL")
         |                                                        ^~~
   include/linux/export.h:86:54: note: expanded from macro '_EXPORT_SYMBOL'
      86 | #define _EXPORT_SYMBOL(sym, license)    __EXPORT_SYMBOL(sym, license, "")
         |                                                         ^~~
   include/linux/export.h:76:21: note: expanded from macro '__EXPORT_SYMBOL'
      76 |         extern typeof(sym) sym;                                 \
         |                            ^~~
   2 errors generated.


vim +/kmsan_handle_dma +339 mm/kmsan/hooks.c

   337	
   338	/* Helper function to handle DMA data transfers. */
 > 339	void kmsan_handle_dma(phys_addr_t phys, size_t size,
   340			      enum dma_data_direction dir, unsigned int attrs)
   341	{
   342		struct page *page = phys_to_page(phys);
   343		u64 page_offset, to_go, addr;
   344	
   345		if (PageHighMem(page))
   346			return;
   347		addr = (u64)page_address(page) + offset_in_page(phys);
   348		/*
   349		 * The kernel may occasionally give us adjacent DMA pages not belonging
   350		 * to the same allocation. Process them separately to avoid triggering
   351		 * internal KMSAN checks.
   352		 */
   353		while (size > 0) {
   354			page_offset = offset_in_page(addr);
   355			to_go = min(PAGE_SIZE - page_offset, (u64)size);
   356			kmsan_handle_dma_page((void *)addr, to_go, dir);
   357			addr += to_go;
   358			size -= to_go;
   359		}
   360	}
   361	EXPORT_SYMBOL_GPL(kmsan_handle_dma);
   362	
   363	void kmsan_handle_dma_sg(struct scatterlist *sg, int nents,
   364				 enum dma_data_direction dir)
   365	{
   366		struct scatterlist *item;
   367		int i;
   368	
   369		for_each_sg(sg, item, nents, i)
   370			kmsan_handle_dma(sg_page(item), item->offset, item->length,
 > 371					 dir);
   372	}
   373	

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