From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Xu, Quan" Subject: Re: [PATCH v2 1/2] IOMMU/spinlock: Fix a bug found in AMD IOMMU initialization. Date: Wed, 9 Mar 2016 12:52:54 +0000 Message-ID: <945CA011AD5F084CBEA3E851C0AB28894B85E23E@SHSMSX101.ccr.corp.intel.com> References: <1457492889-36625-1-git-send-email-quan.xu@intel.com> <1457492889-36625-2-git-send-email-quan.xu@intel.com> <945CA011AD5F084CBEA3E851C0AB28894B85CA85@SHSMSX101.ccr.corp.intel.com> <1457519096.3102.327.camel@citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <1457519096.3102.327.camel@citrix.com> Content-Language: en-US List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xen.org Sender: "Xen-devel" To: Dario Faggioli Cc: "Tian, Kevin" , Jan Beulich , Suravee Suthikulpanit , "xen-devel@lists.xen.org" List-Id: xen-devel@lists.xenproject.org T24gTWFyY2ggMDksIDIwMTYgNjoyNXBtLCA8ZGFyaW8uZmFnZ2lvbGlAY2l0cml4LmNvbT4gd3Jv dGU6DQo+IE9uIFdlZCwgMjAxNi0wMy0wOSBhdCAwNzozMSArMDAwMCwgWHUsIFF1YW4gd3JvdGU6 DQo+ID4gT24gTWFyY2ggMDksIDIwMTYgMToxOXBtLCA8VGlhbiwgS2V2aW4+IHdyb3RlOg0KPiA+ ID4NCj4gPiA+ID4gV2hlbiBpb21tdV9zZXR1cCgpIGlzIGNhbGxlZCBpbiBfX3N0YXJ0X3hlbigp LCBpbnRlcnJ1cHRzIGhhdmUNCj4gPiA+ID4gYWxyZWFkeSBiZWVuIGVuYWJsZWQsIGFuZCBub3Ro aW5nIGRpc2FibGVzICh3aXRob3V0IHJlZW5hYmxpbmcpDQo+ID4gPiA+IHRoZW0gYWdhaW4gaW4g dGhlIHBhdGggdGhhdCBsZWFkcyB0byBjYWxsaW5nDQo+ID4gPiA+IHNldF9pb21tdV9pbnRlcnJ1 cHRfaGFuZGxlcigpLg0KPiA+ID4gPiBUaGVyZSdzDQo+ID4gPiA+IHRoZXJlZm9yZSBubyByaXNr IG9mIHRoZW0gYmVpbmcgZGlzYWJsZWQgYW5kIG5lZWRpbmcgcmVtYWluaW5nIHNvLA0KPiA+ID4g PiBhbmQgaGVuY2Ugbm8gbmVlZCBmb3INCj4gPiA+IG5vIHJpc2sgb2YgdGhlbSBiZWluZyAnZW5h YmxlZCcgc2luY2Ugbm8gb25lIGRpc2FibGVzIHRoZW0gYWdhaW4/DQo+ID4gPg0KPiA+IFllcywN Cj4gPg0KPiBSZWFzb24gd2h5IG9uZSB1c2UgX2lycXNhdmUoKSB3aGVuIGxvY2tpbmcgaXMgYmVj YXVzZSBoZSBkb2Vzbid0IGtub3cNCj4gd2hldGhlciBpbnRlcnJ1cHQgYXJlIGRpc2FibGVkIG9y IG5vdCwgYW5kIHdhbnRzIHRoYXQsIHdoYXRldmVyIHRoZSBzdGF0dXMgaXMNCj4gKGVuYWJsZWQg b3IgZGlzYWJsZWQpLCBpdCByZW1haW5zIHVuY2hhbmdlZCB1cG9uIHVubG9jayAod2hpY2gsIHRo ZXJlZm9yZSwNCj4gZG9lcyB0aGUgX2lycXJlc3RvcmUoKSkuDQo+IA0KPiBJZiB3ZSBhcmUgYWJz b2x1dGVseSBzdXJlIHRoYXQgdGhleSBhcmUgZW5hYmxlZCwgaS5lLiwgdGhlcmUgaXMgbm8gX3Jp c2tfIHRoYXQgdGhleQ0KPiBhcmUgZGlzYWJsZWQsIGl0IHdvdWxkIGJlIG9rIHRvIGp1c3QgdXNl IF9pcnEoKSAoaWYgZGlzYWJsaW5nIGludGVycnVwdCBpcyBuZWNlc3NhcnksDQo+IHdoaWNoIGlz IG5vdCBpbiB0aGlzIGNhc2UsIGJ1dCB0aGF0J3MgYW5vdGhlciB0aGluZykgYW5kIGF2b2lkIHNh dmluZyB0aGUgZmxhZ3MuDQo+IA0KPiBUaGF0J3MgaG93IEkgcmVhZCB0aGUgb3JpZ2luYWwgZGVz Y3JpcHRpb24gKHdoaWNoLCBvZiBjb3Vyc2UsIGRvZXMgbm90IG1lYW4gaXQNCj4gY2FuJ3QgYmUg c2ltcGxpZmllZCkuDQo+IA0KRGFyaW8sIHRoYW5rcyBmb3IgeW91ciBzaGFyZS4NCkkgYXBwcmVj aWF0ZSB5b3UgYWx3YXlzIHNoYXJlIHNvbWUga25vd2xlZGdlLiA6KTopDQpidHcsIHRoZSAiTGlu dXggRGV2aWNlIERyaXZlcnMsIDNyZCBFZGl0aW9uIiBib29rIGFsc28gZGVzY3JpYmUgaXQgY2xl YXJseSwgaHR0cDovL3d3dy5tYWtlbGludXgubmV0L2xkZDMvY2hwLTUtc2VjdC01IA0KDQo+ID4g PiBJIHRoaW5rIGl0IHNob3VsZCBiZSBjbGVhciBlbm91Z2ggdG8gc3RhdGUgdGhhdCBwY2lfZ2V0 X3BkZXYgZG9lc24ndA0KPiA+ID4gcmVxdWlyZSBob2xkaW5nIGxvY2sgd2l0aCBpbnRlcnJ1cHQg ZGlzYWJsZWQgKHNvIHdlIHNob3VsZCB1c2UNCj4gPiA+IHNwaW5fbG9jayBpbiB0aGlzIEFNRCBj YXNlKSwgYW5kIGFkZCB0aGUgZmFjdCB0aGF0IHdoZW4gdGhpcw0KPiA+ID4gZnVuY3Rpb24gaXMg aW52b2tlZCB0aGUgaW50ZXJydXB0IGlzIGluZGVlZCBlbmFibGVkLg0KPiA+ID4NCj4gSSB0b3Rh bGx5IGFncmVlIHdpdGggdGhpcyBkZXNjcmlwdGlvbiEgKENhbiB3ZSB1c2UgdGhhdCBhcyBjaGFu Z2Vsb2c/IDotKSApDQo+IA0KDQpPZiBjb3Vyc2UuIDopDQoNCj4gPiA+ID4gSW4gb3JkZXIgdG8g Zml4IHRoaXMsIHdlIGNhbiB1c2UganVzdCBwbGFpbiBzcGluX2xvY2soKSBhbmQNCj4gPiA+ID4g c3Bpbl91bmxvY2soKSwgaW5zdGVhZCBvZiBzcGluX2xvY2tfaXJxc2F2ZSgpIGFuZA0KPiA+ID4g PiBzcGluX3VubG9ja19pcnFyZXN0b3JlKCkuDQo+ID4gPiBXaGF0IGFib3V0IHJldmlzZSBsaWtl IGJlbG93Pw0KPiA+ID4gLS0NCj4gPiA+DQo+ID4gPiBwY2lkZXZzX2xvY2sgc2hvdWxkIGJlIGhl bGQgd2l0aCBpbnRlcnJ1cHQgZW5hYmxlZC4NCj4gPiBJIGFtIG5vdCBzdXJlIGZvciB0aGlzIHBv aW50Lg0KPiA+DQo+IFdlbGwsIGl0IGlzIHRydWUgdGhhdCBpdCBzaG91bGQuIEFsdG91Z2gsIEkg dGhpbmsgaXQncyBtb3JlIGFjY3VyYXRlIHRvIHNheSB0aGF0LCBhcw0KPiBLZXZpbiBzYXlzIGFi b3ZlLCBpdCAiZG9lc24ndCByZXF1aXJlIGJlaW5nIGhlbGQgd2l0aCBpbnRlcnJ1cHQgZGlzYWJs ZWQiLg0KPiANCj4gPiA+IEhvd2V2ZXIgdGhlcmUgcmVtYWlucyBhbg0KPiA+ID4gZXhjZXB0aW9u IGluIEFNRCBJT01NVSBjb2RlLCB3aGVyZSB0aGUgbG9jayBpcyBhY3F1aXJlZCB3aXRoDQo+ID4g PiBpbnRlcnJ1cHQgZGlzYWJsZWQuIFRoaXMgaW5jb25zaXN0ZW5jeSBjYW4gbGVhZCB0byBkZWFk bG9jay4NCj4gPiA+DQo+IENhbiBpdD8gSW4gdGhlIGNhc2Ugb2YgdGhlIG9jY3VycmVuY2UgYmVp bmcgY2hhbmdlZCBieSB0aGlzIHBhdGNoLCBJIGRvbid0IHRoaW5rIGl0DQo+IGNhbi4NCg0KQmVm b3JlIHRoaXMgcGF0Y2gsIGl0IG1pZ2h0LiBBcyBKYW4gbWVudGlvbmVkLCB0aGF0J3MganVzdCBh IHRoZW9yZXRpY2FsIGNvbmNlcm46IA0KIC1zcGluX2xvY2tfaXJxc2F2ZSgpIGRpc2FibGVzIGlu dGVycnVwdHMgKG9uIHRoZSBsb2NhbCBwcm9jZXNzb3Igb25seSkgYmVmb3JlIHRha2luZyB0aGUg c3BpbmxvY2suIA0KICBTdXBwb3NlZCBpbiBNUCBzeXN0ZW0gKHBDUFUxLCBwQ1BVMiwgLi4uKSwg d2hlbiB0aGUgcENQVTEgY2FsbCB0aGUgc3Bpbl9sb2NrX2lycXNhdmUoKSwNCiAgSXQgZG9lcyBv bmx5IGRpc2FibGUgaW50ZXJydXB0cyBmb3IgcENQVTEsIGFuZCBfbm90XyBmb3Igb3RoZXIgcENQ VW4uDQogLVRoZW4sIGl0IGlzIG1peGluZyBpbnRlcnJ1cHQgZGlzYWJsZWQgYW5kIGVuYWJsZWQg c3BpbmxvY2tzLg0KDQo+IA0KPiA+ID4gVGhlIGZpeCBpcyBzdHJhaWdodGZvcndhcmQgdG8gdXNl IHNwaW5fbG9jayBpbnN0ZWFkLiBBbHNvIGludGVycnVwdA0KPiA+ID4gaGFzIGJlZW4gZW5hYmxl ZCB3aGVuIHRoaXMgZnVuY3Rpb24gaXMgaW52b2tlZCwgc28gd2UncmUgc3VyZQ0KPiA+ID4gY29u c2lzdGVuY3kgYXJvdW5kIHBjaWRldnNfbG9jayBjYW4gYmUgZ3VhcmFudGVlZCBhZnRlciB0aGlz IGZpeC4NCj4gPiBJIHJlYWxseSBhcHByZWNpYXRlIHlvdSBzZW5kIG91dCBhIHJldmlzZWQgb25l LCBidXQgSSB0aGluayBpdCBpcyBub3QNCj4gPiBvbmx5IGZvciBjb25zaXN0ZW5jeS4NCj4gPiDC oF9fV2FpdCBmb3IgRGFyaW8ncyAvIEphbidzIG9waW5pb25zX18uDQo+ID4NCj4gPiBUbyBiZSBo b25lc3QsIHRvIG1lLCBfX215X2NoYW5nbG9nXyBpcyBjb21wbGljYXRlZC4NCj4gPg0KPiBJdCBp cyBjb21wbGljYXRlZC4gSSB0aGluayBpdCBpcyBhIGRldGFpbGVkLCBhbmQgSU1PIGNvcnJlY3Qs IGRlc2NyaXB0aW9uIG9mIHRoZQ0KPiByZWFzb24gd2h5IHRoZSBwYXRjaCBpcyBvay4gVGhhdCBp cyBpbmRlZWQgdGhlIHB1cnBvc2Ugb2YgYSBjaGFuZ2Vsb2cNCj4gKGVzcGVjaWFsbHkgZm9yIHRo ZXNlIGtpbmQgb2YgcGF0Y2hlcyksIGJ1dCBpdCBjZXJ0YWlubHkgY291bGQgYmUNCj4gc3VtbWFy aXplZC9zaW1wbGlmaWVkIGEgYml0Lg0KPiANCj4gSSBndWVzcyBJJ20gYWxzbyBnaXZpbmcgaXQg YSB0cnksIHVzaW5nIHdoYXQgS2V2aW4gd3JvdGUgaW4gdGhlIG1pZGRsZSBvZiBoaXMgZW1haWwN Cj4gYXMgYSBiYXNpcyAod2l0aCBhIHNtYWxsIGFkZGl0aW9uIGFib3V0IGNvbnNpc3RlbmN5IGF0 IHRoZSBlbmQpLg0KPiANCj4gInBjaV9nZXRfcGRldigpIGRvZXNuJ3QgcmVxdWlyZSBpbnRlcnJ1 cHRzIHRvIGJlIGRpc2FibGVkIHdoZW4gdGFraW5nIHRoZQ0KPiBwY2lkZXZzX2xvY2ssIHdoaWNo IHByb3RlY3RzIGl0cyBleGVjdXRpb24uIFNvLCBzcGluX2xvY2soKSBjYW4gYmUgdXNlZCBmb3Ig dGhhdCwNCj4gYW5kIHRoYXQgaXMgd2hhdCBpcyBkb25lIGFsbW9zdCBldmVyeXdoZXJlLg0KPiAN Cj4gQ3VycmVubHR5LCB0aGVyZSBpcyBhbiBleGNlcHRpb24sIGluIGVhcmx5IGJvb3QgSU9NTVUg aW5pdGlhbGl6YXRpb24gb24gQU1EDQo+IHN5c3RlbXMsIHdoZXJlIHNwaW5fbG9ja19pcnFzYXZl KCkgYW5kIHJlc3RvcmUgYXJlIHVzZWQuIEhvd2V2ZXIsIHNpbmNlLCBpbg0KPiB0aGF0IGNhc2Ug dG9vOg0KPiDCoC0gd2UgZG9uJ3QgbmVlZCB0byBkaXNhYmxlIGludGVycnVwdHMgKGZvciBzYW1l IHJlYXNvbnMgc3RhdGVkIGFib3ZlKSwNCj4gwqAtIGludGVycnVwdHMgYXJlIGVuYWJsZWQgYWxy ZWFkeSwgc28gdGhlcmUgaXMgbm8gbmVlZCB0byBzYXZlIGFuZA0KPiDCoCDCoHJlc3RvcmUgZmxh Z3MsDQo+IGp1c3QgY2hhbmdlIGl0IGludG8gdXNpbmcgc3Bpbl9sb2NrKCksIGFzIGV2ZXJ5d2hl cmUuDQo+IA0KPiBOb3RlIHRoYXQsIG1peGluZyBpbnRlcnJ1cHQgZGlzYWJsZWQgYW5kIGVuYWJs ZWQgc3BpbmxvY2tzIGlzIHNvbWV0aGluZyB3ZQ0KPiBkaXNhbGxvdywNCg0KRGFyaW8sIGNvdWxk IHlvdSBzaGFyZSBzb21ldGhpbmcgaW4gZGV0YWlsPw0KUXVhbg0KDQo+IGV4Y2VwdCBmb3IgdmVy eSBzcGVjaWFsIHNpdHVhdGlvbnMuIA0KX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX18KWGVuLWRldmVsIG1haWxpbmcgbGlzdApYZW4tZGV2ZWxAbGlzdHMueGVu Lm9yZwpodHRwOi8vbGlzdHMueGVuLm9yZy94ZW4tZGV2ZWwK