[opencloudos:linux-5.4/lts/5.4.119-20.0009.spr 1010/2441] kernel/dma/mapping.c:557:72: error: macro "alloc_pages" passed 5 arguments, but takes just 2

[opencloudos:linux-5.4/lts/5.4.119-20.0009.spr 1010/2441] kernel/dma/mapping.c:557:72: error: macro "alloc_pages" passed 5 arguments, but takes just 2

[kernel test robot]

tree:   https://gitee.com/OpenCloudOS/OpenCloudOS-Kernel.git linux-5.4/lts/5.4.119-20.0009.spr
head:   63759dec7a1be60e43e98e4257de936b546e4656
commit: 0764432a5ada7a09ce36b0e980ec40340fae2a30 [1010/2441] dma-mapping: add a new dma_alloc_pages API
config: x86_64-allnoconfig (https://download.01.org/0day-ci/archive/20240924/202409240901.O2by61Zq-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/20240924/202409240901.O2by61Zq-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/202409240901.O2by61Zq-lkp@intel.com/

All errors (new ones prefixed by >>):

   In file included from kernel/dma/mapping.c:10:
   include/linux/dma-direct.h:87:7: error: conflicting types for 'dma_direct_alloc_pages'; have 'void *(struct device *, size_t,  dma_addr_t *, gfp_t,  long unsigned int)' {aka 'void *(struct device *, long unsigned int,  long long unsigned int *, unsigned int,  long unsigned int)'}
      87 | void *dma_direct_alloc_pages(struct device *dev, size_t size,
         |       ^~~~~~~~~~~~~~~~~~~~~~
   include/linux/dma-direct.h:82:14: note: previous declaration of 'dma_direct_alloc_pages' with type 'struct page *(struct device *, size_t,  dma_addr_t *, enum dma_data_direction,  gfp_t)' {aka 'struct page *(struct device *, long unsigned int,  long long unsigned int *, enum dma_data_direction,  unsigned int)'}
      82 | struct page *dma_direct_alloc_pages(struct device *dev, size_t size,
         |              ^~~~~~~~~~~~~~~~~~~~~~
   include/linux/dma-direct.h:89:6: error: conflicting types for 'dma_direct_free_pages'; have 'void(struct device *, size_t,  void *, dma_addr_t,  long unsigned int)' {aka 'void(struct device *, long unsigned int,  void *, long long unsigned int,  long unsigned int)'}
      89 | void dma_direct_free_pages(struct device *dev, size_t size, void *cpu_addr,
         |      ^~~~~~~~~~~~~~~~~~~~~
   include/linux/dma-direct.h:84:6: note: previous declaration of 'dma_direct_free_pages' with type 'void(struct device *, size_t,  struct page *, dma_addr_t,  enum dma_data_direction)' {aka 'void(struct device *, long unsigned int,  struct page *, long long unsigned int,  enum dma_data_direction)'}
      84 | void dma_direct_free_pages(struct device *dev, size_t size,
         |      ^~~~~~~~~~~~~~~~~~~~~
   kernel/dma/mapping.c: In function 'dma_alloc_pages':
>> kernel/dma/mapping.c:557:72: error: macro "alloc_pages" passed 5 arguments, but takes just 2
     557 |                 page = ops->alloc_pages(dev, size, dma_handle, dir, gfp);
         |                                                                        ^
   In file included from include/linux/mm.h:10,
                    from include/linux/memblock.h:13,
                    from kernel/dma/mapping.c:8:
   include/linux/gfp.h:540: note: macro "alloc_pages" defined here
     540 | #define alloc_pages(gfp_mask, order) \
         | 
>> kernel/dma/mapping.c:557:22: error: assignment to 'struct page *' from incompatible pointer type 'struct page * (*)(struct device *, size_t,  dma_addr_t *, enum dma_data_direction,  gfp_t)' {aka 'struct page * (*)(struct device *, long unsigned int,  long long unsigned int *, enum dma_data_direction,  unsigned int)'} [-Werror=incompatible-pointer-types]
     557 |                 page = ops->alloc_pages(dev, size, dma_handle, dir, gfp);
         |                      ^
   cc1: some warnings being treated as errors


vim +/alloc_pages +557 kernel/dma/mapping.c

   541	
   542	struct page *dma_alloc_pages(struct device *dev, size_t size,
   543			dma_addr_t *dma_handle, enum dma_data_direction dir, gfp_t gfp)
   544	{
   545		const struct dma_map_ops *ops = get_dma_ops(dev);
   546		struct page *page;
   547	
   548		if (WARN_ON_ONCE(!dev->coherent_dma_mask))
   549			return NULL;
   550		if (WARN_ON_ONCE(gfp & (__GFP_DMA | __GFP_DMA32 | __GFP_HIGHMEM)))
   551			return NULL;
   552	
   553		size = PAGE_ALIGN(size);
   554		if (dma_alloc_direct(dev, ops))
   555			page = dma_direct_alloc_pages(dev, size, dma_handle, dir, gfp);
   556		else if (ops->alloc_pages)
 > 557			page = ops->alloc_pages(dev, size, dma_handle, dir, gfp);
   558		else
   559			return NULL;
   560	
   561		debug_dma_map_page(dev, page, 0, size, dir, *dma_handle);
   562	
   563		return page;
   564	}
   565	EXPORT_SYMBOL_GPL(dma_alloc_pages);
   566	

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