From nobody Wed May 8 16:13:16 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) client-ip=66.175.222.108; envelope-from=bounce+27952+93721+1787277+3901457@groups.io; helo=mail02.groups.io; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce+27952+93721+1787277+3901457@groups.io; arc=fail (BodyHash is different from the expected one) Received: from mail02.groups.io (mail02.groups.io [66.175.222.108]) by mx.zohomail.com with SMTPS id 1663092220679978.9958016534915; Tue, 13 Sep 2022 11:03:40 -0700 (PDT) Return-Path: X-Received: by 127.0.0.2 with SMTP id Z1k3YY1788612xAl22zSXbse; Tue, 13 Sep 2022 11:03:40 -0700 X-Received: from NAM10-BN7-obe.outbound.protection.outlook.com (NAM10-BN7-obe.outbound.protection.outlook.com [40.107.92.86]) by mx.groups.io with SMTP id smtpd.web08.2688.1663061770230252806 for ; Tue, 13 Sep 2022 02:36:11 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=dDBc43P7LvLEswWPQKuXfZ7ISFfQoTNgG+6185qxP1DEyfhVUNNBndIyhBgxjWkPaAlEmewpOlyQP5x/28z+YUTNfeikyQi1c6HQcHRohGD42gx8z0eyRWvu9xbcilIrP2eBQZGlsRr2ZhjusVFrIEsX3t/ZM+iWF4OgJEvH0F6aI+obIVvccHx/43S2JGMpjDVriNc5mro8dZU5HmDcrbKixxOVO5bn9mmFoudFgkubn5z4oSQIBMY2K0TlRRcDij8fiMrkLUVCk+udr//jLcC2tUiK7SPuBIXLUHLOso/1ikESIyPPMUx19RAY9A7HU39hPRLyzDEwdoYVDnvyzQ== 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=Q+JlZukxntGgfvG2+J2uUvo08C+zLT1x1eCzvcEWPuE=; b=JDmEAHHjUH2ttx9DoUZRMIVoasFHAHttgh/DmuSlyXflU9cOIYZ+MxWB1xsNO06gay3UYmpCcto+4fGme4fhliP6BjlAtsfiA07fe0F/Pfm5StuRDWWolNoxomb6xvxFJHpwecBRRGEALqtNh4PbCxYZkZ9b3GXwXlO08aDB9khXU9xvTKihv3ZXjREsiP70jfeUeV3YtEWqZtAdPiat5ANb+IWVWpL2HE1C7d+GbByBJmF6mKk9WmNdL1RJoVqAhhZshH8yp3eD0A3VshibwO9YGg0bmbaVQvYpiyWY4Z8n7JSR+IHqzUB8uR3eyOPed6Ta82mlgqBTr560s7TikQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=ami.com; dmarc=pass action=none header.from=ami.com; dkim=pass header.d=ami.com; arc=none X-Received: from PH0PR10MB5756.namprd10.prod.outlook.com (2603:10b6:510:146::19) by BLAPR10MB5313.namprd10.prod.outlook.com (2603:10b6:208:331::12) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5612.12; Tue, 13 Sep 2022 09:36:08 +0000 X-Received: from PH0PR10MB5756.namprd10.prod.outlook.com ([fe80::b859:eada:b670:6de5]) by PH0PR10MB5756.namprd10.prod.outlook.com ([fe80::b859:eada:b670:6de5%5]) with mapi id 15.20.5612.022; Tue, 13 Sep 2022 09:36:08 +0000 From: =?UTF-8?B?SmFtZXNXYW5nIFvnjovlrrbmmI5dIHZpYSBncm91cHMuaW8=?= To: "devel@edk2.groups.io" CC: Eric Dong , Isaac Oram , Liming Gao Subject: [edk2-devel] [PATCH v2 1/1] BoardModulePkg/BoardBdsHookLib: Fix F7 overwrites Boot0000 Thread-Topic: [PATCH v2 1/1] BoardModulePkg/BoardBdsHookLib: Fix F7 overwrites Boot0000 Thread-Index: AQHYx1RAHieo4gTSRUykmZOZ+v41CQ== Date: Tue, 13 Sep 2022 09:36:07 +0000 Message-ID: <20220913093457.14644-2-jameswang@ami.com> References: <20220913093457.14644-1-jameswang@ami.com> In-Reply-To: <20220913093457.14644-1-jameswang@ami.com> Accept-Language: zh-TW, en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-ms-publictraffictype: Email x-ms-traffictypediagnostic: PH0PR10MB5756:EE_|BLAPR10MB5313:EE_ x-ms-office365-filtering-correlation-id: 9d6ce312-c98b-4ef3-656c-08da956b630e x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam-message-info: xarcr/WYHKI0lFE3x7PWKUEwPHouR3NRECuyjZMT/zG3XCWzDmvHj0ameIpbE+qjSHeK62RqwcYPqzEuNPvcpIegar3AH8qR+tSRY7/mos2pXxseed0sUk1tASlvVYqVMW7P2JlCoLVLnh3NgvBRCv1KNeYYY9jws83MisQCfNSTINQnWwpZfKtt8Kc3w/8G+lZt+hUf4JacGZhRj2BS8l6Hh85sWHZwwlBe17XECKQlpZqh8OovIJR54reEI3rYdh+pPKtUlmR43x5JnSvgLgiXGu8ipLhvBzN4d3KCv6BfIXwKUXrK2YVsxVwfm03kRWeBZ+T4k6O9pe46MFcNeymZ5s15xc6yV3HFGZpwOKPiq6f4lpYaY5AEBnZ5IcP2Cg9EClxBqaqgxGzarr/Z/x5ko0NyDHDlfZFrNdntk08mP/mKcij88y/FpIQseSW8q5i+j9zaXdtkvw/3uSrdd9d1adtMC6M872HnU3XVPMBZofIXns9wZd/aafIAmgkRJLzP7Wmqdj5d6l5tK+kRvM2+NYHnIZzai5HtORLEigWdlnUrjNOYKkvTEXUbzwu89M8U/C16qXTmKeB/MhgwTFcjh57PW73pxl1lumI4Z2Jix/tHRyExYP1El3V2DdQQSpeaGHVs2sH5ui0QPvqU6JRLON+gl/WwcjzRsvQWzJUZNJ8JpDdq/Rpe/Ib/tBGPugkP2TKW11r8ZghGPWHoXmbxJENw5YiQh+fv1EjX+jZA6aU4WLJ9t/cs1DndVG61YAnGDunOK6z1+Wqvazf3qg== x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?iso-2022-jp?B?aDZXMkY0QVZZZjRDWHVibGhyQkVXT0hwSkJrWFJPK0FzNGM2Y1J1akFv?= =?iso-2022-jp?B?dnlyejVKZktrMVZMZC9sRDVnZFdOWmV5WUhUWUhhTEtyMm1jU1AvN2I3?= =?iso-2022-jp?B?OXl2UVdjZ3dOUnhlall2SGY2ZHhEUUp2a2hMaXFYRUFjaW9LTC91ZkIr?= =?iso-2022-jp?B?MVp3S3ljNUo4czJFSk9xSnZhaXhaMUJKdC9SeCtXeVNLaU5WZ3Uyd05U?= =?iso-2022-jp?B?YkRaSG41OUg3SitGL3l0bU5obThDb3FVdTlWOUJlOXlPOFg1OVBkRFc0?= =?iso-2022-jp?B?cHB3QTZicmoyK0tKbGh6K010cllobzc0OGNMWm5pcUNGL1puVDRCb1lw?= =?iso-2022-jp?B?eUhIcFR5Zm1XakpVa0FrakRmckdDZmM3SFdTTFRLMHJsZ2w0dG1rNkVY?= =?iso-2022-jp?B?elJYa2VSY3NqTzFMbGZZNFhOdzFnMVVpL2xNcGtOTU5XVktXdE5XMC82?= =?iso-2022-jp?B?aUd3a3ZLMStpaXkyYkFrS2lXcE51T0VaV0ZWb005a1QyaXpnMm56RHBP?= =?iso-2022-jp?B?UjRPNEN6Qnp1bmFET3FvWXRYdlNTNWJYMXVKd1h0T25LZ1dYYnpXWnpK?= =?iso-2022-jp?B?eGdPdmNFZjlwa2Fqd3RFNkxySEhLSjYwVkVoSU5jMUJLM2NycjdmM2dR?= =?iso-2022-jp?B?bmRpUkJJajh3WndPcENySEp2RmlFMFdxN2NmblNaQ3FxYm95RHpkT29s?= =?iso-2022-jp?B?b3ZqWVhuRHlwOGJPVTdnSG1iOHdXV3FmR2E3VFlFTEFpRG5veXZYUVo2?= =?iso-2022-jp?B?RjlpaW5JRlg2L2hNOXhpLzIxOTZCaVAwUGZLOFR1WHl4R2FTL01PbklX?= =?iso-2022-jp?B?WDBKdG8wWVk3T1RUT0J2Q1BFa3pZOWFrbTI2VU1pV0FjNzg1bTJLbS9P?= =?iso-2022-jp?B?TEZzaUJRMVg1alhpck11eUh5SGtwUTd0c0dlcW5ndkRWamlyV1NrZ2Mz?= =?iso-2022-jp?B?Vlk0eDBiT0NmS1RQd3RJQjI0eW1QVHovWjJPbVdTUUdMb2oyTStUL1Vi?= =?iso-2022-jp?B?KzU5dzFVTTdxcHh1L1lsb2NJTzgrc3BUYkxHRkhXTTZkdG1ORWZydVB4?= =?iso-2022-jp?B?bEU1eERCeVdrdEZzOFpWL3ZQTVBJTWxBVzlQQmgrUnNZaUdPOU9KaGxt?= =?iso-2022-jp?B?Qk5oZ0dDNDY3eE9NOHRHck9yVUxKWWR2bElWZzVMcHFzcFlUUSs0K212?= =?iso-2022-jp?B?T05hWmlpVjcxQmQ2ZjNCdDlkOTNId0phVjZMcHJuWHBkUUpGMXMxbkI3?= =?iso-2022-jp?B?N0FwYWRSK3BNYUFGc1lObTJLZFdqSTBBYW5hZXE5bVVIRmRaRjVmeVVa?= =?iso-2022-jp?B?YlBnM3lkWmxxbldxNElkSm01THppYjZLcHdhdVVXQUMwdHo1ZDRkKzJl?= =?iso-2022-jp?B?ZWllRTN4MXNSYjc0KzJDdk9SMjdvT3pJVlk5Rk1VeE1LSDUvNUZvOU1o?= =?iso-2022-jp?B?WUovSU1FSUJUZkdTL1IxZFREdE1URlE0ZHNzaHF6SWN4K1NSSGZlWnNV?= =?iso-2022-jp?B?eFZSQmo1cVA3dXArU1lVcjhNUmx1YnFQM3BGdzhiYlNXZCtvRHRORkJj?= =?iso-2022-jp?B?ejJ2TnNZUHdFQTZZUWtaZ0hXVjNTMndwV3dLWE9xZGN6RC9EZjhoUzBB?= =?iso-2022-jp?B?Z2diVncwL3hGb0dZdDR3cjRsZVJlSS9sbWJxVkRWQkt6NlY1VXJiNFZs?= =?iso-2022-jp?B?bmNvaDVuZXdpdWwwWDlGaDNXTlA1b0hYbFVHamx4eCtydkViZENHUmdm?= =?iso-2022-jp?B?SC9ZenNPaVMyTFZBQzQzdEhabkFrOVJyMGYxZGtTT200aEdyLzBycDhq?= =?iso-2022-jp?B?cUZ5L01BVll6b1d4TzJMQXZrK3pzQWZWRlAveHVPZDFUUGlmdWVhekM1?= =?iso-2022-jp?B?QXNQejBZRk82akhEbnp4SHVzT3NaUzZTUEx5WWt1eTd4S0xSQ2VLR0I0?= =?iso-2022-jp?B?M3pLaHBrcGFvV0VBTHAwT2NEOXBPQWhXTVQ5VWJZSjlvTkVvcGNOMzZv?= =?iso-2022-jp?B?SFRXQlFDT3p3LzZ6RURKTE8reG1Sdy9NQUJkWFNwbHlQUWF6ckpqZVpr?= =?iso-2022-jp?B?cWsyUURmemRiN0ZsdTlOakI3SXNHVTdoZDcwb2ZlU0g1N0VRckhibEN2?= =?iso-2022-jp?B?eG1NSkpTckNEOU9BbmFVcCsvVVdyL0pEWDhJTWdWTWNqT3BUU3dDRzQ3?= =?iso-2022-jp?B?TjhaSlpmYnc3b1p2VFhDQVdKRzJMZW1FY2tSQS9lUTRGY0lEUGRBN2du?= =?iso-2022-jp?B?ZmxTclF3U3lQc3BxRlNGck1xVVFzdUUycWw0cGV2TytYTGRkOXRNQkNV?= =?iso-2022-jp?B?dlNwUg==?= MIME-Version: 1.0 X-OriginatorOrg: ami.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: PH0PR10MB5756.namprd10.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 9d6ce312-c98b-4ef3-656c-08da956b630e X-MS-Exchange-CrossTenant-originalarrivaltime: 13 Sep 2022 09:36:07.9149 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 27e97857-e15f-486c-b58e-86c2b3040f93 X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: ZlcOt6QpjK743RsvN7W4rsWPtkOiksztRbjWMy8t4D5ipZly5V1bJtNH0hOWZmgEa580mWGKRPvr1U2KlvV6Qg== X-MS-Exchange-Transport-CrossTenantHeadersStamped: BLAPR10MB5313 Precedence: Bulk List-Unsubscribe: List-Subscribe: List-Help: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Reply-To: devel@edk2.groups.io,JamesWang@ami.com X-Gm-Message-State: vw1q6OFb0orfmjsa1T0iGRZvx1787277AA= Content-Language: en-US Content-Transfer-Encoding: quoted-printable DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=groups.io; q=dns/txt; s=20140610; t=1663092220; bh=7UKGD7VvmSHhoMrLLdL80h2kCV8ySs9vfbzp7p7+CkU=; h=CC:Content-Type:Date:From:Reply-To:Subject:To; b=s7l+J0l5/XZK298vL+iu8rzzGnrpyRlj4kF4eT/vTcDtfdViP4IhJjKszAjQN54bAW9 Ng1CPGVpKA1gDC44fYiwxF2XOsA8S+eCpPA0E7E6tdEmTisCPr5yrriW6bbKlzrZhWzcp GQR4V5yccTcIdCB3CiOAOdbjfAVEe7Dmxw8= X-ZohoMail-DKIM: pass (identity @groups.io) X-ZM-MESSAGEID: 1663092222536100006 Content-Type: text/plain; charset="utf-8" Check if mBootMenuOptionNumber is valid or not before calling RegisterBootOptionHotkey to avoid overwriting Boot0000 Cc: Eric Dong Cc: Isaac Oram Cc: Liming Gao Signed-off-by: James Wang --- Platform/Intel/BoardModulePkg/Library/BoardBdsHookLib/BoardBootOption.c | = 17 +++++++++++------ 1 file changed, 11 insertions(+), 6 deletions(-) diff --git a/Platform/Intel/BoardModulePkg/Library/BoardBdsHookLib/BoardBoo= tOption.c b/Platform/Intel/BoardModulePkg/Library/BoardBdsHookLib/BoardBoot= Option.c index 6e3c21f4f052..34c971f3780b 100644 --- a/Platform/Intel/BoardModulePkg/Library/BoardBdsHookLib/BoardBootOption= .c +++ b/Platform/Intel/BoardModulePkg/Library/BoardBdsHookLib/BoardBootOption= .c @@ -461,12 +461,17 @@ RegisterStaticHotkey ( RegisterBootOptionHotkey ((UINT16) mSetupOptionNumber, &F2.Key, TRUE); } - F7.Key.ScanCode =3D SCAN_F7; - F7.Key.UnicodeChar =3D CHAR_NULL; - F7.KeyState.KeyShiftState =3D EFI_SHIFT_STATE_VALID; - F7.KeyState.KeyToggleState =3D 0; - mBootMenuBoot =3D !EnterSetup; - RegisterBootOptionHotkey ((UINT16) mBootMenuOptionNumber, &F7.Key, mBoot= MenuBoot); + // + // Register F7 only when the mBootMenuOptionNumber is valid + // + if (mBootMenuOptionNumber !=3D LoadOptionNumberUnassigned) { + F7.Key.ScanCode =3D SCAN_F7; + F7.Key.UnicodeChar =3D CHAR_NULL; + F7.KeyState.KeyShiftState =3D EFI_SHIFT_STATE_VALID; + F7.KeyState.KeyToggleState =3D 0; + mBootMenuBoot =3D !EnterSetup; + RegisterBootOptionHotkey ((UINT16) mBootMenuOptionNumber, &F7.Key, mBo= otMenuBoot); + } } -- 2.13.2.windows.1 -The information contained in this message may be confidential and propriet= ary to American Megatrends (AMI). This communication is intended to be read= only by the individual or entity to whom it is addressed or by their desig= nee. If the reader of this message is not the intended recipient, you are o= n notice that any distribution of this message, in any form, is strictly pr= ohibited. Please promptly notify the sender by reply e-mail or by telephone= at 770-246-8600, and then delete or destroy all copies of the transmission. -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#93721): https://edk2.groups.io/g/devel/message/93721 Mute This Topic: https://groups.io/mt/93661217/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-