From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=134.134.136.24; helo=mga09.intel.com; envelope-from=eric.dong@intel.com; receiver=edk2-devel@lists.01.org Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) (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 2C51C210DC1BB for ; Wed, 8 Aug 2018 00:39:38 -0700 (PDT) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga004.jf.intel.com ([10.7.209.38]) by orsmga102.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 08 Aug 2018 00:39:38 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.51,456,1526367600"; d="scan'208";a="222887143" Received: from fmsmsx105.amr.corp.intel.com ([10.18.124.203]) by orsmga004.jf.intel.com with ESMTP; 08 Aug 2018 00:39:38 -0700 Received: from fmsmsx154.amr.corp.intel.com (10.18.116.70) by FMSMSX105.amr.corp.intel.com (10.18.124.203) with Microsoft SMTP Server (TLS) id 14.3.319.2; Wed, 8 Aug 2018 00:39:37 -0700 Received: from shsmsx104.ccr.corp.intel.com (10.239.4.70) by FMSMSX154.amr.corp.intel.com (10.18.116.70) with Microsoft SMTP Server (TLS) id 14.3.319.2; Wed, 8 Aug 2018 00:39:37 -0700 Received: from shsmsx102.ccr.corp.intel.com ([169.254.2.226]) by SHSMSX104.ccr.corp.intel.com ([169.254.5.143]) with mapi id 14.03.0319.002; Wed, 8 Aug 2018 15:39:35 +0800 From: "Dong, Eric" To: "Dong, Eric" , "edk2-devel@lists.01.org" CC: Laszlo Ersek , "Ni, Ruiyu" Thread-Topic: [edk2] [Patch 2/2] UefiCpuPkg/CpuS3DataDxe: Change Memory Type and address limitation. Thread-Index: AQHULupY/HMoTseOeE+hO/5tywz73aS1eAug Date: Wed, 8 Aug 2018 07:39:35 +0000 Message-ID: References: <20180808073427.11008-1-eric.dong@intel.com> <20180808073427.11008-3-eric.dong@intel.com> In-Reply-To: <20180808073427.11008-3-eric.dong@intel.com> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Subject: Re: [Patch 2/2] UefiCpuPkg/CpuS3DataDxe: Change Memory Type and address limitation. X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.27 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 08 Aug 2018 07:39:39 -0000 Content-Language: en-US Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 UGxlYXNlIGlnbm9yZSB0aGlzIGNoYW5nZSB3aGljaCBmb3Jnb3QgdG8gY2hhbmdlIHZlcnNpb24g dG8gdjIuDQoNCj4gLS0tLS1PcmlnaW5hbCBNZXNzYWdlLS0tLS0NCj4gRnJvbTogZWRrMi1kZXZl bCBbbWFpbHRvOmVkazItZGV2ZWwtYm91bmNlc0BsaXN0cy4wMS5vcmddIE9uIEJlaGFsZiBPZiBF cmljDQo+IERvbmcNCj4gU2VudDogV2VkbmVzZGF5LCBBdWd1c3QgOCwgMjAxOCAzOjM0IFBNDQo+ IFRvOiBlZGsyLWRldmVsQGxpc3RzLjAxLm9yZw0KPiBDYzogTGFzemxvIEVyc2VrIDxsZXJzZWtA cmVkaGF0LmNvbT47IE5pLCBSdWl5dSA8cnVpeXUubmlAaW50ZWwuY29tPg0KPiBTdWJqZWN0OiBb ZWRrMl0gW1BhdGNoIDIvMl0gVWVmaUNwdVBrZy9DcHVTM0RhdGFEeGU6IENoYW5nZSBNZW1vcnkg VHlwZQ0KPiBhbmQgYWRkcmVzcyBsaW1pdGF0aW9uLg0KPiANCj4gQmVjYXVzZSBDcHVTM0RhdGEg bWVtb3J5IHdpbGwgYmUgY29weSB0byBzbXJhbSBhdCBTbW1SZWFkVG9Mb2NrIHBvaW50LA0KPiBz byB0aGUgbWVtb3J5IHR5cGUgbm8gbmVlZCB0byBiZSBBQ1BJIE5WUyB0eXBlLCBhbHNvIHRoZSBh ZGRyZXNzIG5vdCBsaW1pdCB0bw0KPiBiZWxvdyA0Ry4NCj4gVGhpcyBjaGFuZ2UgcmVtb3ZlIHRo ZSBsaW1pdCBvZiBBQ1BJIE5WUyBtZW1vcnkgdHlwZSBhbmQgYmVsb3cgNEcuDQo+IA0KPiBQYXNz IE9TIGJvb3QgYW5kIHJlc3VtZSBmcm9tIFMzIHRlc3QuDQo+IA0KPiBCdWd6OiBodHRwczovL2J1 Z3ppbGxhLnRpYW5vY29yZS5vcmcvc2hvd19idWcuY2dpP2lkPTk1OQ0KPiANCj4gUmVwb3J0ZWQt Ynk6IE1hcnZpbiBIw6R1c2VyIDxNYXJ2aW4uSGFldXNlckBvdXRsb29rLmNvbT4NCj4gU3VnZ2Vz dGVkLWJ5OiBGYW4gSmVmZiA8dmFuamVmZl85MTlAaG90bWFpbC5jb20+DQo+IENjOiBNYXJ2aW4g SMOkdXNlciA8TWFydmluLkhhZXVzZXJAb3V0bG9vay5jb20+DQo+IENjOiBGYW4gSmVmZiA8dmFu amVmZl85MTlAaG90bWFpbC5jb20+DQo+IENjOiBMYXN6bG8gRXJzZWsgPGxlcnNla0ByZWRoYXQu Y29tPg0KPiBDYzogUnVpeXUgTmkgPHJ1aXl1Lm5pQGludGVsLmNvbT4NCj4gQ29udHJpYnV0ZWQt dW5kZXI6IFRpYW5vQ29yZSBDb250cmlidXRpb24gQWdyZWVtZW50IDEuMQ0KPiBTaWduZWQtb2Zm LWJ5OiBFcmljIERvbmcgPGVyaWMuZG9uZ0BpbnRlbC5jb20+DQo+IC0tLQ0KPiAgVWVmaUNwdVBr Zy9DcHVTM0RhdGFEeGUvQ3B1UzNEYXRhLmMgICAgICB8IDYwICsrKysrKystLS0tLS0tLS0tLS0t LS0tLS0tLS0tLS0tDQo+ICBVZWZpQ3B1UGtnL0NwdVMzRGF0YUR4ZS9DcHVTM0RhdGFEeGUuaW5m IHwgIDEgKw0KPiAgMiBmaWxlcyBjaGFuZ2VkLCAxNCBpbnNlcnRpb25zKCspLCA0NyBkZWxldGlv bnMoLSkNCj4gDQo+IGRpZmYgLS1naXQgYS9VZWZpQ3B1UGtnL0NwdVMzRGF0YUR4ZS9DcHVTM0Rh dGEuYw0KPiBiL1VlZmlDcHVQa2cvQ3B1UzNEYXRhRHhlL0NwdVMzRGF0YS5jDQo+IGluZGV4IGRj Y2I0MDZiOGQuLmQ4ZWI4Yzk3NmYgMTAwNjQ0DQo+IC0tLSBhL1VlZmlDcHVQa2cvQ3B1UzNEYXRh RHhlL0NwdVMzRGF0YS5jDQo+ICsrKyBiL1VlZmlDcHVQa2cvQ3B1UzNEYXRhRHhlL0NwdVMzRGF0 YS5jDQo+IEBAIC0zMSw2ICszMSw3IEBAIFdJVEhPVVQgV0FSUkFOVElFUyBPUiBSRVBSRVNFTlRB VElPTlMgT0YgQU5ZDQo+IEtJTkQsIEVJVEhFUiBFWFBSRVNTIE9SIElNUExJRUQuDQo+ICAjaW5j bHVkZSA8TGlicmFyeS9VZWZpQm9vdFNlcnZpY2VzVGFibGVMaWIuaD4NCj4gICNpbmNsdWRlIDxM aWJyYXJ5L0RlYnVnTGliLmg+DQo+ICAjaW5jbHVkZSA8TGlicmFyeS9NdHJyTGliLmg+DQo+ICsj aW5jbHVkZSA8TGlicmFyeS9NZW1vcnlBbGxvY2F0aW9uTGliLmg+DQo+IA0KPiAgI2luY2x1ZGUg PFByb3RvY29sL01wU2VydmljZS5oPg0KPiAgI2luY2x1ZGUgPEd1aWQvRXZlbnRHcm91cC5oPg0K PiBAQCAtNDUsNDIgKzQ2LDYgQEAgdHlwZWRlZiBzdHJ1Y3Qgew0KPiAgICBJQTMyX0RFU0NSSVBU T1IgICAgICAgICAgIElkdHJQcm9maWxlOw0KPiAgfSBBQ1BJX0NQVV9EQVRBX0VYOw0KPiANCj4g LS8qKg0KPiAtICBBbGxvY2F0ZSBFZmlBQ1BJTWVtb3J5TlZTIGJlbG93IDRHIG1lbW9yeSBhZGRy ZXNzLg0KPiAtDQo+IC0gIFRoaXMgZnVuY3Rpb24gYWxsb2NhdGVzIEVmaUFDUElNZW1vcnlOVlMg YmVsb3cgNEcgbWVtb3J5IGFkZHJlc3MuDQo+IC0NCj4gLSAgQHBhcmFtW2luXSBTaXplICAgU2l6 ZSBvZiBtZW1vcnkgdG8gYWxsb2NhdGUuDQo+IC0NCj4gLSAgQHJldHVybiAgICAgICBBbGxvY2F0 ZWQgYWRkcmVzcyBmb3Igb3V0cHV0Lg0KPiAtDQo+IC0qKi8NCj4gLVZPSUQgKg0KPiAtQWxsb2Nh dGVBY3BpTnZzTWVtb3J5QmVsb3c0RyAoDQo+IC0gIElOIFVJTlROICBTaXplDQo+IC0gICkNCj4g LXsNCj4gLSAgRUZJX1BIWVNJQ0FMX0FERFJFU1MgIEFkZHJlc3M7DQo+IC0gIEVGSV9TVEFUVVMg ICAgICAgICAgICBTdGF0dXM7DQo+IC0gIFZPSUQgICAgICAgICAgICAgICAgICAqQnVmZmVyOw0K PiAtDQo+IC0gIEFkZHJlc3MgPSBCQVNFXzRHQiAtIDE7DQo+IC0gIFN0YXR1cyAgPSBnQlMtPkFs bG9jYXRlUGFnZXMgKA0KPiAtICAgICAgICAgICAgICAgICAgIEFsbG9jYXRlTWF4QWRkcmVzcywN Cj4gLSAgICAgICAgICAgICAgICAgICBFZmlBQ1BJTWVtb3J5TlZTLA0KPiAtICAgICAgICAgICAg ICAgICAgIEVGSV9TSVpFX1RPX1BBR0VTIChTaXplKSwNCj4gLSAgICAgICAgICAgICAgICAgICAm QWRkcmVzcw0KPiAtICAgICAgICAgICAgICAgICAgICk7DQo+IC0gIGlmIChFRklfRVJST1IgKFN0 YXR1cykpIHsNCj4gLSAgICByZXR1cm4gTlVMTDsNCj4gLSAgfQ0KPiAtDQo+IC0gIEJ1ZmZlciA9 IChWT0lEICopKFVJTlROKUFkZHJlc3M7DQo+IC0gIFplcm9NZW0gKEJ1ZmZlciwgU2l6ZSk7DQo+ IC0NCj4gLSAgcmV0dXJuIEJ1ZmZlcjsNCj4gLX0NCj4gLQ0KPiAgLyoqDQo+ICAgIENhbGxiYWNr IGZ1bmN0aW9uIGV4ZWN1dGVkIHdoZW4gdGhlIEVuZE9mRHhlIGV2ZW50IGdyb3VwIGlzIHNpZ25h bGVkLg0KPiANCj4gQEAgLTE1MCw3ICsxMTUsNiBAQCBDcHVTM0RhdGFJbml0aWFsaXplICgNCj4g ICAgRUZJX01QX1NFUlZJQ0VTX1BST1RPQ09MICAgKk1wU2VydmljZXM7DQo+ICAgIFVJTlROICAg ICAgICAgICAgICAgICAgICAgIE51bWJlck9mQ3B1czsNCj4gICAgVUlOVE4gICAgICAgICAgICAg ICAgICAgICAgTnVtYmVyT2ZFbmFibGVkUHJvY2Vzc29yczsNCj4gLSAgVk9JRCAgICAgICAgICAg ICAgICAgICAgICAgKlN0YWNrOw0KPiAgICBVSU5UTiAgICAgICAgICAgICAgICAgICAgICBUYWJs ZVNpemU7DQo+ICAgIENQVV9SRUdJU1RFUl9UQUJMRSAgICAgICAgICpSZWdpc3RlclRhYmxlOw0K PiAgICBVSU5UTiAgICAgICAgICAgICAgICAgICAgICBJbmRleDsNCj4gQEAgLTE3MSwxMCArMTM1 LDcgQEAgQ3B1UzNEYXRhSW5pdGlhbGl6ZSAoDQo+ICAgIC8vDQo+ICAgIE9sZEFjcGlDcHVEYXRh ID0gKEFDUElfQ1BVX0RBVEEgKikgKFVJTlROKSBQY2RHZXQ2NA0KPiAoUGNkQ3B1UzNEYXRhQWRk cmVzcyk7DQo+IA0KPiAtICAvLw0KPiAtICAvLyBBbGxvY2F0ZSBBQ1BJIE5WUyBtZW1vcnkgYmVs b3cgNEcgbWVtb3J5IGZvciB1c2Ugb24gQUNQSSBTMyByZXN1bWUuDQo+IC0gIC8vDQo+IC0gIEFj cGlDcHVEYXRhRXggPSBBbGxvY2F0ZUFjcGlOdnNNZW1vcnlCZWxvdzRHIChzaXplb2YNCj4gKEFD UElfQ1BVX0RBVEFfRVgpKTsNCj4gKyAgQWNwaUNwdURhdGFFeCA9IEFsbG9jYXRlUGFnZXMgKEVG SV9TSVpFX1RPX1BBR0VTIChzaXplb2YNCj4gKyAoQUNQSV9DUFVfREFUQV9FWCkpKTsNCj4gICAg QVNTRVJUIChBY3BpQ3B1RGF0YUV4ICE9IE5VTEwpOw0KPiAgICBBY3BpQ3B1RGF0YSA9ICZBY3Bp Q3B1RGF0YUV4LT5BY3BpQ3B1RGF0YTsNCj4gDQo+IEBAIC0yMTAsMTEgKzE3MSwxNiBAQCBDcHVT M0RhdGFJbml0aWFsaXplICgNCj4gICAgQWNwaUNwdURhdGEtPk10cnJUYWJsZSAgICA9DQo+IChF RklfUEhZU0lDQUxfQUREUkVTUykoVUlOVE4pJkFjcGlDcHVEYXRhRXgtPk10cnJUYWJsZTsNCj4g DQo+ICAgIC8vDQo+IC0gIC8vIEFsbG9jYXRlIHN0YWNrIHNwYWNlIGZvciBhbGwgQ1BVcw0KPiAr ICAvLyBBbGxvY2F0ZSBzdGFjayBzcGFjZSBmb3IgYWxsIENQVXMsIHVzZSBBQ1BJIE5WUyBtZW1v cnkgdHlwZQ0KPiArIGJlY2F1c2UgaXQgd2lsbCAgLy8gbm90IGNvcHkgdG8gc21yYW0gYXQgU21t IHJlYWR5IHRvIGxvY2sgcG9pbnQuDQo+ICAgIC8vDQo+IC0gIFN0YWNrID0gQWxsb2NhdGVBY3Bp TnZzTWVtb3J5QmVsb3c0RyAoTnVtYmVyT2ZDcHVzICogQWNwaUNwdURhdGEtDQo+ID5TdGFja1Np emUpOw0KPiAtICBBU1NFUlQgKFN0YWNrICE9IE5VTEwpOw0KPiAtICBBY3BpQ3B1RGF0YS0+U3Rh Y2tBZGRyZXNzID0gKEVGSV9QSFlTSUNBTF9BRERSRVNTKShVSU5UTilTdGFjazsNCj4gKyAgU3Rh dHVzICA9IGdCUy0+QWxsb2NhdGVQYWdlcyAoDQo+ICsgICAgICAgICAgICAgICAgICAgQWxsb2Nh dGVBbnlQYWdlcywNCj4gKyAgICAgICAgICAgICAgICAgICBFZmlBQ1BJTWVtb3J5TlZTLA0KPiAr ICAgICAgICAgICAgICAgICAgIEVGSV9TSVpFX1RPX1BBR0VTIChOdW1iZXJPZkNwdXMgKiBBY3Bp Q3B1RGF0YS0+U3RhY2tTaXplKSwNCj4gKyAgICAgICAgICAgICAgICAgICAmQWNwaUNwdURhdGEt PlN0YWNrQWRkcmVzcw0KPiArICAgICAgICAgICAgICAgICAgICk7DQo+ICsgIEFTU0VSVF9FRklf RVJST1IgKFN0YXR1cyk7DQo+IA0KPiAgICAvLw0KPiAgICAvLyBHZXQgdGhlIGJvb3QgcHJvY2Vz c29yJ3MgR0RUIGFuZCBJRFQgQEAgLTIyNyw3ICsxOTMsNyBAQA0KPiBDcHVTM0RhdGFJbml0aWFs aXplICgNCj4gICAgLy8NCj4gICAgR2R0U2l6ZSA9IEFjcGlDcHVEYXRhRXgtPkdkdHJQcm9maWxl LkxpbWl0ICsgMTsNCj4gICAgSWR0U2l6ZSA9IEFjcGlDcHVEYXRhRXgtPklkdHJQcm9maWxlLkxp bWl0ICsgMTsNCj4gLSAgR2R0ID0gQWxsb2NhdGVBY3BpTnZzTWVtb3J5QmVsb3c0RyAoR2R0U2l6 ZSArIElkdFNpemUpOw0KPiArICBHZHQgPSBBbGxvY2F0ZVBhZ2VzIChFRklfU0laRV9UT19QQUdF UyAoR2R0U2l6ZSArIElkdFNpemUpKTsNCj4gICAgQVNTRVJUIChHZHQgIT0gTlVMTCk7DQo+ICAg IElkdCA9IChWT0lEICopKChVSU5UTilHZHQgKyBHZHRTaXplKTsNCj4gICAgQ29weU1lbSAoR2R0 LCAoVk9JRCAqKUFjcGlDcHVEYXRhRXgtPkdkdHJQcm9maWxlLkJhc2UsIEdkdFNpemUpOyBAQCAt DQo+IDI0Myw3ICsyMDksNyBAQCBDcHVTM0RhdGFJbml0aWFsaXplICgNCj4gICAgICAvLyBBbGxv Y2F0ZSBidWZmZXIgZm9yIGVtcHR5IFJlZ2lzdGVyVGFibGUgYW5kIFByZVNtbUluaXRSZWdpc3Rl clRhYmxlDQo+IGZvciBhbGwgQ1BVcw0KPiAgICAgIC8vDQo+ICAgICAgVGFibGVTaXplID0gMiAq IE51bWJlck9mQ3B1cyAqIHNpemVvZiAoQ1BVX1JFR0lTVEVSX1RBQkxFKTsNCj4gLSAgICBSZWdp c3RlclRhYmxlID0gKENQVV9SRUdJU1RFUl9UQUJMRQ0KPiAqKUFsbG9jYXRlQWNwaU52c01lbW9y eUJlbG93NEcgKFRhYmxlU2l6ZSk7DQo+ICsgICAgUmVnaXN0ZXJUYWJsZSA9IEFsbG9jYXRlUGFn ZXMgKEVGSV9TSVpFX1RPX1BBR0VTIChUYWJsZVNpemUpKTsNCj4gICAgICBBU1NFUlQgKFJlZ2lz dGVyVGFibGUgIT0gTlVMTCk7DQo+IA0KPiAgICAgIGZvciAoSW5kZXggPSAwOyBJbmRleCA8IE51 bWJlck9mQ3B1czsgSW5kZXgrKykgeyBkaWZmIC0tZ2l0DQo+IGEvVWVmaUNwdVBrZy9DcHVTM0Rh dGFEeGUvQ3B1UzNEYXRhRHhlLmluZg0KPiBiL1VlZmlDcHVQa2cvQ3B1UzNEYXRhRHhlL0NwdVMz RGF0YUR4ZS5pbmYNCj4gaW5kZXggNDgwYzk4ZWJjZC4uYzE2NzMxNTI5YyAxMDA2NDQNCj4gLS0t IGEvVWVmaUNwdVBrZy9DcHVTM0RhdGFEeGUvQ3B1UzNEYXRhRHhlLmluZg0KPiArKysgYi9VZWZp Q3B1UGtnL0NwdVMzRGF0YUR4ZS9DcHVTM0RhdGFEeGUuaW5mDQo+IEBAIC01MSw2ICs1MSw3IEBA DQo+ICAgIERlYnVnTGliDQo+ICAgIEJhc2VMaWINCj4gICAgTXRyckxpYg0KPiArICBNZW1vcnlB bGxvY2F0aW9uTGliDQo+IA0KPiAgW0d1aWRzXQ0KPiAgICBnRWZpRW5kT2ZEeGVFdmVudEdyb3Vw R3VpZCAgICAgICAgICMjIENPTlNVTUVTICAgIyMgRXZlbnQNCj4gLS0NCj4gMi4xNS4wLndpbmRv d3MuMQ0KPiANCj4gX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X18NCj4gZWRrMi1kZXZlbCBtYWlsaW5nIGxpc3QNCj4gZWRrMi1kZXZlbEBsaXN0cy4wMS5vcmcN Cj4gaHR0cHM6Ly9saXN0cy4wMS5vcmcvbWFpbG1hbi9saXN0aW5mby9lZGsyLWRldmVsDQo=