Xen is unconditionally using certain device tree paths to create DOM0 specific node (for instance /psci, /memory and /hypervisor).
Print a warning message on the console to let the user know if we re-use one of these nodes. Note that the content of most of those is very common and they should have already been skipped via the compatible string or typer string. This warning is here to catch unusual device-tree and compatible string that we may not yet support in Xen. Signed-off-by: Julien Grall <julien.gr...@citrix.com> --- Changes in v2: - Update commit message - Update the warning message --- xen/arch/arm/domain_build.c | 16 ++++++++++++++++ 1 file changed, 16 insertions(+) diff --git a/xen/arch/arm/domain_build.c b/xen/arch/arm/domain_build.c index e8f024f..eb93a3a 100644 --- a/xen/arch/arm/domain_build.c +++ b/xen/arch/arm/domain_build.c @@ -1205,6 +1205,13 @@ static int handle_node(struct domain *d, struct kernel_info *kinfo, DT_MATCH_TIMER, { /* sentinel */ }, }; + static const struct dt_device_match reserved_matches[] __initconst = + { + DT_MATCH_PATH("/psci"), + DT_MATCH_PATH("/memory"), + DT_MATCH_PATH("/hypervisor"), + { /* sentinel */ }, + }; struct dt_device_node *child; int res; const char *name; @@ -1252,6 +1259,15 @@ static int handle_node(struct domain *d, struct kernel_info *kinfo, return 0; } + /* + * Xen is using some path for its own purpose. Warn if a node + * already exists with the same path. + */ + if ( dt_match_node(reserved_matches, node) ) + printk(XENLOG_WARNING + "WARNING: Path %s is reserved, skip the node as we may re-use the path.\n", + path); + res = handle_device(d, node); if ( res) return res; -- 2.1.4 _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel