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.31; helo=mga06.intel.com; envelope-from=jian.j.wang@intel.com; receiver=edk2-devel@lists.01.org Received: from mga06.intel.com (mga06.intel.com [134.134.136.31]) (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 2C7A521E25682 for ; Wed, 24 Jan 2018 17:18:03 -0800 (PST) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga006.fm.intel.com ([10.253.24.20]) by orsmga104.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 24 Jan 2018 17:23:31 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.46,409,1511856000"; d="scan'208";a="198585272" Received: from fmsmsx104.amr.corp.intel.com ([10.18.124.202]) by fmsmga006.fm.intel.com with ESMTP; 24 Jan 2018 17:23:31 -0800 Received: from shsmsx151.ccr.corp.intel.com (10.239.6.50) by fmsmsx104.amr.corp.intel.com (10.18.124.202) with Microsoft SMTP Server (TLS) id 14.3.319.2; Wed, 24 Jan 2018 17:23:31 -0800 Received: from shsmsx103.ccr.corp.intel.com ([169.254.4.213]) by SHSMSX151.ccr.corp.intel.com ([169.254.3.218]) with mapi id 14.03.0319.002; Thu, 25 Jan 2018 09:23:25 +0800 From: "Wang, Jian J" To: Laszlo Ersek , "edk2-devel@lists.01.org" CC: "Ni, Ruiyu" , "Dong, Eric" Thread-Topic: [PATCH] UefiCpuPkg/MpInitLib: fix issue in wakeup buffer initialization Thread-Index: AQHTlSgqwbkyQzWfY0GugxjZ5pRD4aODyDSw Date: Thu, 25 Jan 2018 01:23:26 +0000 Message-ID: References: <20180124020845.6096-1-jian.j.wang@intel.com> <890f02ba-c4b3-0bd1-7cf4-8cc9b308861d@redhat.com> In-Reply-To: <890f02ba-c4b3-0bd1-7cf4-8cc9b308861d@redhat.com> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiODRhZDJhMjEtM2JmNS00ZDI3LWEyY2QtYzg1NGZjNzdjMWU3IiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE3LjIuNS4xOCIsIlRydXN0ZWRMYWJlbEhhc2giOiJtSXh4bFZwZlA5UWI1QmM3dVVvRllMckw1QkZNZ3YyR3h1cnF5bHdhQkt0dnY3SXhJcEVydXhLTG5Hb3lLSGtyIn0= x-ctpclassification: CTP_NT dlp-product: dlpe-windows dlp-version: 11.0.0.116 dlp-reaction: no-action x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Subject: Re: [PATCH] UefiCpuPkg/MpInitLib: fix issue in wakeup buffer initialization X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 25 Jan 2018 01:18:03 -0000 Content-Language: en-US Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Q29uc2lkZXJpbmcgdGhhdCB0aGVyZSdzIGEgc2ltaWxhciBhc3NpZ25tZW50IGluICJlbHNlIiBi bG9jaw0KDQogICAgRXhjaGFuZ2VJbmZvLT5Nb2RlVHJhbnNpdGlvbk1lbW9yeSA9IChVSU5UMzIp DQogICAgICAoRXhjaGFuZ2VJbmZvLT5CdWZmZXJTdGFydCArIENwdU1wRGF0YS0+QWRkcmVzc01h cC5Nb2RlVHJhbnNpdGlvbk9mZnNldCk7DQoNCkkgd291bGQgcmF0aGVyIGtlZXAgdGhlIGFzc2ln bm1lbnQgc3RhdGVtZW50IHlvdSBtZW50aW9uZWQgaW5zaWRlICJpZiIgYmxvY2suDQoNCkFjdHVh bGx5IHRoZXJlJ3MgYW5vdGhlciBpc3N1ZSBpbiB0aGlzIHBhcnQgb2YgY29kZS4gVG8gZml4IGl0 LCB0aGUgYXNzaWdubWVudCB0bw0KTW9kZUhpZ2hNZW1vcnkgYW5kIE1vZGVIaWdoU2VnbWVudCB3 aWxsIGJlIG1vdmVkIG91dHNpZGUgdGhlICJpZiIgYmxvY2suDQpGcm9tIHRoaXMgZml4IHBlcnNw ZWN0aXZlLCBpdCdkIGJlIGFsc28gYmV0dGVyIHRvIGtlZXAgYWxsIGFzc2lnbm1lbnRzIHRvDQpN b2RlVHJhbnNpdGlvbk1lbW9yeSBpbnNpZGUgImlmL2Vsc2UiIGJsb2NrLiBZb3UnbGwgc2VlIGl0 IGluIGEgbmV3IHBhdGNoIGxhdGVyLg0KDQpBbnl3YXksIHRoYW5rcyBmb3IgdGhlIGNvbW1lbnRz Lg0KDQpSZWdhcmRzLA0KSmlhbg0KDQoNCj4gLS0tLS1PcmlnaW5hbCBNZXNzYWdlLS0tLS0NCj4g RnJvbTogTGFzemxvIEVyc2VrIFttYWlsdG86bGVyc2VrQHJlZGhhdC5jb21dDQo+IFNlbnQ6IFdl ZG5lc2RheSwgSmFudWFyeSAyNCwgMjAxOCAxMTozMCBQTQ0KPiBUbzogV2FuZywgSmlhbiBKIDxq aWFuLmoud2FuZ0BpbnRlbC5jb20+OyBlZGsyLWRldmVsQGxpc3RzLjAxLm9yZw0KPiBDYzogTmks IFJ1aXl1IDxydWl5dS5uaUBpbnRlbC5jb20+OyBEb25nLCBFcmljIDxlcmljLmRvbmdAaW50ZWwu Y29tPg0KPiBTdWJqZWN0OiBSZTogW1BBVENIXSBVZWZpQ3B1UGtnL01wSW5pdExpYjogZml4IGlz c3VlIGluIHdha2V1cCBidWZmZXINCj4gaW5pdGlhbGl6YXRpb24NCj4gDQo+IE9uIDAxLzI0LzE4 IDAzOjA4LCBKaWFuIEogV2FuZyB3cm90ZToNCj4gPiBUbyBmaXggYW4gaXNzdWUgaW4gd2hpY2gg ZW5hYmxpbmcgTlggZmVhdHVyZSB3aWxsIG1hcmsgdGhlIEFQIHdha2V1cA0KPiA+IGJ1ZmZlciBh cyBub24tZXhlY3V0YWJsZSBhbmQgZmFpbCB0aGUgQVAgaW5pdCwgdGhlIGJ1ZmZlciB3YXMgc3Bs aXQNCj4gPiBpbnRvIHR3byBwYXJ0OiB0aGUgbG93ZXIgcGFydCBpbiBtZW1vcnkgd2l0aGluIDFN QiBhbmQgdGhlIGhpZ2hlciBwYXJ0DQo+ID4gd2l0aGluIGFsbG9jYXRlZCBleGVjdXRhYmxlIG1l bW9yeSAoRWZpQm9vdFNlcnZpY2VzQ29kZSkuIEJ1dCB0aGUNCj4gPiBhZGRyZXNzIG9mIGhpZ2hl ciBwYXJ0IG1lbW9yeSB3YXMgc3RvcmVkIGluIGxvd2VyIHBhcnQgbWVtb3J5LCB3aGljaA0KPiA+ IGlzIGFjdHVhbGx5IHNoYXJlZCB3aXRoIGxlZ2FjeSBjb21wb25lbnRzIGFuZCB3aWxsIGJlIG92 ZXJ3cml0dGVuIGJ5DQo+ID4gTGVnYWN5Qmlvc0R4ZSBkcml2ZXIgaWYgQ1NNIGlzIGVuYWJsZWQu DQo+ID4NCj4gPiBUaGlzIHBhdGNoIGZpeGVzIHRoaXMgaXNzdWUgYnkgc3RvcmluZyB0aGUgYWRk cmVzcyBvZiBoaWdoZXIgcGFydA0KPiA+IG1lbW9yeSBpbiBDcHVNcERhdGEgaW5zdGVhZCBvZiBF eGNoYW5nZUluZm8uDQo+ID4NCj4gPiBDYzogUnVpeXUgTmkgPHJ1aXl1Lm5pQGludGVsLmNvbT4N Cj4gPiBDYzogRXJpYyBEb25nIDxlcmljLmRvbmdAaW50ZWwuY29tPg0KPiA+IENjOiBMYXN6bG8g RXJzZWsgPGxlcnNla0ByZWRoYXQuY29tPg0KPiA+IENvbnRyaWJ1dGVkLXVuZGVyOiBUaWFub0Nv cmUgQ29udHJpYnV0aW9uIEFncmVlbWVudCAxLjENCj4gPiBTaWduZWQtb2ZmLWJ5OiBKaWFuIEog V2FuZyA8amlhbi5qLndhbmdAaW50ZWwuY29tPg0KPiA+IC0tLQ0KPiA+ICBVZWZpQ3B1UGtnL0xp YnJhcnkvTXBJbml0TGliL01wTGliLmMgfCAyMCArKysrKysrKysrLS0tLS0tLS0tLQ0KPiA+ICBV ZWZpQ3B1UGtnL0xpYnJhcnkvTXBJbml0TGliL01wTGliLmggfCAgMSArDQo+ID4gIDIgZmlsZXMg Y2hhbmdlZCwgMTEgaW5zZXJ0aW9ucygrKSwgMTAgZGVsZXRpb25zKC0pDQo+ID4NCj4gPiBkaWZm IC0tZ2l0IGEvVWVmaUNwdVBrZy9MaWJyYXJ5L01wSW5pdExpYi9NcExpYi5jDQo+IGIvVWVmaUNw dVBrZy9MaWJyYXJ5L01wSW5pdExpYi9NcExpYi5jDQo+ID4gaW5kZXggNjIzMTk2OGM3NC4uNDIw MTFkNjIzMSAxMDA2NDQNCj4gPiAtLS0gYS9VZWZpQ3B1UGtnL0xpYnJhcnkvTXBJbml0TGliL01w TGliLmMNCj4gPiArKysgYi9VZWZpQ3B1UGtnL0xpYnJhcnkvTXBJbml0TGliL01wTGliLmMNCj4g PiBAQCAtODIzLDE5ICs4MjMsMjAgQEAgRmlsbEV4Y2hhbmdlSW5mb0RhdGEgKA0KPiA+ICAgIC8v IENvcHkgYWxsIDMyLWJpdCBjb2RlIGFuZCA2NC1iaXQgY29kZSBpbnRvIG1lbW9yeSB3aXRoIHR5 cGUgb2YNCj4gPiAgICAvLyBFZmlCb290U2VydmljZXNDb2RlIHRvIGF2b2lkIHBhZ2UgZmF1bHQg aWYgTlggbWVtb3J5IHByb3RlY3Rpb24gaXMNCj4gZW5hYmxlZC4NCj4gPiAgICAvLw0KPiA+IC0g IGlmIChFeGNoYW5nZUluZm8tPk1vZGVUcmFuc2l0aW9uTWVtb3J5ICE9IDApIHsNCj4gPiArICBp ZiAoQ3B1TXBEYXRhLT5XYWtldXBCdWZmZXJIaWdoICE9IDApIHsNCj4gPiAgICAgIFNpemUgPSBD cHVNcERhdGEtPkFkZHJlc3NNYXAuUmVuZGV6dm91c0Z1bm5lbFNpemUgLQ0KPiA+ICAgICAgICAg ICAgIENwdU1wRGF0YS0+QWRkcmVzc01hcC5Nb2RlVHJhbnNpdGlvbk9mZnNldDsNCj4gPiAgICAg IENvcHlNZW0gKA0KPiA+IC0gICAgICAoVk9JRCAqKShVSU5UTilFeGNoYW5nZUluZm8tPk1vZGVU cmFuc2l0aW9uTWVtb3J5LA0KPiA+ICsgICAgICAoVk9JRCAqKUNwdU1wRGF0YS0+V2FrZXVwQnVm ZmVySGlnaCwNCj4gPiAgICAgICAgQ3B1TXBEYXRhLT5BZGRyZXNzTWFwLlJlbmRlenZvdXNGdW5u ZWxBZGRyZXNzICsNCj4gPiAgICAgICAgQ3B1TXBEYXRhLT5BZGRyZXNzTWFwLk1vZGVUcmFuc2l0 aW9uT2Zmc2V0LA0KPiA+ICAgICAgICBTaXplDQo+ID4gICAgICAgICk7DQo+ID4NCj4gPiAtICAg IEV4Y2hhbmdlSW5mby0+TW9kZUhpZ2hNZW1vcnkgPSBFeGNoYW5nZUluZm8tDQo+ID5Nb2RlVHJh bnNpdGlvbk1lbW9yeTsNCj4gPiAtICAgIEV4Y2hhbmdlSW5mby0+TW9kZUhpZ2hNZW1vcnkgKz0g KFVJTlQzMilFeGNoYW5nZUluZm8tPk1vZGVPZmZzZXQNCj4gLQ0KPiA+IC0gICAgICAgICAgICAg ICAoVUlOVDMyKUNwdU1wRGF0YS0+QWRkcmVzc01hcC5Nb2RlVHJhbnNpdGlvbk9mZnNldDsNCj4g PiArICAgIEV4Y2hhbmdlSW5mby0+TW9kZVRyYW5zaXRpb25NZW1vcnkgPSAoVUlOVDMyKUNwdU1w RGF0YS0NCj4gPldha2V1cEJ1ZmZlckhpZ2g7DQo+ID4gKyAgICBFeGNoYW5nZUluZm8tPk1vZGVI aWdoTWVtb3J5ID0gKFVJTlQzMilDcHVNcERhdGEtDQo+ID5XYWtldXBCdWZmZXJIaWdoICsNCj4g PiArICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAoVUlOVDMyKUV4Y2hhbmdlSW5m by0+TW9kZU9mZnNldCAtDQo+ID4gKyAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg KFVJTlQzMilDcHVNcERhdGEtPkFkZHJlc3NNYXAuTW9kZVRyYW5zaXRpb25PZmZzZXQ7DQo+ID4g ICAgICBFeGNoYW5nZUluZm8tPk1vZGVIaWdoU2VnbWVudCA9IChVSU5UMTYpRXhjaGFuZ2VJbmZv LT5Db2RlU2VnbWVudDsNCj4gPiAgICB9IGVsc2Ugew0KPiA+ICAgICAgRXhjaGFuZ2VJbmZvLT5N b2RlVHJhbnNpdGlvbk1lbW9yeSA9IChVSU5UMzIpDQo+ID4gQEAgLTkxNiwxMSArOTE3LDEwIEBA IEFsbG9jYXRlUmVzZXRWZWN0b3IgKA0KPiA+ICAgICAgQ3B1TXBEYXRhLT5XYWtldXBCdWZmZXIg ICAgICA9IEdldFdha2V1cEJ1ZmZlciAoQXBSZXNldFZlY3RvclNpemUpOw0KPiA+ICAgICAgQ3B1 TXBEYXRhLT5NcENwdUV4Y2hhbmdlSW5mbyA9IChNUF9DUFVfRVhDSEFOR0VfSU5GTyAqKSAoVUlO VE4pDQo+ID4gICAgICAgICAgICAgICAgICAgICAgKENwdU1wRGF0YS0+V2FrZXVwQnVmZmVyICsg Q3B1TXBEYXRhLQ0KPiA+QWRkcmVzc01hcC5SZW5kZXp2b3VzRnVubmVsU2l6ZSk7DQo+ID4gLSAg ICBDcHVNcERhdGEtPk1wQ3B1RXhjaGFuZ2VJbmZvLT5Nb2RlVHJhbnNpdGlvbk1lbW9yeSA9IChV SU5UMzIpDQo+ID4gLSAgICAgICAgICAgICAgICAgICAgR2V0TW9kZVRyYW5zaXRpb25CdWZmZXIg KA0KPiA+IC0gICAgICAgICAgICAgICAgICAgICAgQ3B1TXBEYXRhLT5BZGRyZXNzTWFwLlJlbmRl enZvdXNGdW5uZWxTaXplIC0NCj4gPiAtICAgICAgICAgICAgICAgICAgICAgIENwdU1wRGF0YS0+ QWRkcmVzc01hcC5Nb2RlVHJhbnNpdGlvbk9mZnNldA0KPiA+IC0gICAgICAgICAgICAgICAgICAg ICAgKTsNCj4gPiArICAgIENwdU1wRGF0YS0+V2FrZXVwQnVmZmVySGlnaCAgPSBHZXRNb2RlVHJh bnNpdGlvbkJ1ZmZlciAoDQo+ID4gKyAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg IENwdU1wRGF0YS0+QWRkcmVzc01hcC5SZW5kZXp2b3VzRnVubmVsU2l6ZSAtDQo+ID4gKyAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIENwdU1wRGF0YS0+QWRkcmVzc01hcC5Nb2Rl VHJhbnNpdGlvbk9mZnNldA0KPiA+ICsgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICApOw0KPiA+ICAgIH0NCj4gPiAgICBCYWNrdXBBbmRQcmVwYXJlV2FrZXVwQnVmZmVyIChDcHVN cERhdGEpOw0KPiA+ICB9DQo+ID4gZGlmZiAtLWdpdCBhL1VlZmlDcHVQa2cvTGlicmFyeS9NcElu aXRMaWIvTXBMaWIuaA0KPiBiL1VlZmlDcHVQa2cvTGlicmFyeS9NcEluaXRMaWIvTXBMaWIuaA0K PiA+IGluZGV4IDAyMzJmZTg5NmEuLmU3ZjlhNGRlMGEgMTAwNjQ0DQo+ID4gLS0tIGEvVWVmaUNw dVBrZy9MaWJyYXJ5L01wSW5pdExpYi9NcExpYi5oDQo+ID4gKysrIGIvVWVmaUNwdVBrZy9MaWJy YXJ5L01wSW5pdExpYi9NcExpYi5oDQo+ID4gQEAgLTIwOCw2ICsyMDgsNyBAQCBzdHJ1Y3QgX0NQ VV9NUF9EQVRBIHsNCj4gPiAgICBVSU5UTiAgICAgICAgICAgICAgICAgICAgICAgICAgQ3B1QXBT dGFja1NpemU7DQo+ID4gICAgTVBfQVNTRU1CTFlfQUREUkVTU19NQVAgICAgICAgIEFkZHJlc3NN YXA7DQo+ID4gICAgVUlOVE4gICAgICAgICAgICAgICAgICAgICAgICAgIFdha2V1cEJ1ZmZlcjsN Cj4gPiArICBVSU5UTiAgICAgICAgICAgICAgICAgICAgICAgICAgV2FrZXVwQnVmZmVySGlnaDsN Cj4gPiAgICBVSU5UTiAgICAgICAgICAgICAgICAgICAgICAgICAgQmFja3VwQnVmZmVyOw0KPiA+ ICAgIFVJTlROICAgICAgICAgICAgICAgICAgICAgICAgICBCYWNrdXBCdWZmZXJTaXplOw0KPiA+ DQo+ID4NCj4gDQo+IEFzIGZhciBhcyBJIHNlZSwgYWxsIHRoZSB2YWx1ZXMgc3RvcmVkIGJ5IEZp bGxFeGNoYW5nZUluZm9EYXRhKCkgcmVtYWluDQo+IHRoZSBzYW1lLiBUaGUgZGlmZmVyZW5jZSBp cyB0aGF0ICJNb2RlVHJhbnNpdGlvbk1lbW9yeSIgYmVjb21lcyBhbg0KPiAqb3V0cHV0KiBmaWVs ZCBmb3IgRmlsbEV4Y2hhbmdlSW5mb0RhdGEoKSwgZnJvbSBiZWluZyBhbiBpbnB1dCBmaWVsZC4g V2UNCj4gbm93IGV4cGxpY2l0bHkgc3RvcmUgdGhlIHZhbHVlIHRvIGl0IHRoYXQgd2UgcHJldmlv dXNseSAqYXNzdW1lZCogaXQNCj4gd291bGQgaG9sZCAocHJlc2VydmUpLCBmcm9tIEFsbG9jYXRl UmVzZXRWZWN0b3IoKS4NCj4gDQo+IFRoZSBpbnB1dCBub3cgY29tZXMgdGhyb3VnaCBXYWtldXBC dWZmZXJIaWdoLCBmcm9tIHRoZQ0KPiBBbGxvY2F0ZVJlc2V0VmVjdG9yKCkgZnVuY3Rpb24uDQo+ IA0KPiBUaHVzLCBJIHRoaW5rIHRoZSBwYXRjaCBjb3VsZCBoYXZlIGJlZW4gd3JpdHRlbiBhIGJp dCBtb3JlIHNpbXBseTogd2UNCj4gY291bGQgaGF2ZSBhZGRlZCB0aGUgYXNzaWdubWVudA0KPiAN Cj4gRXhjaGFuZ2VJbmZvLT5Nb2RlVHJhbnNpdGlvbk1lbW9yeSA9IChVSU5UMzIpQ3B1TXBEYXRh LQ0KPiA+V2FrZXVwQnVmZmVySGlnaA0KPiANCj4gYmVmb3JlIHRoZSAiaWYiIGluIEZpbGxFeGNo YW5nZUluZm9EYXRhKCksIGFuZCB0aGVuIHRoZSByZXN0IG9mDQo+IEZpbGxFeGNoYW5nZUluZm9E YXRhKCkgd291bGQgaGF2ZSBuZWVkZWQgbm8gY2hhbmdlcy4gQnV0IHBlcmhhcHMgdGhhdA0KPiB3 b3VsZCBoYXZlIGJlZW4gaGFyZGVyIHRvIHVuZGVyc3RhbmQuDQo+IA0KPiBSZXZpZXdlZC1ieTog TGFzemxvIEVyc2VrIDxsZXJzZWtAcmVkaGF0LmNvbT4NCj4gDQo+IFRoYW5rcw0KPiBMYXN6bG8N Cg==