From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tvrtko Ursulin Subject: Re: [PATCH v1] drm/i915: Fix a false alert of memory leak when free LRC Date: Tue, 24 Nov 2015 10:46:21 +0000 Message-ID: <56543FFD.5040507@linux.intel.com> References: <1445292346-7693-1-git-send-email-yu.dai@intel.com> <1447978226-1588-1-git-send-email-yu.dai@intel.com> <20151120083103.GS17050@phenom.ffwll.local> <5652EBD3.90002@linux.intel.com> <5653938E.20404@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by gabe.freedesktop.org (Postfix) with ESMTP id E69F16E896 for ; Tue, 24 Nov 2015 02:46:29 -0800 (PST) In-Reply-To: <5653938E.20404@intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Yu Dai , Daniel Vetter Cc: intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org Ck9uIDIzLzExLzE1IDIyOjMwLCBZdSBEYWkgd3JvdGU6Cj4gT24gMTEvMjMvMjAxNSAwMjozNCBB TSwgVHZydGtvIFVyc3VsaW4gd3JvdGU6Cj4+IE9uIDIwLzExLzE1IDA4OjMxLCBEYW5pZWwgVmV0 dGVyIHdyb3RlOgo+PiA+IE9uIFRodSwgTm92IDE5LCAyMDE1IGF0IDA0OjEwOjI2UE0gLTA4MDAs IHl1LmRhaUBpbnRlbC5jb20gd3JvdGU6Cj4+ID4+IEZyb206IEFsZXggRGFpIDx5dS5kYWlAaW50 ZWwuY29tPgo+PiA+Pgo+PiA+PiBUaGVyZSBpcyBhIG1lbW9yeSBsZWFrIHdhcm5pbmcgbWVzc2Fn ZSBmcm9tIGk5MTVfZ2VtX2NvbnRleHRfY2xlYW4KPj4gPj4gd2hlbiBHdUMgc3VibWlzc2lvbiBp cyBlbmFibGVkLiBUaGUgcmVhc29uIGlzIHRoYXQgd2hlbiBMUkMgaXMKPj4gPj4gcmVsZWFzZWQs IGl0cyBwcGd0dCBjb3VsZCBiZSBzdGlsbCByZWZlcmVuY2VkLiBUaGUgYXNzdW1wdGlvbiB0aGF0 Cj4+ID4+IGFsbCBWTUFzIGFyZSB1bmJvdW5kIGR1cmluZyByZWxlYXNlIG9mIExSQyBpcyBub3Qg dHJ1ZS4KPj4gPj4KPj4gPj4gdjE6IE1vdmUgdGhlIGNvZGUgaW5zaWRlIGk5MTVfZ2VtX2NvbnRl eHRfY2xlYW4oKSB0byB3aGVyZSBwcGd0dCBpcwo+PiA+PiByZWxlYXNlZCBiZWNhdXNlIGl0IGlz IG5vdCBjbGVhbmluZyBjb250ZXh0IGFueXdheSBidXQgcHBndHQuCj4+ID4+Cj4+ID4+IFNpZ25l ZC1vZmYtYnk6IEFsZXggRGFpIDx5dS5kYWlAaW50ZWwuY29tPgo+PiA+Cj4+ID4gcmV0aXJlX19y ZWFkIGRyb3BzIHRoZSBjdHggKGFuZCBoZW5jZSBwcGd0dCkgcmVmZXJlbmNlIHRvbyBlYXJseSwK Pj4gPiByZXN1bHRpbmcgaW4gdXMgaGl0dGluZyB0aGUgV0FSTklORy4gU2VlIHRoZSBnaWFudCB0 aHJlYWQgd2l0aCBUdnJ0a28sCj4+ID4gQ2hyaXMgYW5kIG1lOgo+PiA+Cj4+ID4gaHR0cDovL3d3 dy5zcGluaWNzLm5ldC9saXN0cy9pbnRlbC1nZngvbXNnNzg5MTguaHRtbAo+PiA+Cj4+ID4gV291 bGQgYmUgZ3JlYXQgaWYgc29tZW9uZSBjb3VsZCB0ZXN0IHRoZSBkaWZmIEkgcG9zdGVkIGluIHRo ZXJlLgo+Pgo+PiBJdCBkb2Vzbid0IHdvcmsgLSBJIGhhdmUgcG9zdGVkIG15IElHVCBzbmlwcGV0 IHdoaWNoIEkgdGhvdWdodAo+PiBleHBsYWluZWQgaXQuCj4KPiBJIHRob3VnaHQgbW92aW5nIHRo ZSBWTUEgbGlzdCBjbGVhbiB1cCBpbnRvIGk5MTVfcHBndHRfcmVsZWFzZSgpIHNob3VsZAo+IHdv cmsuIEhvd2V2ZXIsIGl0IGNyZWF0ZXMgYSBjaGlja2VuICYgZWdnIHByb2JsZW0uIHBwZ3R0X3Jl bGVhc2UoKSByZWx5Cj4gb24gdm1hX3VuYm91bmQoKSB0byBiZSBjYWxsZWQgZmlyc3QgdG8gZGVj cmVhc2UgaXRzIHJlZmNvdW50LiBTbyBjYWxsaW5nCj4gdm1hX3VuYm91bmQoKSBpbnNpZGUgcHBn dHRfcmVsZWFzZSgpIGlzIG5vdCByaWdodC4KPj4gUHJvYmxlbSByZXEgdW5yZWZlcmVuY2UgaW4g b2JqLT5hY3RpdmUgY2FzZS4gV2hlbiBpdCBoaXRzIHRoYXQgcGF0aCBpdAo+PiB3aWxsIG5vdCBt b3ZlIHRoZSBWTUEgdG8gaW5hY3RpdmUgYW5kIHRoZQo+PiBpbnRlbF9leGVjbGlzdHNfcmV0aXJl X3JlcXVlc3RzIHdpbGwgYmUgdGhlIGxhc3QgdW5yZWZlcmVuY2UgZnJvbSB0aGUKPj4gcmV0aXJl IHdvcmtlciB3aGljaCB3aWxsIHRyaWdnZXIgdGhlIFdBUk4uCj4KPiBJIHN0aWxsIHRoaW5rIHRo ZSBwcm9ibGVtIGNvbWVzIGZyb20gdGhlIGFzc3VtcHRpb24gdGhhdCB3aGVuIGxyYyBpcwo+IHJl bGVhc2VkLCBpdHMgYWxsIFZNQXMgc2hvdWxkIGJlIHVuYm91bmQuIFByZWNpc2VseSBJIG1lYW4g dGhlIGNvbW1lbnRzCj4gbWFkZSBmb3IgaTkxNV9nZW1fY29udGV4dF9jbGVhbigpIC0gIlRoaXMg Y29udGV4dCBpcyBnb2luZyBhd2F5IGFuZCB3ZQo+IG5lZWQgdG8gcmVtb3ZlIGFsbCBWTUFzIHN0 aWxsIGFyb3VuZC4iIFJlYWxseSB0aGUgbHJjIGxpZmUgY3ljbGUgaXMKPiBkaWZmZXJlbnQgZnJv bSBwcGd0dCAvIFZNQXMuIENoZWNrIHRoZSBsaW5lIGFmdGVyCj4gaTkxNV9nZW1fY29udGV4dF9j bGVhbigpLiBJdCBpcyBwcGd0dF9wdXQoKS4gSW4gdGhlIGNhc2UgbHJjIGlzIGZyZWVkCj4gZWFy bHksIEl0IHdvbid0IHJlbGVhc2UgcHBndHQgYW55d2F5IGJlY2F1c2UgaXQgaXMgc3RpbGwgcmVm ZXJlbmNlZCBieQo+IFZNQXMuIEFuIGl0IHdpbGwgYmUgZnJlZWQgd2hlbiBubyByZWYgb2YgR0VN IG9iai4KClllcywgYnV0IHRoZSBsYXN0IGlzIGp1c3QgYSBjb25zZXF1ZW5jZSBvZiBob3cgdGhl IGNvZGUgd2FzIGxhaWQgb3V0LCAKbm90IGEgaGFyZCByZXF1aXJlbWVudCBhcyBmYXIgYXMgSSB1 bmRlcnN0YW5kIGl0LgoKV2hlbiBjb250ZXh0IGRlc3RydWN0b3IgcnVucyB0aGF0IG1lYW5zIHR3 byB0aGluZ3M6CgoxLiBHUFUgaXMgZG9uZSB3aXRoIGFsbCBWTUFzIGJlbG9uZ2luZyB0byB0aGUg Vk0uCjIuIFVTZXJzcGFjZSBhbHNvIGNhbm5vdCByZWFjaCB0aGVtIGFueSBtb3JlIGVpdGhlci4K ClNvIFZNQXMgaGF2ZSBubyByZWFzb24gdG8gZXhpc3QgYmV5b25kIHRoYXQgcG9pbnQuIElmIHRo ZXkgZG8sIHRoZXkgY2FuIApiZSBsZWZ0IGRhbmdsaW5nIHVuZGVyIGNlcnRhaW4gY2lyY3Vtc3Rh bmNlcy4gU2VlIGJlbG93LgoKPj4gSSBwb3N0ZWQgYW4gSUdUIHdoaWNoIGhpdHMgdGhhdCAtPgo+ PiBodHRwOi8vcGF0Y2h3b3JrLmZyZWVkZXNrdG9wLm9yZy9wYXRjaC82NTM2OS8KPj4KPj4gQW5k IHBvc3RlZCBvbmUgZ2l2ZSB1cCBvbiB0aGUgYWN0aXZlIFZNQSBtZW0gbGVhayBwYXRjaCAtPgo+ PiBodHRwOi8vcGF0Y2h3b3JrLmZyZWVkZXNrdG9wLm9yZy9wYXRjaC82NTUyOS8KPgo+IFRoaXMg cGF0Y2ggd2lsbCBzaWxlbnQgdGhlIHdhcm5pbmcuIEJ1dCBJIHRoaW5rIHRoZQo+IGk5MTVfZ2Vt X2NvbnRleHRfY2xlYW4oKSBpdHNlbGYgaXMgdW5uZWNlc3NhcnkuIEkgZG9uJ3Qgc2VlIGFueSBp c3N1ZSBieQo+IGRlbGV0aW5nIGl0LiBUaGUgY2hlY2sgb2YgVk1BIGxpc3QgaXMgaW5zaWRlIHBw Z3R0X3JlbGVhc2UoKSBhbmQgdGhlCj4gdW5ib3VuZCBzaG91bGQgYmUgYWxpZ25lZCB0byBHRU0g b2JqJ3MgbGlmZSBjeWNsZSBidXQgbm90IGxyYyBsaWZlIGN5Y2xlLgoKaTkxNV9nZW1fY29udGV4 dF9jbGVhbiBzb2x2ZXMgYSBtZW1vcnkgbGVhayB3aGljaCBpcyBleHBsYWluZWQgaW4gdGhlIApj b21taXQuCgpJZiB0aGVyZSBpcyBhbiBleHRyYSByZWZlcmVuY2Ugb24gdGhlIEdFTSBvYmosIGxp a2UgaW4gdGhlIGZsaW5rIGNhc2UsIApWTUFzIHdpbGwgbm90IGdldCB1bmJvdW5kLgoKQXBhcnQg ZnJvbSB2YXJpb3VzIElHVHMsIHlvdSBjYW4gYWxzbyBkZW1vbnN0cmF0ZSB0aGlzIGxlYWsgd2l0 aCBmYmNvbiAKYW5kIFhvcmcuIEV2ZXJ5IHRpbWUgeW91IHJlLXN0YXJ0IFhvcmcgb25lIFZNQSB3 aWxsIGJlIGxlYWtlZCBzaW5jZSBpdCAKaW1wb3J0cyB0aGUgZmJjb24gZmIgdmlhIGZsaW5rLgoK U2Vjb25kIHBhcnQgb2YgdGhlIHN0b3J5IGlzIHRoYXQgdGhlIFdBUk4gaW4gaTkxNV9nZW1fY29u dGV4dF9jbGVhbiBpcyAKd3JvbmcgYmVjYXVzZSBvZiBob3cgcmV0aXJlbWVudCB3b3Jrcy4gU28g aWYgd2UgcmVtb3ZlZCB0aGUgV0FSTiwgCmNsZWFudXAgc3RpbGwgaGFzIHNvbWUgdmFsdWUgdG8g YXZvaWQgbWVtb3J5IGxlYWsgaW4gdGhlIGFib3ZlIGRlc2NyaWJlZCAKWG9yZyBjYXNlLCBvciBp biBhbnkgY2FzZSB3aGVyZSBmbGluayBpcyBpbiB0aGUgcGljdHVyZSBhbmQgVk1BcyBoYXZlIApi ZWVuIHJldGlyZWQgdG8gaW5hY3RpdmUuCgpCdWcgbGVmdCBzdGFuZGluZyB3b3VsZCBiZSB0aGF0 IGlmIHRoZSBWTUEgaGFwcGVucyB0byBiZSBmbGlua2VkIGFuZCBvbiAKdGhlIGFjdGl2ZSBsaXN0 LCBiZWNhdXNlIG9mIHNheSBiZWluZyBzaGFyZWQgYmV0d2VlbiByaW5ncyBhbmQgY29udGV4dHMs IAppdCB3b3VsZCBzdGlsbCBiZSBsZWFrZWQuIEJ1dCBpdCBpcyBsZXNzIGxlYWtpbmcgdGhhbiB3 aXRob3V0IHRoZSBjbGVhbnVwLgoKSSBwcm9wb3NlZCBhbm90aGVyIHdheSBvZiBmaXhpbmcgdGhh dDogCmh0dHA6Ly9wYXRjaHdvcmsuZnJlZWRlc2t0b3Aub3JnL3BhdGNoLzY1ODYxLwoKPj4gSSBo YXZlIG5vIGlkZWEgeWV0IG9mIEd1QyBpbXBsaWNhdGlvbnMsIEkganVzdCBzcG90dGVkIHRoaXMg cGFyYWxsZWwKPj4gdGhyZWFkLgo+Pgo+PiBBbmQgTWlrYSBoYXMgcHJvcG9zZWQgc29tZXRoaW5n IGludGVyZXN0aW5nIC0gdGhhdCB3ZSBjb3VsZCBqdXN0IGNsZWFuCj4+IHVwIHRoZSBhY3RpdmUg Vk1BIGluIGNvbnRleHQgY2xlYW51cCBzaW5jZSB3ZSBrbm93IGl0IGlzIGEgZmFsc2Ugb25lLgo+ Pgo+PiBIb3dldmVyLCBhZ2FpbiBJIGRvbid0IGtub3cgaG93IHRoYXQgaW50ZXJhY3RzIHdpdGgg dGhlIEd1Qy4gU3VyZWx5IGl0Cj4+IGNhbm5vdCBiZSBmcmVlaW5nIHRoZSBjb250ZXh0IHdpdGgg c3R1ZmYgZ2VudWluZWx5IHN0aWxsIGFjdGl2ZSBpbiB0aGUKPj4gR3VDPwo+Pgo+Cj4gVGhlcmUg aXMgbm8gaW50ZXJhY3RzIHdpdGggR3VDIHRob3VnaC4gSnVzdCB2ZXJ5IGVhc3kgdG8gc2VlIHRo ZSB3YXJuaW5nCj4gd2hlbiBHdUMgaXMgZW5hYmxlZCwgc2F5cyB3aGVuIHJ1biBnZW1fY2xvc2Vf cmFjZS4gVGhlIHJlYXNvbiBpcyB0aGF0Cj4gR3VDIGRvZXMgbm90IHVzZSB0aGUgZXhlY2xpc3Rf cXVldWUgKGV4ZWNsaXN0X3JldGlyZWRfcmVxX2xpc3QpIHdoaWNoIGlzCj4gZGVmZXJyZWQgdG8g cmV0aXJlIHdvcmtlci4gU2FtZSBhcyByaW5nIHN1Ym1pc3Npb24gbW9kZSwgd2hlbiBHdUMgaXMK PiBlbmFibGVkLCB3aGVuZXZlciBkcml2ZXIgc3VibWl0cyBhIG5ldyBiYXRjaCwgaXQgd2lsbCB0 cnkgdG8gcmVsZWFzZQo+IHByZXZpb3VzIHJlcXVlc3QuIEkgZG9uJ3Qga25vdyB3aHkgIGludGVs X2V4ZWNsaXN0c19yZXRpcmVfcmVxdWVzdHMgaXMKPiBub3QgY2FsbGVkIGZvciB0aGlzIGNhc2Uu IFByb2JhYmx5IGJlY2F1c2Ugb2YgdGhlIHVucGluLiBEZWZlcnJpbmcgdGhlCj4gcmV0aXJlbWVu dCBtYXkganVzdCBoaWRlIHRoZSBpc3N1ZS4gSSBiZXQgeW91IHdpbGwgc2VlIHRoZSB3YXJuaW5n IG1vcmUKPiBvZnRlbiBpZiB5b3UgY2hhbmdlIGk5MTVfZ2VtX3JldGlyZV9yZXF1ZXN0c19yaW5n KCkgdG8KPiBpOTE1X2dlbV9yZXRpcmVfcmVxdWVzdHMoKSBpbiBpOTE1X2dlbV9leGVjYnVmZmVy X3Jlc2VydmUoKS4KCkhtbW0uLiBnZW1fY2xvc2VfcmFjZSB1c2VzIGZsaW5rLCBidXQgd2h5IHdv dWxkIGl0IHRyaWdnZXIgY29udGV4dCAKZGVzdHJ1Y3Rpb24gd2l0aCBhY3RpdmUgVk1Bcz8gSG93 IGRvZXMgdGhlIGJhY2t0cmFjZSBsb29rcyBsaWtlIGZyb20gdGhlIApjb250ZXh0IGNsZWFudXAg V0FSTj8KClJlZ2FyZHMsCgpUdnJ0a28KCgoKCgpfX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fXwpJbnRlbC1nZnggbWFpbGluZyBsaXN0CkludGVsLWdmeEBsaXN0 cy5mcmVlZGVza3RvcC5vcmcKaHR0cDovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWlsbWFuL2xp c3RpbmZvL2ludGVsLWdmeAo=