From nobody Fri Nov 29 18:41:07 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) client-ip=192.237.175.120; envelope-from=xen-devel-bounces@lists.xenproject.org; helo=lists.xenproject.org; Authentication-Results: mx.zohomail.com; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; dmarc=fail(p=none dis=none) header.from=arm.com Return-Path: Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) by mx.zohomail.com with SMTPS id 16323200608501004.2828564692558; Wed, 22 Sep 2021 07:14:20 -0700 (PDT) Received: from list by lists.xenproject.org with outflank-mailman.192812.343475 (Exim 4.92) (envelope-from ) id 1mT30h-00048I-9j; Wed, 22 Sep 2021 14:14:07 +0000 Received: by outflank-mailman (output) from mailman id 192812.343475; Wed, 22 Sep 2021 14:14:07 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1mT30h-000483-6U; Wed, 22 Sep 2021 14:14:07 +0000 Received: by outflank-mailman (input) for mailman id 192812; Wed, 22 Sep 2021 14:14:05 +0000 Received: from us1-rack-iad1.inumbo.com ([172.99.69.81]) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1mT30f-0003YB-H3 for xen-devel@lists.xenproject.org; Wed, 22 Sep 2021 14:14:05 +0000 Received: from foss.arm.com (unknown [217.140.110.172]) by us1-rack-iad1.inumbo.com (Halon) with ESMTP id 463e767a-36bb-42b2-89f6-815e4eac9028; Wed, 22 Sep 2021 14:14:02 +0000 (UTC) Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 20DE3113E; Wed, 22 Sep 2021 07:14:02 -0700 (PDT) Received: from e125770.cambridge.arm.com (e125770.cambridge.arm.com [10.1.197.16]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 5AFE93F59C; Wed, 22 Sep 2021 07:14:00 -0700 (PDT) X-Outflank-Mailman: Message body and most headers restored to incoming version X-BeenThere: xen-devel@lists.xenproject.org List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xenproject.org Precedence: list Sender: "Xen-devel" X-Inumbo-ID: 463e767a-36bb-42b2-89f6-815e4eac9028 From: Luca Fancellu To: xen-devel@lists.xenproject.org Cc: bertrand.marquis@arm.com, wei.chen@arm.com, Stefano Stabellini , Julien Grall , Volodymyr Babchuk , Andrew Cooper , George Dunlap , Ian Jackson , Jan Beulich , Wei Liu , =?UTF-8?q?Roger=20Pau=20Monn=C3=A9?= Subject: [PATCH v2 2/2] arm/efi: Use dom0less configuration when using EFI boot Date: Wed, 22 Sep 2021 15:13:41 +0100 Message-Id: <20210922141341.42288-3-luca.fancellu@arm.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20210922141341.42288-1-luca.fancellu@arm.com> References: <20210922141341.42288-1-luca.fancellu@arm.com> X-ZM-MESSAGEID: 1632320061741100001 Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" This patch introduces the support for dom0less configuration when using UEFI boot on ARM, it permits the EFI boot to continue if no dom0 kernel is specified but at least one domU is found. Introduce the new property "uefi,binary" for device tree boot module nodes that are subnode of "xen,domain" compatible nodes. The property holds a string containing the file name of the binary that shall be loaded by the uefi loader from the filesystem. Update efi documentation about how to start a dom0less setup using UEFI Signed-off-by: Luca Fancellu --- Changes in v2: - remove array of struct file - fixed some int types - Made the code use filesystem even when configuration file is skipped. - add documentation of uefi,binary in booting.txt - add documentation on how to boot all configuration for Xen using UEFI in efi.pandoc --- docs/misc/arm/device-tree/booting.txt | 21 +++ docs/misc/efi.pandoc | 203 ++++++++++++++++++++ xen/arch/arm/efi/efi-boot.h | 257 +++++++++++++++++++++++++- xen/arch/x86/efi/efi-boot.h | 6 + xen/common/efi/boot.c | 36 ++-- 5 files changed, 508 insertions(+), 15 deletions(-) diff --git a/docs/misc/arm/device-tree/booting.txt b/docs/misc/arm/device-t= ree/booting.txt index 44cd9e1a9a..bc0f8913db 100644 --- a/docs/misc/arm/device-tree/booting.txt +++ b/docs/misc/arm/device-tree/booting.txt @@ -182,6 +182,13 @@ The kernel sub-node has the following properties: =20 Command line parameters for the guest kernel. =20 +- efi,binary (UEFI boot only) + + Specifies the file name to be loaded by the UEFI boot for this module.= If + this is specified, there is no need to specify the reg property becaus= e it + will be created by the UEFI stub on boot. + This option is needed only when UEFI boot is used. + The ramdisk sub-node has the following properties: =20 - compatible @@ -193,6 +200,13 @@ The ramdisk sub-node has the following properties: Specifies the physical address of the ramdisk in RAM and its length. =20 +- efi,binary (UEFI boot only) + + Specifies the file name to be loaded by the UEFI boot for this module.= If + this is specified, there is no need to specify the reg property becaus= e it + will be created by the UEFI stub on boot. + This option is needed only when UEFI boot is used. + =20 Example =3D=3D=3D=3D=3D=3D=3D @@ -257,6 +271,13 @@ The dtb sub-node should have the following properties: Specifies the physical address of the device tree binary fragment RAM and its length. =20 +- efi,binary (UEFI boot only) + + Specifies the file name to be loaded by the UEFI boot for this module.= If + this is specified, there is no need to specify the reg property becaus= e it + will be created by the UEFI stub on boot. + This option is needed only when UEFI boot is used. + As an example: =20 module@0xc000000 { diff --git a/docs/misc/efi.pandoc b/docs/misc/efi.pandoc index e289c5e7ba..698196e129 100644 --- a/docs/misc/efi.pandoc +++ b/docs/misc/efi.pandoc @@ -167,3 +167,206 @@ sbsign \ --output xen.signed.efi \ xen.unified.efi ``` + +## UEFI boot and dom0less on ARM + +Dom0less feature is supported by ARM and it is possible to use it when Xen= is +started as an EFI application. +The way to specify the domU domains is by Device Tree as specified in the +[dom0less](dom0less.html) documentation page under the "Device Tree +configuration" section, but instead of declaring the reg property in the b= oot +module, the user must specify the "uefi,binary" property containing the na= me +of the binary file that has to be loaded in memory. +The UEFI stub will load the binary in memory and it will add the reg prope= rty +accordingly. + +An example here: + +domU1 { + #address-cells =3D <1>; + #size-cells =3D <1>; + compatible =3D "xen,domain"; + memory =3D <0 0x20000>; + cpus =3D <1>; + vpl011; + + module@1 { + compatible =3D "multiboot,kernel", "multiboot,module"; + uefi,binary =3D "vmlinuz-3.0.31-0.4-xen"; + bootargs =3D "console=3DttyAMA0"; + }; + module@2 { + compatible =3D "multiboot,ramdisk", "multiboot,module"; + uefi,binary =3D "initrd-3.0.31-0.4-xen"; + }; + module@3 { + compatible =3D "multiboot,ramdisk", "multiboot,module"; + uefi,binary =3D "passthrough.dtb"; + }; +}; + +## How to boot different Xen setup using UEFI + +Here the supported user cases for Xen when UEFI boot is used: + + - Boot Xen and Dom0 (minimum required) + - Boot Xen and DomU(s) (true dom0less, only on ARM) + - Boot Xen, Dom0 and DomU(s) (only on ARM) + +### Boot Xen and Dom0 + +This configuration can be started using the Xen configuration file in the +example above. + +### Boot Xen and DomU(s) + +This configuration needs the domU domain(s) specified in the /chosen node, +examples of how to do that are provided by the documentation about dom0less +and the example above shows how to use the "uefi,binary" property to use t= he +UEFI stub for module loading. +Providing the multiboot modules in the device tree, make Xen skip its UEFI +configuration file, if it is needed for some reason, specify the "uefi,cfg= -load" +property in the /chosen node. + +Example 1 of how to boot a true dom0less configuration: + +Xen configuration file: skipped. + +Device tree: + +``` +chosen { + #size-cells =3D <0x1>; + #address-cells =3D <0x1>; + xen,xen-bootargs =3D "" + + domU1 { + #size-cells =3D <0x1>; + #address-cells =3D <0x1>; + compatible =3D "xen,domain"; + cpus =3D <0x1>; + memory =3D <0x0 0xc0000>; + vpl011; + + module@1 { + compatible =3D "multiboot,kernel", "multiboot,module"; + uefi,binary =3D "Image-domu1.bin"; + bootargs =3D "console=3DttyAMA0 root=3D/dev/ram0 rw"; + }; + }; + domU2 { + #size-cells =3D <0x1>; + #address-cells =3D <0x1>; + compatible =3D "xen,domain"; + cpus =3D <0x1>; + memory =3D <0x0 0x100000>; + vpl011; + + module@2 { + compatible =3D "multiboot,kernel", "multiboot,module"; + uefi,binary =3D "Image-domu2.bin"; + bootargs =3D "console=3DttyAMA0 root=3D/dev/ram0 rw"; + }; + }; +}; +``` + +Example 2 of how to boot a true dom0less configuration: + +Xen configuration file: + +``` +[global] +default=3Dxen + +[xen] +options=3D +dtb=3D +``` + +Device tree: + +``` +chosen { + #size-cells =3D <0x1>; + #address-cells =3D <0x1>; + uefi,cfg-load; + + domU1 { + #size-cells =3D <0x1>; + #address-cells =3D <0x1>; + compatible =3D "xen,domain"; + cpus =3D <0x1>; + memory =3D <0x0 0xc0000>; + vpl011; + + module@1 { + compatible =3D "multiboot,kernel", "multiboot,module"; + uefi,binary =3D "Image-domu1.bin"; + bootargs =3D "console=3DttyAMA0 root=3D/dev/ram0 rw"; + }; + }; + domU2 { + #size-cells =3D <0x1>; + #address-cells =3D <0x1>; + compatible =3D "xen,domain"; + cpus =3D <0x1>; + memory =3D <0x0 0x100000>; + vpl011; + + module@2 { + compatible =3D "multiboot,kernel", "multiboot,module"; + uefi,binary =3D "Image-domu2.bin"; + bootargs =3D "console=3DttyAMA0 root=3D/dev/ram0 rw"; + }; + }; +}; +``` + +### Boot Xen, Dom0 and DomU(s) + +This configuration is a mix of the two configuration above, to boot this o= ne +the configuration file must be processed so the /chosen node must have the +"uefi,cfg-load" property. + +Here an example: + +Xen configuration file: + +``` +[global] +default=3Dxen + +[xen] +options=3D +kernel=3Dvmlinuz-3.0.31-0.4-xen [domain 0 command line options] +ramdisk=3Dinitrd-3.0.31-0.4-xen +dtb=3D +``` + +Device tree: + +``` +chosen { + #size-cells =3D <0x1>; + #address-cells =3D <0x1>; + uefi,cfg-load; + + domU1 { + #size-cells =3D <0x1>; + #address-cells =3D <0x1>; + compatible =3D "xen,domain"; + cpus =3D <0x1>; + memory =3D <0x0 0xc0000>; + vpl011; + + module@1 { + compatible =3D "multiboot,kernel", "multiboot,module"; + uefi,binary =3D "Image-domu1.bin"; + bootargs =3D "console=3DttyAMA0 root=3D/dev/ram0 rw"; + }; + }; +}; +``` + + diff --git a/xen/arch/arm/efi/efi-boot.h b/xen/arch/arm/efi/efi-boot.h index 8ceeba4ad1..e2b007ece0 100644 --- a/xen/arch/arm/efi/efi-boot.h +++ b/xen/arch/arm/efi/efi-boot.h @@ -8,9 +8,43 @@ #include #include =20 +typedef struct { + char *name; + unsigned int name_len; + EFI_PHYSICAL_ADDRESS addr; + UINTN size; +} dom0less_module_name; + +/* + * Binaries will be translated into bootmodules, the maximum number for th= em is + * MAX_MODULES where we should remove a unit for Xen and one for Xen DTB + */ +#define MAX_DOM0LESS_MODULES (MAX_MODULES - 2) +static struct file __initdata dom0less_file; +static dom0less_module_name __initdata dom0less_modules[MAX_DOM0LESS_MODUL= ES]; +static unsigned int __initdata dom0less_modules_available =3D + MAX_DOM0LESS_MODULES; +static unsigned int __initdata dom0less_modules_idx; + +#define ERROR_DOM0LESS_FILE_NOT_FOUND (-1) + void noreturn efi_xen_start(void *fdt_ptr, uint32_t fdt_size); void __flush_dcache_area(const void *vaddr, unsigned long size); =20 +static int get_dom0less_file_index(const char *name, unsigned int name_len= ); +static unsigned int allocate_dom0less_file(EFI_FILE_HANDLE dir_handle, + const char *name, + unsigned int name_len); +static void handle_dom0less_module_node(EFI_FILE_HANDLE dir_handle, + int module_node_offset, + int reg_addr_cells, + int reg_size_cells); +static void handle_dom0less_domain_node(EFI_FILE_HANDLE dir_handle, + int domain_node, + int addr_cells, + int size_cells); +static bool efi_arch_check_dom0less_boot(EFI_FILE_HANDLE dir_handle); + #define DEVICE_TREE_GUID \ {0xb1b621d5, 0xf19c, 0x41a5, {0x83, 0x0b, 0xd9, 0x15, 0x2c, 0x69, 0xaa, 0x= e0}} =20 @@ -552,8 +586,218 @@ static void __init efi_arch_handle_module(const struc= t file *file, kernel.size) < 0 ) blexit(L"Unable to set reg property."); } - else + else if ( file !=3D &dom0less_file ) + /* + * If file is not a dom0 module file and it's not a domU module, + * stop here. + */ blexit(L"Unknown module type"); + + /* + * dom0less_modules_available is decremented here because for each dom0 + * file added, there will be an additional bootmodule, so the number + * of dom0less module files will be decremented because there is + * a maximum amount of bootmodules that can be loaded. + */ + dom0less_modules_available--; +} + +/* + * This function checks for a binary previously loaded with a give name, it + * returns the index of the file in the dom0less_files array or a negative + * number if no file with that name is found. + */ +static int __init get_dom0less_file_index(const char *name, + unsigned int name_len) +{ + unsigned int i; + int ret =3D ERROR_DOM0LESS_FILE_NOT_FOUND; + + for (i =3D 0; i < dom0less_modules_idx; i++) + { + dom0less_module_name *mod =3D &dom0less_modules[i]; + if ( (mod->name_len =3D=3D name_len) && + (strncmp(mod->name, name, name_len) =3D=3D 0) ) + { + ret =3D i; + break; + } + } + return ret; +} + +/* + * This function allocates a binary and keeps track of its name, it + * returns the index of the file in the dom0less_files array. + */ +static unsigned int __init allocate_dom0less_file(EFI_FILE_HANDLE dir_hand= le, + const char *name, + unsigned int name_len) +{ + dom0less_module_name* file_name; + union string module_name; + unsigned int ret_idx; + + /* + * Check if there is any space left for a domU module, the variable + * dom0less_modules_available is updated each time we use read_file(..= .) + * successfully. + */ + if ( !dom0less_modules_available ) + blexit(L"No space left for domU modules"); + + module_name.s =3D (char*) name; + ret_idx =3D dom0less_modules_idx; + + /* Save at this index the name of this binary */ + file_name =3D &dom0less_modules[ret_idx]; + + if ( efi_bs->AllocatePool(EfiLoaderData, (name_len + 1) * sizeof(char), + (void**)&file_name->name) !=3D EFI_SUCCESS ) + blexit(L"Error allocating memory for dom0less binary name"); + + /* Save name and length of the binary in the data structure */ + strlcpy(file_name->name, name, name_len); + file_name->name_len =3D name_len; + + /* Load the binary in memory */ + read_file(dir_handle, s2w(&module_name), &dom0less_file, NULL); + + /* Save address and size */ + file_name->addr =3D dom0less_file.addr; + file_name->size =3D dom0less_file.size; + + /* s2w(...) allocates some memory, free it */ + efi_bs->FreePool(module_name.w); + + dom0less_modules_idx++; + + return ret_idx; +} + +/* + * This function checks for the presence of the uefi,binary property in the + * module, if found it loads the binary as dom0less module and sets the ri= ght + * address for the reg property into the module DT node. + */ +static void __init handle_dom0less_module_node(EFI_FILE_HANDLE dir_handle, + int module_node_offset, + int reg_addr_cells, + int reg_size_cells) +{ + const void *uefi_name_prop; + char mod_string[24]; /* Placeholder for module@ + a 64-bit number + \0= */ + int uefi_name_len, file_idx; + dom0less_module_name *file; + + /* Read uefi,binary property to get the file name. */ + uefi_name_prop =3D fdt_getprop(fdt, module_node_offset, "uefi,binary", + &uefi_name_len); + + if ( !uefi_name_prop ) + /* Property not found */ + return; + + file_idx =3D get_dom0less_file_index(uefi_name_prop, uefi_name_len); + if (file_idx < 0) + file_idx =3D allocate_dom0less_file(dir_handle, uefi_name_prop, + uefi_name_len); + + file =3D &dom0less_modules[file_idx]; + + snprintf(mod_string, sizeof(mod_string), "module@%"PRIx64, file->addr); + + /* Rename the module to be module@{address} */ + if ( fdt_set_name(fdt, module_node_offset, mod_string) < 0 ) + blexit(L"Unable to add domU ramdisk FDT node."); + + if ( fdt_set_reg(fdt, module_node_offset, reg_addr_cells, reg_size_cel= ls, + file->addr, file->size) < 0 ) + blexit(L"Unable to set reg property."); +} + +/* + * This function checks for boot modules under the domU guest domain node + * in the DT. + */ +static void __init handle_dom0less_domain_node(EFI_FILE_HANDLE dir_handle, + int domain_node, + int addr_cells, + int size_cells) +{ + int module_node; + /* + * Check for nodes compatible with multiboot,{kernel,ramdisk,device-tr= ee} + * inside this node + */ + for ( module_node =3D fdt_first_subnode(fdt, domain_node); + module_node > 0; + module_node =3D fdt_next_subnode(fdt, module_node) ) + if ( (fdt_node_check_compatible(fdt, module_node, + "multiboot,kernel") =3D=3D 0) || + (fdt_node_check_compatible(fdt, module_node, + "multiboot,ramdisk") =3D=3D 0) || + (fdt_node_check_compatible(fdt, module_node, + "multiboot,device-tree") =3D=3D 0)= ) + /* The compatible is one of the strings above, check the modul= e */ + handle_dom0less_module_node(dir_handle, module_node, addr_cell= s, + size_cells); +} + +/* + * This function checks for xen domain nodes under the /chosen node for po= ssible + * domU guests to be loaded. + */ +static bool __init efi_arch_check_dom0less_boot(EFI_FILE_HANDLE dir_handle) +{ + int chosen; + int addr_len, size_len; + unsigned int i =3D 0; + + /* Check for the chosen node in the current DTB */ + chosen =3D setup_chosen_node(fdt, &addr_len, &size_len); + if ( chosen < 0 ) + blexit(L"Unable to setup chosen node"); + + /* Check for nodes compatible with xen,domain under the chosen node */ + for ( int node =3D fdt_first_subnode(fdt, chosen); + node > 0; + node =3D fdt_next_subnode(fdt, node) ) + { + int addr_cells, size_cells, len; + const struct fdt_property *prop; + + if ( fdt_node_check_compatible(fdt, node, "xen,domain") !=3D 0 ) + continue; + + /* Get or set #address-cells and #size-cells */ + prop =3D fdt_get_property(fdt, node, "#address-cells", &len); + if ( !prop ) + blexit(L"#address-cells not found in domain node."); + + addr_cells =3D fdt32_to_cpu(*((uint32_t *)prop->data)); + + prop =3D fdt_get_property(fdt, node, "#size-cells", &len); + if ( !prop ) + blexit(L"#size-cells not found in domain node."); + + size_cells =3D fdt32_to_cpu(*((uint32_t *)prop->data)); + + /* Found a node with compatible xen,domain; handle this node. */ + handle_dom0less_domain_node(dir_handle, node, addr_cells, size_cel= ls); + } + + /* Free dom0less file names if any */ + for ( ; i < dom0less_modules_idx; i++ ) + { + /* Free dom0less binary names */ + efi_bs->FreePool(dom0less_modules[i].name); + } + + if ( dom0less_modules_idx > 0 ) + return true; + + return false; } =20 static void __init efi_arch_cpu(void) @@ -562,8 +806,19 @@ static void __init efi_arch_cpu(void) =20 static void __init efi_arch_blexit(void) { + unsigned int i =3D 0; + if ( dtbfile.need_to_free ) efi_bs->FreePages(dtbfile.addr, PFN_UP(dtbfile.size)); + /* Free dom0less files if any */ + for ( ; i < dom0less_modules_idx; i++ ) + { + /* Free dom0less binary names */ + efi_bs->FreePool(dom0less_modules[i].name); + /* Free dom0less binaries */ + efi_bs->FreePages(dom0less_modules[i].addr, + PFN_UP(dom0less_modules[i].size)); + } if ( memmap ) efi_bs->FreePool(memmap); } diff --git a/xen/arch/x86/efi/efi-boot.h b/xen/arch/x86/efi/efi-boot.h index 9b0cc29aae..950fdf16b7 100644 --- a/xen/arch/x86/efi/efi-boot.h +++ b/xen/arch/x86/efi/efi-boot.h @@ -678,6 +678,12 @@ static void __init efi_arch_handle_module(const struct= file *file, efi_bs->FreePool(ptr); } =20 +static bool __init efi_arch_check_dom0less_boot(EFI_FILE_HANDLE dir_handle) +{ + /* x86 doesn't support dom0less */ + return false; +} + static void __init efi_arch_cpu(void) { uint32_t eax =3D cpuid_eax(0x80000000); diff --git a/xen/common/efi/boot.c b/xen/common/efi/boot.c index 758f9d74d2..7d8734199e 100644 --- a/xen/common/efi/boot.c +++ b/xen/common/efi/boot.c @@ -1127,15 +1127,16 @@ efi_start(EFI_HANDLE ImageHandle, EFI_SYSTEM_TABLE = *SystemTable) static EFI_GUID __initdata shim_lock_guid =3D SHIM_LOCK_PROTOCOL_GUID; EFI_LOADED_IMAGE *loaded_image; EFI_STATUS status; - unsigned int i, argc; - CHAR16 **argv, *file_name, *cfg_file_name =3D NULL, *options =3D NULL; + unsigned int i, argc =3D 0; + CHAR16 **argv, *file_name =3D NULL, *cfg_file_name =3D NULL, *options = =3D NULL; UINTN gop_mode =3D ~0; EFI_SHIM_LOCK_PROTOCOL *shim_lock; EFI_GRAPHICS_OUTPUT_PROTOCOL *gop =3D NULL; union string section =3D { NULL }, name; bool base_video =3D false; - const char *option_str; + const char *option_str =3D NULL; bool use_cfg_file; + EFI_FILE_HANDLE dir_handle; =20 __set_bit(EFI_BOOT, &efi_flags); __set_bit(EFI_LOADER, &efi_flags); @@ -1216,9 +1217,11 @@ efi_start(EFI_HANDLE ImageHandle, EFI_SYSTEM_TABLE *= SystemTable) =20 efi_arch_relocate_image(0); =20 + /* Get the file system interface. */ + dir_handle =3D get_parent_handle(loaded_image, &file_name); + if ( use_cfg_file ) { - EFI_FILE_HANDLE dir_handle; UINTN depth, cols, rows, size; =20 size =3D cols =3D rows =3D depth =3D 0; @@ -1229,9 +1232,6 @@ efi_start(EFI_HANDLE ImageHandle, EFI_SYSTEM_TABLE *S= ystemTable) =20 gop =3D efi_get_gop(); =20 - /* Get the file system interface. */ - dir_handle =3D get_parent_handle(loaded_image, &file_name); - /* Read and parse the config file. */ if ( read_section(loaded_image, L"config", &cfg, NULL) ) PrintStr(L"Using builtin config file\r\n"); @@ -1285,14 +1285,13 @@ efi_start(EFI_HANDLE ImageHandle, EFI_SYSTEM_TABLE = *SystemTable) efi_bs->FreePool(name.w); } =20 - if ( !name.s ) - blexit(L"No Dom0 kernel image specified."); - efi_arch_cfg_file_early(loaded_image, dir_handle, section.s); =20 - option_str =3D split_string(name.s); + if ( name.s ) + option_str =3D split_string(name.s); =20 - if ( !read_section(loaded_image, L"kernel", &kernel, option_str) ) + if ( !read_section(loaded_image, L"kernel", &kernel, option_str) && + name.s ) { read_file(dir_handle, s2w(&name), &kernel, option_str); efi_bs->FreePool(name.w); @@ -1361,12 +1360,21 @@ efi_start(EFI_HANDLE ImageHandle, EFI_SYSTEM_TABLE = *SystemTable) efi_bs->FreePages(cfg.addr, PFN_UP(cfg.size)); cfg.addr =3D 0; =20 - dir_handle->Close(dir_handle); - if ( gop && !base_video ) gop_mode =3D efi_find_gop_mode(gop, cols, rows, depth); } =20 + /* + * Check if a proper configuration is provided to start Xen: + * - Dom0 specified (minimum required) + * - Dom0 and DomU(s) specified + * - DomU(s) specified + */ + if ( !efi_arch_check_dom0less_boot(dir_handle) && !kernel.addr ) + blexit(L"No Dom0 kernel image specified."); + + dir_handle->Close(dir_handle); + efi_arch_edd(); =20 /* XXX Collect EDID info. */ --=20 2.17.1