[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [XEN v2 09/11] xen/arm: Introduce ARM_PA_32 to support 32 bit physical address
- To: Ayan Kumar Halder <ayan.kumar.halder@xxxxxxx>
- From: Jan Beulich <jbeulich@xxxxxxxx>
- Date: Wed, 18 Jan 2023 09:50:52 +0100
- Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=suse.com; dmarc=pass action=none header.from=suse.com; dkim=pass header.d=suse.com; arc=none
- 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=YMP6Ax4MV+910RQf3bz23z9fo9ecWxfLuUkM/UYhYZ0=; b=h5DKwP4GQumEmHWa2z7W6aim23xTxgcOQtIGW2YzYby+t07XM8U3vaqEjltWNOKQ3IHWjMbka7zt8v0l0FLPU3ezZNxjmoX8rwiYFGlnpgCbo3eiMyirngsee7J/SPzHq2BB6Wapd/dBad1PyyYo/Fznswoq+qyhfgQmeHxdGlHiJQcadvGjgde6OSbiRn7PNCX8KCxNKC6Y0wZ1eofGbaRGlzZIXAeSAnGSgnjwVnALYU6KSgmktJ1rMXTSD13He1vDKhYxj8GDkDxDSqJH8xP48sFMvopaZh5ls3XjSFUtiBOae1awxGLQwfceM2lKOORyKgQItdhULFVbcIb/Wg==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=ni5F42P38v1bF85DzMiPZOrUlWu/vqAbeO/q3SKWpNV7HQWHSwU/M4wdFq3WFGkk7Nr39e9OA2F7a8p3tN2D1I3Yze1uLZjwa93q2DOBZuixGFliDDxvxAUAtoYExJiomd7o3Gg5zqS3bl2WP1Pbf6yhOCygQSsz6y8s7aRa+/AfRue9QgvIm4n55CY+skLV3M3J8cPJYG+Ox8NSHXZdbI8aH1cjg7SlclqrLqZH5IvNe6E1m/hCwOg/eQLKbrOH09PSkgTINCRbUEhmBd+4pK3MQcW2KMhc1lREW40iWfysC3irRlsIZkir+eUFo8S0p4fmXvCvfaMEUj86gqy8sg==
- Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=suse.com;
- Cc: sstabellini@xxxxxxxxxx, stefano.stabellini@xxxxxxx, julien@xxxxxxx, Volodymyr_Babchuk@xxxxxxxx, bertrand.marquis@xxxxxxx, xen-devel@xxxxxxxxxxxxxxxxxxxx
- Delivery-date: Wed, 18 Jan 2023 08:51:13 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
On 17.01.2023 18:43, Ayan Kumar Halder wrote:
> --- a/xen/arch/arm/include/asm/types.h
> +++ b/xen/arch/arm/include/asm/types.h
> @@ -37,9 +37,16 @@ typedef signed long long s64;
> typedef unsigned long long u64;
> typedef u32 vaddr_t;
> #define PRIvaddr PRIx32
> +#if defined(CONFIG_ARM_PA_32)
> +typedef u32 paddr_t;
> +#define INVALID_PADDR (~0UL)
> +#define PADDR_SHIFT BITS_PER_LONG
> +#define PRIpaddr PRIx32
> +#else
With our plan to consolidate basic type definitions into xen/types.h
the use of ARM_PA_32 is problematic: Preferably we'd have an arch-
independent Kconfig setting, much like Linux'es PHYS_ADDR_T_64BIT
(I don't think we should re-use the name directly, as we have no
phys_addr_t type), which each arch selects (or not) accordingly.
Perhaps architectures already selecting 64BIT wouldn't need to do
this explicitly, and instead 64BIT could select the new setting
(or the new setting could default to Y when 64BIT=y).
Jan
|
