Hi Jan,

On 23/09/2024 11:23, Jan Beulich wrote:
On 18.09.2024 19:51, Ayan Kumar Halder wrote:
Secondary cpus initialization is not yet supported. Thus, we print an
appropriate message and put the secondary cpus in WFE state.

Signed-off-by: Ayan Kumar Halder <[email protected]>
---
Changes from :-

v1 - 1. NR_CPUS is defined as 1 for MPU
Not quite, what you do is ...

--- a/xen/arch/Kconfig
+++ b/xen/arch/Kconfig
@@ -11,6 +11,7 @@ config NR_CPUS
        default "8" if ARM && RCAR3
        default "4" if ARM && QEMU
        default "4" if ARM && MPSOC
+       default "1" if ARM && MPU
        default "128" if ARM
        help
          Controls the build-time size of various arrays and bitmaps
... merely set the default. I wonder how useful that is, the more that
- as per the description - this is temporary only anyway.

Yes, I can enforce a build time check like this.

--- a/xen/arch/arm/setup.c
+++ b/xen/arch/arm/setup.c
@@ -295,6 +295,12 @@ void asmlinkage __init start_xen(unsigned long fdt_paddr)
     struct domain *d;
     int rc, i;

+    /*
+     * Currently MPU does not support SMP.
+     */
+#ifdef CONFIG_MPU
+    BUILD_BUG_ON(NR_CPUS > 1);
+#endif
     dcache_line_bytes = read_dcache_line_bytes();

Does this look ok ?

- Ayan


Reply via email to