From: Stefano Stabellini Date: Tue, 31 Jul 2018 23:27:49 +0000 (-0700) Subject: xen/arm: move evtchn_allocate call out of make_hypervisor_node X-Git-Tag: 4.12.0-rc1~778 X-Git-Url: http://xenbits.xensource.com/gitweb?a=commitdiff_plain;h=9d61c62188cfc0cfaf324e19a5a941090d431b4f;p=people%2Fdwmw2%2Fxen.git xen/arm: move evtchn_allocate call out of make_hypervisor_node In the case of domUs, evtchn_irq is allocated by arch_domain_create and set to GUEST_EVTCHN_PPI. To make make_hypervisor_node more reusable, move the call to evtchn_allocate out of make_hypervisor_node, to the dom0 specific caller (handle_node). Signed-off-by: Stefano Stabellini Reviewed-by: Julien Grall --- diff --git a/xen/arch/arm/domain_build.c b/xen/arch/arm/domain_build.c index 0177492b1c..2ba0e34868 100644 --- a/xen/arch/arm/domain_build.c +++ b/xen/arch/arm/domain_build.c @@ -619,11 +619,7 @@ static int __init make_hypervisor_node(struct domain *d, if ( res ) return res; - /* - * It is safe to allocate the event channel here because all the - * PPIs used by the hardware domain have been registered. - */ - evtchn_allocate(d); + BUG_ON(d->arch.evtchn_irq == 0); /* * Interrupt event channel upcall: @@ -1293,6 +1289,12 @@ static int __init handle_node(struct domain *d, struct kernel_info *kinfo, int addrcells = dt_child_n_addr_cells(node); int sizecells = dt_child_n_size_cells(node); + /* + * It is safe to allocate the event channel here because all the + * PPIs used by the hardware domain have been registered. + */ + evtchn_allocate(d); + /* * The hypervisor node should always be created after all nodes * from the host DT have been parsed.