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=jian.j.wang@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 1C02E2117CE8C for ; Mon, 22 Oct 2018 00:23:48 -0700 (PDT) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga003.jf.intel.com ([10.7.209.27]) by fmsmga104.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 22 Oct 2018 00:23:48 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.54,411,1534834800"; d="scan'208";a="93938842" Received: from fmsmsx103.amr.corp.intel.com ([10.18.124.201]) by orsmga003.jf.intel.com with ESMTP; 22 Oct 2018 00:23:47 -0700 Received: from fmsmsx122.amr.corp.intel.com (10.18.125.37) by FMSMSX103.amr.corp.intel.com (10.18.124.201) with Microsoft SMTP Server (TLS) id 14.3.319.2; Mon, 22 Oct 2018 00:23:47 -0700 Received: from shsmsx101.ccr.corp.intel.com (10.239.4.153) by fmsmsx122.amr.corp.intel.com (10.18.125.37) with Microsoft SMTP Server (TLS) id 14.3.319.2; Mon, 22 Oct 2018 00:23:47 -0700 Received: from shsmsx103.ccr.corp.intel.com ([169.254.4.224]) by SHSMSX101.ccr.corp.intel.com ([169.254.1.46]) with mapi id 14.03.0319.002; Mon, 22 Oct 2018 15:23:44 +0800 From: "Wang, Jian J" To: Laszlo Ersek , "edk2-devel@lists.01.org" CC: "Zeng, Star" , "Kinney, Michael D" , "Yao, Jiewen" , "Ni, Ruiyu" Thread-Topic: [PATCH 2/3] UefiCpuPkg/CpuDxe: fix an infinite loop issue Thread-Index: AQHUZ6FJPVUm8+rx1US9R2/kJrIA7aUq3ipQ Date: Mon, 22 Oct 2018 07:23:44 +0000 Message-ID: References: <20181019015013.7488-1-jian.j.wang@intel.com> <20181019015013.7488-3-jian.j.wang@intel.com> <10148d39-039c-49e0-ba5a-83287f47c18b@redhat.com> In-Reply-To: <10148d39-039c-49e0-ba5a-83287f47c18b@redhat.com> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiMWY0MWFmODMtNTcxMC00ZTA3LThkMzMtOGUxOWEzOWEwOWU2IiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE3LjEwLjE4MDQuNDkiLCJUcnVzdGVkTGFiZWxIYXNoIjoiSEVPRUFGa1hDVjVQNUtXZnZQbTRBcXU1cWtYU1ZzTHc0bVZXelwvNXRwYWJpbm43VGRVakJaMGR3dVU0YlY2Z3YifQ== x-ctpclassification: CTP_NT dlp-product: dlpe-windows dlp-version: 11.0.400.15 dlp-reaction: no-action x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Subject: Re: [PATCH 2/3] UefiCpuPkg/CpuDxe: fix an infinite loop issue 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: Mon, 22 Oct 2018 07:23:49 -0000 Content-Language: en-US Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 TGFzemxvLA0KDQpUaGFua3MgZm9yIHRoZSBjb21tZW50cy4NCg0KUmVnYXJkcywNCkppYW4NCg0K DQo+IC0tLS0tT3JpZ2luYWwgTWVzc2FnZS0tLS0tDQo+IEZyb206IExhc3psbyBFcnNlayBbbWFp bHRvOmxlcnNla0ByZWRoYXQuY29tXQ0KPiBTZW50OiBGcmlkYXksIE9jdG9iZXIgMTksIDIwMTgg Nzo0NiBQTQ0KPiBUbzogV2FuZywgSmlhbiBKIDxqaWFuLmoud2FuZ0BpbnRlbC5jb20+OyBlZGsy LWRldmVsQGxpc3RzLjAxLm9yZw0KPiBDYzogWmVuZywgU3RhciA8c3Rhci56ZW5nQGludGVsLmNv bT47IEtpbm5leSwgTWljaGFlbCBEDQo+IDxtaWNoYWVsLmQua2lubmV5QGludGVsLmNvbT47IFlh bywgSmlld2VuIDxqaWV3ZW4ueWFvQGludGVsLmNvbT47IE5pLCBSdWl5dQ0KPiA8cnVpeXUubmlA aW50ZWwuY29tPg0KPiBTdWJqZWN0OiBSZTogW1BBVENIIDIvM10gVWVmaUNwdVBrZy9DcHVEeGU6 IGZpeCBhbiBpbmZpbml0ZSBsb29wIGlzc3VlDQo+IA0KPiBPbiAxMC8xOS8xOCAwMzo1MCwgSmlh biBKIFdhbmcgd3JvdGU6DQo+ID4gVGhlIFVBRiAoVXNlLUFmdGVyLUZyZWUpIG1lbW9yeSBkZXRl Y3Rpb24gZmVhdHVyZSB3aWxsIGNhdXNlIGFuDQo+ID4gaW5maW5pdGUgY2FsbGluZyBvZiBJbml0 aWFsaXplUGFnZVRhYmxlUG9vbCgpLiBUaGlzIGlzIGR1ZSB0byBhDQo+ID4gZmFjdCB0aGF0IEFs bG9jYXRlQWxpZ25lZFBhZ2VzKCkgaXMgdXNlZCB0byBhbGxvY2F0ZSBwYWdlIHRhYmxlDQo+ID4g cG9vbCBtZW1vcnkuIFRoaXMgZnVuY3Rpb24gd2lsbCBtb3N0IGxpa2VseSBjYWxsIGdCUy0+RnJl ZVBhZ2VzDQo+ID4gdG8gZnJlZSB1bmFsaWduZWQgcGFnZXMgYW5kIHRoZW4gY2F1c2UgYW5vdGhl ciByb3VuZCBvZiBwYWdlDQo+ID4gYXR0cmlidXRlcyBjaGFuZ2UsIGxpa2UgYmVsb3cNCj4gPg0K PiA+ICAgIEZyZWVQYWdlcygpIDw9PT09PT09PT09PT09PT18DQo+ID4gPT4gU2V0TWVtb3J5QXR0 cmlidXRlcygpICAgICAgIHwNCj4gDQo+IFRoaXMgc2hvdWxkIGxpa2VseSBiZSAiU2V0TWVtb3J5 U3BhY2VBdHRyaWJ1dGVzIiAodGhlIERYRSBzZXJ2aWNlKSwgb3IgZWxzZQ0KPiAiQ3B1U2V0TWVt b3J5QXR0cmlidXRlcyIgKHRoZSB1bmRlcmx5aW5nIENwdUR4ZSBmdW5jdGlvbiBuYW1lKS4NCj4g DQoNCllvdSdyZSByaWdodC4gSSdsbCBjaGFuZ2UgaXQuDQoNCj4gPiA9PiA8b3V0IG9mIHBhZ2Ug dGFibGU+ICAgICAgICAgfA0KPiA+ID0+IEluaXRpYWxpemVQYWdlVGFibGVQb29sKCkgICB8DQo+ ID4gPT4gQWxsb2NhdGVBbGlnbmVkUGFnZXMoKSAgICAgIHwNCj4gPiA9PiBGcmVlUGFnZXMoKSA9 PT09PT09PT09PT09PT09fA0KPiA+DQo+ID4gVGhlIHNvbHV0aW9uIGlzIGFkZCBhIGxvY2sgaW4g cGFnZSB0YWJsZSBwb29sIGFsbG9jYXRpb24gZnVuY3Rpb24NCj4gPiBhbmQgZmFpbCBhbnkgb3Ro ZXIgcmVxdWVzdHMgaWYgaXQgaGFzIG5vdCBiZWVuIGRvbmUuDQo+IA0KPiBPSywgYnV0IHdoYXQg aXMgdGhlIGVuZCByZXN1bHQ/IEluaXRpYWxpemVQYWdlVGFibGVQb29sKCkgd2lsbCByZXR1cm4g RkFMU0UuIEhvdw0KPiBmYXIgYmFjayB1cCBpcyB0aGF0IGVycm9yIHByb3BhZ2F0ZWQ/IFRvIHdo YXQgY29tcG9uZW50cyB3aWxsIHRoZSBlcnJvciBiZQ0KPiB2aXNpYmxlPw0KPiANCj4gQlRXLCBJ J3ZlIGZvdW5kIHRoZSBmb2xsb3dpbmcgY29tbWVudCBpbiBDcHVTZXRNZW1vcnlBdHRyaWJ1dGVz KCk6DQo+IA0KPiAgIC8vDQo+ICAgLy8gRHVyaW5nIG1lbW9yeSBhdHRyaWJ1dGVzIHVwZGF0aW5n LCBuZXcgcGFnZXMgbWF5IGJlIGFsbG9jYXRlZCB0byBzZXR1cA0KPiAgIC8vIHNtYWxsZXIgZ3Jh bnVsYXJpdHkgb2YgcGFnZSB0YWJsZS4gUGFnZSBhbGxvY2F0aW9uIGFjdGlvbiBtaWdodCB0aGVu IGNhdXNlDQo+ICAgLy8gYW5vdGhlciBjYWxsaW5nIG9mIENwdVNldE1lbW9yeUF0dHJpYnV0ZXMo KSByZWN1cnNpdmVseSwgZHVlIHRvIG1lbW9yeQ0KPiAgIC8vIHByb3RlY3Rpb24gcG9saWN5IGNv bmZpZ3VyZWQgKHN1Y2ggYXMgUGNkRHhlTnhNZW1vcnlQcm90ZWN0aW9uUG9saWN5KS4NCj4gICAv LyBTaW5jZSB0aGlzIGRyaXZlciB3aWxsIGFsd2F5cyBwcm90ZWN0IG1lbW9yeSB1c2VkIGFzIHBh Z2UgdGFibGUgYnkgaXRzZWxmLA0KPiAgIC8vIHRoZXJlJ3Mgbm8gbmVlZCB0byBhcHBseSBwcm90 ZWN0aW9uIHBvbGljeSByZXF1ZXN0ZWQgZnJvbSBtZW1vcnkgc2VydmljZS4NCj4gICAvLyBTbyBp dCdzIHNhZmUgdG8ganVzdCByZXR1cm4gRUZJX1NVQ0NFU1MgaWYgdGhpcyB0aW1lIG9mIGNhbGxp bmcgaXMgY2F1c2VkDQo+ICAgLy8gYnkgcGFnZSB0YWJsZSBtZW1vcnkgYWxsb2NhdGlvbi4NCj4g ICAvLw0KPiANCj4gSXMgdGhlIGN1cnJlbnQgYXJndW1lbnQgc2ltaWxhcj8gSSB0aGluayBpdCBz aG91bGQgYmUgZG9jdW1lbnRlZCBzb21laG93Lg0KPiANCg0KTm8sIEkgZG9uJ3QgdGhpbmsgdGhl eSdyZSB0aGUgc2ltaWxhci4gVGhlIGlzc3VlIEkgZW5jb3VudGVyZWQgaGVyZSBpcyB0aGF0IHRo ZSBjb2RlDQp0cmllcyB0byBzZXQgZnJlZWQgbWVtb3J5IGFzIG5vdC1wcmVzZW50IGJ1dCB0cmFw cGVkIGluIGRlYWQgbG9vcC4gVGhlIG9ubHkNCmNvbnNlcXVlbmNlIGhlcmUgaXMgdGhhdCB0aGUg ZnJlZWQgcGFnZXMgaW4gQWxsb2NhdGVBbGlnbmVkUGFnZXMoKSBjYW5ub3QgYmUNCnNldCBhcyBu b3QtcHJlc2VudC4gQnV0IGl0J3Mgb2sgYmVjYXVzZSB0aGV5J3JlIGp1c3QgYWxsb2NhdGVkIGFu ZCBoYXZlbid0IGJlZW4NCnVzZWQgYnkgYW55IG90aGVyIGNvZGUuDQoNCj4gPg0KPiA+IENjOiBM YXN6bG8gRXJzZWsgPGxlcnNla0ByZWRoYXQuY29tPg0KPiA+IENjOiBTdGFyIFplbmcgPHN0YXIu emVuZ0BpbnRlbC5jb20+DQo+ID4gQ2M6IE1pY2hhZWwgRCBLaW5uZXkgPG1pY2hhZWwuZC5raW5u ZXlAaW50ZWwuY29tPg0KPiA+IENjOiBKaWV3ZW4gWWFvIDxqaWV3ZW4ueWFvQGludGVsLmNvbT4N Cj4gPiBDYzogUnVpeXUgTmkgPHJ1aXl1Lm5pQGludGVsLmNvbT4NCj4gPiBDb250cmlidXRlZC11 bmRlcjogVGlhbm9Db3JlIENvbnRyaWJ1dGlvbiBBZ3JlZW1lbnQgMS4xDQo+ID4gU2lnbmVkLW9m Zi1ieTogSmlhbiBKIFdhbmcgPGppYW4uai53YW5nQGludGVsLmNvbT4NCj4gPiAtLS0NCj4gPiAg VWVmaUNwdVBrZy9DcHVEeGUvQ3B1UGFnZVRhYmxlLmMgfCAxMiArKysrKysrKysrKysNCj4gPiAg MSBmaWxlIGNoYW5nZWQsIDEyIGluc2VydGlvbnMoKykNCj4gPg0KPiA+IGRpZmYgLS1naXQgYS9V ZWZpQ3B1UGtnL0NwdUR4ZS9DcHVQYWdlVGFibGUuYw0KPiBiL1VlZmlDcHVQa2cvQ3B1RHhlL0Nw dVBhZ2VUYWJsZS5jDQo+ID4gaW5kZXggMzNlOGVlMmQyYy4uMjE0NWU2MjNmYSAxMDA2NDQNCj4g PiAtLS0gYS9VZWZpQ3B1UGtnL0NwdUR4ZS9DcHVQYWdlVGFibGUuYw0KPiA+ICsrKyBiL1VlZmlD cHVQa2cvQ3B1RHhlL0NwdVBhZ2VUYWJsZS5jDQo+ID4gQEAgLTEwMCw2ICsxMDAsNyBAQCBQQUdF X0FUVFJJQlVURV9UQUJMRSBtUGFnZUF0dHJpYnV0ZVRhYmxlW10gPSB7DQo+ID4gIH07DQo+ID4N Cj4gPiAgUEFHRV9UQUJMRV9QT09MICAgICAgICAgICAgICAgICAgICptUGFnZVRhYmxlUG9vbCA9 IE5VTEw7DQo+ID4gK0VGSV9MT0NLICAgICAgICAgICAgICAgICAgICAgICAgICBtUGFnZVRhYmxl UG9vbExvY2sgPQ0KPiBFRklfSU5JVElBTElaRV9MT0NLX1ZBUklBQkxFIChUUExfTk9USUZZKTsN Cj4gDQo+IFdoeSBkb2VzIHRoaXMgaGF2ZSB0byBiZSBhbiAiRUZJX0xPQ0siPyBDYW4ndCB3ZSBq dXN0IHVzZSBhIGdsb2JhbCB2YXJpYWJsZT8gKEkNCj4gZG9uJ3QgdW5kZXJzdGFuZCB3aHkgbWVz c2luZyB3aXRoIHRoZSBUUEwgaXMgbmVjZXNzYXJ5LikNCj4gDQo+IEluIGZhY3QsIEkgdG90YWxs eSBkb24ndCB1bmRlcnN0YW5kIHRoZSBwb2ludCBvZiBFZmlBY3F1aXJlTG9jaygpLiBJZiB3ZSBo YXZlIHR3bw0KPiBpbmRlcGVuZGVudCByZXNvdXJjZXMsIGVhY2ggcHJvdGVjdGVkIHdpdGggaXRz IG93biBzZXBhcmF0ZSBsb2NrLCB0aGVuIHdoeSBkbw0KPiBib3RoIGxvY2tzIHNoYXJlIHRoZSBz eXN0ZW0td2lkZSBUUEwgYmV0d2VlbiBlYWNoIG90aGVyPw0KPiANCg0KTWF5YmUgeW91J3JlIHJp Z2h0LiBMb2NrIGlzIGEgYml0IG92ZXJraWxsLiBJJ2xsIHRyeSBhIGdsb2JhbCB0byBmaW5kIG91 dCBpZiBpdCdzIG9rLg0KDQo+IA0KPiA+ICBQQUdFX1RBQkxFX0xJQl9QQUdJTkdfQ09OVEVYVCAg ICAgbVBhZ2luZ0NvbnRleHQ7DQo+ID4gIEVGSV9TTU1fQkFTRTJfUFJPVE9DT0wgICAgICAgICAg ICAqbVNtbUJhc2UyID0gTlVMTDsNCj4gPg0KPiA+IEBAIC0xMDQ1LDYgKzEwNDYsMTIgQEAgSW5p dGlhbGl6ZVBhZ2VUYWJsZVBvb2wgKA0KPiA+ICB7DQo+ID4gICAgVk9JRCAgICAgICAgICAgICAg ICAgICAgICAqQnVmZmVyOw0KPiA+ICAgIEJPT0xFQU4gICAgICAgICAgICAgICAgICAgSXNNb2Rp ZmllZDsNCj4gPiArICBFRklfU1RBVFVTICAgICAgICAgICAgICAgIFN0YXR1czsNCj4gPiArDQo+ ID4gKyAgU3RhdHVzID0gRWZpQWNxdWlyZUxvY2tPckZhaWwgKCZtUGFnZVRhYmxlUG9vbExvY2sp Ow0KPiA+ICsgIGlmIChFRklfRVJST1IgKFN0YXR1cykpIHsNCj4gPiArICAgIHJldHVybiBGQUxT RTsNCj4gPiArICB9DQo+ID4NCj4gPiAgICAvLw0KPiA+ICAgIC8vIEFsd2F5cyByZXNlcnZlIGF0 IGxlYXN0IFBBR0VfVEFCTEVfUE9PTF9VTklUX1BBR0VTLCBpbmNsdWRpbmcgb25lDQo+IHBhZ2Ug Zm9yDQo+ID4gQEAgLTEwNTYsNyArMTA2MywxMCBAQCBJbml0aWFsaXplUGFnZVRhYmxlUG9vbCAo DQo+ID4gICAgQnVmZmVyID0gQWxsb2NhdGVBbGlnbmVkUGFnZXMgKFBvb2xQYWdlcywgUEFHRV9U QUJMRV9QT09MX0FMSUdOTUVOVCk7DQo+ID4gICAgaWYgKEJ1ZmZlciA9PSBOVUxMKSB7DQo+ID4g ICAgICBERUJVRyAoKERFQlVHX0VSUk9SLCAiRVJST1I6IE91dCBvZiBhbGlnbmVkIHBhZ2VzXHJc biIpKTsNCj4gPiArICAgIEVmaVJlbGVhc2VMb2NrICgmbVBhZ2VUYWJsZVBvb2xMb2NrKTsNCj4g DQo+IEkgZmVlbCB0aGF0IGl0IHdvdWxkIGJlIHNhZmVyIHRvIGludHJvZHVjZSBhICJEb25lIiBs YWJlbCBhdCB0aGUgYm90dG9tIG9mIHRoZQ0KPiBmdW5jdGlvbiwgYW5kIHJlbGVhc2UgdGhlIGxv Y2sgdGhlcmUuDQo+IA0KPiAoQWdhaW4sIEknbSBub3Qgc3VyZSB3aHkgdGhpcyBoYXMgdG8gYmUg YSAibG9jayIuKQ0KPiANCg0KQWdyZWUuIEknbGwgdXBkYXRlIHRoaXMgcGFydCBvZiBsb2dpYy4N Cg0KPiA+ICAgICAgcmV0dXJuIEZBTFNFOw0KPiA+ICsgIH0gZWxzZSB7DQo+ID4gKyAgICBERUJV RyAoKERFQlVHX0lORk8sICJQYWdpbmc6IGFkZGVkICVkIHBhZ2VzIHRvIHBhZ2UgdGFibGUgcG9v bFxyXG4iLA0KPiBQb29sUGFnZXMpKTsNCj4gDQo+IFBsZWFzZSBkb24ndCBwcmludCBVSU5UTiB2 YWx1ZXMgd2l0aCAlZC4gQ2FzdCB0aGVtIHRvIFVJTlQ2NCBhbmQgbG9nIHRoZW0NCj4gd2l0aCAl bHUuDQo+IA0KDQpZb3UgZ290IG1lIGFnYWluLiBUaGF0J3MgYSBzaGFtZTooIEJ1dCB0aGFua3Mg Zm9yIHBvaW50IGl0IG91dC4NCg0KPiA+ICAgIH0NCj4gPg0KPiA+ICAgIC8vDQo+ID4gQEAgLTEw OTIsNiArMTEwMiw4IEBAIEluaXRpYWxpemVQYWdlVGFibGVQb29sICgNCj4gPiAgICAgICk7DQo+ ID4gICAgQVNTRVJUIChJc01vZGlmaWVkID09IFRSVUUpOw0KPiA+DQo+ID4gKyAgRWZpUmVsZWFz ZUxvY2sgKCZtUGFnZVRhYmxlUG9vbExvY2spOw0KPiA+ICsNCj4gPiAgICByZXR1cm4gVFJVRTsN Cj4gPiAgfQ0KPiA+DQo+ID4NCj4gDQo+IFRoYW5rcw0KPiBMYXN6bG8NCg==