All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: Ard Biesheuvel <ardb+git@google.com>
Cc: llvm@lists.linux.dev, oe-kbuild-all@lists.linux.dev
Subject: Re: [PATCH v4 08/11] x86/sev: Split off startup code from core code
Date: Sat, 12 Apr 2025 05:46:25 +0800	[thread overview]
Message-ID: <202504120536.izfqm3jS-lkp@intel.com> (raw)
In-Reply-To: <20250410134117.3713574-21-ardb+git@google.com>

Hi Ard,

kernel test robot noticed the following build errors:

[auto build test ERROR on tip/master]
[also build test ERROR on next-20250411]
[cannot apply to tip/x86/core tip/x86/mm efi/next tip/auto-latest linus/master v6.15-rc1]
[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/Ard-Biesheuvel/x86-asm-Make-rip_rel_ptr-usable-from-fPIC-code/20250410-214836
base:   tip/master
patch link:    https://lore.kernel.org/r/20250410134117.3713574-21-ardb%2Bgit%40google.com
patch subject: [PATCH v4 08/11] x86/sev: Split off startup code from core code
config: x86_64-buildonly-randconfig-005-20250411 (https://download.01.org/0day-ci/archive/20250412/202504120536.izfqm3jS-lkp@intel.com/config)
compiler: clang version 20.1.2 (https://github.com/llvm/llvm-project 58df0ef89dd64126512e4ee27b4ac3fd8ddf6247)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250412/202504120536.izfqm3jS-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/202504120536.izfqm3jS-lkp@intel.com/

All errors (new ones prefixed by >>):

>> arch/x86/boot/compressed/sev.c:263:6: error: call to undeclared function 'vmgexit_psc'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
     263 |         if (vmgexit_psc(boot_ghcb, desc))
         |             ^
>> arch/x86/boot/compressed/sev.c:266:2: error: call to undeclared function 'pvalidate_pages'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
     266 |         pvalidate_pages(desc);
         |         ^
   arch/x86/boot/compressed/sev.c:266:2: note: did you mean 'pvalidate_4k_page'?
   arch/x86/boot/compressed/../../coco/sev/shared.c:1227:20: note: 'pvalidate_4k_page' declared here
    1227 | static void __head pvalidate_4k_page(unsigned long vaddr, unsigned long paddr,
         |                    ^
   2 errors generated.


vim +/vmgexit_psc +263 arch/x86/boot/compressed/sev.c

597cfe48212a3f arch/x86/boot/compressed/sev-es.c Joerg Roedel 2020-09-07  232  
6c3211796326a9 arch/x86/boot/compressed/sev.c    Tom Lendacky 2023-06-06  233  static phys_addr_t __snp_accept_memory(struct snp_psc_desc *desc,
6c3211796326a9 arch/x86/boot/compressed/sev.c    Tom Lendacky 2023-06-06  234  				       phys_addr_t pa, phys_addr_t pa_end)
6c3211796326a9 arch/x86/boot/compressed/sev.c    Tom Lendacky 2023-06-06  235  {
6c3211796326a9 arch/x86/boot/compressed/sev.c    Tom Lendacky 2023-06-06  236  	struct psc_hdr *hdr;
6c3211796326a9 arch/x86/boot/compressed/sev.c    Tom Lendacky 2023-06-06  237  	struct psc_entry *e;
6c3211796326a9 arch/x86/boot/compressed/sev.c    Tom Lendacky 2023-06-06  238  	unsigned int i;
6c3211796326a9 arch/x86/boot/compressed/sev.c    Tom Lendacky 2023-06-06  239  
6c3211796326a9 arch/x86/boot/compressed/sev.c    Tom Lendacky 2023-06-06  240  	hdr = &desc->hdr;
6c3211796326a9 arch/x86/boot/compressed/sev.c    Tom Lendacky 2023-06-06  241  	memset(hdr, 0, sizeof(*hdr));
6c3211796326a9 arch/x86/boot/compressed/sev.c    Tom Lendacky 2023-06-06  242  
6c3211796326a9 arch/x86/boot/compressed/sev.c    Tom Lendacky 2023-06-06  243  	e = desc->entries;
6c3211796326a9 arch/x86/boot/compressed/sev.c    Tom Lendacky 2023-06-06  244  
6c3211796326a9 arch/x86/boot/compressed/sev.c    Tom Lendacky 2023-06-06  245  	i = 0;
6c3211796326a9 arch/x86/boot/compressed/sev.c    Tom Lendacky 2023-06-06  246  	while (pa < pa_end && i < VMGEXIT_PSC_MAX_ENTRY) {
6c3211796326a9 arch/x86/boot/compressed/sev.c    Tom Lendacky 2023-06-06  247  		hdr->end_entry = i;
6c3211796326a9 arch/x86/boot/compressed/sev.c    Tom Lendacky 2023-06-06  248  
6c3211796326a9 arch/x86/boot/compressed/sev.c    Tom Lendacky 2023-06-06  249  		e->gfn = pa >> PAGE_SHIFT;
6c3211796326a9 arch/x86/boot/compressed/sev.c    Tom Lendacky 2023-06-06  250  		e->operation = SNP_PAGE_STATE_PRIVATE;
6c3211796326a9 arch/x86/boot/compressed/sev.c    Tom Lendacky 2023-06-06  251  		if (IS_ALIGNED(pa, PMD_SIZE) && (pa_end - pa) >= PMD_SIZE) {
6c3211796326a9 arch/x86/boot/compressed/sev.c    Tom Lendacky 2023-06-06  252  			e->pagesize = RMP_PG_SIZE_2M;
6c3211796326a9 arch/x86/boot/compressed/sev.c    Tom Lendacky 2023-06-06  253  			pa += PMD_SIZE;
6c3211796326a9 arch/x86/boot/compressed/sev.c    Tom Lendacky 2023-06-06  254  		} else {
6c3211796326a9 arch/x86/boot/compressed/sev.c    Tom Lendacky 2023-06-06  255  			e->pagesize = RMP_PG_SIZE_4K;
6c3211796326a9 arch/x86/boot/compressed/sev.c    Tom Lendacky 2023-06-06  256  			pa += PAGE_SIZE;
6c3211796326a9 arch/x86/boot/compressed/sev.c    Tom Lendacky 2023-06-06  257  		}
6c3211796326a9 arch/x86/boot/compressed/sev.c    Tom Lendacky 2023-06-06  258  
6c3211796326a9 arch/x86/boot/compressed/sev.c    Tom Lendacky 2023-06-06  259  		e++;
6c3211796326a9 arch/x86/boot/compressed/sev.c    Tom Lendacky 2023-06-06  260  		i++;
6c3211796326a9 arch/x86/boot/compressed/sev.c    Tom Lendacky 2023-06-06  261  	}
6c3211796326a9 arch/x86/boot/compressed/sev.c    Tom Lendacky 2023-06-06  262  
6c3211796326a9 arch/x86/boot/compressed/sev.c    Tom Lendacky 2023-06-06 @263  	if (vmgexit_psc(boot_ghcb, desc))
6c3211796326a9 arch/x86/boot/compressed/sev.c    Tom Lendacky 2023-06-06  264  		sev_es_terminate(SEV_TERM_SET_LINUX, GHCB_TERM_PSC);
6c3211796326a9 arch/x86/boot/compressed/sev.c    Tom Lendacky 2023-06-06  265  
6c3211796326a9 arch/x86/boot/compressed/sev.c    Tom Lendacky 2023-06-06 @266  	pvalidate_pages(desc);
6c3211796326a9 arch/x86/boot/compressed/sev.c    Tom Lendacky 2023-06-06  267  
6c3211796326a9 arch/x86/boot/compressed/sev.c    Tom Lendacky 2023-06-06  268  	return pa;
6c3211796326a9 arch/x86/boot/compressed/sev.c    Tom Lendacky 2023-06-06  269  }
6c3211796326a9 arch/x86/boot/compressed/sev.c    Tom Lendacky 2023-06-06  270  

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

  reply	other threads:[~2025-04-11 21:47 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-04-10 13:41 [PATCH v4 00/11] x86: Refactor and consolidate startup code Ard Biesheuvel
2025-04-10 13:41 ` [PATCH v4 01/11] x86/asm: Make rip_rel_ptr() usable from fPIC code Ard Biesheuvel
2025-04-12 14:33   ` [tip: x86/boot] " tip-bot2 for Ard Biesheuvel
2025-04-10 13:41 ` [PATCH v4 02/11] x86/boot: Move the early GDT/IDT setup code into startup/ Ard Biesheuvel
2025-04-12 14:33   ` [tip: x86/boot] " tip-bot2 for Ard Biesheuvel
2025-04-10 13:41 ` [PATCH v4 03/11] x86/boot: Move early kernel mapping " Ard Biesheuvel
2025-04-12 14:33   ` [tip: x86/boot] " tip-bot2 for Ard Biesheuvel
2025-04-10 13:41 ` [PATCH v4 04/11] x86/boot: Drop RIP_REL_REF() uses from early mapping code Ard Biesheuvel
2025-04-12 14:33   ` [tip: x86/boot] " tip-bot2 for Ard Biesheuvel
2025-04-10 13:41 ` [PATCH v4 05/11] x86/boot: Move early SME init code into startup/ Ard Biesheuvel
2025-04-12 14:33   ` [tip: x86/boot] " tip-bot2 for Ard Biesheuvel
2025-04-10 13:41 ` [PATCH v4 06/11] x86/boot: Drop RIP_REL_REF() uses from SME startup code Ard Biesheuvel
2025-04-12 14:33   ` [tip: x86/boot] " tip-bot2 for Ard Biesheuvel
2025-04-10 13:41 ` [PATCH v4 07/11] x86/sev: Prepare for splitting off early SEV code Ard Biesheuvel
2025-04-12 14:33   ` [tip: x86/boot] " tip-bot2 for Ard Biesheuvel
2025-04-10 13:41 ` [PATCH v4 08/11] x86/sev: Split off startup code from core code Ard Biesheuvel
2025-04-11 21:46   ` kernel test robot [this message]
2025-04-12 12:23   ` Ingo Molnar
2025-04-12 18:47     ` Ingo Molnar
2025-04-12 20:08       ` Ingo Molnar
2025-04-12 20:24         ` Ard Biesheuvel
2025-04-12 20:50           ` Ingo Molnar
2025-04-10 13:41 ` [PATCH v4 09/11] x86/boot: Move SEV startup code into startup/ Ard Biesheuvel
2025-04-10 13:41 ` [PATCH v4 10/11] x86/boot: Drop RIP_REL_REF() uses from early SEV code Ard Biesheuvel
2025-04-10 13:41 ` [PATCH v4 11/11] x86/asm: Retire RIP_REL_REF() Ard Biesheuvel
2025-04-11 19:15 ` [PATCH v4 00/11] x86: Refactor and consolidate startup code Borislav Petkov

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=202504120536.izfqm3jS-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=ardb+git@google.com \
    --cc=llvm@lists.linux.dev \
    --cc=oe-kbuild-all@lists.linux.dev \
    /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.