From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=2607:f8b0:400d:c0d::241; helo=mail-qt0-x241.google.com; envelope-from=gsomlo@gmail.com; receiver=edk2-devel@lists.01.org Received: from mail-qt0-x241.google.com (mail-qt0-x241.google.com [IPv6:2607:f8b0:400d:c0d::241]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 1D1AE2258AEFA for ; Fri, 16 Mar 2018 08:23:32 -0700 (PDT) Received: by mail-qt0-x241.google.com with SMTP id g60so11244849qtd.11 for ; Fri, 16 Mar 2018 08:29:58 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=JX/q3ZekhejSBa67yJnaPQrKaMngfHV5cmZRkLfjRDA=; b=i3cxaWKVaTSoAV5J3y5x8KFUKpATBBtZYvr4LHLr2LmQIdkKnv3QCVaRmCYU7yVmSx Djz2WsArkzV4k/SWwMJ5UmpYo/plV2lnpFwAJjs/ahKvBu5j2jhvbrkkV5JY1xiLerX+ q+TVk/YJrB4cMPbdXYOy8BSHXK117sUCm6MQhFGiMBVKlQuw9a8pZ4dj6/9xlsPzKie6 +cCXPmGknOqZN8rUzBmeBWmrzgs9ZO429DWYom0wjqCx9W08HWid4q/K7NFEM95rpSvo wYFuCF2yw9CC1EzIltpDm2FSmMvf9LiYGyVd55qjWz73gjGRfQU/7Y+dS49EguNdLMoE 0gFA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=JX/q3ZekhejSBa67yJnaPQrKaMngfHV5cmZRkLfjRDA=; b=FNj4DGcWd0CZI8tZoFQbHFsk7jAYuWebbf7rkca8zX61y45/AMqjZjuL9SsbUbpD+x /t39kD0TFkVJvtIhYetVf4QE0LQ8xZjr/LQAA0jVlPl3qNPT1MHeX9L2hq5CB+cSRzzW WbmpUBnINg7kMxLj4kLJ2n7P7GGroWrM6RVXB6xbjIwb0rFVttmg4qc7Q48w8t7nfaQL 7SjuRg51a3JQCI0tx3DIEX+txaa+WxlOOBCbY/9lpiKgt5DwnjnD0ig5V3pxOSdusJwQ w7UAFG2SNerhVjdxa9WTjqj/vJKcqJVROQAMQYdWH+AfcF8VDykjJoj+VCdAuIhpsI3n Knnw== X-Gm-Message-State: AElRT7FVNqUOVRTjuUS9e8h/DmzN6V3feDdE5abQCW/JzeCJzGjPNe4M poaFzH5p3Z8x7iOKoVss1MU= X-Google-Smtp-Source: AG47ELsGzdZX2Nm0ZYEcZ8fogiU5ib2THHwseTkiMm8rlACwQFFkrIh8IVV2SJb/chC0gEfwoPjfnA== X-Received: by 10.200.12.136 with SMTP id n8mr3296444qti.167.1521214197463; Fri, 16 Mar 2018 08:29:57 -0700 (PDT) Received: from hedwig.ini.cmu.edu (HEDWIG.INI.CMU.EDU. [128.2.16.51]) by smtp.gmail.com with ESMTPSA id j18sm6015389qtc.46.2018.03.16.08.29.56 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Fri, 16 Mar 2018 08:29:56 -0700 (PDT) Date: Fri, 16 Mar 2018 11:29:54 -0400 From: "Gabriel L. Somlo" To: Laszlo Ersek Cc: edk2-devel-01 , "Richard W.M. Jones" , Ard Biesheuvel , Jordan Justen , Xiang Zheng Message-ID: <20180316152953.GB2263@hedwig.ini.cmu.edu> References: <20180315190258.6580-1-lersek@redhat.com> MIME-Version: 1.0 In-Reply-To: <20180315190258.6580-1-lersek@redhat.com> X-Clacks-Overhead: GNU Terry Pratchett User-Agent: Mutt/1.9.1 (2017-09-22) Subject: Re: [PATCH 0/5] ArmVirtPkg, OvmfPkg: improve firmware duration of direct kernel boot X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 16 Mar 2018 15:23:33 -0000 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Thu, Mar 15, 2018 at 08:02:53PM +0100, Laszlo Ersek wrote: > (Copying Rich, Xiang and Gabriel for testing requests below.) > > Repo: https://github.com/lersek/edk2.git > Branch: kernel_before_bootdevs > > After the recent series "OvmfPkg, ArmVirtQemu: leaner platform BDS > policy for connecting devices", I'm picking up another earlier idea -- a > direct kernel boot does not need devices such as disks and NICs to be > bound by UEFI. > > I tested this series extensively on QEMU, in OVMF (IA32X64) and > ArmVirtQemu (AARCH64), both with and without direct kernel boot. I > compared the logs in all sensible relations within a given architecture. > > Rich, can you please test this on ARM64, with guestfish/libguestfs? > Please attach a good number of disks at once on the command line, and > compare the appliance's boot time between (e.g.) RHEL7's > "/usr/share/AAVMF/AAVMF_CODE.fd" and the following binary (after > decompression): > > https://people.redhat.com/lersek/kernel_before_bootdevs-991e2f2f-64cf-4566-b933-919928e2aa6b/QEMU_EFI.fd.padded.xz > > (That binary corresponds to the branch linked above, cross-built from > x86_64 with "aarch64-linux-gnu-gcc (GCC) 6.1.1 20160621 (Red Hat Cross > 6.1.1-2)", using the following options: > > export GCC5_AARCH64_PREFIX=aarch64-linux-gnu- > build --cmd-len=65536 --hash -t GCC5 -b DEBUG -a AARCH64 \ > -p ArmVirtPkg/ArmVirtQemu.dsc -D DEBUG_PRINT_ERROR_LEVEL=0x80000000 > > and then it was padded with zeroes to 64MB.) > > If you have good results, please respond with your Tested-by (which I'll > apply to patch 1/5, since that's the one that matters for the ARM64 > target). > > Xiang, if you use guestfish (or else direct kernel boot) occasionally, > then similar testing would be very welcome from your side too. > > Gabriel, I'm CC'ing you on patch 4/5, because it affects code that you > had originally written. Can you please regression-test this series with > your usual OVMF environment / guest(s)? With the series applied on top of all my out-of-tree macboot patches, OSX (10.12) boots just as well as before, so no regressions as far as I'm able to tell! whole series: Tested-by: Gabriel Somlo Thanks, --G > Cc: "Gabriel L. Somlo" > Cc: "Richard W.M. Jones" > Cc: Ard Biesheuvel > Cc: Jordan Justen > Cc: Xiang Zheng > > Thanks everyone! > Laszlo > > Laszlo Ersek (5): > ArmVirtPkg/PlatformBootManagerLib: return to "-kernel before boot > devices" > OvmfPkg/PlatformBootManagerLib: wrap overlong lines in "BdsPlatform.c" > OvmfPkg/PlatformBootManagerLib: rejuvenate old-style function comments > OvmfPkg/PlatformBootManagerLib: hoist PciAcpiInitialization() > OvmfPkg/PlatformBootManagerLib: process "-kernel" before boot devices > > ArmVirtPkg/Library/PlatformBootManagerLib/PlatformBm.c | 16 +- > OvmfPkg/Library/PlatformBootManagerLib/BdsPlatform.c | 262 ++++++++++---------- > 2 files changed, 136 insertions(+), 142 deletions(-) > > -- > 2.14.1.3.gb7cf6e02401b >