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

Re: [PATCH v3 6/7] xen: introduce Kconfig ARCH_PAGING_MEMPOOL


  • To: Jan Beulich <jbeulich@xxxxxxxx>
  • From: Luca Fancellu <Luca.Fancellu@xxxxxxx>
  • Date: Wed, 19 Mar 2025 12:18:06 +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=lTjs+LiMA1PvXz9SzeaXguR5ZE40Z+DHXv6zDQ66l3E=; b=CqcfTP+6GlWfGpukE9NrRUBtXtOeH8SDZZ6hB93PkdZiqjFFz6NITSipq1p9dplNHXVAs/uVXAGso4EVEBkfwATR/tw2EpN9AsPGW1NakhyYHL06PB5sHgATlWk+JGiveLDy6FY4dNsjYW85B7mAeCMuETGQOpkI7ysen4KPl5F6gmqsgKxYmCnkYIvP59sX07kcE2L/C+eGjJp8tVqagIDANx2JMwtc90HkxMHv3AjcwrYI8krY+CE9FVWv3NmK0jDGgE63PilQ6X/1a160e3T8r6bKnvTPOQ3oB/jzlPettZ7w3KaNu5RdtsNuNLsVFR4cNf0vAdBmBNjPB2cg9Q==
  • 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=lTjs+LiMA1PvXz9SzeaXguR5ZE40Z+DHXv6zDQ66l3E=; b=FfBtanI/6A32eWLRy0+dy076NDnv6KJHDZJV3ZqX/d40sgZ36lv8BQy0M3sf/WQLNvpnVx+DYMGEsYbnZrQJcoW1T+b5D82WvoMcCmBaAKK4SByAZG89KA9X7U/cwveDisnyB+dyhQl4QmMwYAOTfbR1pxqsqueChVSS1LNtn1gdlKCTGuleUw/cOyanq/4OarW8ViDkfczS35LYlgdUtcyXoaUJ37WZtkRAgDp+eD8UZNHctfeO258/ix3UD4kAEi0a6Ysr4BMh+jicQFYZuPynvy7pIh5mlL5MyGNZh8w4jqsTyFVjNMB7fvZ2oLOjId9pfmBqPayk7w94EYV9JQ==
  • Arc-seal: i=2; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=pass; b=HED6QeFKrm25Bt2KDc7//sLGVP49jutywOBt+Cjj0NW028KrqgrWddIofqBrdtHfvz+sLaA0B6zeFRBfaRYb9eRoSZG/etTOR0u7etoXl3bUXC0u/kfW/yq/iKcVuff4/J+kj8ynhZwCiC9EStNACyR9/h6wk83Lgk/bytKV4nNOFLMLflJ9jbTE9a0A/kU1v3eWZv6KwG6ynZx9te/D9OuOJCEi03OcWRhwfR/+RODeXn9AIptOI1NVZYoJd/3fGdMh+urx3hkuEBFc74nUyRgVHSfXblQAmCyCMA2AeBofqVZAvbxCuiP3hTiq2vN6LetxBmCQ1SX+gixS58MrBA==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=ejAJsd38CbX9ikEODYaoCNINz7exWi7CGOyhhznGpcuDgxqCdt8BkbORDVSM2EyW/8crEUwtsjjljeAWWMGmDng8GtrFROIy7kib3OgVX7YxmP3XfMbU7SG/Np8m3Sltr+v6d19S4WVQYcP55nfZr/NwwsssRPsq/X3sfyvnf+2oDII4oSPa+tYtzOlKPAe0BZIDsHa24PbDpCiO3OBRnEwG2xlYWyq38SGMjUAIt/iNQmD3ib0oNl6g82DRqGwiYFmRgoJuz5DdGGhKX702d73uS2bLN7wFQX3hCDHR2Wphrf4SsSQqtL/W8IBJLCQkyKEZmHqmWbXptZc4cZbzAw==
  • Authentication-results-original: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
  • Cc: Oleksii Kurochko <oleksii.kurochko@xxxxxxxxx>, Penny Zheng <Penny.Zheng@xxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Julien Grall <julien@xxxxxxx>, Bertrand Marquis <Bertrand.Marquis@xxxxxxx>, Michal Orzel <michal.orzel@xxxxxxx>, Volodymyr Babchuk <Volodymyr_Babchuk@xxxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Anthony PERARD <anthony.perard@xxxxxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>, Alistair Francis <alistair.francis@xxxxxxx>, Bob Eshleman <bobbyeshleman@xxxxxxxxx>, Connor Davis <connojdavis@xxxxxxxxx>, Wei Chen <Wei.Chen@xxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>
  • Delivery-date: Wed, 19 Mar 2025 12:18:44 +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: AQHbl3hIuq5Ui4sTKky0PpKNFdMoBrN43e6AgAF5B4CAAAvlgA==
  • Thread-topic: [PATCH v3 6/7] xen: introduce Kconfig ARCH_PAGING_MEMPOOL

Hi Jan,

> On 19 Mar 2025, at 11:35, Jan Beulich <jbeulich@xxxxxxxx> wrote:
> 
> On 18.03.2025 14:05, Oleksii Kurochko wrote:
>> 
>> On 3/17/25 9:07 PM, Luca Fancellu wrote:
>>> From: Penny Zheng<Penny.Zheng@xxxxxxx>
>>> 
>>> ARM MPU system doesn't need to use paging memory pool, as MPU memory
>>> mapping table at most takes only one 4KB page, which is enough to
>>> manage the maximum 255 MPU memory regions, for all EL2 stage 1
>>> translation and EL1 stage 2 translation.
>>> 
>>> Introduce ARCH_PAGING_MEMPOOL Kconfig common symbol, selected for Arm
>>> MMU systems, x86 and RISC-V.
>>> 
>>> Wrap the code inside 'construct_domU' that deal with p2m paging
>>> allocation in a new function 'domain_p2m_set_allocation', protected
>>> by ARCH_PAGING_MEMPOOL, this is done in this way to prevent polluting
>>> the former function with #ifdefs and improve readability
>>> 
>>> Introduce arch_{get,set}_paging_mempool_size stubs for architecture
>>> with !ARCH_PAGING_MEMPOOL.
>>> 
>>> Remove 'struct paging_domain' from Arm 'struct arch_domain' when the
>>> field is not required.
>>> 
>>> Signed-off-by: Penny Zheng<penny.zheng@xxxxxxx>
>>> Signed-off-by: Wei Chen<wei.chen@xxxxxxx>
>>> Signed-off-by: Luca Fancellu<luca.fancellu@xxxxxxx>
>>> ---
>>> v3 changes:
>>>  - Introduced ARCH_PAGING_MEMPOOL instead of HAS_PAGING_MEMPOOL
>>> v2 changes:
>>>  - make Kconfig HAS_PAGING_MEMPOOL common
>>>  - protect also "xen,domain-p2m-mem-mb" reading with HAS_PAGING_MEMPOOL
>>>  - do not define p2m_teardown{_allocation} in this patch
>>>  - change commit message
>>> ---
>>>  xen/arch/arm/Kconfig              |  1 +
>>>  xen/arch/arm/dom0less-build.c     | 74 ++++++++++++++++++++-----------
>>>  xen/arch/arm/include/asm/domain.h |  2 +
>>>  xen/arch/riscv/Kconfig            |  1 +
>>>  xen/arch/x86/Kconfig              |  1 +
>>>  xen/common/Kconfig                |  3 ++
>>>  xen/include/xen/domain.h          | 17 +++++++
>>>  7 files changed, 73 insertions(+), 26 deletions(-)
>> 
>> For RISC-V:
>>  Reviewed-by: Oleksii Kurochko<oleksii.kurochko@xxxxxxxxx>
> 
> Mind me asking then why RISC-V needs this at this point? The stubs surely
> were added to address some build issue, not because they are actively
> meaningful?

sorry I’m not a RISC-V expert, I saw the stub and I thought the architecture 
wanted to have them implemented.

If not, is it possible to let the RISC-V people handle that separately? I’ll be 
off until 31st of March and this patch is
useful for Ayan to introduce a building status for arm32.

Cheers,
Luca

> 
> Jan


 


Rackspace

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