[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH 05/19] arm: Remove dependencies with membank(s) definitions from setup.h


  • To: Stefano Stabellini <sstabellini@xxxxxxxxxx>
  • From: Alejandro Vallejo <agarciav@xxxxxxx>
  • Date: Mon, 2 Jun 2025 16:07:45 +0200
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=kernel.org smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none (0)
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=VRduVKyxGWOIw04J5RNaOAAl08qw4iMH6VZDnrnwcVg=; b=d6ODP1Fs1npgbd3yAUFgezsnDD0KrY6aQlbXpH9a40bx110sfSdVtDoOyyHYSAXkN6jXnaQ7L/JfmjCZU+1ClR9WqJMNHVcDWiJ+x6vDy1OTeIC6xnl0/JCrT3fchBdtXLMc9w5rC1CiWBCyH/C/8ICD7uYh+RCxBXEHtDCUAKw9hyu3alj8oyVC2X24zB8XLVtwyrQxPAcWK8pry66vMIf/TjSOaljMz5afHAmFkKtQlxnNhioIkiIag7R0TS8vJCgYggjmElczrR0HvYOhzkVCPfjViooTCsxiyvWsXqs8tkA3a9zKztw+BFFL4GTO1MCL2mI1SkIiWODSRRsBhw==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=Y7d5grMKwYOQfwC9YS4Wqr+19iwlgYbkjPypw0o6TQFnFWmunL5GHCSqrz2VnJEf9k4fE8exmQGle32Dk52m5K0eO64HXxPMamcRsRPSdUV2cLi5bPivv2X7EDwiA1wn4aZqEM3Pb+430ZLYu/fWnEGOn9ABBIRLy2h2VaPZ8+zHU/OpXFmHsiFNWpCXL3elZ1I7lYAgMWAsp60YJg/KR+Nf4efxZkcBYnFkNkIWOCBml8yQxyOUf4YfjoGpSwOccbv82OY/5ztZ+2P/uhFcRc+drbCmAoJ95msyPojGNLDGdqquaTPnRnKi7c1hctnds2Z2xCV/xRXTpLJ0rVvZ8A==
  • Cc: <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Julien Grall <julien@xxxxxxx>, "Bertrand Marquis" <bertrand.marquis@xxxxxxx>, Michal Orzel <michal.orzel@xxxxxxx>, Volodymyr Babchuk <Volodymyr_Babchuk@xxxxxxxx>, "Daniel P. Smith" <dpsmith@xxxxxxxxxxxxxxxxxxxx>
  • Delivery-date: Mon, 02 Jun 2025 14:08:27 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On Sat May 31, 2025 at 2:51 AM CEST, Stefano Stabellini wrote:
> On Fri, 30 May 2025, Alejandro Vallejo wrote:
>> ... as they can be forward-declared changing arrays for pointers in the 
>> function
>> declarations.
>> 
>> No functional change.
>> 
>> Signed-off-by: Alejandro Vallejo <agarciav@xxxxxxx>
>> ---
>>  xen/arch/arm/include/asm/setup.h | 10 ++++++----
>>  1 file changed, 6 insertions(+), 4 deletions(-)
>> 
>> diff --git a/xen/arch/arm/include/asm/setup.h 
>> b/xen/arch/arm/include/asm/setup.h
>> index 6cf272c160..0f9e531a34 100644
>> --- a/xen/arch/arm/include/asm/setup.h
>> +++ b/xen/arch/arm/include/asm/setup.h
>> @@ -3,7 +3,6 @@
>>  
>>  #include <public/version.h>
>>  #include <asm/p2m.h>
>> -#include <xen/bootfdt.h>
>>  #include <xen/device_tree.h>
>
> This change breaks the build on ARM:
>
>   CC      xsm/xsm_policy.o
> xsm/xsm_policy.c: In function ‘xsm_dt_policy_init’:
> xsm/xsm_policy.c:71:30: error: implicit declaration of function 
> ‘boot_module_find_by_kind’ [-Werror=implicit-function-declaration]
>    71 |     struct bootmodule *mod = boot_module_find_by_kind(BOOTMOD_XSM);
>       |                              ^~~~~~~~~~~~~~~~~~~~~~~~
> xsm/xsm_policy.c:71:30: error: nested extern declaration of 
> ‘boot_module_find_by_kind’ [-Werror=nested-externs]
> xsm/xsm_policy.c:71:55: error: ‘BOOTMOD_XSM’ undeclared (first use in this 
> function)
>    71 |     struct bootmodule *mod = boot_module_find_by_kind(BOOTMOD_XSM);
>       |                                                       ^~~~~~~~~~~
> xsm/xsm_policy.c:71:55: note: each undeclared identifier is reported only 
> once for each function it appears in
> xsm/xsm_policy.c:74:22: error: dereferencing pointer to incomplete type 
> ‘struct bootmodule’
>    74 |     if ( !mod || !mod->size )
>       |                      ^~
> cc1: all warnings being treated as errors
> make[2]: *** [Rules.mk:249: xsm/xsm_policy.o] Error 1
> make[1]: *** [build.mk:72: xsm] Error 2
> make: *** [Makefile:619: xen] Error 2
>
> The rest looks OK

Ugh. The series passed regular CI, but it must not have had XSM compiled-in. 
I'll
sort that out and re-send.

Cheers,
Alejandro



 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.