From: kernel test robot <lkp@intel.com>
To: Paolo Bonzini <pbonzini@redhat.com>
Cc: oe-kbuild-all@lists.linux.dev
Subject: Re: [PATCH v3 04/14] x86/virt/tdx: Add SEAMCALL wrappers for TDX page cache management
Date: Wed, 29 Jan 2025 07:04:12 +0800 [thread overview]
Message-ID: <202501290644.uFIc4hv8-lkp@intel.com> (raw)
In-Reply-To: <20250115160912.617654-5-pbonzini@redhat.com>
Hi Paolo,
kernel test robot noticed the following build errors:
[auto build test ERROR on tip/x86/tdx]
[also build test ERROR on kvm/queue kvm/next linus/master v6.13 next-20250128]
[cannot apply to kvm/linux-next]
[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/Paolo-Bonzini/x86-virt-tdx-Add-SEAMCALL-wrappers-for-TDX-KeyID-management/20250116-001537
base: tip/x86/tdx
patch link: https://lore.kernel.org/r/20250115160912.617654-5-pbonzini%40redhat.com
patch subject: [PATCH v3 04/14] x86/virt/tdx: Add SEAMCALL wrappers for TDX page cache management
config: x86_64-randconfig-076-20250128 (https://download.01.org/0day-ci/archive/20250129/202501290644.uFIc4hv8-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/20250129/202501290644.uFIc4hv8-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/202501290644.uFIc4hv8-lkp@intel.com/
All errors (new ones prefixed by >>):
In file included from arch/x86/include/asm/page.h:89,
from arch/x86/include/asm/processor.h:20,
from arch/x86/include/asm/timex.h:5,
from include/linux/timex.h:67,
from include/linux/clocksource.h:13,
from arch/x86/kernel/kvmclock.c:6:
arch/x86/include/asm/tdx.h: In function 'mk_keyed_paddr':
>> include/asm-generic/memory_model.h:38:58: error: 'vmemmap' undeclared (first use in this function); did you mean 'mem_map'?
38 | #define __page_to_pfn(page) (unsigned long)((page) - vmemmap)
| ^~~~~~~
include/asm-generic/memory_model.h:64:21: note: in expansion of macro '__page_to_pfn'
64 | #define page_to_pfn __page_to_pfn
| ^~~~~~~~~~~~~
include/asm-generic/memory_model.h:70:31: note: in expansion of macro 'page_to_pfn'
70 | unsigned long __pfn = page_to_pfn(page); \
| ^~~~~~~~~~~
arch/x86/include/asm/tdx.h:148:15: note: in expansion of macro 'page_to_phys'
148 | ret = page_to_phys(page);
| ^~~~~~~~~~~~
include/asm-generic/memory_model.h:38:58: note: each undeclared identifier is reported only once for each function it appears in
38 | #define __page_to_pfn(page) (unsigned long)((page) - vmemmap)
| ^~~~~~~
include/asm-generic/memory_model.h:64:21: note: in expansion of macro '__page_to_pfn'
64 | #define page_to_pfn __page_to_pfn
| ^~~~~~~~~~~~~
include/asm-generic/memory_model.h:70:31: note: in expansion of macro 'page_to_pfn'
70 | unsigned long __pfn = page_to_pfn(page); \
| ^~~~~~~~~~~
arch/x86/include/asm/tdx.h:148:15: note: in expansion of macro 'page_to_phys'
148 | ret = page_to_phys(page);
| ^~~~~~~~~~~~
--
In file included from arch/x86/include/asm/page.h:89,
from arch/x86/include/asm/processor.h:20,
from arch/x86/kernel/cpu/hypervisor.c:26:
arch/x86/include/asm/tdx.h: In function 'mk_keyed_paddr':
include/asm-generic/memory_model.h:38:58: error: 'vmemmap' undeclared (first use in this function)
38 | #define __page_to_pfn(page) (unsigned long)((page) - vmemmap)
| ^~~~~~~
include/asm-generic/memory_model.h:64:21: note: in expansion of macro '__page_to_pfn'
64 | #define page_to_pfn __page_to_pfn
| ^~~~~~~~~~~~~
include/asm-generic/memory_model.h:70:31: note: in expansion of macro 'page_to_pfn'
70 | unsigned long __pfn = page_to_pfn(page); \
| ^~~~~~~~~~~
arch/x86/include/asm/tdx.h:148:15: note: in expansion of macro 'page_to_phys'
148 | ret = page_to_phys(page);
| ^~~~~~~~~~~~
include/asm-generic/memory_model.h:38:58: note: each undeclared identifier is reported only once for each function it appears in
38 | #define __page_to_pfn(page) (unsigned long)((page) - vmemmap)
| ^~~~~~~
include/asm-generic/memory_model.h:64:21: note: in expansion of macro '__page_to_pfn'
64 | #define page_to_pfn __page_to_pfn
| ^~~~~~~~~~~~~
include/asm-generic/memory_model.h:70:31: note: in expansion of macro 'page_to_pfn'
70 | unsigned long __pfn = page_to_pfn(page); \
| ^~~~~~~~~~~
arch/x86/include/asm/tdx.h:148:15: note: in expansion of macro 'page_to_phys'
148 | ret = page_to_phys(page);
| ^~~~~~~~~~~~
In file included from arch/x86/include/asm/bug.h:99,
from include/linux/bug.h:5,
from include/linux/alloc_tag.h:8,
from include/linux/percpu.h:5,
from include/linux/context_tracking_state.h:5,
from include/linux/hardirq.h:5,
from include/linux/interrupt.h:11,
from arch/x86/include/asm/kvm_para.h:7,
from arch/x86/include/asm/hypervisor.h:37,
from arch/x86/kernel/cpu/hypervisor.c:27:
>> include/asm-generic/memory_model.h:72:23: error: implicit declaration of function 'pfn_valid' [-Werror=implicit-function-declaration]
72 | WARN_ON_ONCE(!pfn_valid(__pfn)); \
| ^~~~~~~~~
include/asm-generic/bug.h:111:32: note: in definition of macro 'WARN_ON_ONCE'
111 | int __ret_warn_on = !!(condition); \
| ^~~~~~~~~
arch/x86/include/asm/tdx.h:148:15: note: in expansion of macro 'page_to_phys'
148 | ret = page_to_phys(page);
| ^~~~~~~~~~~~
cc1: some warnings being treated as errors
vim +38 include/asm-generic/memory_model.h
8f6aac419bd590 Christoph Lameter 2007-10-16 35
af901ca181d92a André Goddard Rosa 2009-11-14 36 /* memmap is virtually contiguous. */
8f6aac419bd590 Christoph Lameter 2007-10-16 37 #define __pfn_to_page(pfn) (vmemmap + (pfn))
32272a26974d20 Martin Schwidefsky 2008-12-25 @38 #define __page_to_pfn(page) (unsigned long)((page) - vmemmap)
8f6aac419bd590 Christoph Lameter 2007-10-16 39
a117e66ed45ac0 KAMEZAWA Hiroyuki 2006-03-27 40 #elif defined(CONFIG_SPARSEMEM)
a117e66ed45ac0 KAMEZAWA Hiroyuki 2006-03-27 41 /*
1a49123b343461 Zhang Yanfei 2013-10-03 42 * Note: section's mem_map is encoded to reflect its start_pfn.
a117e66ed45ac0 KAMEZAWA Hiroyuki 2006-03-27 43 * section[i].section_mem_map == mem_map's address - start_pfn;
a117e66ed45ac0 KAMEZAWA Hiroyuki 2006-03-27 44 */
67de648211fa04 Andy Whitcroft 2006-06-23 45 #define __page_to_pfn(pg) \
aa462abe8aaf21 Ian Campbell 2011-08-17 46 ({ const struct page *__pg = (pg); \
a117e66ed45ac0 KAMEZAWA Hiroyuki 2006-03-27 47 int __sec = page_to_section(__pg); \
f05b6284ee5d3b Randy Dunlap 2007-02-10 48 (unsigned long)(__pg - __section_mem_map_addr(__nr_to_section(__sec))); \
a117e66ed45ac0 KAMEZAWA Hiroyuki 2006-03-27 49 })
a117e66ed45ac0 KAMEZAWA Hiroyuki 2006-03-27 50
67de648211fa04 Andy Whitcroft 2006-06-23 51 #define __pfn_to_page(pfn) \
a117e66ed45ac0 KAMEZAWA Hiroyuki 2006-03-27 52 ({ unsigned long __pfn = (pfn); \
a117e66ed45ac0 KAMEZAWA Hiroyuki 2006-03-27 53 struct mem_section *__sec = __pfn_to_section(__pfn); \
a117e66ed45ac0 KAMEZAWA Hiroyuki 2006-03-27 54 __section_mem_map_addr(__sec) + __pfn; \
a117e66ed45ac0 KAMEZAWA Hiroyuki 2006-03-27 55 })
bb1c50d3967f69 Mike Rapoport 2021-06-28 56 #endif /* CONFIG_FLATMEM/SPARSEMEM */
67de648211fa04 Andy Whitcroft 2006-06-23 57
012dcef3f05838 Christoph Hellwig 2015-08-07 58 /*
012dcef3f05838 Christoph Hellwig 2015-08-07 59 * Convert a physical address to a Page Frame Number and back
012dcef3f05838 Christoph Hellwig 2015-08-07 60 */
8f235d1a3eb719 Chen Gang 2016-01-14 61 #define __phys_to_pfn(paddr) PHYS_PFN(paddr)
ae4f976968896f Tyler Baker 2015-09-19 62 #define __pfn_to_phys(pfn) PFN_PHYS(pfn)
012dcef3f05838 Christoph Hellwig 2015-08-07 63
67de648211fa04 Andy Whitcroft 2006-06-23 64 #define page_to_pfn __page_to_pfn
67de648211fa04 Andy Whitcroft 2006-06-23 65 #define pfn_to_page __pfn_to_page
a117e66ed45ac0 KAMEZAWA Hiroyuki 2006-03-27 66
3e25d5a49f99b7 Christoph Hellwig 2024-10-23 67 #ifdef CONFIG_DEBUG_VIRTUAL
3e25d5a49f99b7 Christoph Hellwig 2024-10-23 68 #define page_to_phys(page) \
3e25d5a49f99b7 Christoph Hellwig 2024-10-23 69 ({ \
3e25d5a49f99b7 Christoph Hellwig 2024-10-23 70 unsigned long __pfn = page_to_pfn(page); \
3e25d5a49f99b7 Christoph Hellwig 2024-10-23 71 \
3e25d5a49f99b7 Christoph Hellwig 2024-10-23 @72 WARN_ON_ONCE(!pfn_valid(__pfn)); \
3e25d5a49f99b7 Christoph Hellwig 2024-10-23 73 PFN_PHYS(__pfn); \
3e25d5a49f99b7 Christoph Hellwig 2024-10-23 74 })
3e25d5a49f99b7 Christoph Hellwig 2024-10-23 75 #else
c5c3238d9b8cee Christoph Hellwig 2024-10-23 76 #define page_to_phys(page) PFN_PHYS(page_to_pfn(page))
3e25d5a49f99b7 Christoph Hellwig 2024-10-23 77 #endif /* CONFIG_DEBUG_VIRTUAL */
c5c3238d9b8cee Christoph Hellwig 2024-10-23 78 #define phys_to_page(phys) pfn_to_page(PHYS_PFN(phys))
c5c3238d9b8cee Christoph Hellwig 2024-10-23 79
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
next prev parent reply other threads:[~2025-01-28 23:04 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-01-15 16:08 [PATCH v3 00/14] x86/virt/tdx: Add SEAMCALL wrappers for KVM Paolo Bonzini
2025-01-15 16:08 ` [PATCH v3 01/14] x86/virt/tdx: Add SEAMCALL wrappers for TDX KeyID management Paolo Bonzini
2025-01-15 16:09 ` [PATCH v3 02/14] x86/virt/tdx: Add SEAMCALL wrappers for TDX TD creation Paolo Bonzini
2025-01-15 16:09 ` [PATCH v3 03/14] x86/virt/tdx: Add SEAMCALL wrappers for TDX vCPU creation Paolo Bonzini
2025-01-15 16:09 ` [PATCH v3 04/14] x86/virt/tdx: Add SEAMCALL wrappers for TDX page cache management Paolo Bonzini
2025-01-15 16:38 ` Dave Hansen
2025-01-28 23:04 ` kernel test robot [this message]
2025-01-15 16:09 ` [PATCH v3 05/14] x86/virt/tdx: Add SEAMCALL wrappers for TDX VM/vCPU field access Paolo Bonzini
2025-01-15 16:09 ` [PATCH v3 06/14] x86/virt/tdx: Add SEAMCALL wrappers for TDX flush operations Paolo Bonzini
2025-01-15 16:09 ` [PATCH v3 07/14] x86/virt/tdx: Add SEAMCALL wrapper tdh_mem_sept_add() to add SEPT pages Paolo Bonzini
2025-01-15 16:09 ` [PATCH v3 08/14] x86/virt/tdx: Add SEAMCALL wrappers to add TD private pages Paolo Bonzini
2025-01-15 16:09 ` [PATCH v3 09/14] x86/virt/tdx: Add SEAMCALL wrappers to manage TDX TLB tracking Paolo Bonzini
2025-01-15 16:09 ` [PATCH v3 10/14] x86/virt/tdx: Add SEAMCALL wrappers to remove a TD private page Paolo Bonzini
2025-01-15 16:09 ` [PATCH v3 11/14] x86/virt/tdx: Add SEAMCALL wrappers for TD measurement of initial contents Paolo Bonzini
2025-01-15 16:09 ` [PATCH v3 12/14] x86/virt/tdx: Add SEAMCALL wrapper to enter/exit TDX guest Paolo Bonzini
2025-01-15 17:14 ` Adrian Hunter
2025-01-15 16:09 ` [PATCH v3 13/14] x86/virt/tdx: Read essential global metadata for KVM Paolo Bonzini
2025-01-15 16:09 ` [PATCH v3 14/14] x86/virt/tdx: Add tdx_guest_keyid_alloc/free() to alloc and free TDX guest KeyID Paolo Bonzini
2025-01-15 16:39 ` [PATCH v3 00/14] x86/virt/tdx: Add SEAMCALL wrappers for KVM Dave Hansen
2025-01-15 19:14 ` Edgecombe, Rick P
2025-01-15 19:36 ` Paolo Bonzini
2025-01-15 20:06 ` Edgecombe, Rick P
2025-01-16 22:11 ` Edgecombe, Rick P
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=202501290644.uFIc4hv8-lkp@intel.com \
--to=lkp@intel.com \
--cc=oe-kbuild-all@lists.linux.dev \
--cc=pbonzini@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.