While preparing the CPU-config changes for upstream, I first of all added checks regarding CPU usage to our config checker. This confirmed what I saw in manual review before: a lack of consistency in the core assignment to permit running linux and inmate demo in parallel (on >2 cores systems).
The more I ran the checker on older configs, the more issues I found. Many are resolved - but not all. This adds a new script to scan all compiled configs, at least rool cells, inmate and linux demos, and run the checker on them. So, you can check what is still broken: # scripts/check-configs -q ... Checking imx8dxl.cell imx8dxl-inmate-demo.cell... FAILED ... Checking imx8qm.cell imx8qm-inmate-demo.cell imx8qm-linux-demo.cell... FAILED ... Checking k3-j7200-evm.cell k3-j7200-evm-inmate-demo.cell... FAILED Checking k3-j7200-evm.cell k3-j7200-evm-linux-demo.cell... FAILED Checking k3-j721e-evm.cell k3-j721e-evm-inmate-demo.cell... FAILED Checking k3-j721e-evm.cell k3-j721e-evm-linux-demo.cell... FAILED Please check, some issues look fatal (mapping GICD/R...). Jan CC: Alice Guo <[email protected]> CC: Jiafei Pan <[email protected]> CC: Lad Prabhakar <[email protected]> CC: Matt Ranostay <[email protected]> CC: Peng Fan <[email protected]> CC: Ralf Ramsauer <[email protected]> Jan Kiszka (14): pyjailhouse: config_parser: Build CPU set from cell configuration tools: jailhouse-config-check: Add CPU set checks scripts: Add config checker script configs: arm: dts: Silence style warnings configs: arm: jetson-tk1: Move inmate demo to core 1 configs: arm: jetson-tk1: Avoid memory region overlap configs: arm64: Consistently move inmate demo to 2 core configs: arm64: imx8dxl: Mark 32-bit demo also as 64-bit config configs: arm: emtrion-rzg1{h,m}: Fix config checker findings configs: arm64: jetson-tx{1,2}: Fix config checker findings configs: arm64: ls1046a: Resolve overlap between two RAM regions configs: arm64: ultra96/zcu102: Carve out SMMU from MMIO region configs: arm64: k3-am654: Rename inmate demo ci: Update to actions/checkout@v3 and actions/cache@v3 .github/workflows/main.yaml | 4 +- configs/arm/dts/inmate-bananapi.dts | 6 +- configs/arm/dts/inmate-orangepi0.dts | 4 +- configs/arm/emtrion-rzg1h.c | 42 +++------ configs/arm/emtrion-rzg1m.c | 21 ++--- configs/arm/jetson-tk1-inmate-demo.c | 2 +- configs/arm/jetson-tk1.c | 2 +- configs/arm64/dts/inmate-k3-am625-sk.dts | 7 -- configs/arm64/imx8dxl-inmate-demo.c | 3 +- configs/arm64/imx8mm-inmate-demo.c | 2 +- configs/arm64/imx8mn-inmate-demo.c | 2 +- configs/arm64/imx8mp-inmate-demo.c | 2 +- configs/arm64/imx8mq-inmate-demo.c | 2 +- configs/arm64/imx8qm-inmate-demo.c | 2 +- configs/arm64/jetson-tx1-inmate-demo.c | 2 +- configs/arm64/jetson-tx1.c | 2 +- configs/arm64/jetson-tx2.c | 11 +-- configs/arm64/k3-am625-sk-inmate-demo.c | 2 +- configs/arm64/k3-am625-sk-linux-demo.c | 2 +- ...mate-demo.c => k3-am654-idk-inmate-demo.c} | 2 +- configs/arm64/ls1046a-rdb.c | 2 +- .../arm64/miriac-sbc-ls1046a-inmate-demo.c | 2 +- configs/arm64/renesas-r8a774a1-inmate-demo.c | 2 +- configs/arm64/ultra96.c | 13 ++- configs/arm64/zynqmp-zcu102-inmate-demo.c | 2 +- configs/arm64/zynqmp-zcu102.c | 13 ++- pyjailhouse/config_parser.py | 12 ++- scripts/check-configs | 90 +++++++++++++++++++ tools/jailhouse-config-check | 26 ++++++ 29 files changed, 193 insertions(+), 91 deletions(-) rename configs/arm64/{k3-am654-inmate-demo.c => k3-am654-idk-inmate-demo.c} (99%) create mode 100755 scripts/check-configs -- 2.35.3 -- You received this message because you are subscribed to the Google Groups "Jailhouse" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To view this discussion on the web visit https://groups.google.com/d/msgid/jailhouse-dev/cover.1674415741.git.jan.kiszka%40siemens.com.
