From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=45.249.212.35; helo=huawei.com; envelope-from=huangming23@huawei.com; receiver=edk2-devel@lists.01.org Received: from huawei.com (unknown [45.249.212.35]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 66C9C2215BD93 for ; Mon, 29 Jan 2018 00:49:50 -0800 (PST) Received: from DGGEMS404-HUB.china.huawei.com (unknown [172.30.72.58]) by Forcepoint Email with ESMTP id 1817D7E0FC82C; Mon, 29 Jan 2018 16:55:17 +0800 (CST) Received: from [127.0.0.1] (10.61.17.224) by DGGEMS404-HUB.china.huawei.com (10.3.19.204) with Microsoft SMTP Server id 14.3.361.1; Mon, 29 Jan 2018 16:55:08 +0800 To: Ard Biesheuvel References: <1516287703-35516-1-git-send-email-huangming23@huawei.com> <1516287703-35516-14-git-send-email-huangming23@huawei.com> <20180123102320.7rloxdthhq2njbvu@bivouac.eciton.net> <2ced3021-fd0e-7551-c476-4468c915f82e@huawei.com> CC: Leif Lindholm , Ming Huang , linaro-uefi , "edk2-devel@lists.01.org" , Graeme Gregory , , wanghuiqiang , Jason Zhang , Mengfanrong , From: "Huangming (Mark)" Message-ID: <3f9a89b8-7558-f9a3-a262-59860b2e98a9@huawei.com> Date: Mon, 29 Jan 2018 16:55:07 +0800 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.2.0 MIME-Version: 1.0 In-Reply-To: X-Originating-IP: [10.61.17.224] X-CFilter-Loop: Reflected Subject: Re: [PATCH edk2-platforms v1 13/14] Hisilicon/Library: Add OsBootLib 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: Mon, 29 Jan 2018 08:49:51 -0000 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit On 2018/1/27 18:37, Ard Biesheuvel wrote: > On 27 January 2018 at 01:47, Huangming (Mark) wrote: >> >> >> On 2018/1/23 18:23, Leif Lindholm wrote: >>> On Thu, Jan 18, 2018 at 11:01:42PM +0800, Ming Huang wrote: >>>> OsBootLib can create OS option after upgrade firmware. >>> >>> I will respond more strongly that Ard did: >>> >>> I have seen functionality like this implemented in publicly available >>> systems - laptops, desktops. >>> Without exception, they end up in bug reports saying "my system >>> refuses to boot after installation/upgrade". >>> Without exception, they add to existing negative perceptions of UEFI >>> in general in certain market spaces. >>> >>> Presumably this is trying to address a real problem you have faced. >>> Please bring this issue to the table for discussion, so that we can >>> agree on an appropriate way of resolving it. >>> >>> Regardless, this code will not be included in 18.02. >>> >>> / >>> Leif >>> >>> . >>> >> >> The problem is that OS boot option is lost after upgrade firmware. > > Why is that? There is no need to clear the variable store if you > upgrade the executable image. If you fix this issue, you don't need > this patch. > Ok, retaining the variable store can solve the problem also. But retaining the variable store have some issues, like,if the struct stored in variable is different between new firmware and old firmware, this situation may cause a problem. If OsBootLib is not needed for community, It will use for internal project in hisilicon. Thanks. >> It is inconvenient for using. OsBootLib can help this. >> >> OsBootLib retain the options installed by OS, and create OS boot option >> after upgrade firmware if grub file is existed in EFI partition and in mUefiOsBootFiles, >> and delete redundant options in the same GPT. >> > > . > -- Best Regards, Ming