From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=192.55.52.136; helo=mga12.intel.com; envelope-from=hao.a.wu@intel.com; receiver=edk2-devel@lists.01.org Received: from mga12.intel.com (mga12.intel.com [192.55.52.136]) (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 B962C211C280D for ; Wed, 30 Jan 2019 21:53:15 -0800 (PST) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by fmsmga106.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 30 Jan 2019 21:53:15 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.56,543,1539673200"; d="scan'208";a="139409132" Received: from fmsmsx108.amr.corp.intel.com ([10.18.124.206]) by fmsmga002.fm.intel.com with ESMTP; 30 Jan 2019 21:53:15 -0800 Received: from FMSMSX109.amr.corp.intel.com (10.18.116.9) by FMSMSX108.amr.corp.intel.com (10.18.124.206) with Microsoft SMTP Server (TLS) id 14.3.408.0; Wed, 30 Jan 2019 21:53:14 -0800 Received: from shsmsx151.ccr.corp.intel.com (10.239.6.50) by fmsmsx109.amr.corp.intel.com (10.18.116.9) with Microsoft SMTP Server (TLS) id 14.3.408.0; Wed, 30 Jan 2019 21:53:14 -0800 Received: from shsmsx104.ccr.corp.intel.com ([169.254.5.102]) by SHSMSX151.ccr.corp.intel.com ([169.254.3.172]) with mapi id 14.03.0415.000; Thu, 31 Jan 2019 13:53:12 +0800 From: "Wu, Hao A" To: "Ni, Ray" , "edk2-devel@lists.01.org" CC: "Wang, Jian J" , "Zeng, Star" Thread-Topic: [PATCH v2 10/12] MdeModulePkg/SmmLockBox(PEI): Remove an ASSERT in RestoreLockBox() Thread-Index: AQHUuQ+Umx2KxtsFXUi3Hxy+T15InaXIWXoAgACGeXA= Date: Thu, 31 Jan 2019 05:53:12 +0000 Message-ID: References: <20190131024854.4880-1-hao.a.wu@intel.com> <20190131024854.4880-11-hao.a.wu@intel.com> <3c7c8767-23c8-e8a1-ef35-a03e37968ddb@Intel.com> In-Reply-To: <3c7c8767-23c8-e8a1-ef35-a03e37968ddb@Intel.com> Accept-Language: zh-CN, en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Subject: Re: [PATCH v2 10/12] MdeModulePkg/SmmLockBox(PEI): Remove an ASSERT in RestoreLockBox() X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 31 Jan 2019 05:53:16 -0000 Content-Language: en-US Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 PiAtLS0tLU9yaWdpbmFsIE1lc3NhZ2UtLS0tLQ0KPiBGcm9tOiBOaSwgUmF5DQo+IFNlbnQ6IFRo dXJzZGF5LCBKYW51YXJ5IDMxLCAyMDE5IDE6NTAgUE0NCj4gVG86IFd1LCBIYW8gQTsgZWRrMi1k ZXZlbEBsaXN0cy4wMS5vcmcNCj4gQ2M6IFdhbmcsIEppYW4gSjsgTmksIFJheTsgWmVuZywgU3Rh cg0KPiBTdWJqZWN0OiBSZTogW1BBVENIIHYyIDEwLzEyXSBNZGVNb2R1bGVQa2cvU21tTG9ja0Jv eChQRUkpOiBSZW1vdmUgYW4NCj4gQVNTRVJUIGluIFJlc3RvcmVMb2NrQm94KCkNCj4gDQo+IE9u IDEvMzEvMjAxOSAxMDo0OCBBTSwgSGFvIFd1IHdyb3RlOg0KPiA+IFRoaXMgY29tbWl0IGlzIG91 dCBvZiB0aGUgc2NvcGUgZm9yIEJaLTE0MDkuIEl0IGlzIGEgcmVmaW5lbWVudCBmb3IgdGhlDQo+ ID4gUEVJIGxpYnJhcnkgaW5zdGFuY2Ugd2l0aGluIFNtbUxvY2tCb3hMaWIuDQo+ID4NCj4gPiBG b3IgdGhlIGJlbG93IEFTU0VSVCBzdGF0ZW1lbnQgd2l0aGluIGZ1bmN0aW9uIFJlc3RvcmVMb2Nr Qm94KCk6DQo+ID4gICAgU3RhdHVzID0gU21tQ29tbXVuaWNhdGlvblBwaS0+Q29tbXVuaWNhdGUg KA0KPiA+ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgU21tQ29tbXVuaWNhdGlv blBwaSwNCj4gPiAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICZDb21tQnVmZmVy WzBdLA0KPiA+ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgJkNvbW1TaXplDQo+ ID4gICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICApOw0KPiA+ICAgIGlmIChTdGF0 dXMgPT0gRUZJX05PVF9TVEFSVEVEKSB7DQo+ID4gICAgICAvLw0KPiA+ICAgICAgLy8gUGVpIFNN TSBjb21tdW5pY2F0aW9uIG5vdCByZWFkeSB5ZXQsIHNvIHdlIGFjY2VzcyBTTVJBTSBkaXJlY3Rs eQ0KPiA+ICAgICAgLy8NCj4gPiAgICAgIERFQlVHICgoREVCVUdfSU5GTywgIlNtbUxvY2tCb3hQ ZWlMaWIgQ29tbXVuaWNhdGUgLSAoJXIpXG4iLA0KPiBTdGF0dXMpKTsNCj4gPiAgICAgIFN0YXR1 cyA9IEludGVybmFsUmVzdG9yZUxvY2tCb3hGcm9tU21yYW0gKEd1aWQsIEJ1ZmZlciwgTGVuZ3Ro KTsNCj4gPiAgICAgIExvY2tCb3hQYXJhbWV0ZXJSZXN0b3JlLT5IZWFkZXIuUmV0dXJuU3RhdHVz ID0gKFVJTlQ2NClTdGF0dXM7DQo+ID4gICAgICBpZiAoTGVuZ3RoICE9IE5VTEwpIHsNCj4gPiAg ICAgICAgTG9ja0JveFBhcmFtZXRlclJlc3RvcmUtPkxlbmd0aCA9IChVSU5UNjQpKkxlbmd0aDsN Cj4gPiAgICAgIH0NCj4gPiAgICB9DQo+ID4gICAgQVNTRVJUX0VGSV9FUlJPUiAoU3RhdHVzKTsN Cj4gPg0KPiA+IEl0IGlzIHBvc3NpYmxlIGZvciBwcmV2aW91cyBjb2RlcyB0byByZXR1cm4gYW4g ZXJyb3Igc3RhdHVzIHRoYXQgaXMNCj4gPiBwb3NzaWJsZSBmb3IgaGFwcGVuLiBPbmUgZXhhbXBs ZSBpcyB0aGF0LCB3aGVuIHRoZSAnaWYnIHN0YXRlbWVudA0KPiA+ICdpZiAoU3RhdHVzID09IEVG SV9OT1RfU1RBUlRFRCkgeycgaXMgZW50ZXJlZCwgZnVuY3Rpb24NCj4gPiBJbnRlcm5hbFJlc3Rv cmVMb2NrQm94RnJvbVNtcmFtKCkgaXMgcG9zc2libGUgdG8gcmV0dXJuDQo+ICdCVUZGRVJfVE9P X1NNQUxMJw0KPiA+IGlmIHRoZSBjYWxsZXIgb2YgUmVzdG9yZUxvY2tCb3goKSBwcm92aWRlcyBh IGJ1ZmZlciB0aGF0IGlzIHRvbyBzbWFsbCB0bw0KPiA+IGhvbGQgdGhlIGNvbnRlbnQgb2YgTG9j a0JveC4NCj4gPg0KPiA+IFRodXMsIHRoaXMgY29tbWl0IHdpbGwgcmVtb3ZlIHRoZSBBU1NFUlQg aGVyZS4NCj4gPg0KPiA+IFBsZWFzZSBub3RlIHRoYXQgdGhlIGN1cnJlbnQgaW1wbGVtZW50YXRp b24gb2YgUmVzdG9yZUxvY2tCb3goKSBpcw0KPiA+IGhhbmRsaW5nIHRoZSBhYm92ZS1tZW50aW9u ZWQgZXJyb3IgY2FzZSBwcm9wZXJseSwgc28gbm8gYWRkaXRpb25hbCBlcnJvcg0KPiA+IGhhbmRs aW5nIGNvZGVzIGFyZSBuZWVkZWQgaGVyZS4NCj4gPg0KPiA+IENjOiBKaWFuIEogV2FuZyA8amlh bi5qLndhbmdAaW50ZWwuY29tPg0KPiA+IENjOiBSYXkgTmkgPHJheS5uaUBpbnRlbC5jb20+DQo+ ID4gQ2M6IFN0YXIgWmVuZyA8c3Rhci56ZW5nQGludGVsLmNvbT4NCj4gPiBDb250cmlidXRlZC11 bmRlcjogVGlhbm9Db3JlIENvbnRyaWJ1dGlvbiBBZ3JlZW1lbnQgMS4xDQo+ID4gU2lnbmVkLW9m Zi1ieTogSGFvIFd1IDxoYW8uYS53dUBpbnRlbC5jb20+DQo+ID4gLS0tDQo+ID4gICBNZGVNb2R1 bGVQa2cvTGlicmFyeS9TbW1Mb2NrQm94TGliL1NtbUxvY2tCb3hQZWlMaWIuYyB8IDIgKy0NCj4g PiAgIDEgZmlsZSBjaGFuZ2VkLCAxIGluc2VydGlvbigrKSwgMSBkZWxldGlvbigtKQ0KPiA+DQo+ ID4gZGlmZiAtLWdpdCBhL01kZU1vZHVsZVBrZy9MaWJyYXJ5L1NtbUxvY2tCb3hMaWIvU21tTG9j a0JveFBlaUxpYi5jDQo+IGIvTWRlTW9kdWxlUGtnL0xpYnJhcnkvU21tTG9ja0JveExpYi9TbW1M b2NrQm94UGVpTGliLmMNCj4gPiBpbmRleCA5ZjczNDgwMDcwLi45ZDdiNGMzNzA2IDEwMDY0NA0K PiA+IC0tLSBhL01kZU1vZHVsZVBrZy9MaWJyYXJ5L1NtbUxvY2tCb3hMaWIvU21tTG9ja0JveFBl aUxpYi5jDQo+ID4gKysrIGIvTWRlTW9kdWxlUGtnL0xpYnJhcnkvU21tTG9ja0JveExpYi9TbW1M b2NrQm94UGVpTGliLmMNCj4gPiBAQCAtNjIzLDcgKzYyMyw3IEBAIFJlc3RvcmVMb2NrQm94ICgN Cj4gPiAgICAgICAgIExvY2tCb3hQYXJhbWV0ZXJSZXN0b3JlLT5MZW5ndGggPSAoVUlOVDY0KSpM ZW5ndGg7DQo+ID4gICAgICAgfQ0KPiA+ICAgICB9DQo+ID4gLSAgQVNTRVJUX0VGSV9FUlJPUiAo U3RhdHVzKTsNCj4gPiArICAvL0FTU0VSVF9FRklfRVJST1IgKFN0YXR1cyk7DQoNCkl0IGNvbWVz IHRvIG1lIHRoYXQgdGhlIGFib3ZlIGxpbmUgc2hvdWxkIGJlIHJlbW92ZWQgcmF0aGVyIHRoYW4g YmVpbmcNCmNvbW1lbnRlZC4gV2lsbCBoYW5kbGUgaW4gdGhlIG5leHQgc2VyaWVzLg0KDQpCZXN0 IFJlZ2FyZHMsDQpIYW8gV3UNCg0KPiA+DQo+ID4gICAgIGlmIChMZW5ndGggIT0gTlVMTCkgew0K PiA+ICAgICAgICpMZW5ndGggPSAoVUlOVE4pTG9ja0JveFBhcmFtZXRlclJlc3RvcmUtPkxlbmd0 aDsNCj4gPg0KPiBSZXZpZXdlZC1ieTogUmF5IE5pIDxyYXkubmlAaW50ZWwuY29tPg0KPiANCj4g LS0NCj4gVGhhbmtzLA0KPiBSYXkNCg==