[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH for-4.19] xen/arm: static-shmem: fix "gbase/pbase used uninitialized" build failure
- To: Michal Orzel <michal.orzel@xxxxxxx>
- From: Bertrand Marquis <Bertrand.Marquis@xxxxxxx>
- Date: Wed, 19 Jun 2024 09:02:35 +0000
- Accept-language: en-GB, en-US
- Arc-authentication-results: i=2; mx.microsoft.com 1; spf=pass (sender ip is 63.35.35.123) smtp.rcpttodomain=lists.xenproject.org smtp.mailfrom=arm.com; dmarc=pass (p=none sp=none pct=100) action=none header.from=arm.com; dkim=pass (signature was verified) header.d=arm.com; arc=pass (0 oda=1 ltdi=1 spf=[1,1,smtp.mailfrom=arm.com] dkim=[1,1,header.d=arm.com] dmarc=[1,1,header.from=arm.com])
- Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=arm.com; dmarc=pass action=none header.from=arm.com; dkim=pass header.d=arm.com; arc=none
- Arc-message-signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; 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=Dg8wFKGvi0pJ6iFSBlBGiM/4Rp2i3FxBb1Sx8wo+uGA=; b=WEO3q9QBLlkz2k/0VdE1eupLrmLWDQvq4Ir+BjBiGOmdG7eyJr4/ZPGDB2FSFF2TqDCmPRtTmQF6I3AJyAznRLrV2EUBbHQRayy3XEcTFQIJpyu2xl8MbfOulKTAQL824F65O2dUj9MGFBGFsgAD7XXYDLRzT+8Mn2dDsNyfgKNe0NA3ve2Ar2z08Q9KJvnHBGH565UwWRVBy/vULzI4KhwJnfg3L/IsLLrqBmYTFMLJpGAnyL7qeumkxBUZHpO6LpNvJWwCTX29CLuy/9c5Umnk8xk83wraXFqeebXyrB10vyzedQt2pt/VA8fAvn0n4g9f7URVzTSRaX2LJ0FHUw==
- Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; 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=Dg8wFKGvi0pJ6iFSBlBGiM/4Rp2i3FxBb1Sx8wo+uGA=; b=G1cJH2IZ4BbjlrhSXEg8A3Ve/mv3B8mbwsjBxuCyvkwSWzxZIM54e/wd5yeRJZQthl5m8GyGtjkrBHPz9yWZdJ6aP6CgB2MJoKr/APCSkbLB2oPB/Bkhg4JFD2l0IyOUFSVgk32ndIdnksvMMgfE1uEN61fruniYV9wrLXAYWs0PUl51jXqm4qDXeooYc8fHk8u5v7yiS1cDJyRwCwNvJYEjW0Olh1lyPA4RRkJnkwvh40UakUw+PNqOqNX+Z+x95OOMBJ3TiLroZAXFDNcpStlpAfWsrFnWWRp3iNKZ6oFFrapI7qd+DieMILJdcBkI/AyjDghO7SX/+hHBqBDMQA==
- Arc-seal: i=2; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=pass; b=Df/cpKb00GgJeb7wMCAyoCtgxFw78VB8dPAFA9+LLEio4PaUNlMT/yaXIQERZvhEnz9hUEI5X7D4vaR9EnzRGAXvc4ZEX01MZySJUlW4+Y+ocLIRDbOcgaQunq6Hozj01mFsYPYTgvOL/Mtb4tYZ3Do27/4xErlcCXCHgD/3EErgyjKOk8K2x7Z2VZaO6QnXaM4TrlM7HHarSee60eKPcGcvkfOMjm+sEK18/kud+Bome7ZFA7IF/ZyvmqWfXvacdktS+a0DewIlInnM6KH4mTUGTy8TiAF0DZMkh1XQTLQ/y2u8TEaAS8ZIL+BhRNOFDqK8hQYWZYvQyJgvmT0+Xg==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=cbVVfGK9mvbr+hFDApZAdyTdvICPFvLVGg3tJNVpV2dcLgUFJjTJkxuQggryn1YRbB3HHTXKfcw8Bh6v4frp13okupsdIzGHWh+YxgYaqUvUSEYOjIxXj4V8TEFNw2+UwWAPEKOzXAHs06yfuYpU5iazjpzQEtp41IuTXe44EXgCHoAFdqWiizCZ+k2IB7lKkKRIFmrLV+4WkexVIeqxT8aiaBmwPEmy0QL8KkVuJ0XJlQDO0d4jQGRA4SSoyjAEN+hyLjtIV3jDWUtL+k5ZR2ILLhGwspDbVzXrmMtU/T9NupBma+9K5bFj3IYY2lL6393tanoWV0xjVpgaWDjVmg==
- Authentication-results-original: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
- Cc: Xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Julien Grall <julien@xxxxxxx>, Volodymyr Babchuk <Volodymyr_Babchuk@xxxxxxxx>, Oleksii <oleksii.kurochko@xxxxxxxxx>
- Delivery-date: Wed, 19 Jun 2024 09:02:51 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
- Nodisclaimer: true
- Original-authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
- Thread-index: AQHawhSD1mJnwmtkoE+LLMyZ3OtgsbHOyqWA
- Thread-topic: [PATCH for-4.19] xen/arm: static-shmem: fix "gbase/pbase used uninitialized" build failure
Hi,
Adding Oleksii for Release ack.
Cheers
Bertrand
> On 19 Jun 2024, at 08:46, Michal Orzel <michal.orzel@xxxxxxx> wrote:
>
> Building Xen with CONFIG_STATIC_SHM=y results in a build failure:
>
> arch/arm/static-shmem.c: In function 'process_shm':
> arch/arm/static-shmem.c:327:41: error: 'gbase' may be used uninitialized
> [-Werror=maybe-uninitialized]
> 327 | if ( is_domain_direct_mapped(d) && (pbase != gbase) )
> arch/arm/static-shmem.c:305:17: note: 'gbase' was declared here
> 305 | paddr_t gbase, pbase, psize;
>
> This is because the commit cb1ddafdc573 adds a check referencing
> gbase/pbase variables which were not yet assigned a value. Fix it.
>
> Fixes: cb1ddafdc573 ("xen/arm/static-shmem: Static-shmem should be
> direct-mapped for direct-mapped domains")
> Signed-off-by: Michal Orzel <michal.orzel@xxxxxxx>
> ---
> Rationale for 4.19: this patch fixes a build failure reported by CI:
> https://gitlab.com/xen-project/xen/-/jobs/7131807878
> ---
> xen/arch/arm/static-shmem.c | 13 +++++++------
> 1 file changed, 7 insertions(+), 6 deletions(-)
>
> diff --git a/xen/arch/arm/static-shmem.c b/xen/arch/arm/static-shmem.c
> index c434b96e6204..cd48d2896b7e 100644
> --- a/xen/arch/arm/static-shmem.c
> +++ b/xen/arch/arm/static-shmem.c
> @@ -324,12 +324,6 @@ int __init process_shm(struct domain *d, struct
> kernel_info *kinfo,
> printk("%pd: static shared memory bank not found: '%s'", d,
> shm_id);
> return -ENOENT;
> }
> - if ( is_domain_direct_mapped(d) && (pbase != gbase) )
> - {
> - printk("%pd: physical address 0x%"PRIpaddr" and guest address
> 0x%"PRIpaddr" are not direct-mapped.\n",
> - d, pbase, gbase);
> - return -EINVAL;
> - }
>
> pbase = boot_shm_bank->start;
> psize = boot_shm_bank->size;
> @@ -353,6 +347,13 @@ int __init process_shm(struct domain *d, struct
> kernel_info *kinfo,
> /* guest phys address is after host phys address */
> gbase = dt_read_paddr(cells + addr_cells, addr_cells);
>
> + if ( is_domain_direct_mapped(d) && (pbase != gbase) )
> + {
> + printk("%pd: physical address 0x%"PRIpaddr" and guest
> address 0x%"PRIpaddr" are not direct-mapped.\n",
> + d, pbase, gbase);
> + return -EINVAL;
> + }
> +
> for ( i = 0; i < PFN_DOWN(psize); i++ )
> if ( !mfn_valid(mfn_add(maddr_to_mfn(pbase), i)) )
> {
> --
> 2.25.1
>
|
