[linux-next:master 11234/11516] include/linux/pfn.h:12:43: error: passing argument 1 of 'kho_restore_page' makes pointer from integer without a cast

[linux-next:master 11234/11516] include/linux/pfn.h:12:43: error: passing argument 1 of 'kho_restore_page' makes pointer from integer without a cast

[kernel test robot]

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
head:   bf2602a3cb2381fb1a04bf1c39a290518d2538d1
commit: faccf4f8536bbd064a1d25fde98da1c5a25474a2 [11234/11516] kho: add support for preserving vmalloc allocations
config: x86_64-randconfig-004-20250922 (https://download.01.org/0day-ci/archive/20250923/202509230547.orS14pYx-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/20250923/202509230547.orS14pYx-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/202509230547.orS14pYx-lkp@intel.com/

All errors (new ones prefixed by >>):

   In file included from include/asm-generic/memory_model.h:5,
                    from arch/x86/include/asm/page.h:89,
                    from arch/x86/include/asm/thread_info.h:12,
                    from include/linux/thread_info.h:60,
                    from include/linux/spinlock.h:60,
                    from include/linux/wait.h:9,
                    from include/linux/wait_bit.h:8,
                    from include/linux/fs.h:7,
                    from include/linux/debugfs.h:15,
                    from kernel/kexec_handover.c:13:
   kernel/kexec_handover.c: In function 'kho_restore_pages':
>> include/linux/pfn.h:12:43: error: passing argument 1 of 'kho_restore_page' makes pointer from integer without a cast [-Wint-conversion]
      12 | #define PFN_PHYS(x)     ((phys_addr_t)(x) << PAGE_SHIFT)
         |                         ~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~
         |                                           |
         |                                           long long unsigned int
   kernel/kexec_handover.c:273:54: note: in expansion of macro 'PFN_PHYS'
     273 |                 struct page *page = kho_restore_page(PFN_PHYS(pfn));
         |                                                      ^~~~~~~~
   kernel/kexec_handover.c:214:43: note: expected 'struct page *' but argument is of type 'long long unsigned int'
     214 | static void kho_restore_page(struct page *page, unsigned int order)
         |                              ~~~~~~~~~~~~~^~~~
   kernel/kexec_handover.c:273:37: error: too few arguments to function 'kho_restore_page'
     273 |                 struct page *page = kho_restore_page(PFN_PHYS(pfn));
         |                                     ^~~~~~~~~~~~~~~~
   kernel/kexec_handover.c:214:13: note: declared here
     214 | static void kho_restore_page(struct page *page, unsigned int order)
         |             ^~~~~~~~~~~~~~~~


vim +/kho_restore_page +12 include/linux/pfn.h

947d0496cf3e12 Jeremy Fitzhardinge 2008-09-11   8  
22a9835c350782 Dave Hansen         2006-03-27   9  #define PFN_ALIGN(x)	(((unsigned long)(x) + (PAGE_SIZE - 1)) & PAGE_MASK)
22a9835c350782 Dave Hansen         2006-03-27  10  #define PFN_UP(x)	(((x) + PAGE_SIZE-1) >> PAGE_SHIFT)
22a9835c350782 Dave Hansen         2006-03-27  11  #define PFN_DOWN(x)	((x) >> PAGE_SHIFT)
947d0496cf3e12 Jeremy Fitzhardinge 2008-09-11 @12  #define PFN_PHYS(x)	((phys_addr_t)(x) << PAGE_SHIFT)
8f235d1a3eb719 Chen Gang           2016-01-14  13  #define PHYS_PFN(x)	((unsigned long)((x) >> PAGE_SHIFT))
22a9835c350782 Dave Hansen         2006-03-27  14  

:::::: The code at line 12 was first introduced by commit
:::::: 947d0496cf3e12ebfa70b3eaf561c25403247ce9 generic: make PFN_PHYS explicitly return phys_addr_t

:::::: TO: Jeremy Fitzhardinge <jeremy@goop.org>
:::::: CC: Ingo Molnar <mingo@elte.hu>

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