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.93; helo=mga11.intel.com; envelope-from=paul.a.lohr@intel.com; receiver=edk2-devel@lists.01.org Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) (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 2F9332116DA1E for ; Mon, 22 Oct 2018 20:14:22 -0700 (PDT) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga002.jf.intel.com ([10.7.209.21]) by fmsmga102.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 22 Oct 2018 20:12:59 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.54,414,1534834800"; d="scan'208";a="102444265" Received: from fmsmsx105.amr.corp.intel.com ([10.18.124.203]) by orsmga002.jf.intel.com with ESMTP; 22 Oct 2018 20:12:58 -0700 Received: from fmsmsx118.amr.corp.intel.com ([169.254.1.230]) by FMSMSX105.amr.corp.intel.com ([169.254.4.63]) with mapi id 14.03.0319.002; Mon, 22 Oct 2018 20:12:58 -0700 From: "Lohr, Paul A" To: Laszlo Ersek , "Ni, Ruiyu" , "edk2-devel@lists.01.org" CC: "Kinney, Michael D" , "Yao, Jiewen" Thread-Topic: [edk2] [PATCH] MdeModulePkg/PiSmmIpl: Do not reset SMRAM to UC when CPU driver runs Thread-Index: AQHUahPKIbNaSTcWXkOJpjfEQUVWn6UsJ0nA Date: Tue, 23 Oct 2018 03:12:57 +0000 Message-ID: References: <20181022090333.95988-1-ruiyu.ni@intel.com> In-Reply-To: Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiNzU3NDczOWMtM2U0ZC00YjRkLThiNDItNzZjYmJjM2FiYTg2IiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE3LjEwLjE4MDQuNDkiLCJUcnVzdGVkTGFiZWxIYXNoIjoiVjkybHc2aExMUHltekJBbXRmVU1GUHU4MENqYlFaMTh6cDZaU2YwcVBIN1FTXC9oMUZHWlNTS3lJbUIrYTN3eG8ifQ== x-ctpclassification: CTP_NT dlp-product: dlpe-windows dlp-version: 11.0.400.15 dlp-reaction: no-action x-originating-ip: [10.1.200.106] MIME-Version: 1.0 Subject: Re: [PATCH] MdeModulePkg/PiSmmIpl: Do not reset SMRAM to UC when CPU driver runs 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: Tue, 23 Oct 2018 03:14:22 -0000 Content-Language: en-US Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 SGVsbG8sDQoNCkNvZGUgdG8gcmVtb3ZlIFNNUkFNID0gVUMgKGxpbmUgNjUwLWlzaCkgbG9va3Mg Z29vZC4gIEkgd291bGQgc3VnZ2VzdCBhZGRpbmcgc29tZSBkZWJ1ZyBjb21tZW50cyBpbiB0aGUg YXJlYSBpdCB3YXMgcmVtb3ZlZC4gIFRoYW5rcy4NCg0KUGVyICM0LCBJIGFsc28gInRoaW5rIiB0 aGUgc2Vjb25kIFNNUkFNID0gVUMgc2hvdWxkIGJlIHJlbW92ZWQsIGluIGFkZGl0aW9uIHRvIHRo ZSBTTVJBTSA9IFdCIGluIHRoZSBzYW1lIGFyZWEuIEJ1dCB0aGlzIGlzIHVuZGVyIGludmVzdGln YXRpb24gbm93LiAgIEV4cGVjdCBhbiB1cGRhdGUgaW4gMjQtNDggaG91cnMuDQoNClBlciAjNSwg SSB0b28gZG8gbm90IGxpa2UgdGhlICJDUFUgQVAiIGNvbW1lbnQsIGFzIEkgaW5pdGlhbGx5IHRo b3VnaHQgaXQgcmVmZXJyZWQgdG8gQXBwbGljYXRpb24gUHJvY2Vzc29yICDwn5iKICBZZXMsIHBs ZWFzZSBjbGVhbiB0aGF0IEFQIGNvbW1lbnQgdXAhDQoNCg0KUGF1bCBBLiBMb2hyIOKAkyBTZXJ2 ZXIgRmlybXdhcmUgRW5hYmxpbmcNCjUxMi4yMzkuOTA3MyAoY2VsbCkNCjUxMi43OTQuNTA0NCAo d29yaykNCg0KLS0tLS1PcmlnaW5hbCBNZXNzYWdlLS0tLS0NCkZyb206IGVkazItZGV2ZWwgPGVk azItZGV2ZWwtYm91bmNlc0BsaXN0cy4wMS5vcmc+IE9uIEJlaGFsZiBPZiBMYXN6bG8gRXJzZWsN ClNlbnQ6IE1vbmRheSwgT2N0b2JlciAyMiwgMjAxOCA5OjMwIEFNDQpUbzogTmksIFJ1aXl1IDxy dWl5dS5uaUBpbnRlbC5jb20+OyBlZGsyLWRldmVsQGxpc3RzLjAxLm9yZw0KQ2M6IEtpbm5leSwg TWljaGFlbCBEIDxtaWNoYWVsLmQua2lubmV5QGludGVsLmNvbT47IFlhbywgSmlld2VuIDxqaWV3 ZW4ueWFvQGludGVsLmNvbT4NClN1YmplY3Q6IFJlOiBbZWRrMl0gW1BBVENIXSBNZGVNb2R1bGVQ a2cvUGlTbW1JcGw6IERvIG5vdCByZXNldCBTTVJBTSB0byBVQyB3aGVuIENQVSBkcml2ZXIgcnVu cw0KDQpPbiAxMC8yMi8xOCAxMTowMywgUnVpeXUgTmkgd3JvdGU6DQo+IFRvZGF5J3MgUGlTbW1J cGwgaW1wbGVtZW50YXRpb24gaW5pdGlhbGx5IHNldHMgU01SQU0gdG8gV0IgdG8gc3BlZWQgdXAg DQo+IHRoZSBTTU0gY29yZS9tb2R1bGVzIGxvYWRpbmcgYmVmb3JlIFNNTSBDUFUgZHJpdmVyIHJ1 bnMuDQo+IFdoZW4gU01NIENQVSBkcml2ZXIgcnVucywgUGlTbW1JcGwgcmVzZXRzIHRoZSBTTVJB TSB0byBVQy4gSXQncyBkb25lIA0KPiBpbiBTbW1JcGxEeGVEaXNwYXRjaEV2ZW50Tm90aWZ5KCku IENPTU1fQlVGRkVSX1NNTV9ESVNQQVRDSF9SRVNUQVJUIGlzIA0KPiByZXR1cm5lZCBmcm9tIFNN TSBjb3JlIHRoYXQgU01NIENQVSBkcml2ZXIgaXMganVzdCBkaXNwYXRjaGVkLg0KPiANCj4gU2lu Y2Ugbm93IHRoZSBTTVJSIGlzIHdpZGVseSB1c2VkIHRvIGNvbnRyb2wgdGhlIFNNUkFNIGNhY2hl IHNldHRpbmcuDQo+IEl0J3Mgbm90IG5lZWRlZCB0byByZXNldCB0aGUgU01SQU0gdG8gVUMgYW55 bW9yZS4NCj4gDQo+IENvbnRyaWJ1dGVkLXVuZGVyOiBUaWFub0NvcmUgQ29udHJpYnV0aW9uIEFn cmVlbWVudCAxLjENCj4gU2lnbmVkLW9mZi1ieTogUnVpeXUgTmkgPHJ1aXl1Lm5pQGludGVsLmNv bT4NCj4gQ2M6IEppZXdlbiBZYW8gPGppZXdlbi55YW9AaW50ZWwuY29tPg0KPiBDYzogTWljaGFl bCBLaW5uZXkgPG1pY2hhZWwuZC5raW5uZXlAaW50ZWwuY29tPg0KPiAtLS0NCj4gIE1kZU1vZHVs ZVBrZy9Db3JlL1BpU21tQ29yZS9QaVNtbUlwbC5jIHwgMTMgLS0tLS0tLS0tLS0tLQ0KPiAgMSBm aWxlIGNoYW5nZWQsIDEzIGRlbGV0aW9ucygtKQ0KPiANCj4gZGlmZiAtLWdpdCBhL01kZU1vZHVs ZVBrZy9Db3JlL1BpU21tQ29yZS9QaVNtbUlwbC5jIA0KPiBiL01kZU1vZHVsZVBrZy9Db3JlL1Bp U21tQ29yZS9QaVNtbUlwbC5jDQo+IGluZGV4IGY4Y2JlMTcwNGIuLmRjMGQ5YTcwYjAgMTAwNjQ0 DQo+IC0tLSBhL01kZU1vZHVsZVBrZy9Db3JlL1BpU21tQ29yZS9QaVNtbUlwbC5jDQo+ICsrKyBi L01kZU1vZHVsZVBrZy9Db3JlL1BpU21tQ29yZS9QaVNtbUlwbC5jDQo+IEBAIC02NzIsMTkgKzY3 Miw2IEBAIFNtbUlwbER4ZURpc3BhdGNoRXZlbnROb3RpZnkgKA0KPiAgICAgICAgcmV0dXJuOw0K PiAgICAgIH0NCj4gIA0KPiAtICAgIC8vDQo+IC0gICAgLy8gQXR0ZW1wdCB0byByZXNldCBTTVJB TSBjYWNoZWFiaWxpdHkgdG8gVUMNCj4gLSAgICAvLyBBc3N1bWUgQ1BVIEFQIGlzIGF2YWlsYWJs ZSBhdCB0aGlzIHRpbWUNCj4gLSAgICAvLw0KPiAtICAgIFN0YXR1cyA9IGdEUy0+U2V0TWVtb3J5 U3BhY2VBdHRyaWJ1dGVzKA0KPiAtICAgICAgICAgICAgICAgICAgICBtU21yYW1DYWNoZUJhc2Us DQo+IC0gICAgICAgICAgICAgICAgICAgIG1TbXJhbUNhY2hlU2l6ZSwNCj4gLSAgICAgICAgICAg ICAgICAgICAgRUZJX01FTU9SWV9VQw0KPiAtICAgICAgICAgICAgICAgICAgICApOw0KPiAtICAg IGlmIChFRklfRVJST1IgKFN0YXR1cykpIHsNCj4gLSAgICAgIERFQlVHICgoREVCVUdfV0FSTiwg IlNNTSBJUEwgZmFpbGVkIHRvIHJlc2V0IFNNUkFNIHdpbmRvdyB0byBFRklfTUVNT1JZX1VDXG4i KSk7DQo+IC0gICAgfQ0KPiAtDQo+ICAgICAgLy8NCj4gICAgICAvLyBDbG9zZSBhbGwgU01SQU0g cmFuZ2VzIHRvIHByb3RlY3QgU01SQU0NCj4gICAgICAvLw0KPiANCg0KSSB2YWd1ZWx5IHJlbWVt YmVyIHRoaXMgY29kZSBmcm9tIGNvbW1pdCBiMDdlYTRjMTk4YTQgKCJNZGVNb2R1bGVQa2c6DQpT bW1JcGxFbnRyeSgpOiBkb24ndCBzdXBwcmVzcyBTTU0gY29yZSBzdGFydHVwIGZhaWx1cmUiLCAy MDE1LTA1LTEyKS4gU28gaGVyZSBJJ20gYXNraW5nIGp1c3Qgb3V0IG9mIGN1cmlvc2l0eSAtLSBi ZWNhdXNlIFNNUlIgaXMgbm90IG5lY2Vzc2FyeSB0byBoYW5kbGUgb24gUUVNVS9LVk0uDQoNCkkg YXNzdW1lIHRoYXQgdGhlIG9yaWdpbmFsIFVDIHNldHRpbmcgd2FzIHBhcnQgb2YgdGhlIFNNUkFN IHByb3RlY3Rpb24gdGhhdCBzdGFydHMgYXQgdGhlIG5leHQgbGluZSAtLSB0aGUgY29tbWVudCBp cyB2aXNpYmxlIGluIHRoZSBjb250ZXh0LCBzYXlpbmcgIkNsb3NlIGFsbCBTTVJBTSByYW5nZXMg dG8gcHJvdGVjdCBTTVJBTSIuIFRoZSBjb21taXQgbWVzc2FnZSBzdWdnZXN0cyB0aGF0IGV4cGxp Y2l0IFNNUlIgbWFuYWdlbWVudCBoYXMgbm93IHRha2VuIHRoYXQgcm9sZS4NCg0KDQooMSkgQ2Fu IHlvdSBhZGQgbW9yZSBkZXRhaWxzIHRvIHRoZSBjb21taXQgbWVzc2FnZSB3aGVyZSB0aGF0IGV4 cGxpY2l0IG1hbmFnZW1lbnQgaGFwcGVucz8gSS5lLiwgd2hhdCBtb2R1bGVzIGNhbGwgd2hhdCBs aWJyYXJ5IGludGVyZmFjZXM/IEkgYmVsaWV2ZSB0aGUgcmVsZXZhbnQgQVBJcyBhcmUgZnJvbSBT bW1DcHVGZWF0dXJlc0xpYi4NCg0KDQooMikgQWxzbywgY2FuIHlvdSBlbGFib3JhdGUgb24gIndp ZGVseSI/IERvZXMgdGhhdCBtZWFuDQoNCiAgaW4gbW9zdCBlZGsyIG1vZHVsZXMgZm9yIHdoaWNo IFNNUkFNIGNhY2hpbmcgaXMgcmVsZXZhbnQNCg0Kb3INCg0KICBpbiBtb3N0IGVkazItYmFzZWQg cGxhdGZvcm0gZmlybXdhcmUgdGhhdCBpcyBzaGlwcGVkIGluIHByb2R1Y3Rpb24NCg0KPw0KDQoN CigzKSBZb3UgZm9yZ290IHRvIG1lbnRpb24gdGhlIGJ1Z3ppbGxhIGluIHRoZSBjb21taXQgbWVz c2FnZToNCg0KICBodHRwczovL2J1Z3ppbGxhLnRpYW5vY29yZS5vcmcvc2hvd19idWcuY2dpP2lk PTEyNjgNCg0KDQooNCkgRnJvbSBjb21taXQgYjA3ZWE0YzE5OGE0IHRoYXQgSSBtZW50aW9uIGFi b3ZlLCBpdCBzZWVtcyB0aGF0IHdlIGhhdmUgYW5vdGhlciBpbnN0YW5jZSBvZiB0aGlzIFVDIHNl dHRpbmcgaW4gdGhlIFNNTSBJUEwgcnVudGltZSBEWEUgZHJpdmVyOyBuYW1lbHkgaW4gdGhlIFNt bUlwbEVudHJ5KCkgZnVuY3Rpb24uIFNob3VsZCB3ZSByZW1vdmUgdGhhdCB0b28/DQoNCk5vdywg SSB1bmRlcnN0YW5kIHRoYXQgdGhlIHNpdHVhdGlvbiBhdCB0aGF0IHNpdGUgaXMgZGlmZmVyZW50 LCBiZWNhdXNlIGluIHRoYXQgY2FzZSwgd2UgYXJlIHJlc2V0dGluZyB0aGUgY2FjaGVhYmlsaXR5 IGFmdGVyICpmYWlsdXJlKiB0byBsYXVuY2ggdGhlIFNNTSBDb3JlLCBzbyB0aGVyZSBtYXkgaGF2 ZSBiZWVuIG5vIGNoYW5jZSBmb3IgYW55IG1vZHVsZSB0byBtYW5hZ2UgU01SUi4gQW5kIHNvIHdl IGhhdmUgdG8gcm9sbCBiYWNrIG91ciBvd24gaW5pdGlhbCBXQiBzZXR0aW5nLiBEbyBJIHVuZGVy c3RhbmQgdGhhdCByaWdodD8NCg0KDQooNSkgSSBsaWtlIHRoYXQgdGhpcyBwYXRjaCByZW1vdmVz IGEgIkNQVSBBUCIgY29tbWVudCwgYmVjYXVzZSB0aGUgY29tbWVudCBpcyBtaXNsZWFkaW5nLiBJ biB0aGlzIGNvbnRleHQsICJDUFUgQVAiIG1lYW5zICJDUFUgQXJjaGl0ZWN0dXJhbCBQcm90b2Nv bCIgKHdoaWNoIHVuZGVybGllcw0KZ0RTLT5TZXRNZW1vcnlTcGFjZUF0dHJpYnV0ZXMoKSksIGFu ZCBub3QgIkFwcGxpY2F0aW9uIFByb2Nlc3NvciIuIEl0J3MNCmdvb2QgdGhhdCB0aGUgcGF0Y2gg cmVtb3ZlcyBzdWNoIGEgY29uZnVzaW5nIGNvbW1lbnQuDQoNCkhvd2V2ZXIsIG90aGVyIHBvdGVu dGlhbCBhYnVzZXMgb2YgdGhlIHNob3J0aGFuZCAiQ1BVIEFQIiByZW1haW4gaW4gdGhlIGNvZGUu IFdvdWxkIHlvdSBjb25zaWRlciBzdWJtaXR0aW5nIGEgc2VwYXJhdGUgcGF0Y2ggdGhhdCByZXBs YWNlcyBhbGwgcmVtYWluaW5nIGluc3RhbmNlcyBvZiAiQ1BVIEFQIiB3aXRoICJDUFUgQXJjaCBQ cm90b2NvbCIsIGluICJNZGVNb2R1bGVQa2cvQ29yZS9QaVNtbUNvcmUvUGlTbW1JcGwuYyI/DQoN Cg0KLi4uIEkgb3JpZ2luYWxseSBpbnRlbmRlZCB0byB0ZXN0IHRoaXMgcGF0Y2ggYXQgb25jZSwg YnV0IGdpdmVuIG15IHF1ZXN0aW9uICg0KSwgeW91IG1pZ2h0IHdhbnQgdG8gbW9kaWZ5IHRoZSBj b2RlIGFzIHdlbGwsIG5vdCBqdXN0IHRoZSBjb21taXQgbWVzc2FnZSwgYW5kIHNvIEkgZmlndXJl ZCBJJ2QgcG9zdHBvbmUgdGhlIHRlc3RpbmcgdW50aWwgeW91ciBhbnN3ZXIuDQoNClRoYW5rcyEN Ckxhc3psbw0KX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18N CmVkazItZGV2ZWwgbWFpbGluZyBsaXN0DQplZGsyLWRldmVsQGxpc3RzLjAxLm9yZw0KaHR0cHM6 Ly9saXN0cy4wMS5vcmcvbWFpbG1hbi9saXN0aW5mby9lZGsyLWRldmVsDQo=