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.120; helo=mga04.intel.com; envelope-from=star.zeng@intel.com; receiver=edk2-devel@lists.01.org Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) (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 08C322034A7BD for ; Mon, 23 Oct 2017 19:15:46 -0700 (PDT) Received: from fmsmga006.fm.intel.com ([10.253.24.20]) by fmsmga104.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 23 Oct 2017 19:19:29 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.43,424,1503385200"; d="scan'208";a="166713598" Received: from fmsmsx108.amr.corp.intel.com ([10.18.124.206]) by fmsmga006.fm.intel.com with ESMTP; 23 Oct 2017 19:19:30 -0700 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.319.2; Mon, 23 Oct 2017 19:19:30 -0700 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.319.2; Mon, 23 Oct 2017 19:19:29 -0700 Received: from shsmsx102.ccr.corp.intel.com ([169.254.2.175]) by SHSMSX151.ccr.corp.intel.com ([169.254.3.218]) with mapi id 14.03.0319.002; Tue, 24 Oct 2017 10:19:27 +0800 From: "Zeng, Star" To: Laszlo Ersek , edk2-devel-01 CC: "Dong, Eric" , "Yao, Jiewen" , "Zeng, Star" Thread-Topic: [PATCH] MdeModulePkg/Variable/RuntimeDxe: delete & lock MOR in the absence of SMM Thread-Index: AQHTQcsymAp4aNQ81UGwQZlp6QvOb6LdGeHQ//+1AQCAFYpFEA== Date: Tue, 24 Oct 2017 02:19:27 +0000 Message-ID: <0C09AFA07DD0434D9E2A0C6AEB0483103B9ABA16@shsmsx102.ccr.corp.intel.com> References: <20171010132443.11383-1-lersek@redhat.com> <0C09AFA07DD0434D9E2A0C6AEB0483103B980020@shsmsx102.ccr.corp.intel.com> <948376a4-9609-b78f-3618-a1b45d1f0da4@redhat.com> In-Reply-To: <948376a4-9609-b78f-3618-a1b45d1f0da4@redhat.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] MdeModulePkg/Variable/RuntimeDxe: delete & lock MOR in the absence of SMM X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 24 Oct 2017 02:15:47 -0000 Content-Language: en-US Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 SSBwcmVmZXIgdG8gc2V0IHRoZSBhdHRyaWJ1dGVzIHRvIDAgZm9yIHRoZSBNb3JMb2NrIGRlbGV0 aW9uIGluIHRoZSBzYW1lIHBhdGNoLg0KDQpJdCBpcyBmaW5lIHRvIGhhdmUgZW1wdHkgTW9yTG9j a0luaXRBdEVuZE9mRHhlKCkgaW4gVGNnTW9yTG9ja0R4ZS5jIHdpdGggeW91ciBleHBsYW5hdGlv bi4NCg0KDQpUaGFua3MsDQpTdGFyDQotLS0tLU9yaWdpbmFsIE1lc3NhZ2UtLS0tLQ0KRnJvbTog TGFzemxvIEVyc2VrIFttYWlsdG86bGVyc2VrQHJlZGhhdC5jb21dIA0KU2VudDogV2VkbmVzZGF5 LCBPY3RvYmVyIDExLCAyMDE3IDE6MjIgQU0NClRvOiBaZW5nLCBTdGFyIDxzdGFyLnplbmdAaW50 ZWwuY29tPjsgZWRrMi1kZXZlbC0wMSA8ZWRrMi1kZXZlbEBsaXN0cy4wMS5vcmc+DQpDYzogRG9u ZywgRXJpYyA8ZXJpYy5kb25nQGludGVsLmNvbT47IFlhbywgSmlld2VuIDxqaWV3ZW4ueWFvQGlu dGVsLmNvbT4NClN1YmplY3Q6IFJlOiBbUEFUQ0hdIE1kZU1vZHVsZVBrZy9WYXJpYWJsZS9SdW50 aW1lRHhlOiBkZWxldGUgJiBsb2NrIE1PUiBpbiB0aGUgYWJzZW5jZSBvZiBTTU0NCg0KT24gMTAv MTAvMTcgMTU6NTQsIFplbmcsIFN0YXIgd3JvdGU6DQo+IENvdWxkIHlvdSBoZWxwIG1ha2UgdGhl IGNvZGUgY29uc2lzdGVudCB0byBjYWxsDQo+IFZhcmlhYmxlU2VydmljZVNldFZhcmlhYmxlKCkg Zm9yIHRoZSBBdHRyaWJ1dGVzPyBPbmUgb3JpZ2luYWwgZm9yIA0KPiBNb3JMb2NrIGlzIHVzaW5n IHJlYWwgYXR0cmlidXRlcywgdGhlIG5ldyB5b3UgYWRkZWQgZm9yIE1vciB3aWxsIHVzZSAwIA0K PiBhdHRyaWJ1dGVzLiBIb3cgYWJvdXQgdG8gYm90aCB1c2UgMCBhdHRyaWJ1dGVzLg0KDQpTdXJl LCBJIGNhbiBkbyB0aGF0IC0tIGRvIHlvdSB3YW50IG1lIHRvIHNldCB0aGUgYXR0cmlidXRlcyB0 byAwIGZvciB0aGUgTW9yTG9jayBkZWxldGlvbiBpbiB0aGUgc2FtZSBwYXRjaCwgb3Igc2hvdWxk IEkgZG8gaXQgaW4gYSBzZXBhcmF0ZSBwYXRjaD8NCg0KPiBBbm90aGVyIHF1ZXN0aW9uLCB3aHkg ZW1wdHkgTW9yTG9ja0luaXRBdEVuZE9mRHhlKCkgbmVlZHMgdG8gYmUgDQo+IGltcGxlbWVudGVk IGluIFRjZ01vckxvY2tEeGUuYyBhbmQgY2FsbGVkIGluIFZhcmlhYmxlRHhlLmM/DQoNCkJlY2F1 c2Ugb3RoZXJ3aXNlIHdlIHdvdWxkIGJyZWFrIHRoZSBwcm9taXNlIHdlIG1ha2UgaW4NCiJQcml2 aWxlZ2VQb2x5bW9ycGhpYy5oIjoNCg0KPiAgIFBvbHltb3JwaGljIGZ1bmN0aW9ucyB0aGF0IGFy ZSBjYWxsZWQgZnJvbSBib3RoIHRoZSBwcml2aWxlZ2VkIGRyaXZlciAoaS5lLiwNCj4gICB0aGUg RFhFX1NNTSB2YXJpYWJsZSBtb2R1bGUpIGFuZCB0aGUgbm9uLXByaXZpbGVnZWQgZHJpdmVycyAo aS5lLiwgb25lIG9yDQo+ICAgYm90aCBvZiB0aGUgRFhFX1JVTlRJTUUgdmFyaWFibGUgbW9kdWxl cykuDQo+DQo+ICAgRWFjaCBvZiB0aGVzZSBmdW5jdGlvbnMgaGFzIHR3byBpbXBsZW1lbnRhdGlv bnMsIGFwcHJvcHJpYXRlIGZvciBwcml2aWxlZ2VkDQo+ICAgdnMuIG5vbi1wcml2aWxlZ2VkIGRy aXZlciBjb2RlLg0KDQpJZiBJIGRvbid0IGltcGxlbWVudCBNb3JMb2NrSW5pdEF0RW5kT2ZEeGUo KSBmb3IgVmFyaWFibGVSdW50aW1lRHhlLCB0aGVuIHdlJ2xsIGhhdmUgYSBkZWNsYXJhdGlvbiB3 aXRoIG5vIGRlZmluaXRpb24gaW4gVmFyaWFibGVSdW50aW1lRHhlLCBhbmQgb25seSBvbmUgZGVm aW5pdGlvbiBpbiB0b3RhbC4NCg0KVGhlIGVtcHR5IGZ1bmN0aW9uIGNhbGwgc2hvdWxkIGJlIG9w dGltaXplZCBvdXQgaW4gUkVMRUFTRSBidWlsZHMgYW55d2F5IChhc3N1bWluZyBMVE8gaXMgdXNl ZCkuDQoNCkl0J3MgZWFzeSB0byByZW1vdmUgdGhlIGVtcHR5IGRlZmluaXRpb24gYW5kIHRoZSBj YWxscyB0byBpdCwgYnV0IHRoZW4gdGhlIHB1cnBvc2Ugb2YgIlByaXZpbGVnZVBvbHltb3JwaGlj LmgiIGJlY29tZXMgbGVzcyBjbGVhci4NCg0KRm9yIGNvbXBsZXRlIGNsYXJpdHksIEkgd291bGQg aGF2ZSBoYWQgdG8gaW50cm9kdWNlICp0d28qIG5ldyBoZWFkZXINCmZpbGVzOg0KDQotIG9uZSBm b3IgU2VjdXJlQm9vdEhvb2soKSwgTW9yTG9ja0luaXQoKSBhbmQNCiAgU2V0VmFyaWFibGVDaGVj a0hhbmRsZXJNb3IoKSAtLSB0byBiZSB1c2VkIGJ5IGJvdGggU01NIGFuZCBub24tU01NLA0KDQot IGFuZCBhbm90aGVyIGhlYWRlciBmaWxlIGZvciBqdXN0IE1vckxvY2tJbml0QXRFbmRPZkR4ZSgp IC0tIHRvIGJlIHVzZWQNCiAgb25seSBieSBTTU0uIChUaGlzIGlzIG5lY2Vzc2FyeSBiZWNhdXNl IHRoZSBjYWxsIGlzIG1hZGUgZnJvbQ0KICAiVmFyaWFibGVTbW0uYyIgdG8gIlRjZ01vckxvY2tT bW0uYyIpLg0KDQpJIHRob3VnaHQgdGhhdCBpbnRyb2R1Y2luZyB0d28gbmV3IGhlYWRlciBmaWxl cyB3b3VsZCBub3QgYmUgYWNjZXB0ZWQsIHNvIEkgb3B0ZWQgZm9yIG9uZSBoZWFkZXIgZmlsZSBv bmx5LiBCdXQgdGhlbiBJIGZlbHQgdGhhdCB0aGUgZW1wdHkNCk1vckxvY2tJbml0QXRFbmRPZkR4 ZSgpIGhvb2sgc2hvdWxkIGJlIGNvcnJlY3RseSBpbXBsZW1lbnRlZCBmb3IgVmFyaWFibGVSdW50 aW1lRHhlIHRvby4NCg0KVGhhbmtzLA0KTGFzemxvDQoNCg0KPg0KPg0KPiBUaGFua3MsDQo+IFN0 YXINCj4gLS0tLS1PcmlnaW5hbCBNZXNzYWdlLS0tLS0NCj4gRnJvbTogTGFzemxvIEVyc2VrIFtt YWlsdG86bGVyc2VrQHJlZGhhdC5jb21dDQo+IFNlbnQ6IFR1ZXNkYXksIE9jdG9iZXIgMTAsIDIw MTcgOToyNSBQTQ0KPiBUbzogZWRrMi1kZXZlbC0wMSA8ZWRrMi1kZXZlbEBsaXN0cy4wMS5vcmc+ DQo+IENjOiBEb25nLCBFcmljIDxlcmljLmRvbmdAaW50ZWwuY29tPjsgWWFvLCBKaWV3ZW4gDQo+ IDxqaWV3ZW4ueWFvQGludGVsLmNvbT47IFplbmcsIFN0YXIgPHN0YXIuemVuZ0BpbnRlbC5jb20+ DQo+IFN1YmplY3Q6IFtQQVRDSF0gTWRlTW9kdWxlUGtnL1ZhcmlhYmxlL1J1bnRpbWVEeGU6IGRl bGV0ZSAmIGxvY2sgTU9SIA0KPiBpbiB0aGUgYWJzZW5jZSBvZiBTTU0NCj4NCj4gVmFyaWFibGVS dW50aW1lRHhlIGRlbGV0ZXMgYW5kIGxvY2tzIHRoZSBNb3JMb2NrIHZhcmlhYmxlIGluIA0KPiBN b3JMb2NrSW5pdCgpLCB3aXRoIHRoZSBhcmd1bWVudCB0aGF0IGFueSBwcm90ZWN0aW9uIHByb3Zp ZGVkIGJ5IA0KPiBNb3JMb2NrIGNhbiBiZSBjaXJjdW12ZW50ZWQgaWYgTW9yTG9jayBjYW4gYmUg b3ZlcndyaXR0ZW4gYnkgDQo+IHVucHJpdmlsZWdlZCBjb2RlIChpLmUuLCBvdXRzaWRlIG9mIFNN TSkuDQo+DQo+IEV4dGVuZCB0aGUgYXJndW1lbnQgYW5kIHRoZSBsb2dpYyB0byB0aGUgTU9SIHZh cmlhYmxlLCB3aGljaCBpcyANCj4gc3VwcG9zZWQgdG8gYmUgcHJvdGVjdGVkIGJ5IE1vckxvY2su DQo+DQo+IFRoaXMgY2hhbmdlIHdhcyBzdWdnZXN0ZWQgYnkgU3RhcjsgaXQgaXMgaW5zcGlyZWQg YnkgZWFybGllciANCj4gVmFyaWFibGVTbW0gY29tbWl0IGZkYThmNjMxZWRiYiAoIk1kZU1vZHVs ZVBrZy9WYXJpYWJsZS9SdW50aW1lRHhlOiANCj4gZGVsZXRlIGFuZCBsb2NrIE9TLWNyZWF0ZWQg TU9SIHZhcmlhYmxlIiwgMjAxNy0xMC0wMykuDQo+DQo+IENjOiBFcmljIERvbmcgPGVyaWMuZG9u Z0BpbnRlbC5jb20+DQo+IENjOiBKaWV3ZW4gWWFvIDxqaWV3ZW4ueWFvQGludGVsLmNvbT4NCj4g Q2M6IFN0YXIgWmVuZyA8c3Rhci56ZW5nQGludGVsLmNvbT4NCj4gU3VnZ2VzdGVkLWJ5OiBTdGFy IFplbmcgPHN0YXIuemVuZ0BpbnRlbC5jb20+DQo+IENvbnRyaWJ1dGVkLXVuZGVyOiBUaWFub0Nv cmUgQ29udHJpYnV0aW9uIEFncmVlbWVudCAxLjENCj4gU2lnbmVkLW9mZi1ieTogTGFzemxvIEVy c2VrIDxsZXJzZWtAcmVkaGF0LmNvbT4NCj4gLS0tDQo+DQo+IE5vdGVzOg0KPiAgICAgUmVwbzog ICBodHRwczovL2dpdGh1Yi5jb20vbGVyc2VrL2VkazIuZ2l0DQo+ICAgICBCcmFuY2g6IGRlbF9h bmRfbG9ja19tb3Jfd2l0aG91dF9zbW0NCj4NCj4gIE1kZU1vZHVsZVBrZy9Vbml2ZXJzYWwvVmFy aWFibGUvUnVudGltZUR4ZS9UY2dNb3JMb2NrRHhlLmMgfCAyNCANCj4gKysrKysrKysrKysrKysr KysrKysNCj4gIDEgZmlsZSBjaGFuZ2VkLCAyNCBpbnNlcnRpb25zKCspDQo+DQo+IGRpZmYgLS1n aXQgDQo+IGEvTWRlTW9kdWxlUGtnL1VuaXZlcnNhbC9WYXJpYWJsZS9SdW50aW1lRHhlL1RjZ01v ckxvY2tEeGUuYyANCj4gYi9NZGVNb2R1bGVQa2cvVW5pdmVyc2FsL1ZhcmlhYmxlL1J1bnRpbWVE eGUvVGNnTW9yTG9ja0R4ZS5jDQo+IGluZGV4IDcxNDJlMmRhMjA3My4uMTYxMGM3YWEwNzA2IDEw MDY0NA0KPiAtLS0gYS9NZGVNb2R1bGVQa2cvVW5pdmVyc2FsL1ZhcmlhYmxlL1J1bnRpbWVEeGUv VGNnTW9yTG9ja0R4ZS5jDQo+ICsrKyBiL01kZU1vZHVsZVBrZy9Vbml2ZXJzYWwvVmFyaWFibGUv UnVudGltZUR4ZS9UY2dNb3JMb2NrRHhlLmMNCj4gQEAgLTY5LDI5ICs2OSw1MyBAQCBFRklfU1RB VFVTDQo+ICBNb3JMb2NrSW5pdCAoDQo+ICAgIFZPSUQNCj4gICAgKQ0KPiAgew0KPiAgICAvLw0K PiAgICAvLyBBbHdheXMgY2xlYXIgdmFyaWFibGUgdG8gcmVwb3J0IHVuc3VwcG9ydGVkIHRvIE9T Lg0KPiAgICAvLyBUaGUgcmVhc29uIGlzIHRoYXQgdGhlIERYRSB2ZXJzaW9uIGlzIG5vdCBwcm9w ZXIgdG8gcHJvdmlkZSAqcHJvdGVjdGlvbiouDQo+ICAgIC8vIEJJT1Mgc2hvdWxkIHVzZSBTTU0g dmVyc2lvbiB2YXJpYWJsZSBkcml2ZXIgdG8gcHJvdmlkZSBzdWNoIGNhcGFiaWxpdHkuDQo+ICAg IC8vDQo+ICAgIFZhcmlhYmxlU2VydmljZVNldFZhcmlhYmxlICgNCj4gICAgICBNRU1PUllfT1ZF UldSSVRFX1JFUVVFU1RfQ09OVFJPTF9MT0NLX05BTUUsDQo+ICAgICAgJmdFZmlNZW1vcnlPdmVy d3JpdGVSZXF1ZXN0Q29udHJvbExvY2tHdWlkLA0KPiAgICAgIEVGSV9WQVJJQUJMRV9OT05fVk9M QVRJTEUgfCBFRklfVkFSSUFCTEVfQk9PVFNFUlZJQ0VfQUNDRVNTIHwgRUZJX1ZBUklBQkxFX1JV TlRJTUVfQUNDRVNTLA0KPiAgICAgIDAsDQo+ICAgICAgTlVMTA0KPiAgICAgICk7DQo+DQo+ICAg IC8vDQo+ICAgIC8vIE5lZWQgc2V0IHRoaXMgdmFyaWFibGUgdG8gYmUgcmVhZC1vbmx5IHRvIHBy ZXZlbnQgb3RoZXIgbW9kdWxlIHNldCBpdC4NCj4gICAgLy8NCj4gICAgVmFyaWFibGVMb2NrUmVx dWVzdFRvTG9jayAoJm1WYXJpYWJsZUxvY2ssIA0KPiBNRU1PUllfT1ZFUldSSVRFX1JFUVVFU1Rf Q09OVFJPTF9MT0NLX05BTUUsIA0KPiAmZ0VmaU1lbW9yeU92ZXJ3cml0ZVJlcXVlc3RDb250cm9s TG9ja0d1aWQpOw0KPiArDQo+ICsgIC8vDQo+ICsgIC8vIFRoZSBNT1IgdmFyaWFibGUgY2FuIGVm ZmVjdGl2ZWx5IGltcHJvdmUgcGxhdGZvcm0gc2VjdXJpdHkgb25seSANCj4gKyB3aGVuIHRoZSAg Ly8gTW9yTG9jayB2YXJpYWJsZSBwcm90ZWN0cyB0aGUgTU9SIHZhcmlhYmxlLiBJbiB0dXJuIA0K PiArIE1vckxvY2sgY2Fubm90IGJlIG1hZGUgIC8vIHNlY3VyZSB3aXRob3V0IFNNTSBzdXBwb3J0 IGluIHRoZSBwbGF0Zm9ybSBmaXJtd2FyZSAoc2VlIGFib3ZlKS4NCj4gKyAgLy8NCj4gKyAgLy8g VGh1cywgZGVsZXRlIHRoZSBNT1IgdmFyaWFibGUsIHNob3VsZCBpdCBleGlzdCBmb3IgYW55IHJl YXNvbiANCj4gKyAoc29tZSBPU2VzICAvLyBhcmUga25vd24gdG8gY3JlYXRlIE1PUiB1bmludGVu dGlvbmFsbHksIGluIGFuIA0KPiArIGF0dGVtcHQgdG8gc2V0IGl0KSwgdGhlbiAgLy8gYWxzbyBs b2NrIHRoZSBNT1IgdmFyaWFibGUsIGluIG9yZGVyIHRvIA0KPiArIHByZXZlbnQgb3RoZXIgbW9k dWxlcyBmcm9tICAvLyBjcmVhdGluZyBpdC4NCj4gKyAgLy8NCj4gKyAgVmFyaWFibGVTZXJ2aWNl U2V0VmFyaWFibGUgKA0KPiArICAgIE1FTU9SWV9PVkVSV1JJVEVfUkVRVUVTVF9WQVJJQUJMRV9O QU1FLA0KPiArICAgICZnRWZpTWVtb3J5T3ZlcndyaXRlQ29udHJvbERhdGFHdWlkLA0KPiArICAg IDAsICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAvLyBBdHRyaWJ1dGVzDQo+ ICsgICAgMCwgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIC8vIERhdGFTaXpl DQo+ICsgICAgTlVMTCAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIC8vIERhdGEN Cj4gKyAgICApOw0KPiArICBWYXJpYWJsZUxvY2tSZXF1ZXN0VG9Mb2NrICgNCj4gKyAgICAmbVZh cmlhYmxlTG9jaywNCj4gKyAgICBNRU1PUllfT1ZFUldSSVRFX1JFUVVFU1RfVkFSSUFCTEVfTkFN RSwNCj4gKyAgICAmZ0VmaU1lbW9yeU92ZXJ3cml0ZUNvbnRyb2xEYXRhR3VpZA0KPiArICAgICk7 DQo+ICsNCj4gICAgcmV0dXJuIEVGSV9TVUNDRVNTOw0KPiAgfQ0KPg0KPiAgLyoqDQo+ICAgIERl bGF5ZWQgaW5pdGlhbGl6YXRpb24gZm9yIE1PUiBDb250cm9sIExvY2sgYXQgRW5kT2ZEeGUuDQo+ DQo+ICAgIFRoaXMgZnVuY3Rpb24gcGVyZm9ybXMgYW55IG9wZXJhdGlvbnMgcXVldWVkIGJ5IE1v ckxvY2tJbml0KCkuDQo+ICAqKi8NCj4NCg0K