[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 <ayankuma@xxxxxxx>
- From: Luca Fancellu <Luca.Fancellu@xxxxxxx>
- Date: Tue, 4 Mar 2025 13:09:57 +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=U8WJT6ccS9Y9oaYaix56tGOCJVW7hAmA9S8Xsf8GRaA=; b=JnXgw8CyHetTkW8d8xvfQ98RZUE0hPMGsCpQS6TT8XQIDzKWflnA75VL7sI+VfFnmUzZxQcaKID0fuec0TK1soRBVoqjxRrwHoa56Dwl/9DjF1I8oNUU8B4m/fhCc9da+bZbx9u16Nkr/DbUV/SACpyIZdz9G5jaJWy9mxlc+OGX9TVxK7kjSPoCC3fSKJyxz9fK1ZVmoVlqT1x1GCRLh/nEmcdZ1ed+hPJF3+gahctHz5zfJpdli9IXPrc8+9B2V1cdHPbVCDOb7/iYda2jtw+4cyHj0M9qJJgVP0D53mEEk8rovsjoYtQcU8lWZ5CWYoHS1gmcGGeNy8pO2rp2lg==
- 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=U8WJT6ccS9Y9oaYaix56tGOCJVW7hAmA9S8Xsf8GRaA=; b=JBGC7yMYyzv8MyNsDmysl/vkbTEG77gZlVhejJi1BaRtKc5LUtEQuKXai3QWJ1ResppwEbWL1B6SWXIU3/e0k7AP2Fv0bHk53K1J717+05F79M5gHXmRDqCiTMsngh29x2u5tYSNF8smT79x3nRhotxP2ZafvPsB9TaAQUHeNdge/aL3V7Q0v0q7kLiNtRbIr/Uy9bzZWIm2evA7yuYD6IAMTpRpKXTBsF4Z4by4wTN4km29+x2Fba8z1rYSD9QozeWWEVIi7JueTt20599onUmFMUWNuPHa/7LHiWA7OcWSluoaLn69ii2JtpuZyi0SSad5sSbsogtw4a8DSXdhYw==
- Arc-seal: i=2; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=pass; b=XV6VPxpGihc0i2A+ssLcvu1azxH6lnbcwCml+N3xaPbezLkCgf0Fg31sBZqKaoeVrwoRu4lS2BfYS07X74iDyk7jQ5IBjvsmWxU3GUc/y2IInF3AJ/qhbunwVtechRjldxNxLfVjk+cs1VQ+pMjZFBr3U0udzNvXv9sJHQicxHotCAa8IICI/7JEcAGOtNzOGmN2bgSiC2RqOD2cLeiL+UCdardOUYxV2UdfN/oW4SiFUPhYeVXFcXM8oQCwUfWAkdFS2j4iULnukknNTGQnRrAuZv06MyC8vc51392GCWYymM55rfEJs1YNAkbibsinHn6Si81LLfAtcazOSxn/sA==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=mQWUe5u4Zra6qL19bxrbR0YYEvZjMr/7j0H3XvOL3UTJlkWD9BJXVICumqcyvzDM6hXCGUoiQGHVgJTYyMRDst3SohWm7xcqEKcEUKe3eYE+yOhFXcB8ZSfDZY4MN6tCqfKsl/tbwX7KgVRMZq/n9xrhSUPUIE0rTDhFn8+NWaIaMgn1d5avKPeo9V32eRzI6fqM22Jb/2kB8XGk0bDsdI8e/tnJ9imlgvbvDveqiQbxUUC1yeAdY5aIHDAJVpv2xD5Oxz+i+qUCpsJzmvpMqLW1jkZNo/U1/sds57Z2TtCUMVcQLfvxOFCnWhxEP6BVDx1o/E28ZzZPvzWzR3rY9w==
- Authentication-results-original: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
- Cc: Julien Grall <julien@xxxxxxx>, Ayan Kumar Halder <ayan.kumar.halder@xxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Bertrand Marquis <Bertrand.Marquis@xxxxxxx>, Michal Orzel <michal.orzel@xxxxxxx>, Volodymyr Babchuk <Volodymyr_Babchuk@xxxxxxxx>
- Delivery-date: Tue, 04 Mar 2025 13:10:20 +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: AQHbdzpt64DSSlVqrUGsnEaCrB85rbM6YXOAgB5TtQCAAK0QgIADM9QAgAAUVICABnR7gA==
- Thread-topic: [PATCH 3/5] xen/arm: mpu: Move some of the definitions to common file
> On 28 Feb 2025, at 10:34, Ayan Kumar Halder <ayankuma@xxxxxxx> wrote:
>
> Hi Julien/Luca,
>
> On 28/02/2025 09:22, Julien Grall wrote:
>>
>>
>> On 26/02/2025 08:28, Luca Fancellu wrote:
>>> Hi Julien,
>>
>> Hi Luca,
>>
>>>
>>>>>>
>>>>>> #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/ ?
>>>>
>>>> Do you mean include/asm/mpu? or something different?
>>>
>>> Yes, sorry typo, I mean include/asm/mpu/mpu.h
>>
>> Thanks for the clarification. I don't have a strong opinion either way. I
>> will let Ayan decide.
> Can I leave as it is for the time being ?
>
> I mean I will create "xen/arch/arm/include/asm/mpu/" directory when I know
> there will be more files.
>
> Let me know what you suggest.
>
> - Ayan
Ok, as agreed on Matrix that Ayan would like to go with include/asm/mpu.h:
Reviewed-by: Luca Fancellu <luca.fancellu@xxxxxxx>
>
>>
>> Cheers,
>>
|