[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 3/5] xen/arm: mpu: Move some of the definitions to common file
- To: Ayan Kumar Halder <ayan.kumar.halder@xxxxxxx>
- From: Luca Fancellu <Luca.Fancellu@xxxxxxx>
- Date: Thu, 6 Feb 2025 15:01:39 +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=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=tWPZxCELBs2R1/ChJWb1T8mSgOW396uZbGTcuivp3Wg=; b=XILQbSRyl3s2aaQ1/+WuA/9yr+XNL4iVEFMimTJQJHZq/AAx755jen8XzI3zB7e2hoQLaQ9pEEjD0sJaKlgplZhbsoTRU35k5AqsrIfmnICMZet7rZ5EOdtFeNNTgW7fB/UoHV5FNXAxj31SMssso4/lQJbhs9IEyXD3PL2bOTrGBYWcQjhj/6mJW5+CV+BqEBU8T+obUtE/RZ6wqdaTJ+M3LfFEbpHDlpvoFPKuE9EUsa9VDiE6hB4AHI27P9GXAsRob5QCynvlOW4RP8kVBZ86oLv9AM4AlQpd+EgHFeI9mBNKTwxFfwFDBUUZeCvTX5wd1LDgtDk7cIi4+p810w==
- 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=tWPZxCELBs2R1/ChJWb1T8mSgOW396uZbGTcuivp3Wg=; b=CiLJZxz9xxn2trpfmmRxdnpdTLYYO8HW70EJzRPHCYUQcgYlz326V6VevlKoEtFvRoy+iHV9TvX+UJnkqeT1NR7XUQp9+lCKJ/9VIHXZ60D7F7delWapJ0/zpMBmRTUnnCaE/eGQWEXx04ErIspfLJ4pMBzKZDtjsGuK/dP4S28u4g3YuQadZDWbgy1VZY64Or0NoNkARyrMQX//4VekTx5YWq47kAuafO1coA2GPxV9ZCbKwB6VUWoW98RG09vkPWxqDWH9jVMBpX4MrV9HgFkLAmpaQyOXbhEKvUKMZDZwr4l4Hp74AobuI3rDgqfz5784FNoH4BxEUwukxZCPVw==
- Arc-seal: i=2; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=pass; b=UVt9LGtshASSPNs87lShPUW64ImgDFylz4oA9ULdSRd/tjBNP76/wGn/jBDHmBTumdFKQyILPltMpJh91VZ3BS7F8pd0LvyF56AbMG52cfkpoK7XWqbPH3jc7t+Uh0G7OCkb5PRblsg96iao9lt+Z7t2k/Kn7GsdWywqUoZdYMIoxVHl8ZI+bMoC0g1hmDS3nWjRAp+CJghNiPcL7hHSKfeGafU9cmhLScZTtSfAzdUFhffvmRDkOEJA4EETQHefQbhhndBijEtSLGLeK1V7k3jBBP3QIIsjpxIfiB7kT6jQcrPJdVGR5povMQ9AvMvT242lSN0vClDvEtCH6C1D4Q==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=rLnCemUJ8bV9kn5VjTrV1bd0JAZtU/ZqW50PLXNV8cigV9w0Y0+XFs5utRI6iHifIK/X440Qrxeyflub4BsuSb1S8xEjs/GOp5pfusv69H6AAVk6svbzHClABplHgShzWFe/QL+JE/dPJtAXxD+IRbKniGAOPNdwLDgSTwCstE47CWuzxaynetqXPJ4y30NfnLGaTjn85FEcPAbI20+1mOluSY93IMBvi5LETjoXyFuNJpPyVxrAwCDRtsnXGTOhNcVUXPt145ybW+Tkt1cmnCpqsK/KPWKXHEeUJ8ugO62d/yNXps5VKUl8wf09fkWVBNrMHD7qbOyee1y1pZLI9w==
- Authentication-results-original: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
- Cc: "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Julien Grall <julien@xxxxxxx>, Bertrand Marquis <Bertrand.Marquis@xxxxxxx>, Michal Orzel <michal.orzel@xxxxxxx>, Volodymyr Babchuk <Volodymyr_Babchuk@xxxxxxxx>
- Delivery-date: Thu, 06 Feb 2025 15:02:05 +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: AQHbdzpt64DSSlVqrUGsnEaCrB85rbM6YXOA
- Thread-topic: [PATCH 3/5] xen/arm: mpu: Move some of the definitions to common file
Hi Ayan,
> On 4 Feb 2025, at 19:23, Ayan Kumar Halder <ayan.kumar.halder@xxxxxxx> wrote:
>
> For AArch32, refer to ARM DDI 0568A.c ID110520.
> MPU_REGION_SHIFT is same between AArch32 and AArch64 (HPRBAR).
> Also, NUM_MPU_REGIONS_SHIFT is same between AArch32 and AArch64
> (HMPUIR).
>
> Signed-off-by: Ayan Kumar Halder <ayan.kumar.halder@xxxxxxx>
> ---
> xen/arch/arm/arm64/mpu/head.S | 2 +-
> xen/arch/arm/include/asm/early_printk.h | 2 +-
> xen/arch/arm/include/asm/{arm64 => }/mpu.h | 6 +++---
> 3 files changed, 5 insertions(+), 5 deletions(-)
> rename xen/arch/arm/include/asm/{arm64 => }/mpu.h (87%)
>
> diff --git a/xen/arch/arm/arm64/mpu/head.S b/xen/arch/arm/arm64/mpu/head.S
> index e4f2021f45..7b659aa42b 100644
> --- a/xen/arch/arm/arm64/mpu/head.S
> +++ b/xen/arch/arm/arm64/mpu/head.S
> @@ -3,7 +3,7 @@
> * Start-of-day code for an Armv8-R MPU system.
> */
>
> -#include <asm/arm64/mpu.h>
> +#include <asm/mpu.h>
> #include <asm/early_printk.h>
>
> /* Backgroud region enable/disable */
> diff --git a/xen/arch/arm/include/asm/early_printk.h
> b/xen/arch/arm/include/asm/early_printk.h
> index 219705a8b6..644fd0fcfb 100644
> --- a/xen/arch/arm/include/asm/early_printk.h
> +++ b/xen/arch/arm/include/asm/early_printk.h
> @@ -11,7 +11,7 @@
> #define __ARM_EARLY_PRINTK_H__
>
> #include <xen/page-size.h>
> -#include <asm/arm64/mpu.h>
> +#include <asm/mpu.h>
> #include <asm/fixmap.h>
>
> #ifdef CONFIG_EARLY_PRINTK
> diff --git a/xen/arch/arm/include/asm/arm64/mpu.h
> b/xen/arch/arm/include/asm/mpu.h
Why not in include/mpu/ ?
Cheers,
Luca
|
