From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 192.55.52.151, mailfrom: ray.ni@intel.com) Received: from mga17.intel.com (mga17.intel.com [192.55.52.151]) by groups.io with SMTP; Thu, 26 Sep 2019 11:44:23 -0700 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga006.fm.intel.com ([10.253.24.20]) by fmsmga107.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 26 Sep 2019 11:44:23 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.64,552,1559545200"; d="scan'208";a="390825374" Received: from fmsmsx105.amr.corp.intel.com ([10.18.124.203]) by fmsmga006.fm.intel.com with ESMTP; 26 Sep 2019 11:44:23 -0700 Received: from FMSMSX109.amr.corp.intel.com (10.18.116.9) by FMSMSX105.amr.corp.intel.com (10.18.124.203) with Microsoft SMTP Server (TLS) id 14.3.439.0; Thu, 26 Sep 2019 11:44:23 -0700 Received: from shsmsx108.ccr.corp.intel.com (10.239.4.97) by fmsmsx109.amr.corp.intel.com (10.18.116.9) with Microsoft SMTP Server (TLS) id 14.3.439.0; Thu, 26 Sep 2019 11:44:07 -0700 Received: from shsmsx104.ccr.corp.intel.com ([169.254.5.32]) by SHSMSX108.ccr.corp.intel.com ([169.254.8.146]) with mapi id 14.03.0439.000; Fri, 27 Sep 2019 02:44:06 +0800 From: "Ni, Ray" To: "devel@edk2.groups.io" , "lersek@redhat.com" CC: "Dong, Eric" Subject: Re: [edk2-devel] [PATCH 1/2] UefiCpuPkg/PiSmmCpu: Remove hard code when getting physical line size Thread-Topic: [edk2-devel] [PATCH 1/2] UefiCpuPkg/PiSmmCpu: Remove hard code when getting physical line size Thread-Index: AQHVdJNw5cE4lB/dokiOyK7FoZ/XkKc+S2ig Date: Thu, 26 Sep 2019 18:44:05 +0000 Message-ID: <734D49CCEBEEF84792F5B80ED585239D5C2F3890@SHSMSX104.ccr.corp.intel.com> References: <20190926000904.187532-1-ray.ni@intel.com> <20190926000904.187532-2-ray.ni@intel.com> <6230e7c3-f23a-bb53-bebc-aa7fd406d61f@redhat.com> In-Reply-To: <6230e7c3-f23a-bb53-bebc-aa7fd406d61f@redhat.com> Accept-Language: en-US, zh-CN X-MS-Has-Attach: X-MS-TNEF-Correlator: x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiNjU5NGNjY2UtOTNiZi00ZTY3LWJiYjItMDkwNWY2YTM2NTdiIiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE3LjEwLjE4MDQuNDkiLCJUcnVzdGVkTGFiZWxIYXNoIjoiQzF5QmgwUStESVY4dmQyXC9VTXpMU0hSUU1SN204bGZsSkRcL25VNWd5UkdEcHJLWTh2U1B2QmpqVHBPTVMyS09xIn0= x-ctpclassification: CTP_NT dlp-product: dlpe-windows dlp-version: 11.2.0.6 dlp-reaction: no-action x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Return-Path: ray.ni@intel.com Content-Language: en-US Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 TGFzemxvLA0KSSBhZ3JlZSB3aXRoIHlvdXIgY29tbWVudHMuDQpJIHdpbGw6DQoxLiBzZXBhcmF0 ZSB0aGUgcGF0Y2ggaW50byAyDQoyLiByZW1vdmUgdGhlIHVubmVlZGVkICJlbHNlIiBhZnRlciBn ZXR0aW5nIGZyb20gSE9CLg0KDQpUaGFua3MsDQpSYXkNCg0KPiAtLS0tLU9yaWdpbmFsIE1lc3Nh Z2UtLS0tLQ0KPiBGcm9tOiBkZXZlbEBlZGsyLmdyb3Vwcy5pbyA8ZGV2ZWxAZWRrMi5ncm91cHMu aW8+IE9uIEJlaGFsZiBPZiBMYXN6bG8gRXJzZWsNCj4gU2VudDogVGh1cnNkYXksIFNlcHRlbWJl ciAyNiwgMjAxOSAxMDo1NCBBTQ0KPiBUbzogTmksIFJheSA8cmF5Lm5pQGludGVsLmNvbT47IGRl dmVsQGVkazIuZ3JvdXBzLmlvDQo+IENjOiBEb25nLCBFcmljIDxlcmljLmRvbmdAaW50ZWwuY29t Pg0KPiBTdWJqZWN0OiBSZTogW2VkazItZGV2ZWxdIFtQQVRDSCAxLzJdIFVlZmlDcHVQa2cvUGlT bW1DcHU6IFJlbW92ZSBoYXJkIGNvZGUgd2hlbiBnZXR0aW5nIHBoeXNpY2FsIGxpbmUgc2l6ZQ0K PiANCj4gSGkgUmF5LA0KPiANCj4gT24gMDkvMjYvMTkgMDI6MDksIFJheSBOaSB3cm90ZToNCj4g PiBUaGUgY29kZSByZXBsYWNlcyB0aGUgaGFyZCBjb2RlIHdpdGggbWFjcm9zIGRlZmluZWQgaW4N Cj4gPiBNZGVQa2dcSW5jbHVkZVxSZWdpc3RlclxJbnRlbFxDcHVJZC5oLg0KPiA+DQo+ID4gTm8g ZnVuY3Rpb25hbGl0eSBpbXBhY3QuDQo+ID4NCj4gPiBTaWduZWQtb2ZmLWJ5OiBSYXkgTmkgPHJh eS5uaUBpbnRlbC5jb20+DQo+ID4gQ2M6IEVyaWMgRG9uZyA8ZXJpYy5kb25nQGludGVsLmNvbT4N Cj4gPiBDYzogTGFzemxvIEVyc2VrIDxsZXJzZWtAcmVkaGF0LmNvbT4NCj4gPiAtLS0NCj4gPiAg VWVmaUNwdVBrZy9QaVNtbUNwdUR4ZVNtbS9YNjQvUGFnZVRibC5jIHwgMjQgKysrKysrKysrKyst LS0tLS0tLS0tLS0tDQo+ID4gIDEgZmlsZSBjaGFuZ2VkLCAxMSBpbnNlcnRpb25zKCspLCAxMyBk ZWxldGlvbnMoLSkNCj4gPg0KPiA+IGRpZmYgLS1naXQgYS9VZWZpQ3B1UGtnL1BpU21tQ3B1RHhl U21tL1g2NC9QYWdlVGJsLmMgYi9VZWZpQ3B1UGtnL1BpU21tQ3B1RHhlU21tL1g2NC9QYWdlVGJs LmMNCj4gPiBpbmRleCBlNWM0Nzg4YzEzLi5iOGU5NWJmNmVkIDEwMDY0NA0KPiA+IC0tLSBhL1Vl ZmlDcHVQa2cvUGlTbW1DcHVEeGVTbW0vWDY0L1BhZ2VUYmwuYw0KPiA+ICsrKyBiL1VlZmlDcHVQ a2cvUGlTbW1DcHVEeGVTbW0vWDY0L1BhZ2VUYmwuYw0KPiA+IEBAIC0xNTEsMzAgKzE1MSwyOCBA QCBHZXRTdWJFbnRyaWVzTnVtICgNCj4gPiAgICBAcmV0dXJuIHRoZSBtYXhpbXVtIHN1cHBvcnQg YWRkcmVzcy4NCj4gPiAgKiovDQo+ID4gIFVJTlQ4DQo+ID4gLUNhbGN1bGF0ZU1heGltdW1TdXBw b3J0QWRkcmVzcyAoDQo+ID4gK0dldFBoeXNpY2FsQWRkcmVzc0JpdHMgKA0KPiA+ICAgIFZPSUQN Cj4gPiAgICApDQo+ID4gIHsNCj4gPiAtICBVSU5UMzIgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgUmVnRWF4Ow0KPiA+IC0gIFVJTlQ4ICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICBQaHlzaWNhbEFkZHJlc3NCaXRzOw0KPiA+IC0gIFZPSUQgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAqSG9iOw0KPiA+ICsgIENQVUlE X1ZJUl9QSFlfQUREUkVTU19TSVpFX0VBWCAgICAgICAgICAgICAgVmlyUGh5QWRkcmVzc1NpemU7 DQo+ID4gKyAgVUlOVDMyICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICBNYXhF eHRlbmRlZEZ1bmN0aW9uSWQ7DQo+ID4gKyAgVk9JRCAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAqSG9iOw0KPiA+DQo+ID4gICAgLy8NCj4gPiAgICAvLyBHZXQgcGh5c2lj YWwgYWRkcmVzcyBiaXRzIHN1cHBvcnRlZC4NCj4gPiAgICAvLw0KPiA+ICAgIEhvYiA9IEdldEZp cnN0SG9iIChFRklfSE9CX1RZUEVfQ1BVKTsNCj4gPiAgICBpZiAoSG9iICE9IE5VTEwpIHsNCj4g PiAtICAgIFBoeXNpY2FsQWRkcmVzc0JpdHMgPSAoKEVGSV9IT0JfQ1BVICopIEhvYiktPlNpemVP Zk1lbW9yeVNwYWNlOw0KPiA+ICsgICAgcmV0dXJuICgoRUZJX0hPQl9DUFUgKikgSG9iKS0+U2l6 ZU9mTWVtb3J5U3BhY2U7DQo+ID4gICAgfSBlbHNlIHsNCj4gPiAtICAgIEFzbUNwdWlkICgweDgw MDAwMDAwLCAmUmVnRWF4LCBOVUxMLCBOVUxMLCBOVUxMKTsNCj4gPiAtICAgIGlmIChSZWdFYXgg Pj0gMHg4MDAwMDAwOCkgew0KPiA+IC0gICAgICBBc21DcHVpZCAoMHg4MDAwMDAwOCwgJlJlZ0Vh eCwgTlVMTCwgTlVMTCwgTlVMTCk7DQo+ID4gLSAgICAgIFBoeXNpY2FsQWRkcmVzc0JpdHMgPSAo VUlOVDgpIFJlZ0VheDsNCj4gPiAtICAgIH0gZWxzZSB7DQo+ID4gLSAgICAgIFBoeXNpY2FsQWRk cmVzc0JpdHMgPSAzNjsNCj4gPiArICAgIEFzbUNwdWlkIChDUFVJRF9FWFRFTkRFRF9GVU5DVElP TiwgJk1heEV4dGVuZGVkRnVuY3Rpb25JZCwgTlVMTCwgTlVMTCwgTlVMTCk7DQo+ID4gKyAgICBp ZiAoTWF4RXh0ZW5kZWRGdW5jdGlvbklkIDwgQ1BVSURfVklSX1BIWV9BRERSRVNTX1NJWkUpIHsN Cj4gPiArICAgICAgcmV0dXJuIDM2Ow0KPiA+ICAgICAgfQ0KPiA+ICsgICAgQXNtQ3B1aWQgKENQ VUlEX1ZJUl9QSFlfQUREUkVTU19TSVpFLCAmVmlyUGh5QWRkcmVzc1NpemUuVWludDMyLCBOVUxM LCBOVUxMLCBOVUxMKTsNCj4gPiArICAgIHJldHVybiAoVUlOVDgpIFZpclBoeUFkZHJlc3NTaXpl LkJpdHMuUGh5c2ljYWxBZGRyZXNzQml0czsNCj4gPiAgICB9DQo+ID4gLSAgcmV0dXJuIFBoeXNp Y2FsQWRkcmVzc0JpdHM7DQo+ID4gIH0NCj4gDQo+IEkgd291bGQgcHJlZmVyIGlmIHlvdSBzZXBh cmF0ZWQNCj4gLSB0aGUgcmVwbGFjZW1lbnQgb2YgdGhlIG1hZ2ljIGNvbnN0YW50cyB3aXRoIG1h Y3JvcywNCj4gLSBmcm9tIHJlb3JnYW5pemluZyB0aGUgY29udHJvbCBmbG93Lg0KPiANCj4gRXZl biBpZiB3ZSBrZWVwIGJvdGggY2hhbmdlcyBpbiB0aGUgc2FtZSBwYXRjaCwgdGhlIHJlc3VsdGFu dCBjb250cm9sDQo+IGZsb3cgaXMgbm90IG9wdGltYWwuIFdoZXJlIHlvdSByZXR1cm4gU2l6ZU9m TWVtb3J5U3BhY2UsIHRoZXJlIHNob3VsZCBiZQ0KPiBubyAiZWxzZSIgYnJhbmNoIGFmdGVyIC0t IHRoZSByZXN0IG9mIHRoZSBjb2RlIHNob3VsZCBiZSB1bi1pbmRlbnRlZCBieQ0KPiBvbmUgbGV2 ZWwsIGluc3RlYWQuDQo+IA0KPiBUaGFua3MNCj4gTGFzemxvDQo+IA0KPiA+DQo+ID4gIC8qKg0K PiA+IEBAIC0zNTQsNyArMzUyLDcgQEAgU21tSW5pdFBhZ2VUYWJsZSAoDQo+ID4gICAgbUNwdVNt bVJlc3RyaWN0ZWRNZW1vcnlBY2Nlc3MgPSBQY2RHZXRCb29sIChQY2RDcHVTbW1SZXN0cmljdGVk TWVtb3J5QWNjZXNzKTsNCj4gPiAgICBtMUdQYWdlVGFibGVTdXBwb3J0ICAgICAgICAgICA9IElz MUdQYWdlU3VwcG9ydCAoKTsNCj4gPiAgICBtNUxldmVsUGFnaW5nTmVlZGVkICAgICAgICAgICA9 IElzNUxldmVsUGFnaW5nTmVlZGVkICgpOw0KPiA+IC0gIG1QaHlzaWNhbEFkZHJlc3NCaXRzICAg ICAgICAgID0gQ2FsY3VsYXRlTWF4aW11bVN1cHBvcnRBZGRyZXNzICgpOw0KPiA+ICsgIG1QaHlz aWNhbEFkZHJlc3NCaXRzICAgICAgICAgID0gR2V0UGh5c2ljYWxBZGRyZXNzQml0cyAoKTsNCj4g PiAgICBQYXRjaEluc3RydWN0aW9uWDg2IChnUGF0Y2g1TGV2ZWxQYWdpbmdOZWVkZWQsIG01TGV2 ZWxQYWdpbmdOZWVkZWQsIDEpOw0KPiA+ICAgIERFQlVHICgoREVCVUdfSU5GTywgIjVMZXZlbFBh Z2luZyBOZWVkZWQgICAgICAgICAgICAgLSAlZFxuIiwgbTVMZXZlbFBhZ2luZ05lZWRlZCkpOw0K PiA+ICAgIERFQlVHICgoREVCVUdfSU5GTywgIjFHUGFnZVRhYmxlIFN1cHBvcnQgICAgICAgICAg ICAgLSAlZFxuIiwgbTFHUGFnZVRhYmxlU3VwcG9ydCkpOw0KPiA+DQo+IA0KPiANCj4gDQoNCg==