From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jason Gunthorpe Subject: Re: [PATCH hmm 00/15] Consolidate the mmu notifier interval_tree and locking Date: Mon, 21 Oct 2019 13:57:51 +0000 Message-ID: <20191021135744.GA25164@mellanox.com> References: <20191015181242.8343-1-jgg@ziepe.ca> <20191016160444.GB3430@mellanox.com> <2df298e2-ee91-ef40-5da9-2bc1af3a17be@gmail.com> <2046e0b4-ba05-0683-5804-e9bbf903658d@amd.com> <20191018203608.GA5670@mellanox.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: Content-Language: en-US Content-ID: <65E636A7AEB8FE479F728A3DE81C272C@eurprd05.prod.outlook.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: "Koenig, Christian" Cc: Andrea Arcangeli , "Yang, Philip" , Ralph Campbell , "linux-rdma@vger.kernel.org" , John Hubbard , "Kuehling, Felix" , "amd-gfx@lists.freedesktop.org" , "linux-mm@kvack.org" , Jerome Glisse , "dri-devel@lists.freedesktop.org" , Ben Skeggs List-Id: dri-devel@lists.freedesktop.org T24gU3VuLCBPY3QgMjAsIDIwMTkgYXQgMDI6MjE6NDJQTSArMDAwMCwgS29lbmlnLCBDaHJpc3Rp YW4gd3JvdGU6DQo+IEFtIDE4LjEwLjE5IHVtIDIyOjM2IHNjaHJpZWIgSmFzb24gR3VudGhvcnBl Og0KPiA+IE9uIFRodSwgT2N0IDE3LCAyMDE5IGF0IDA0OjQ3OjIwUE0gKzAwMDAsIEtvZW5pZywg Q2hyaXN0aWFuIHdyb3RlOg0KPiA+DQo+ID4+PiBnZXRfdXNlcl9wYWdlcy9obW1fcmFuZ2VfZmF1 bHQoKSBhbmQgaW52YWxpZGF0ZV9yYW5nZV9zdGFydCgpIGJvdGggYXJlDQo+ID4+PiBjYWxsZWQg d2hpbGUgaG9sZGluZyBtbS0+bWFwX3NlbSwgc28gdGhleSBhcmUgYWx3YXlzIHNlcmlhbGl6ZWQu DQo+ID4+IE5vdCBldmVuIHJlbW90ZWx5Lg0KPiA+Pg0KPiA+PiBGb3IgY2FsbGluZyBnZXRfdXNl cl9wYWdlcygpL2htbV9yYW5nZV9mYXVsdCgpIHlvdSBvbmx5IG5lZWQgdG8gaG9sZCB0aGUNCj4g Pj4gbW1hcF9zZW0gaW4gcmVhZCBtb2RlLg0KPiA+IFJpZ2h0DQo+ID4gICANCj4gPj4gQW5kIElJ UkMgaW52YWxpZGF0ZV9yYW5nZV9zdGFydCgpIGlzIHNvbWV0aW1lcyBjYWxsZWQgd2l0aG91dCBo b2xkaW5nDQo+ID4+IHRoZSBtbWFwX3NlbSBhdCBhbGwuDQo+ID4gWWVwDQo+ID4gICANCj4gPj4g U28gYWdhaW4gaG93IGFyZSB0aGV5IHNlcmlhbGl6ZWQ/DQo+ID4gVGhlICdkcml2ZXIgbG9jaycg dGhpbmcgZG9lcyBpdCwgcmVhZCB0aGUgaG1tIGRvY3VtZW50YXRpb24sIHRoZSBobW0NCj4gPiBh cHByb2FjaCBpcyBiYXNpY2FsbHkgdGhlIG9ubHkgYXBwcm9hY2ggdGhhdCB3YXMgY29ycmVjdCBv ZiBhbGwgdGhlDQo+ID4gZHJpdmVycy4uDQo+IA0KPiBXZWxsIHRoYXQncyB3aGF0IEkndmUgZGlk LCBidXQgd2hhdCBITU0gZG9lcyBzdGlsbCBkb2Vzbid0IGxvb2tzIGNvcnJlY3QgDQo+IHRvIG1l Lg0KDQpJdCBoYXMgYSBidWcsIGJ1dCB0aGUgYmFzaWMgZmxvdyBzZWVtcyB0byB3b3JrLg0KDQpo dHRwczovL3BhdGNod29yay5rZXJuZWwub3JnL3BhdGNoLzExMTkxDQoNCj4gPiBTbyBsb25nIGFz IHRoZSAnZHJpdmVyIGxvY2snIGlzIGhlbGQgdGhlIHJhbmdlIGNhbm5vdCBiZWNvbWUNCj4gPiBp bnZhbGlkYXRlZCBhcyB0aGUgJ2RyaXZlciBsb2NrJyBwcmV2ZW50cyBwcm9ncmVzcyBvZiBpbnZh bGlkYXRpb24uDQo+IA0KPiBDb3JyZWN0LCBidXQgdGhlIHByb2JsZW0gaXMgaXQgZG9lc24ndCB3 YWl0IGZvciBvbmdvaW5nIG9wZXJhdGlvbnMgdG8gDQo+IGNvbXBsZXRlLg0KPg0KPiBTZWUgSSdt IHRhbGtpbmcgYWJvdXQgdGhlIGZvbGxvd2luZyBjYXNlOg0KPiANCj4gVGhyZWFkIEHCoMKgwqAg VGhyZWFkIEINCj4gaW52YWxpZGF0ZV9yYW5nZV9zdGFydCgpDQo+ICAgICAgICAgICAgICAgICAg ICAgIG1tdV9yYW5nZV9yZWFkX2JlZ2luKCkNCj4gICAgICAgICAgICAgICAgICAgICAgZ2V0X3Vz ZXJfcGFnZXMoKS9obW1fcmFuZ2VfZmF1bHQoKQ0KPiAgICAgICAgICAgICAgICAgICAgICBncmFi X2RyaXZlcl9sb2NrKCkNCj4gVXBkYXRpbmcgdGhlIHB0ZXMNCj4gaW52YWxpZGF0ZV9yYW5nZV9l bmQoKQ0KPiANCj4gQXMgZmFyIGFzIEkgY2FuIHNlZSBpbiBpbnZhbGlkYXRlX3JhbmdlX3N0YXJ0 KCkgdGhlIGRyaXZlciBsb2NrIGlzIHRha2VuIA0KPiB0byBtYWtlIHN1cmUgdGhhdCB3ZSBjYW4n dCBzdGFydCBhbnkgaW52YWxpZGF0aW9uIHdoaWxlIHRoZSBkcml2ZXIgaXMgDQo+IHVzaW5nIHRo ZSBwYWdlcyBmb3IgYSBjb21tYW5kIHN1Ym1pc3Npb24uDQoNCkFnYWluLCB0aGlzIHVzZXMgdGhl IHNlcWxvY2sgbGlrZSBzY2hlbWUgKmFuZCogdGhlIGRyaXZlciBsb2NrLg0KDQpJbiB0aGlzIGNh c2UgYWZ0ZXIgZ3JhYl9kcml2ZXJfbG9jaygpIG1tdV9yYW5nZV9yZWFkX3JldHJ5KCkgd2lsbA0K cmV0dXJuIGZhbHNlIGlmIFRocmVhZCBBIGhhcyBwcm9ncmVzc2VkIHRvICd1cGRhdGluZyB0aGUg cHRlcy4NCg0KRm9yIGluc3RhbmNlIGhlcmUgaXMgaG93IHRoZSBjb25jdXJyZW5jeSByZXNvbHZl cyBmb3IgcmV0cnk6DQoNCiAgICAgICBDUFUxICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICBDUFUyDQogICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgc2VxID0gbW11X3Jhbmdl X3JlYWRfYmVnaW4oKQ0KaW52YWxpZGF0ZV9yYW5nZV9zdGFydCgpDQogIGludmFsaWRhdGVfc2Vx KysgDQogICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgZ2V0X3VzZXJfcGFnZXMoKS9o bW1fcmFuZ2VfZmF1bHQoKQ0KICBncmFiX2RyaXZlcl9sb2NrKCkNCiAgdW5ncmFiX2RyaXZlcl9s b2NrKCkNCiAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICBncmFiX2RyaXZlcl9sb2Nr KCkNCiAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICBzZXEgIT0gaW52YWxpZGF0ZV9z ZXEsIHNvIHJldHJ5DQoNCiAgVXBkYXRpbmcgdGhlIHB0ZXMNCg0KaW52YWxpZGF0ZV9yYW5nZV9l bmQoKQ0KICBpbnZhbGlkYXRlX3NlcSsrDQoNCg0KQW5kIGhlcmUgaXMgaG93IGl0IHJlc29sdmVz IGZvciBibG9ja2luZzoNCg0KICAgICAgIENQVTEgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgIENQVTINCiAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICBzZXEgPSBtbXVfcmFu Z2VfcmVhZF9iZWdpbigpDQppbnZhbGlkYXRlX3JhbmdlX3N0YXJ0KCkNCiAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICBnZXRfdXNlcl9wYWdlcygpL2htbV9yYW5nZV9mYXVsdCgpDQoN CiAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICBncmFiX2RyaXZlcl9sb2NrKCkNCiAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICBzZXEgPT0gaW52YWxpZGF0ZV9zZXEsIHNv IGNvbml0bnVlDQogIGludmFsaWRhdGVfc2VxKysgDQogICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgdW5ncmFiX2RyaXZlcl9sb2NrKCkNCiAgZ3JhYl9kcml2ZXJfbG9jaygpDQogIC8v IENhbm5vdCBwcm9ncmVzcyB0byAnVXBkYXRpbmcgdGhlIHB0ZXMnIHdoaWxlIHRoZSBkcml2ZV9s b2NrIGlzIGhlbGQNCiAgdW5ncmFiX2RyaXZlcl9sb2NrKCkNCiAgVXBkYXRpbmcgdGhlIHB0ZXMN Cg0KaW52YWxpZGF0ZV9yYW5nZV9lbmQoKQ0KICBpbnZhbGlkYXRlX3NlcSsrDQoNCkZvciB0aGUg YWJvdmUgSSd2ZSBzaW1wbGlmaWVkIHRoZSBtZWNoYW5pY3Mgb2YgdGhlIGludmFsaWRhdGVfc2Vx LCB5b3UNCm5lZWQgdG8gbG9vayB0aHJvdWdoIHRoZSBwYXRjaCB0byBzZWUgaG93IGl0IGFjdHVh bGx5IHdvcmtzLg0KDQo+IFdlbGwgd2UgZG9uJ3QgdXBkYXRlIHRoZSBzZXFsb2NrIGFmdGVyIHRo ZSB1cGRhdGUgdG8gdGhlIHByb3RlY3RlZCBkYXRhIA0KPiBzdHJ1Y3R1cmUgKHRoZSBwYWdlIHRh YmxlKSBoYXBwZW5lZCwgYnV0IHJhdGhlciBiZWZvcmUgdGhhdC4NCg0KPz8/IFRoaXMgaXMgd2hh dCBtbl9pdHJlZV9pbnZfZW5kKCkgZG9lcywgaXQgaXMgY2FsbGVkIGJ5DQppbnZhbGlkYXRlX3Jh bmdlX2VuZA0KDQo+IFRoYXQgZG9lc24ndCBsb29rcyBsaWtlIHRoZSBub3JtYWwgcGF0dGVyIGZv ciBhIHNlcWxvY2sgdG8gbWUgYW5kIGFzIGZhciANCj4gYXMgSSBjYW4gc2VlIHRoYXQgaXMgcXVp dGUgYSBidWcgaW4gdGhlIEhNTSBkZXNpZ24vbG9naWMuDQoNCldlbGwsIGhtbSBoYXMgYSBidWcg YmVjYXVzZSBpdCBkb2Vzbid0IHVzZSBhIHNlcWxvY2sgcGF0dGVybiwgc2VlIHRoZQ0KYWJvdmUg VVJMLg0KDQpPbmUgb2YgdGhlIG1vdGl2YXRpb25zIGZvciB0aGlzIHdvcmsgaXMgdG8gc3F1YXNo IHRoYXQgYnVnIGJ5IGFkZGluZyBhDQpzZXFsb2NrIGxpa2UgcGF0dGVybi4gQnV0IHRoZSBiYXNp YyBobW0gZmxvdyBhbmQgY29sbGlzaW9uLXJldHJ5DQphcHByb2FjaCBzZWVtcyBzb3VuZC4NCg0K RG8geW91IHNlZSBhIHByb2JsZW0gd2l0aCB0aGlzIHBhdGNoPw0KDQpKYXNvbg0KX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVsIG1haWxpbmcg bGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlzdHMuZnJlZWRl c2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVs