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.88; helo=mga01.intel.com; envelope-from=eric.dong@intel.com; receiver=edk2-devel@lists.01.org Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) (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 5B3B9210C1ECD for ; Wed, 25 Jul 2018 04:17:52 -0700 (PDT) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga008.fm.intel.com ([10.253.24.58]) by fmsmga101.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 25 Jul 2018 04:17:51 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.51,401,1526367600"; d="scan'208";a="57751024" Received: from fmsmsx106.amr.corp.intel.com ([10.18.124.204]) by fmsmga008.fm.intel.com with ESMTP; 25 Jul 2018 04:15:31 -0700 Received: from fmsmsx115.amr.corp.intel.com (10.18.116.19) by FMSMSX106.amr.corp.intel.com (10.18.124.204) with Microsoft SMTP Server (TLS) id 14.3.319.2; Wed, 25 Jul 2018 04:15:31 -0700 Received: from shsmsx103.ccr.corp.intel.com (10.239.4.69) by fmsmsx115.amr.corp.intel.com (10.18.116.19) with Microsoft SMTP Server (TLS) id 14.3.319.2; Wed, 25 Jul 2018 04:15:31 -0700 Received: from shsmsx102.ccr.corp.intel.com ([169.254.2.124]) by SHSMSX103.ccr.corp.intel.com ([169.254.4.100]) with mapi id 14.03.0319.002; Wed, 25 Jul 2018 19:15:29 +0800 From: "Dong, Eric" To: Laszlo Ersek , "edk2-devel@lists.01.org" CC: "Ni, Ruiyu" Thread-Topic: [edk2] [Patch v3 1/3] UefiCpuPkg/MpInitLib: Remove redundant CpuStateFinished State. Thread-Index: AQHUI+xX5hPSyQ/Pk0eksieN7znpvaSfPd2AgACKSJA= Date: Wed, 25 Jul 2018 11:15:29 +0000 Message-ID: References: <20180725075020.240-1-eric.dong@intel.com> <20180725075020.240-2-eric.dong@intel.com> <191141df-1a1a-0228-9055-86fa8982287e@redhat.com> In-Reply-To: <191141df-1a1a-0228-9055-86fa8982287e@redhat.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 v3 1/3] UefiCpuPkg/MpInitLib: Remove redundant CpuStateFinished State. 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, 25 Jul 2018 11:17:52 -0000 Content-Language: en-US Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 SGkgTGFzemxvLA0KDQoNCj4gLS0tLS1PcmlnaW5hbCBNZXNzYWdlLS0tLS0NCj4gRnJvbTogTGFz emxvIEVyc2VrIFttYWlsdG86bGVyc2VrQHJlZGhhdC5jb21dDQo+IFNlbnQ6IFdlZG5lc2RheSwg SnVseSAyNSwgMjAxOCA2OjU1IFBNDQo+IFRvOiBEb25nLCBFcmljIDxlcmljLmRvbmdAaW50ZWwu Y29tPjsgZWRrMi1kZXZlbEBsaXN0cy4wMS5vcmcNCj4gQ2M6IE5pLCBSdWl5dSA8cnVpeXUubmlA aW50ZWwuY29tPg0KPiBTdWJqZWN0OiBSZTogW2VkazJdIFtQYXRjaCB2MyAxLzNdIFVlZmlDcHVQ a2cvTXBJbml0TGliOiBSZW1vdmUgcmVkdW5kYW50DQo+IENwdVN0YXRlRmluaXNoZWQgU3RhdGUu DQo+IA0KPiBIaSBFcmljLA0KPiANCj4gT24gMDcvMjUvMTggMDk6NTAsIEVyaWMgRG9uZyB3cm90 ZToNCj4gPiBDdXJyZW50IENQVSBzdGF0ZSBkZWZpbml0aW9uIGluY2x1ZGUgQ3B1U3RhdGVJZGxl IGFuZCBDcHVTdGF0ZUZpbmlzaGVkLg0KPiA+IEFmdGVyIGludmVzdGlnYXRpb24sIGN1cnJlbnQg Y29kZSBjYW4gdXNlIENwdVN0YXRlSWRsZSB0byByZXBsYWNlIHRoZQ0KPiA+IENwdVN0YXRlRmlu aXNoZWQuIEl0IHdpbGwgcmVkdWNlIHRoZSBzdGF0ZSBudW1iZXIgYW5kIGVhc3kgZm9yIG1haW50 ZW5hbmNlLg0KPiA+DQo+ID4gQ2M6IExhc3psbyBFcnNlayA8bGVyc2VrQHJlZGhhdC5jb20+DQo+ ID4gQ2M6IFJ1aXl1IE5pIDxydWl5dS5uaUBpbnRlbC5jb20+DQo+ID4gQ29udHJpYnV0ZWQtdW5k ZXI6IFRpYW5vQ29yZSBDb250cmlidXRpb24gQWdyZWVtZW50IDEuMQ0KPiA+IFNpZ25lZC1vZmYt Ynk6IEVyaWMgRG9uZyA8ZXJpYy5kb25nQGludGVsLmNvbT4NCj4gPiAtLS0NCj4gPiAgVWVmaUNw dVBrZy9MaWJyYXJ5L01wSW5pdExpYi9NcExpYi5jIHwgMTggKysrKysrKystLS0tLS0tLS0tDQo+ ID4gVWVmaUNwdVBrZy9MaWJyYXJ5L01wSW5pdExpYi9NcExpYi5oIHwgIDEgLQ0KPiA+ICAyIGZp bGVzIGNoYW5nZWQsIDggaW5zZXJ0aW9ucygrKSwgMTEgZGVsZXRpb25zKC0pDQo+IA0KPiBBZnRl ciBsb29raW5nIG92ZXIgdGhpcyBwYXRjaCwgaXQgc2VlbXMgdGhhdCB5b3UgYXJlIHByZXNlcnZp bmcgdGhlDQo+IENwdVN0YXRlUmVhZHkgZW51bSBjb25zdGFudCwgcmVsYXRpdmUgdG86DQo+IA0K PiAgIGh0dHA6Ly9taWQubWFpbC1hcmNoaXZlLmNvbS8yMDE4MDYyODExMjkyMC41Mjk2LTEtZXJp Yy5kb25nQGludGVsLmNvbQ0KPiANCj4gSG93ZXZlciwgYmFzZWQgb24geW91ciBhbmFseXNpcyBp bg0KPiANCj4gICBodHRwOi8vbWlkLm1haWwtDQo+IGFyY2hpdmUuY29tL0VEMDc3OTMwQzI1ODg4 NEJCQ0I0NTBEQjczN0U2NjIyNEFDNUE0NTNAc2hzbXN4MTAyLg0KPiBjY3IuY29ycC5pbnRlbC5j b20NCj4gDQo+IGlzbid0IGl0IHN0aWxsIHBvc3NpYmxlIHRvIHJ1biBpbnRvIHRoZSBleGFjdCBz YW1lIGlzc3VlPyAoTmFtZWx5LCBCU1AgdGhpbmtzIHRoZQ0KPiBBUCBoYXMgZ29uZSB0aHJvdWdo IElkbGUgLT4gQnVzeSAtPiBJZGxlLCBidXQgdGhlIEFQIGhhcyBuZXZlciBhY3R1YWxseSBsZWZ0 DQo+IElkbGU/KQ0KPiANCj4gSG0sIHdhaXQsIGlzIGl0IHRoZSBjYXNlIHRoYXQgdGhlIEJTUCBm aXJzdCBzZXRzIFJlYWR5LCBhbmQgc28gaWYgdGhlIGNoZWNrIGZvciBhbg0KPiBBUCByZXR1cm5z IElkbGUsIGl0IGltcGxpZXMgdGhlIEFQIG11c3QgaGF2ZSBnb25lIHRocm91Z2g6DQo+IA0KPiAg IElkbGUgLS0tLT4gUmVhZHkgLS0tLT4gQnVzeSAtLS0tPiBJZGxlDQo+IA0KPiA/DQoNCkNvcnJl Y3QhIFRoZSBSZWFkeSBzdGF0ZSBpcyBiZWdpbiBzdGF0ZSBhbmQgdGhlIElkbGUgaXMgdGhlIGZp bmlzaCBzdGF0ZS4NCg0KPiANCj4gSWYgdGhpcyBpcyBjb3JyZWN0LCBjYW4geW91IHBsZWFzZSBp bmNsdWRlIHRoZSBmb2xsb3dpbmcgaW4gdGhlIGNvbW1pdA0KPiBtZXNzYWdlOg0KPiANCj4gPiBC ZWZvcmUgdGhpcyBwYXRjaCwgdGhlIHN0YXRlIHRyYW5zaXRpb25zIGZvciBhbiBBUCBhcmU6DQo+ ID4NCj4gPiAgIElkbGUgLS0tLT4gUmVhZHkgLS0tLT4gQnVzeSAtLS0tPiBGaW5pc2hlZCAtLS0t PiBJZGxlDQo+ID4gICAgICAgIFtCU1BdICAgICAgIFtBUF0gICAgICAgW0FQXSAgICAgICAgICAg W0JTUF0NCj4gPg0KPiA+IEFmdGVyIHRoZSBwYXRjaCwgdGhlIHN0YXRlIHRyYW5zaXRpb25zIGZv ciBhbiBBUCBhcmU6DQo+ID4NCj4gPiAgIElkbGUgLS0tLT4gUmVhZHkgLS0tLT4gQnVzeSAtLS0t PiBJZGxlDQo+ID4gICAgICAgIFtCU1BdICAgICAgIFtBUF0gICAgICAgW0FQXQ0KPiANCj4gRG8g eW91IGFncmVlPw0KDQpHb29kIHN1Z2dlc3Rpb24sICBJIHdpbGwgaW5jbHVkZSB0aGlzIGluZm8g aW4gdGhlIGNvbW1pdCBtZXNzYWdlLg0KDQo+IA0KPiBJIGhhdmUgYW5vdGhlciBxdWVzdGlvbjoN Cj4gDQo+IE9uIDA3LzI1LzE4IDA5OjUwLCBFcmljIERvbmcgd3JvdGU6DQo+ID4gZGlmZiAtLWdp dCBhL1VlZmlDcHVQa2cvTGlicmFyeS9NcEluaXRMaWIvTXBMaWIuYw0KPiA+IGIvVWVmaUNwdVBr Zy9MaWJyYXJ5L01wSW5pdExpYi9NcExpYi5jDQo+ID4gaW5kZXggYzgyYjk4NTk0My4uZmYwOWEw ZTllNyAxMDA2NDQNCj4gPiAtLS0gYS9VZWZpQ3B1UGtnL0xpYnJhcnkvTXBJbml0TGliL01wTGli LmMNCj4gPiArKysgYi9VZWZpQ3B1UGtnL0xpYnJhcnkvTXBJbml0TGliL01wTGliLmMNCj4gPiBA QCAtNjk2LDcgKzY5Niw3IEBAIEFwV2FrZXVwRnVuY3Rpb24gKA0KPiA+ICAgICAgICAgICAgICB9 DQo+ID4gICAgICAgICAgICB9DQo+ID4gICAgICAgICAgfQ0KPiA+IC0gICAgICAgIFNldEFwU3Rh dGUgKCZDcHVNcERhdGEtPkNwdURhdGFbUHJvY2Vzc29yTnVtYmVyXSwNCj4gQ3B1U3RhdGVGaW5p c2hlZCk7DQo+ID4gKyAgICAgICAgU2V0QXBTdGF0ZSAoJkNwdU1wRGF0YS0+Q3B1RGF0YVtQcm9j ZXNzb3JOdW1iZXJdLA0KPiA+ICsgQ3B1U3RhdGVJZGxlKTsNCj4gPiAgICAgICAgfQ0KPiA+ICAg ICAgfQ0KPiA+DQo+ID4gQEAgLTEzNTIsMTggKzEzNTIsMTcgQEAgQ2hlY2tUaGlzQVAgKA0KPiA+ ICAgIENwdURhdGEgICA9ICZDcHVNcERhdGEtPkNwdURhdGFbUHJvY2Vzc29yTnVtYmVyXTsNCj4g Pg0KPiA+ICAgIC8vDQo+ID4gLSAgLy8gIENoZWNrIHRoZSBDUFUgc3RhdGUgb2YgQVAuIElmIGl0 IGlzIENwdVN0YXRlRmluaXNoZWQsIHRoZW4gdGhlIEFQIGhhcw0KPiBmaW5pc2hlZCBpdHMgdGFz ay4NCj4gPiArICAvLyAgQ2hlY2sgdGhlIENQVSBzdGF0ZSBvZiBBUC4gSWYgaXQgaXMgQ3B1U3Rh dGVJZGxlLCB0aGVuIHRoZSBBUCBoYXMNCj4gZmluaXNoZWQgaXRzIHRhc2suDQo+ID4gICAgLy8g IE9ubHkgQlNQIGFuZCBjb3JyZXNwb25kaW5nIEFQIGFjY2VzcyB0aGlzIHVuaXQgb2YgQ1BVIERh dGEuDQo+ID4gVGhpcyBtZWFucyB0aGUgQVAgd2lsbCBub3QgbW9kaWZ5IHRoZQ0KPiA+IC0gIC8v ICB2YWx1ZSBvZiBzdGF0ZSBhZnRlciBzZXR0aW5nIHRoZSBpdCB0byBDcHVTdGF0ZUZpbmlzaGVk LCBzbyBCU1AgY2FuIHNhZmVseQ0KPiBtYWtlIHVzZSBvZiBpdHMgdmFsdWUuDQo+ID4gKyAgLy8g IHZhbHVlIG9mIHN0YXRlIGFmdGVyIHNldHRpbmcgdGhlIGl0IHRvIENwdVN0YXRlSWRsZSwgc28g QlNQIGNhbiBzYWZlbHkNCj4gbWFrZSB1c2Ugb2YgaXRzIHZhbHVlLg0KPiA+ICAgIC8vDQo+ID4g ICAgLy8NCj4gPiAgICAvLyBJZiB0aGUgQVAgZmluaXNoZXMgZm9yIFN0YXJ0dXBUaGlzQVAoKSwg cmV0dXJuIEVGSV9TVUNDRVNTLg0KPiA+ICAgIC8vDQo+ID4gLSAgaWYgKEdldEFwU3RhdGUoQ3B1 RGF0YSkgPT0gQ3B1U3RhdGVGaW5pc2hlZCkgew0KPiA+ICsgIGlmIChHZXRBcFN0YXRlKENwdURh dGEpID09IENwdVN0YXRlSWRsZSkgew0KPiA+ICAgICAgaWYgKENwdURhdGEtPkZpbmlzaGVkICE9 IE5VTEwpIHsNCj4gPiAgICAgICAgKihDcHVEYXRhLT5GaW5pc2hlZCkgPSBUUlVFOw0KPiA+ICAg ICAgfQ0KPiA+IC0gICAgU2V0QXBTdGF0ZSAoQ3B1RGF0YSwgQ3B1U3RhdGVJZGxlKTsNCj4gPiAg ICAgIHJldHVybiBFRklfU1VDQ0VTUzsNCj4gPiAgICB9IGVsc2Ugew0KPiA+ICAgICAgLy8NCj4g PiBAQCAtMTQyMCwxNCArMTQxOSwxMyBAQCBDaGVja0FsbEFQcyAoDQo+ID4NCj4gPiAgICAgIENw dURhdGEgPSAmQ3B1TXBEYXRhLT5DcHVEYXRhW1Byb2Nlc3Nvck51bWJlcl07DQo+ID4gICAgICAv Lw0KPiA+IC0gICAgLy8gQ2hlY2sgdGhlIENQVSBzdGF0ZSBvZiBBUC4gSWYgaXQgaXMgQ3B1U3Rh dGVGaW5pc2hlZCwgdGhlbiB0aGUgQVAgaGFzDQo+IGZpbmlzaGVkIGl0cyB0YXNrLg0KPiA+ICsg ICAgLy8gQ2hlY2sgdGhlIENQVSBzdGF0ZSBvZiBBUC4gSWYgaXQgaXMgQ3B1U3RhdGVJZGxlLCB0 aGVuIHRoZSBBUCBoYXMNCj4gZmluaXNoZWQgaXRzIHRhc2suDQo+ID4gICAgICAvLyBPbmx5IEJT UCBhbmQgY29ycmVzcG9uZGluZyBBUCBhY2Nlc3MgdGhpcyB1bml0IG9mIENQVSBEYXRhLiBUaGlz DQo+IG1lYW5zIHRoZSBBUCB3aWxsIG5vdCBtb2RpZnkgdGhlDQo+ID4gLSAgICAvLyB2YWx1ZSBv ZiBzdGF0ZSBhZnRlciBzZXR0aW5nIHRoZSBpdCB0byBDcHVTdGF0ZUZpbmlzaGVkLCBzbyBCU1Ag Y2FuDQo+IHNhZmVseSBtYWtlIHVzZSBvZiBpdHMgdmFsdWUuDQo+ID4gKyAgICAvLyB2YWx1ZSBv ZiBzdGF0ZSBhZnRlciBzZXR0aW5nIHRoZSBpdCB0byBDcHVTdGF0ZUlkbGUsIHNvIEJTUCBjYW4g c2FmZWx5DQo+IG1ha2UgdXNlIG9mIGl0cyB2YWx1ZS4NCj4gPiAgICAgIC8vDQo+ID4gLSAgICBp ZiAoR2V0QXBTdGF0ZShDcHVEYXRhKSA9PSBDcHVTdGF0ZUZpbmlzaGVkKSB7DQo+ID4gKyAgICBp ZiAoR2V0QXBTdGF0ZShDcHVEYXRhKSA9PSBDcHVTdGF0ZUlkbGUpIHsNCj4gPiAgICAgICAgQ3B1 TXBEYXRhLT5SdW5uaW5nQ291bnQgKys7DQo+ID4gICAgICAgIENwdU1wRGF0YS0+Q3B1RGF0YVtQ cm9jZXNzb3JOdW1iZXJdLldhaXRpbmcgPSBGQUxTRTsNCj4gPiAtICAgICAgU2V0QXBTdGF0ZShD cHVEYXRhLCBDcHVTdGF0ZUlkbGUpOw0KPiA+DQo+ID4gICAgICAgIC8vDQo+ID4gICAgICAgIC8v IElmIGluIFNpbmdsZSBUaHJlYWQgbW9kZSwgdGhlbiBzZWFyY2ggZm9yIHRoZSBuZXh0IHdhaXRp bmcgQVAgZm9yDQo+IGV4ZWN1dGlvbi4NCj4gDQo+IFRoaXMgcGFydCBvZiB0aGUgY29kZSwgYWZ0 ZXIgdGhlIHBhdGNoLCBkb2VzIG5vdCBzZWVtIGlkZW1wb3RlbnQ7IGluIG90aGVyDQo+IHdvcmRz LCBpZiB0aGUgQlNQIGNhbGxzIENoZWNrQWxsQVBzKCkgbXVsdGlwbGUgdGltZXMsIHRoZW4gUnVu bmluZ0NvdW50IHdpbGwNCj4gYmUgaW5jcmVhc2VkIGV2ZXJ5IHRpbWUuIEJlZm9yZSB0aGUgcGF0 Y2gsIHRoaXMgd2Fzbid0IHRoZSBjYXNlLCBiZWNhdXNlIGFmdGVyDQo+IHRoZSBGaW5pc2hlZCAt PiBJZGxlIHRyYW5zaXRpb24sIHRoZSBpbmNyZW1lbnQgd291bGRuJ3QgYmUgcmVhY2hlZCBhZ2Fp bi4NCj4gDQo+IEhtbW0sIHdhaXQsIEknbSB3cm9uZzogd2Ugc2V0IHRoZSBXYWl0aW5nIGZpZWxk IHRvIEZBTFNFIGFzIHdlbGwsIHNvIGF0IHRoZQ0KPiBuZXh0IGNhbGwgdG8gQ2hlY2tBbGxBUHMo KSwgd2UnbGwgdGFrZSB0aGUgZWFybHkgImNvbnRpbnVlIiBicmFuY2guDQo+IExvb2tzIE9LIGFm dGVyIGFsbC4NCj4gDQoNClllcywgd2UgaGF2ZSB0d28gZmxhZ3MgaGVyZS4gIFdhaXRpbmcgZmxh Z3MgbWVhbnMgdGhlIEFQIHdpbGwgZG8gdGhlIHRhc2suIFN0YXRlIGZsYWcgbWVhbnMgd2hldGhl ciB0aGUgdGFzayBoYXMgZmluaXNoZWQuDQpCb3RoIGZsYWdzIHdpbGwgYmUgY2hlY2tlZCBhbmQg dXBkYXRlZC4NCg0KPiBJJ2xsIGZvbGxvdyB1cCB3aXRoIHRlc3QgcmVzdWx0cy4NCj4gDQo+IFRo YW5rcywNCj4gTGFzemxvDQo+IA0KPiA+IEBAIC0xOTIzLDcgKzE5MjEsNyBAQCBTd2l0Y2hCU1BX b3JrZXIgKA0KPiA+ICAgIC8vDQo+ID4gICAgLy8gV2FpdCBmb3Igb2xkIEJTUCBmaW5pc2hlZCBB UCB0YXNrDQo+ID4gICAgLy8NCj4gPiAtICB3aGlsZSAoR2V0QXBTdGF0ZSAoJkNwdU1wRGF0YS0+ Q3B1RGF0YVtDYWxsZXJOdW1iZXJdKSAhPQ0KPiA+IENwdVN0YXRlRmluaXNoZWQpIHsNCj4gPiAr ICB3aGlsZSAoR2V0QXBTdGF0ZSAoJkNwdU1wRGF0YS0+Q3B1RGF0YVtDYWxsZXJOdW1iZXJdKSAh PQ0KPiA+ICsgQ3B1U3RhdGVJZGxlKSB7DQo+ID4gICAgICBDcHVQYXVzZSAoKTsNCj4gPiAgICB9 DQo+ID4NCj4gPiBkaWZmIC0tZ2l0IGEvVWVmaUNwdVBrZy9MaWJyYXJ5L01wSW5pdExpYi9NcExp Yi5oDQo+ID4gYi9VZWZpQ3B1UGtnL0xpYnJhcnkvTXBJbml0TGliL01wTGliLmgNCj4gPiBpbmRl eCA5ZDBiODY2ZDA5Li45NjJiY2U2ODVkIDEwMDY0NA0KPiA+IC0tLSBhL1VlZmlDcHVQa2cvTGli cmFyeS9NcEluaXRMaWIvTXBMaWIuaA0KPiA+ICsrKyBiL1VlZmlDcHVQa2cvTGlicmFyeS9NcElu aXRMaWIvTXBMaWIuaA0KPiA+IEBAIC04NSw3ICs4NSw2IEBAIHR5cGVkZWYgZW51bSB7DQo+ID4g ICAgQ3B1U3RhdGVJZGxlLA0KPiA+ICAgIENwdVN0YXRlUmVhZHksDQo+ID4gICAgQ3B1U3RhdGVC dXN5LA0KPiA+IC0gIENwdVN0YXRlRmluaXNoZWQsDQo+ID4gICAgQ3B1U3RhdGVEaXNhYmxlZA0K PiA+ICB9IENQVV9TVEFURTsNCj4gPg0KPiA+DQoNCg==