From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jason Gunthorpe Subject: Re: [PATCH v3 hmm 11/12] mm/hmm: Remove confusing comment and logic from hmm_release Date: Wed, 19 Jun 2019 08:56:32 -0300 Message-ID: <20190619115632.GC9360@ziepe.ca> References: <20190614004450.20252-1-jgg@ziepe.ca> <20190614004450.20252-12-jgg@ziepe.ca> <20190615142106.GK17724@infradead.org> <20190618004509.GE30762@ziepe.ca> <20190618053733.GA25048@infradead.org> <20190619080705.GA5164@infradead.org> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mail-qt1-x842.google.com (mail-qt1-x842.google.com [IPv6:2607:f8b0:4864:20::842]) by gabe.freedesktop.org (Postfix) with ESMTPS id 736FF6E379 for ; Wed, 19 Jun 2019 11:56:34 +0000 (UTC) Received: by mail-qt1-x842.google.com with SMTP id n11so19431440qtl.5 for ; Wed, 19 Jun 2019 04:56:34 -0700 (PDT) Content-Disposition: inline In-Reply-To: <20190619080705.GA5164@infradead.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Christoph Hellwig Cc: Andrea Arcangeli , "Yang, Philip" , Ralph Campbell , "linux-rdma@vger.kernel.org" , John Hubbard , "Kuehling, Felix" , "dri-devel@lists.freedesktop.org" , "linux-mm@kvack.org" , Jerome Glisse , "amd-gfx@lists.freedesktop.org" , Ben Skeggs List-Id: dri-devel@lists.freedesktop.org T24gV2VkLCBKdW4gMTksIDIwMTkgYXQgMDE6MDc6MDVBTSAtMDcwMCwgQ2hyaXN0b3BoIEhlbGx3 aWcgd3JvdGU6Cj4gT24gV2VkLCBKdW4gMTksIDIwMTkgYXQgMTI6NTM6NTVBTSArMDAwMCwgS3Vl aGxpbmcsIEZlbGl4IHdyb3RlOgo+ID4gVGhpcyBjb2RlIGlzIGRlcml2ZWQgZnJvbSBvdXIgb2xk IE1NVSBub3RpZmllciBjb2RlLiBCZWZvcmUgSE1NIHdlIHVzZWQgCj4gPiB0byByZWdpc3RlciBh IHNpbmdsZSBNTVUgbm90aWZpZXIgcGVyIG1tX3N0cnVjdCBhbmQgbG9vayB1cCB2aXJ0dWFsIAo+ ID4gYWRkcmVzcyByYW5nZXMgdGhhdCBoYWQgYmVlbiByZWdpc3RlcmVkIGZvciBtaXJyb3Jpbmcg dmlhIGRyaXZlciBBUEkgCj4gPiBjYWxscy4gVGhlIGlkZWEgd2FzIHRvIHJldXNlIGEgc2luZ2xl IE1NVSBub3RpZmllciBmb3IgdGhlIGxpZmUgdGltZSBvZiAKPiA+IHRoZSBwcm9jZXNzLiBJdCB3 b3VsZCByZW1haW4gcmVnaXN0ZXJlZCB1bnRpbCB3ZSBnb3QgYSBub3RpZmllcl9yZWxlYXNlLgo+ ID4gCj4gPiBobW1fbWlycm9yIHRvb2sgdGhlIHBsYWNlIG9mIHRoYXQgd2hlbiB3ZSBjb252ZXJ0 ZWQgdGhlIGNvZGUgdG8gSE1NLgo+ID4gCj4gPiBJIHN1cHBvc2Ugd2UgY291bGQgZGVzdHJveSB0 aGUgbWlycm9yIGVhcmxpZXIsIHdoZW4gd2UgaGF2ZSBubyBtb3JlIAo+ID4gcmVnaXN0ZXJlZCB2 aXJ0dWFsIGFkZHJlc3MgcmFuZ2VzLCBhbmQgY3JlYXRlIGEgbmV3IG9uZSBpZiBuZWVkZWQgbGF0 ZXIuCj4gCj4gSSBkaWRuJ3Qgd3JpdGUgdGhlIGNvZGUsIGJ1dCBpZiB5b3UgbG9vayBhdCBobW1f bWlycm9yIGl0IGFscmVhZHkgaXMKPiBhIG11bHRpcGxleGVyIG92ZXIgdGhlIG1tdSBub3RpZmll ciwgYW5kIHRoZSBpbnRlbnQgY2xlYXJseSBzZWVtcyB0aGF0Cj4geW91IHJlZ2lzdGVyIG9uZSBw ZXIgcmFuZ2UgdGhhdCB5b3Ugd2FudCB0byBtaXJyb3IsIGFuZCBub3QgbXVsdGlwbGV4Cj4gaXQg b25jZSBhZ2Fpbi4gIEluIG90aGVyIHdvcmRzIC0gSSB0aGluayBlYWNoIGFtZGdwdV9tbl9ub2Rl IHNob3VsZAo+IHByb2JhYmx5IGhhdmUgaXRzIG93biBobW1fbWlycm9yLiAgQW5kIHdoaWxlIHRo ZSBhbWRncHVfbW5fbm9kZSBvYmplY3RzCj4gYXJlIGN1cnJlbnRseSBzdG9yZWQgaW4gYW4gaW50 ZXJ2YWwgdHJlZSBpdCBzZWVtcyBsaWtlIHRoZXkgYXJlIG9ubHkKPiBsaW5lYXJseSBpdGVyYXRl ZCBhbnl3YXksIHNvIGEgbGlzdCBhY3R1YWxseSBzZWVtcyBwcmV0dHkgc3VpdGFibGUuICBJZgo+ IG5vdCB3ZSBuZWVkIHRvIGltcHJvdmUgdGhlIGNvcmUgZGF0YSBzdHJ1Y3R1cmVzIGluc3RlYWQg b2Ygd29ya2luZwo+IGFyb3VuZCB0aGVtLgoKVGhpcyBsb29rcyBhIGxvdCBsaWtlIHRoZSBPRFAg Y29kZSAoYW1kZ3B1X21uX25vZGUgPT0gaWJfdW1lbV9vZHApCgpUaGUgaW50ZXJ2YWwgdHJlZSBp cyB0byBxdWlja2x5IGZpbmQgdGhlIGRyaXZlciBvYmplY3QocykgdGhhdCBoYXZlCnRoZSB2aXJ0 dWFsIHBhZ2VzIGR1cmluZyBpbnZhbGlkYXRpb246CgpzdGF0aWMgaW50IGFtZGdwdV9tbl9zeW5j X3BhZ2V0YWJsZXNfZ2Z4KHN0cnVjdCBobW1fbWlycm9yICptaXJyb3IsCiAgICAgICAgICAgICAg ICAgICAgICAgIGNvbnN0IHN0cnVjdCBobW1fdXBkYXRlICp1cGRhdGUpCnsKICAgICAgICBpdCA9 IGludGVydmFsX3RyZWVfaXRlcl9maXJzdCgmYW1uLT5vYmplY3RzLCBzdGFydCwgZW5kKTsKICAg ICAgICB3aGlsZSAoaXQpIHsKICAgICAgICAgICAgICAgIFsuLl0KICAgICAgICAgICAgICAgIGFt ZGdwdV9tbl9pbnZhbGlkYXRlX25vZGUobm9kZSwgc3RhcnQsIGVuZCk7CgpBbmQgZm9sbG93aW5n IHRoZSBPRFAgbW9kZWwgdGhlcmUgc2hvdWxkIGJlIGEgc2luZ2xlIGhtbV9taXJyb3IgcGVyLW1t Cih1c2VyIGNhbiBmb3JrIGFuZCBzdHVmZiwgdGhpcyBpcyBzb21ldGhpbmcgSSB3YW50IHRvIGhh dmUgY29yZSBjb2RlCmhlbHAgd2l0aCkuIAoKVGhlIGhtbV9taXJyb3IgY2FuIGVpdGhlciBleGlz dCBzbyBsb25nIGFzIG9iamVjdHMgZXhpc3QsIG9yIGl0IGNhbgpleGlzdCB1bnRpbCB0aGUgY2hh cmRldiBpcyBjbG9zZWQgLSBidXQgbmV2ZXIgbG9uZ2VyIHRoYW4gdGhlCmNoYXJkZXYncyBsaWZl dGltZS4KCk1heWJlIHdlIHNob3VsZCBiZSBjb25zaWRlcmluZyBwcm92aWRpbmcgYSBtbXUgbm90 aWZpZXIgJiBpbnRlcnZhbAp0cmVlICYgbG9jayBhYnN0cmFjdGlvbiBzaW5jZSBPRFAgJiBBTUQg YXJlIHZlcnkgc2ltaWxhciBoZXJlLi4KCkphc29uCl9fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fCmRyaS1kZXZlbCBtYWlsaW5nIGxpc3QKZHJpLWRldmVsQGxp c3RzLmZyZWVkZXNrdG9wLm9yZwpodHRwczovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWlsbWFu L2xpc3RpbmZvL2RyaS1kZXZlbA==