From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jerome Glisse Subject: Re: [PATCH 2/3] drm/ttm: fix object deallocation to properly fill in the page pool. Date: Tue, 7 Jul 2015 13:41:59 -0400 Message-ID: <20150707174158.GB12288@gmail.com> References: <1407901926-24516-1-git-send-email-j.glisse@gmail.com> <1407901926-24516-3-git-send-email-j.glisse@gmail.com> <559A4641.1000408@daenzer.net> <20150706161025.GA3489@gmail.com> <559B7421.50101@daenzer.net> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mail-qg0-f47.google.com (mail-qg0-f47.google.com [209.85.192.47]) by gabe.freedesktop.org (Postfix) with ESMTPS id D46A06E739 for ; Tue, 7 Jul 2015 10:42:03 -0700 (PDT) Received: by qgii30 with SMTP id i30so87931399qgi.1 for ; Tue, 07 Jul 2015 10:42:02 -0700 (PDT) Content-Disposition: inline In-Reply-To: <559B7421.50101@daenzer.net> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Michel =?iso-8859-1?Q?D=E4nzer?= Cc: Thomas Hellstrom , Jack.Xiao@amd.com, dri-devel@lists.freedesktop.org, Konrad Rzeszutek Wilk List-Id: dri-devel@lists.freedesktop.org T24gVHVlLCBKdWwgMDcsIDIwMTUgYXQgMDM6Mzk6MjlQTSArMDkwMCwgTWljaGVsIETDpG56ZXIg d3JvdGU6Cj4gT24gMDcuMDcuMjAxNSAwMToxMCwgSmVyb21lIEdsaXNzZSB3cm90ZToKPiA+IE9u IE1vbiwgSnVsIDA2LCAyMDE1IGF0IDA2OjExOjI5UE0gKzA5MDAsIE1pY2hlbCBEw6RuemVyIHdy b3RlOgo+ID4+Cj4gPj4gSGkgSsOpcsO0bWUsCj4gPj4KPiA+PiBPbiAxMy4wOC4yMDE0IDEyOjUy LCBKw6lyw7RtZSBHbGlzc2Ugd3JvdGU6Cj4gPj4+IEZyb206IErDqXLDtG1lIEdsaXNzZSA8amds aXNzZUByZWRoYXQuY29tPgo+ID4+Pgo+ID4+PiBDdXJyZW50IGNvZGUgbmV2ZXIgYWxsb3dlZCB0 aGUgcGFnZSBwb29sIHRvIGFjdHVhbHkgZmlsbCBpbiBhbnl3YXkuIFRoaXMgZml4Cj4gPj4+IGl0 IGFuZCBhbHNvIGFsbG93IGl0IHRvIGdyb3cgb3ZlciBpdHMgbGltaXQgdW50aWwgaXQgZ3JvdyBi ZXlvbmQgdGhlIGJhdGNoCj4gPj4+IHNpemUgZm9yIGFsbG9jYXRpb24gYW5kIGRlYWxsb2NhdGlv bi4KPiA+Pj4KPiA+Pj4gU2lnbmVkLW9mZi1ieTogSsOpcsO0bWUgR2xpc3NlIDxqZ2xpc3NlQHJl ZGhhdC5jb20+Cj4gPj4+IFJldmlld2VkLWJ5OiBNYXJpbyBLbGVpbmVyIDxtYXJpby5rbGVpbmVy LmRlQGdtYWlsLmNvbT4KPiA+Pj4gVGVzdGVkLWJ5OiBNaWNoZWwgRMOkbnplciA8bWljaGVsQGRh ZW56ZXIubmV0Pgo+ID4+PiBDYzogVGhvbWFzIEhlbGxzdHJvbSA8dGhlbGxzdHJvbUB2bXdhcmUu Y29tPgo+ID4+PiBDYzogS29ucmFkIFJ6ZXN6dXRlayBXaWxrIDxrb25yYWQud2lsa0BvcmFjbGUu Y29tPgo+ID4+PiAtLS0KPiA+Pj4gIGRyaXZlcnMvZ3B1L2RybS90dG0vdHRtX3BhZ2VfYWxsb2Nf ZG1hLmMgfCA5ICsrLS0tLS0tLQo+ID4+PiAgMSBmaWxlIGNoYW5nZWQsIDIgaW5zZXJ0aW9ucygr KSwgNyBkZWxldGlvbnMoLSkKPiA+Pj4KPiA+Pj4gZGlmZiAtLWdpdCBhL2RyaXZlcnMvZ3B1L2Ry bS90dG0vdHRtX3BhZ2VfYWxsb2NfZG1hLmMgYi9kcml2ZXJzL2dwdS9kcm0vdHRtL3R0bV9wYWdl X2FsbG9jX2RtYS5jCj4gPj4+IGluZGV4IGM5NmRiNDMuLmEwNzZmZjMgMTAwNjQ0Cj4gPj4+IC0t LSBhL2RyaXZlcnMvZ3B1L2RybS90dG0vdHRtX3BhZ2VfYWxsb2NfZG1hLmMKPiA+Pj4gKysrIGIv ZHJpdmVycy9ncHUvZHJtL3R0bS90dG1fcGFnZV9hbGxvY19kbWEuYwo+ID4+PiBAQCAtOTUzLDE0 ICs5NTMsOSBAQCB2b2lkIHR0bV9kbWFfdW5wb3B1bGF0ZShzdHJ1Y3QgdHRtX2RtYV90dCAqdHRt X2RtYSwgc3RydWN0IGRldmljZSAqZGV2KQo+ID4+PiAgCX0gZWxzZSB7Cj4gPj4+ICAJCXBvb2wt Pm5wYWdlc19mcmVlICs9IGNvdW50Owo+ID4+PiAgCQlsaXN0X3NwbGljZSgmdHRtX2RtYS0+cGFn ZXNfbGlzdCwgJnBvb2wtPmZyZWVfbGlzdCk7Cj4gPj4+IC0JCW5wYWdlcyA9IGNvdW50Owo+ID4+ PiAtCQlpZiAocG9vbC0+bnBhZ2VzX2ZyZWUgPiBfbWFuYWdlci0+b3B0aW9ucy5tYXhfc2l6ZSkg ewo+ID4+PiArCQlpZiAocG9vbC0+bnBhZ2VzX2ZyZWUgPj0gKF9tYW5hZ2VyLT5vcHRpb25zLm1h eF9zaXplICsKPiA+Pj4gKwkJCQkJICBOVU1fUEFHRVNfVE9fQUxMT0MpKQo+ID4+PiAgCQkJbnBh Z2VzID0gcG9vbC0+bnBhZ2VzX2ZyZWUgLSBfbWFuYWdlci0+b3B0aW9ucy5tYXhfc2l6ZTsKPiA+ Pj4gLQkJCS8qIGZyZWUgYXQgbGVhc3QgTlVNX1BBR0VTX1RPX0FMTE9DIG51bWJlciBvZiBwYWdl cwo+ID4+PiAtCQkJICogdG8gcmVkdWNlIGNhbGxzIHRvIHNldF9tZW1vcnlfd2IgKi8KPiA+Pj4g LQkJCWlmIChucGFnZXMgPCBOVU1fUEFHRVNfVE9fQUxMT0MpCj4gPj4+IC0JCQkJbnBhZ2VzID0g TlVNX1BBR0VTX1RPX0FMTE9DOwo+ID4+PiAtCQl9Cj4gPj4+ICAJfQo+ID4+PiAgCXNwaW5fdW5s b2NrX2lycXJlc3RvcmUoJnBvb2wtPmxvY2ssIGlycV9mbGFncyk7Cj4gPj4+ICAKPiA+Pj4KPiA+ Pgo+ID4+IENvbGxlYWd1ZXMgb2YgbWluZSBoYXZlIG1lYXN1cmVkIHNpZ25pZmljYW50IHBlcmZv cm1hbmNlIGdhaW5zIGZvciBzb21lCj4gPj4gd29ya2xvYWRzIHdpdGggdGhpcyBwYXRjaC4gV2l0 aG91dCBpdCwgYSBsb3Qgb2YgQ1BVIGN5Y2xlcyBhcmUgc3BlbnQKPiA+PiBjaGFuZ2luZyB0aGUg Y2FjaGluZyBhdHRyaWJ1dGVzIG9mIHBhZ2VzIG9uIGFsbG9jYXRpb24uCj4gPj4KPiA+PiBOb3Rl IHRoYXQgdGhlIHBlcmZvcm1hbmNlIGVmZmVjdCBzZWVtcyB0byBtb3N0bHkgZGlzYXBwZWFyIHdo ZW4gYXBwbHlpbmcKPiA+PiBwYXRjaCAxIGFzIHdlbGwsIHNvIGFwcGFyZW50bHkgNjRNQiBpcyB0 b28gc21hbGwgZm9yIHRoZSBtYXggcG9vbCBzaXplLgo+ID4+Cj4gPj4gSXMgdGhlcmUgYW55IGNo YW5jZSB0aGlzIHBhdGNoIGNvdWxkIGJlIGFwcGxpZWQgd2l0aG91dCB0aGUKPiA+PiBjb250cm92 ZXJzaWFsIHBhdGNoIDM/IElmIG5vdCwgZG8geW91IGhhdmUgaWRlYXMgZm9yIGFkZHJlc3Npbmcg dGhlCj4gPj4gY29uY2VybnMgcmFpc2VkIGFnYWluc3QgcGF0Y2ggMz8KPiA+IAo+ID4gV2Fob3Us IG5vdyBpIG5lZWQgdG8gZmluZCB0aGUga2V5cyB0byB0aGUgRGVMb3JlYW4gdG8gdHJhdmVsIGJh Y2sgaW4gdGltZS4KPiA+IAo+ID4gVGhpcyBwYXRjaCBpcyBhIGZpeCBhbmQgc2hvdWxkIGJlIGFw cGxpZWQgd2l0aG91dCAxIG9yIDMuIEJlY2F1c2UgdG9kYXkKPiA+IGJhc2ljbHkgdGhlIHBvb2wg aXMgYWx3YXlzIGVtcHRpZWQgYW5kIG5ldmVyIGZpbGwgdXAuIEJ1dCBhcyBUaG9tYXMgcG9pbnRl ZAo+ID4gb3V0IHRoZXJlIGlzIGFscmVhZHkgYml0IHRvbyBtdWNoIHBvb2wgYWNjcm9zcyB0aGUg c3RhY2suIFByb3BlciBzb2x1dGlvbgo+ID4gd291bGQgYmUgdG8gd29yayBzb21ldGhpbmcgaW5z aWRlIHRoZSBtbSBsZXZlbCBvciB0aGUgYXJjaGl0ZWN0dXJlIChpIGFzc3VtZQo+ID4gQU1EIGlz IG1vc3RseSBpbnRlcmVzdGVkIGluIHg4NiBvbiB0aGF0IGZyb250KS4KPiA+IAo+ID4gSGVyZSB0 aGUgd2hvbGUgaXNzdWUgaXMgcmVhbGx5IGFib3V0IGFsbG9jYXRpbmcgcGFnZSB3aXRoIFdDL1VD IGNhY2hlCj4gPiBwcm9wZXJ0aWVzLiBDaGFuZ2luZyBjYWNoZSBwcm9wZXJ0aWVzIG9uIHBhZ2Ug aXMgcmVhbGx5IGNvc3RseSBvbiBzZXZlcmFsCj4gPiBsZXZlbCwgbGlrZSB0aGUga2VybmVsIG5l ZWRzIHRvIGJyZWFrIHRoZSBodWdlIGxpbmVhciBtYXBwaW5nIGFuZCBwb3B1bGF0ZQo+ID4gbG93 ZXIgbGV2ZWwgdG8gcmVtYXAgdGhlIHBhZ2Ugd2l0aCBwcm9wZXIgY2FjaGUgYXR0cmlidXRlIGlu c2lkZSB0aGUga2VybmVsCj4gPiBtYXBwaW5nLgo+ID4gCj4gPiBBcyBmYXIgYXMgaSByZW1lbWJl ciB0aGUga2VybmVsIG5ldmVyIGdvZXMgYmFjayB0byBodWdlIHBhZ2UgbWFwcGluZyB3aGVuCj4g PiByZXN0b3JpbmcgcGFnZSBjYWNoZSBhdHRyaWJ1dGUsIHdoaWNoIG1lYW5pbmcgdGhhdCBsaXR0 bGUgYnkgbGl0dGUgd2l0aAo+ID4gdXB0aW1lIHlvdSBsb29zZSB0aGUgd2hvbGUgaHVnZSBwYWdl IG1hcHBpbmcgYmVuZWZpdCBmb3IgZXZlcnl0aGluZyBhbmQgeW91Cj4gPiB3YXN0ZSBtb3JlIG1l bW9yeS4KPiAKPiBUaGF0IHNvdW5kcyBwcmV0dHkgYmFkLCBidXQgdGhpcyBwYXRjaCBtaWdodCBh Y3R1YWxseSBoZWxwIGEgbGl0dGxlIGZvcgo+IHRoYXQgYnkgcmVkdWNpbmcgdGhlIGFtb3VudCBv ZiBodWdlIHBhZ2UgbWFwcGluZ3MgdGhhdCBuZWVkIHRvIGJlIGJyb2tlbiB1cD8KCk5vdCByZWFs bHksIGZvciBsaW1pdGluZyBodWdlIHBhZ2UgbWFwcGluZyBicmVha2FnZSB5b3Ugd291bGQgbmVl ZCB0byBhbGxvY2F0ZQpwYWdlIGluIHNhbWUgcGh5c2ljYWwgY2x1c3RlciBzbyB0aGF0IG9ubHkg MSBodWdlIHBhZ2UgbWFwcGluZyBuZWVkcyB0byBiZQpicm9rZW4uIEl0IHdvdWxkIGJlIGEgYml0 IGxpa2UgRE1BMzIgb3IgRE1BMTYgcGh5c2ljYWwgcmFuZ2UuIFNvIHRoaXMgd291bGQKb2J2aW91 c2x5IG5lZWQgc29tZSB3b3JrIGF0IHRoZSBNTSBsZXZlbC4gQXQgdHRtIGxldmVsIHRoaXMgY2Fu IGJlIG1vcmUgb3IKbGVzcyBpbXBsZW1lbnRlZCB1c2luZyBHRlBfRE1BMzIgZmxhZyBvbiBwYWdl IGFsbG9jYXRpb24gYnV0IGF0IHRoZSBzYW1lIHRpbWUKZG9pbmcgdGhhdCB5b3Uga2luZCBvZiBw dXQgbW9yZSBwcmVzc3VyZSBvbiB0aGUgZmlyc3QgNEcgb2YgbWVtb3J5IGFuZCBpCnRoaW5rIG5v d2FkYXlzIHdpdGggdGhlIHdob2xlIHdlYmJyb3dzZXIgY29uc3VtaW5nIHNldmVyYWwgR0Igb2Yg dGV4dHVyZSB5b3UKcHJvYmFibHkgZG8gbm90IHdhbnQgdG8gZG8gdGhhdC4KCgo+ID4gSW4gbWVh bnRpbWUgaSB0aGluayB3ZSBuZWVkIHRvIGFwcGx5IHRoaXMgcGF0Y2ggYXMgaXQgaXMgcmVhbGx5 IGEgZml4IHRvCj4gPiBtYWtlIHRoZSBjb2RlIGFjdHVhbGx5IGRvIHdoYXQgdGhlIGNvbW1lbnQg YW5kIGRlc2lnbiBwcmV0ZW5kcyBpdCBkb2VzLgo+IAo+IEkgYWdyZWUuCj4gCj4gQlRXLCBtYXli ZSBpdCBzaG91bGQgYmUgc3BsaXQgdXAgaW50byB0aGUgYWN0dWFsIGZpeCAocmVtb3ZpbmcgdGhl Cj4gbnBhZ2VzIGFzc2lnbm1lbnQpIGFuZCB0aGUgTlVNX1BBR0VTX1RPX0FMTE9DIHJlbGF0ZWQg c2ltcGxpZmljYXRpb24/CgpUaGlzIHdvdWxkIG1ha2UgMiByZWFsbHkgc21hbGwgcGF0Y2gsIHBh dGNoIGlzIHNtYWxsIGFzIGl0IGlzIDopIEJ1dCB3aHkKbm90LgoKQ2hlZXJzLApKw6lyw7RtZQpf X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpkcmktZGV2ZWwg bWFpbGluZyBsaXN0CmRyaS1kZXZlbEBsaXN0cy5mcmVlZGVza3RvcC5vcmcKaHR0cDovL2xpc3Rz LmZyZWVkZXNrdG9wLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2RyaS1kZXZlbAo=