Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix issues with Xen hypervisor and xenvm board for main branch #82450

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

firscity
Copy link
Contributor

@firscity firscity commented Dec 2, 2024

This PR contains 2 fixes for problems that I faced during build and run of samples for xenvm GICv3 board:

  • xenvm GICv3 board missing dedicated defconfig file and now does not use generic xenvm_defconfig
  • all variations of Xen boards/snippets crashes during boot due to issue introduced by Do not use SYS_INIT in arch/ #75677, event channel initialization can not be called this early (please see the commit message).

During migration to HWMv2 separate specific defconfig file was removed
for GICv3 version of virtual xenvm boards. It worked fine before
commit 0be0d21 ("cmake: modules: extensions: Revert using common
board files") significantly changed build behavior, but did not return
previously removed file. This led to build/runtime issues, when some of
the Kconfig options were not selected.

Return GICv3 specific defconfig to board directory to fix configuration
problems.

Signed-off-by: Dmytro Firsov <[email protected]>
This reverts commit 7c90f1b.

Xen initialization maps enlighten page to Zephyr memory and also
initializes Xen event channels. It is used for communication between
Xen domains and based on interrupt connected to domain virtual GIC.
Moving event channel initialization to arch_kernel_init() make it call
irq_enable() when GIC is not initialized. Since GIC is initialized
on PRE_KERNEL_1 stage, this lead to fatal error during boot.

Revert these changes to make xenvm operable again.

Signed-off-by: Dmytro Firsov <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: ARM64 ARM (64-bit) Architecture area: Xen Platform
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants