From mboxrd@z Thu Jan 1 00:00:00 1970 From: zhoucm1 Subject: Re: [PATCH] drm/amdgpu: fix memory leak in wait_all_fence Date: Fri, 7 Apr 2017 17:03:57 +0800 Message-ID: <58E755FD.1080501@amd.com> References: <1491536178-5978-1-git-send-email-David1.Zhou@amd.com> <058491a9-73a6-eddf-73fb-aaac066f9b3e@vodafone.de> <58E751D5.50707@amd.com> <6aa1afcf-b2d3-ea84-b08c-3520fffb0b7c@vodafone.de> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <6aa1afcf-b2d3-ea84-b08c-3520fffb0b7c-ANTagKRnAhcb1SvskN2V4Q@public.gmane.org> List-Id: Discussion list for AMD gfx List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: amd-gfx-bounces-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org Sender: "amd-gfx" To: =?UTF-8?B?Q2hyaXN0aWFuIEvDtm5pZw==?= , amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org, Ken.Wang-5C7GfCeVMHo@public.gmane.org CgpPbiAyMDE35bm0MDTmnIgwN+aXpSAxNjo1NSwgQ2hyaXN0aWFuIEvDtm5pZyB3cm90ZToKPiBB bSAwNy4wNC4yMDE3IHVtIDEwOjQ2IHNjaHJpZWIgemhvdWNtMToKPj4KPj4KPj4gT24gMjAxN+W5 tDA05pyIMDfml6UgMTY6MzYsIENocmlzdGlhbiBLw7ZuaWcgd3JvdGU6Cj4+PiBBbSAwNy4wNC4y MDE3IHVtIDA1OjM2IHNjaHJpZWIgQ2h1bm1pbmcgWmhvdToKPj4+PiBDaGFuZ2UtSWQ6IEliM2Uy NzFlMDBlNDlmMTAxNTJjMWIzZWFjZTk4MWE2YmY3ODgyMGRlCj4+Pj4gU2lnbmVkLW9mZi1ieTog Q2h1bm1pbmcgWmhvdSA8RGF2aWQxLlpob3VAYW1kLmNvbT4KPj4+Cj4+PiBOQUssIHRoYXQgd2ls bCBhbGxvY2F0ZSBhbiBhcnJheSBmb3IgdGhlIGZlbmNlIGFnYWluLCB3aGljaCB3ZSAKPj4+IHdh bnRlZCB0byBhdm9pZC4KPj4gSSBkb24ndCBnb3QgeW91ciBtZWFucywgdGhlICoqYXJyYXkganVz dCB0byBzdG9yZSBmZW5jZSB0ZW1wb3JhcnksIAo+PiBmcmVlZCBhdCB0aGUgZW5kLgo+Cj4gWWVh aCwgYnV0IHRoYXQgaXMgdW5uZWNlc3NhcnkuIFdlIGNyZWF0ZWQgdGhpcyBmdW5jdGlvbiB0byBq dXN0IGF2b2lkIAo+IHRoYXQuCj4KPj4KPj4+Cj4+PiBXZSBzaG91bGQganVzdCBkcm9wIHRoZSBm ZW5jZSByZWZlcmVuY2UgZGlyZWN0bHkgYWZ0ZXIgd2FpdGluZyBmb3IgaXQuCj4+IEhvdyB0byBo YW5kbGUgZXJyIGNhc2UgaW4gdGhlIG1pZGRsZS4KPgo+IGp1c3QgcHV0IGEgZmVuY2VfcHV0KCkg ZGlyZWN0bHkgYWZ0ZXIgZmVuY2Vfd2FpdF90aW1lb3V0KCkuIFdlIGRvbid0IAo+IG5lZWQgdG8g a2VlcCBhIHJlZmVyZW5jZSB0byB0aGUgZmVuY2UgdGlsbCB0aGUgZW5kLgpJIHNlZSB5b3VyIG1l YW4sIHdpbGwgc2VuZCBWMiBzb29uLgoKUmVnYXJkcywKRGF2aWQgWmhvdQo+Cj4gUmVnYXJkcywK PiBDaHJpc3RpYW4uCj4KPj4KPj4gUmVnYXJkcywKPj4gRGF2aWQgWmhvdQo+Pj4KPj4+IFJlZ2Fy ZHMsCj4+PiBDaHJpc3RpYW4uCj4+Pgo+Pj4+IC0tLQo+Pj4+ICAgZHJpdmVycy9ncHUvZHJtL2Ft ZC9hbWRncHUvYW1kZ3B1X2NzLmMgfCAyNSAKPj4+PiArKysrKysrKysrKysrKysrKysrLS0tLS0t Cj4+Pj4gICAxIGZpbGUgY2hhbmdlZCwgMTkgaW5zZXJ0aW9ucygrKSwgNiBkZWxldGlvbnMoLSkK Pj4+Pgo+Pj4+IGRpZmYgLS1naXQgYS9kcml2ZXJzL2dwdS9kcm0vYW1kL2FtZGdwdS9hbWRncHVf Y3MuYyAKPj4+PiBiL2RyaXZlcnMvZ3B1L2RybS9hbWQvYW1kZ3B1L2FtZGdwdV9jcy5jCj4+Pj4g aW5kZXggZGUxYzRjMy4uZDg0MjQ1MiAxMDA2NDQKPj4+PiAtLS0gYS9kcml2ZXJzL2dwdS9kcm0v YW1kL2FtZGdwdS9hbWRncHVfY3MuYwo+Pj4+ICsrKyBiL2RyaXZlcnMvZ3B1L2RybS9hbWQvYW1k Z3B1L2FtZGdwdV9jcy5jCj4+Pj4gQEAgLTEyMTYsMjIgKzEyMTYsMjggQEAgc3RhdGljIGludCBh bWRncHVfY3Nfd2FpdF9hbGxfZmVuY2VzKHN0cnVjdCAKPj4+PiBhbWRncHVfZGV2aWNlICphZGV2 LAo+Pj4+ICAgICAgICAgICAgICAgICAgICAgICAgc3RydWN0IGRybV9hbWRncHVfZmVuY2UgKmZl bmNlcykKPj4+PiAgIHsKPj4+PiAgICAgICB1aW50MzJfdCBmZW5jZV9jb3VudCA9IHdhaXQtPmlu LmZlbmNlX2NvdW50Owo+Pj4+ICsgICAgc3RydWN0IGZlbmNlICoqYXJyYXk7Cj4+Pj4gICAgICAg dW5zaWduZWQgaW50IGk7Cj4+Pj4gICAgICAgbG9uZyByID0gMTsKPj4+PiAgICsgICAgYXJyYXkg PSBrY2FsbG9jKGZlbmNlX2NvdW50LCBzaXplb2Yoc3RydWN0IGZlbmNlICopLCAKPj4+PiBHRlBf S0VSTkVMKTsKPj4+PiArCj4+Pj4gKyAgICBpZiAoYXJyYXkgPT0gTlVMTCkKPj4+PiArICAgICAg ICByZXR1cm4gLUVOT01FTTsKPj4+PiAgICAgICBmb3IgKGkgPSAwOyBpIDwgZmVuY2VfY291bnQ7 IGkrKykgewo+Pj4+ICAgICAgICAgICBzdHJ1Y3QgZmVuY2UgKmZlbmNlOwo+Pj4+ICAgICAgICAg ICB1bnNpZ25lZCBsb25nIHRpbWVvdXQgPSAKPj4+PiBhbWRncHVfZ2VtX3RpbWVvdXQod2FpdC0+ aW4udGltZW91dF9ucyk7Cj4+Pj4gICAgICAgICAgICAgZmVuY2UgPSBhbWRncHVfY3NfZ2V0X2Zl bmNlKGFkZXYsIGZpbHAsICZmZW5jZXNbaV0pOwo+Pj4+IC0gICAgICAgIGlmIChJU19FUlIoZmVu Y2UpKQo+Pj4+IC0gICAgICAgICAgICByZXR1cm4gUFRSX0VSUihmZW5jZSk7Cj4+Pj4gLSAgICAg ICAgZWxzZSBpZiAoIWZlbmNlKQo+Pj4+ICsgICAgICAgIGlmIChJU19FUlIoZmVuY2UpKSB7Cj4+ Pj4gKyAgICAgICAgICAgIHIgPSBQVFJfRVJSKGZlbmNlKTsKPj4+PiArICAgICAgICAgICAgZ290 byBlcnI7Cj4+Pj4gKyAgICAgICAgfSBlbHNlIGlmICghZmVuY2UpCj4+Pj4gICAgICAgICAgICAg ICBjb250aW51ZTsKPj4+PiAtCj4+Pj4gKyAgICAgICAgYXJyYXlbaV0gPSBmZW5jZTsKPj4+PiAg ICAgICAgICAgciA9IGtjbF9mZW5jZV93YWl0X3RpbWVvdXQoZmVuY2UsIHRydWUsIHRpbWVvdXQp Owo+Pj4+ICAgICAgICAgICBpZiAociA8IDApCj4+Pj4gLSAgICAgICAgICAgIHJldHVybiByOwo+ Pj4+ICsgICAgICAgICAgICBnb3RvIGVycjsKPj4+PiAgICAgICAgICAgICBpZiAociA9PSAwKQo+ Pj4+ICAgICAgICAgICAgICAgYnJlYWs7Cj4+Pj4gQEAgLTEyNDAsNyArMTI0NiwxNCBAQCBzdGF0 aWMgaW50IGFtZGdwdV9jc193YWl0X2FsbF9mZW5jZXMoc3RydWN0IAo+Pj4+IGFtZGdwdV9kZXZp Y2UgKmFkZXYsCj4+Pj4gICAgICAgbWVtc2V0KHdhaXQsIDAsIHNpemVvZigqd2FpdCkpOwo+Pj4+ ICAgICAgIHdhaXQtPm91dC5zdGF0dXMgPSAociA+IDApOwo+Pj4+ICAgLSAgICByZXR1cm4gMDsK Pj4+PiArICAgIHIgPSAwOwo+Pj4+ICsKPj4+PiArZXJyOgo+Pj4+ICsgICAgZm9yIChpID0gMDsg aSA8IGZlbmNlX2NvdW50OyBpKyspCj4+Pj4gKyAgICAgICAgZmVuY2VfcHV0KGFycmF5W2ldKTsK Pj4+PiArICAgIGtmcmVlKGFycmF5KTsKPj4+PiArCj4+Pj4gKyAgICByZXR1cm4gcjsKPj4+PiAg IH0KPj4+PiAgICAgLyoqCj4+Pgo+Pj4KPj4KPj4gX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX18KPj4gYW1kLWdmeCBtYWlsaW5nIGxpc3QKPj4gYW1kLWdmeEBs aXN0cy5mcmVlZGVza3RvcC5vcmcKPj4gaHR0cHM6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFp bG1hbi9saXN0aW5mby9hbWQtZ2Z4Cj4KPgoKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX18KYW1kLWdmeCBtYWlsaW5nIGxpc3QKYW1kLWdmeEBsaXN0cy5mcmVl ZGVza3RvcC5vcmcKaHR0cHM6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5m by9hbWQtZ2Z4Cg==