[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v5 1/2] arm/efi: Use dom0less configuration when using EFI boot

  • To: Luca Fancellu <luca.fancellu@xxxxxxx>
  • From: Jan Beulich <jbeulich@xxxxxxxx>
  • Date: Mon, 11 Oct 2021 10:11:22 +0200
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=suse.com; dmarc=pass action=none header.from=suse.com; dkim=pass header.d=suse.com; arc=none
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=hj7UsMq9SrEe2FV3McSS9jWXqS6uB0dcy2UGnm64cuI=; b=O4sbrZjGOqKUwNRY4DYhOfFVOqKpRo4wqG86zr3daFIlnUAdgh6jGCDE/Gfvdl55ghvmIl/Ump1wY8ZgxuJ2NxprimNRhiyTTgxXHs1YU7Phzfo7rkYtPr5V35uN273DNMONPuvFevOHAFz+PWXLr1Ikbw3xjlJ8pFItkRbsuXonbtLwdbaeYWGbImn0bLDrjYTaZTkunVr2CxHuWL+7TowiafnlUW1l6dQ3JprW2KvvpFi1dy8HC+J8jMm4HB/LOY4T+Sk5pOnlHA9d3VCnx50TllVGa532bWRfHtXJ7AsGgT/G9aZz+t01Jl7t0tIBvGLYKy2z7CjyRDzylCUb6g==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=g2yjGgvmYJnxbtwdVg+1DKS72yRO8qci97VjTz4V80O9wVXboVWDYdFeUZdCp1gjt1ScbD5JOelwArWhVDLTlgKZRJu6SGjOS30x0IU3soRV5blFn2e0X97b60ezTVcBYzOTda0l26BFzAD9/YMwRN8Qa77aDcP9ktyrvEGJ8DPrKYxfMKBZWC2NpqHDfGViZqoget5QtA2tNhGR3XUf6IcMFAEe7gq4OEupoJbD1XSPGQknXfxv3rhV4o/PxvPPmWuyi67rVJ3HGXQdp02x4382ZJaFqzE47r/pfmiJP941lcTbHPumUWiNX+V6JIbN+SycOs277BGqpznsiun99A==
  • Authentication-results: lists.xenproject.org; dkim=none (message not signed) header.d=none;lists.xenproject.org; dmarc=none action=none header.from=suse.com;
  • Cc: bertrand.marquis@xxxxxxx, wei.chen@xxxxxxx, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Julien Grall <julien@xxxxxxx>, Volodymyr Babchuk <Volodymyr_Babchuk@xxxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, George Dunlap <george.dunlap@xxxxxxxxxx>, Ian Jackson <iwj@xxxxxxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxxx
  • Delivery-date: Mon, 11 Oct 2021 08:11:54 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On 11.10.2021 10:03, Luca Fancellu wrote:
> 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 "xen,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.
> Introduce a new call efi_check_dt_boot(...) called during EFI boot
> that checks for module to be loaded using device tree.
> Architectures that don't support device tree don't have to
> provide this function.
> Update efi documentation about how to start a dom0less
> setup using UEFI
> Signed-off-by: Luca Fancellu <luca.fancellu@xxxxxxx>
> Reviewed-by: Bertrand Marquis <bertrand.marquis@xxxxxxx>
> Reviewed-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>

Did you get indication that these are fine to retain with ...

> ---
> Changes in v5:
> - Removed unneeded variable initialization
> - Fixed comment
> - Fixed error message for the absence of an initial domain kernel
> - changed efi_arch_check_dt_boot to efi_check_dt_boot and add
> a stub if CONFIG_HAS_DEVICE_TREE is not declared, updated commit
> message about the call introduction in the EFI boot flow.

... all of these changes? Every individual change may be minor enough,
but their sum makes me wonder. If so (or if at least one of the two
gets re-offered)
Acked-by: Jan Beulich <jbeulich@xxxxxxxx>
albeit preferably with ...

> --- a/xen/common/efi/boot.c
> +++ b/xen/common/efi/boot.c
> @@ -166,6 +166,13 @@ static void __init PrintErr(const CHAR16 *s)
>      StdErr->OutputString(StdErr, (CHAR16 *)s );
>  }
> +static inline int __init efi_check_dt_boot(EFI_FILE_HANDLE dir_handle)

... the "inline" here dropped. We don't normally add this outside of
headers, leaving it to the compiler to decide. In headers it's wanted
to avoid "defined by never used" style warnings.




Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.