From mboxrd@z Thu Jan 1 00:00:00 1970 From: Laurent Pinchart Subject: Re: [PATCH v5 12/19] drm: writeback: Cleanup job ownership handling when queuing job Date: Thu, 21 Feb 2019 23:56:09 +0200 Message-ID: <20190221215609.GB3485@pendragon.ideasonboard.com> References: <20190221103212.28764-1-laurent.pinchart+renesas@ideasonboard.com> <20190221103212.28764-13-laurent.pinchart+renesas@ideasonboard.com> <20190221104225.GJ3451@pendragon.ideasonboard.com> <20190221160238.ezdlfhtlyz3reko3@DESKTOP-E1NTVVP.localdomain> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from perceval.ideasonboard.com (perceval.ideasonboard.com [213.167.242.64]) by gabe.freedesktop.org (Postfix) with ESMTPS id 023B889364 for ; Thu, 21 Feb 2019 21:56:15 +0000 (UTC) Content-Disposition: inline In-Reply-To: <20190221160238.ezdlfhtlyz3reko3@DESKTOP-E1NTVVP.localdomain> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Brian Starkey Cc: Laurent Pinchart , Liviu Dudau , Kieran Bingham , "dri-devel@lists.freedesktop.org" , "james qian wang (Arm Technology China)" , nd List-Id: dri-devel@lists.freedesktop.org SGkgQnJpYW4sCgpPbiBUaHUsIEZlYiAyMSwgMjAxOSBhdCAwNDowMjozN1BNICswMDAwLCBCcmlh biBTdGFya2V5IHdyb3RlOgo+IE9uIFRodSwgRmViIDIxLCAyMDE5IGF0IDEyOjQyOjI1UE0gKzAy MDAsIExhdXJlbnQgUGluY2hhcnQgd3JvdGU6Cj4gPiBPbiBUaHUsIEZlYiAyMSwgMjAxOSBhdCAx MjozMjowNVBNICswMjAwLCBMYXVyZW50IFBpbmNoYXJ0IHdyb3RlOgo+ID4+IFRoZSBkcm1fd3Jp dGViYWNrX3F1ZXVlX2pvYigpIGZ1bmN0aW9uIHRha2VzIG93bmVyc2hpcCBvZiB0aGUgcGFzc2Vk IGpvYgo+ID4+IGFuZCByZXF1aXJlcyB0aGUgY2FsbGVyIHRvIG1hbnVhbGx5IHNldCB0aGUgY29u bmVjdG9yIHN0YXRlCj4gPj4gd3JpdGViYWNrX2pvYiBwb2ludGVyIHRvIE5VTEwuIFRvIHNpbXBs aWZ5IGRyaXZlcnMgYW5kIGF2b2lkIGVycm9ycwo+ID4+IChzdWNoIGFzIHRoZSBtaXNzaW5nIE5V TEwgc2V0IGluIHRoZSB2YzQgZHJpdmVyKSwgcGFzcyB0aGUgY29ubmVjdG9yCj4gPj4gc3RhdGUg cG9pbnRlciB0byB0aGUgZnVuY3Rpb24gaW5zdGVhZCBvZiB0aGUgam9iIHBvaW50ZXIsIGFuZCBz ZXQgdGhlCj4gPj4gd3JpdGViYWNrX2pvYiBwb2ludGVyIHRvIE5VTEwgaW50ZXJuYWxseS4KPiAK PiBMR1RNLCBpdCB3YXMgYSBtaXN0YWtlIG5vdCBkb2luZyBpdCBsaWtlIHRoaXMgZnJvbSB0aGUg c3RhcnQuCj4gCj4gSSBkbyBoYXZlIG9uZSBzdWdnZXN0aW9uIGJlbG93LCBidXQgZWl0aGVyIHdh eToKPiAKPiBSZXZpZXdlZC1ieTogQnJpYW4gU3RhcmtleSA8YnJpYW4uc3RhcmtleUBhcm0uY29t Pgo+IAo+ID4+IFNpZ25lZC1vZmYtYnk6IExhdXJlbnQgUGluY2hhcnQgPGxhdXJlbnQucGluY2hh cnQrcmVuZXNhc0BpZGVhc29uYm9hcmQuY29tPgo+ID4+IC0tLQo+ID4+ICBkcml2ZXJzL2dwdS9k cm0vYXJtL21hbGlkcF9tdy5jIHwgIDMgKy0tCj4gPj4gIGRyaXZlcnMvZ3B1L2RybS9kcm1fd3Jp dGViYWNrLmMgfCAxNSArKysrKysrKysrLS0tLS0KPiA+PiAgZHJpdmVycy9ncHUvZHJtL3ZjNC92 YzRfdHhwLmMgICB8ICAyICstCj4gPj4gIGluY2x1ZGUvZHJtL2RybV93cml0ZWJhY2suaCAgICAg fCAgMiArLQo+ID4+ICA0IGZpbGVzIGNoYW5nZWQsIDEzIGluc2VydGlvbnMoKyksIDkgZGVsZXRp b25zKC0pCj4gPj4gCj4gPj4gZGlmZiAtLWdpdCBhL2RyaXZlcnMvZ3B1L2RybS9hcm0vbWFsaWRw X213LmMgYi9kcml2ZXJzL2dwdS9kcm0vYXJtL21hbGlkcF9tdy5jCj4gPj4gaW5kZXggMDQxYTY0 ZGM3MTY3Li44NzYyNzIxOWNlM2IgMTAwNjQ0Cj4gPj4gLS0tIGEvZHJpdmVycy9ncHUvZHJtL2Fy bS9tYWxpZHBfbXcuYwo+ID4+ICsrKyBiL2RyaXZlcnMvZ3B1L2RybS9hcm0vbWFsaWRwX213LmMK PiA+PiBAQCAtMjUyLDggKzI1Miw3IEBAIHZvaWQgbWFsaWRwX213X2F0b21pY19jb21taXQoc3Ry dWN0IGRybV9kZXZpY2UgKmRybSwKPiA+PiAgCQkJCSAgICAgJm13X3N0YXRlLT5hZGRyc1swXSwK PiA+PiAgCQkJCSAgICAgbXdfc3RhdGUtPmZvcm1hdCk7Cj4gPj4gIAo+ID4+IC0JCWRybV93cml0 ZWJhY2tfcXVldWVfam9iKG13X2Nvbm4sIGNvbm5fc3RhdGUtPndyaXRlYmFja19qb2IpOwo+ID4+ IC0JCWNvbm5fc3RhdGUtPndyaXRlYmFja19qb2IgPSBOVUxMOwo+ID4+ICsJCWRybV93cml0ZWJh Y2tfcXVldWVfam9iKG13X2Nvbm4sIGNvbm5fc3RhdGUpOwo+ID4+ICAJCWh3ZGV2LT5ody0+ZW5h YmxlX21lbXdyaXRlKGh3ZGV2LCBtd19zdGF0ZS0+YWRkcnMsCj4gPj4gIAkJCQkJICAgbXdfc3Rh dGUtPnBpdGNoZXMsIG13X3N0YXRlLT5uX3BsYW5lcywKPiA+PiAgCQkJCQkgICBmYi0+d2lkdGgs IGZiLT5oZWlnaHQsIG13X3N0YXRlLT5mb3JtYXQsCj4gPj4gZGlmZiAtLWdpdCBhL2RyaXZlcnMv Z3B1L2RybS9kcm1fd3JpdGViYWNrLmMgYi9kcml2ZXJzL2dwdS9kcm0vZHJtX3dyaXRlYmFjay5j Cj4gPj4gaW5kZXggYzIwZTZmZTAwY2IzLi4zMzhiOTkzZDdjOWYgMTAwNjQ0Cj4gPj4gLS0tIGEv ZHJpdmVycy9ncHUvZHJtL2RybV93cml0ZWJhY2suYwo+ID4+ICsrKyBiL2RyaXZlcnMvZ3B1L2Ry bS9kcm1fd3JpdGViYWNrLmMKPiA+PiBAQCAtMjQyLDExICsyNDIsMTIgQEAgRVhQT1JUX1NZTUJP TChkcm1fd3JpdGViYWNrX2Nvbm5lY3Rvcl9pbml0KTsKPiA+PiAgLyoqCj4gPj4gICAqIGRybV93 cml0ZWJhY2tfcXVldWVfam9iIC0gUXVldWUgYSB3cml0ZWJhY2sgam9iIGZvciBsYXRlciBzaWdu YWxsaW5nCj4gPj4gICAqIEB3Yl9jb25uZWN0b3I6IFRoZSB3cml0ZWJhY2sgY29ubmVjdG9yIHRv IHF1ZXVlIGEgam9iIG9uCj4gPj4gLSAqIEBqb2I6IFRoZSBqb2IgdG8gcXVldWUKPiA+PiArICog QGNvbm5fc3RhdGU6IFRoZSBjb25uZWN0b3Igc3RhdGUgY29udGFpbmluZyB0aGUgam9iIHRvIHF1 ZXVlCj4gPj4gICAqCj4gPj4gLSAqIFRoaXMgZnVuY3Rpb24gYWRkcyBhIGpvYiB0byB0aGUgam9i X3F1ZXVlIGZvciBhIHdyaXRlYmFjayBjb25uZWN0b3IuIEl0Cj4gPj4gLSAqIHNob3VsZCBiZSBj b25zaWRlcmVkIHRvIHRha2Ugb3duZXJzaGlwIG9mIHRoZSB3cml0ZWJhY2sgam9iLCBhbmQgc28g YW55IG90aGVyCj4gPj4gLSAqIHJlZmVyZW5jZXMgdG8gdGhlIGpvYiBtdXN0IGJlIGNsZWFyZWQg YWZ0ZXIgY2FsbGluZyB0aGlzIGZ1bmN0aW9uLgo+ID4+ICsgKiBUaGlzIGZ1bmN0aW9uIGFkZHMg dGhlIGpvYiBjb250YWluZWQgaW4gQGNvbm5fc3RhdGUgdG8gdGhlIGpvYl9xdWV1ZSBmb3IgYQo+ ID4+ICsgKiB3cml0ZWJhY2sgY29ubmVjdG9yLiBJdCB0YWtlcyBvd25lcnNoaXAgb2YgdGhlIHdy aXRlYmFjayBqb2IgYW5kIHNldHMgdGhlCj4gPj4gKyAqIEBjb25uX3N0YXRlLT53cml0ZWJhY2tf am9iIHRvIE5VTEwsIGFuZCBzbyBubyBhY2Nlc3MgdG8gdGhlIGpvYiBtYXkgYmUKPiA+PiArICog cGVyZm9ybWVkIGJ5IHRoZSBjYWxsZXIgYWZ0ZXIgdGhpcyBmdW5jdGlvbiByZXR1cm5zLgo+ID4+ ICAgKgo+ID4+ICAgKiBEcml2ZXJzIG11c3QgZW5zdXJlIHRoYXQgZm9yIGEgZ2l2ZW4gd3JpdGVi YWNrIGNvbm5lY3Rvciwgam9icyBhcmUgcXVldWVkIGluCj4gPj4gICAqIGV4YWN0bHkgdGhlIHNh bWUgb3JkZXIgYXMgdGhleSB3aWxsIGJlIGNvbXBsZXRlZCBieSB0aGUgaGFyZHdhcmUgKGFuZAo+ ID4+IEBAIC0yNTgsMTAgKzI1OSwxNCBAQCBFWFBPUlRfU1lNQk9MKGRybV93cml0ZWJhY2tfY29u bmVjdG9yX2luaXQpOwo+ID4+ICAgKiBTZWUgYWxzbzogZHJtX3dyaXRlYmFja19zaWduYWxfY29t cGxldGlvbigpCj4gPj4gICAqLwo+ID4+ICB2b2lkIGRybV93cml0ZWJhY2tfcXVldWVfam9iKHN0 cnVjdCBkcm1fd3JpdGViYWNrX2Nvbm5lY3RvciAqd2JfY29ubmVjdG9yLAo+ID4+IC0JCQkgICAg IHN0cnVjdCBkcm1fd3JpdGViYWNrX2pvYiAqam9iKQo+ID4+ICsJCQkgICAgIHN0cnVjdCBkcm1f Y29ubmVjdG9yX3N0YXRlICpjb25uX3N0YXRlKQo+ID4+ICB7Cj4gPj4gKwlzdHJ1Y3QgZHJtX3dy aXRlYmFja19qb2IgKmpvYjsKPiA+PiAgCXVuc2lnbmVkIGxvbmcgZmxhZ3M7Cj4gPj4gIAo+ID4+ ICsJam9iID0gY29ubl9zdGF0ZS0+d3JpdGViYWNrX2pvYjsKPiAKPiBXaGF0IGRvIHlvdSB0aGlu ayBhYm91dCBhZGRpbmcgYSBkZWZlbnNpdmUgV0FSTl9PTigham9iKT8KCkkgZXhwZWN0IHRoaXMg ZnVuY3Rpb24gdG8gYmUgY2FsbGVkIGZyb20gYW4gYXRvbWljIGNvbW1pdCBoYW5kbGVyIGZvcgp0 aGUgd3JpdGViYWNrIGpvYiwgd2hpY2ggd2lsbCBuZWVkIHRvIGFjY2VzcyB0aGUgam9iJ3MgY29u dGVudHMgKGluCnBhcnRpY3VsYXIgdGhlIGZyYW1lYnVmZmVyKS4gSSB0aHVzIHRoaW5rIHdlJ2xs IG5ldmVyIGJlIGNhbGxlZCB3aXRoIGEKTlVMTCBqb2IgaGVyZSwgc28gYSBXQVJOX09OIHdvdWxk IG9ubHkgY29uc3VtZSBhIGJpdCBvZiBDUFUgdGltZSBhbmQKbWVtb3J5IHdpdGhvdXQgYWRkaW5n IGFueSBzYWZlZ3VhcmQuIElmIHlvdXIgYW5hbHlzaXMgZGlmZmVycyBhbmQgeW91CnN0aWxsIHRo aW5rIHRoZXJlJ3MgYSByaXNrLCBJJ2xsIGFkZCBpdC4KCj4gPj4gKwljb25uX3N0YXRlLT53cml0 ZWJhY2tfam9iID0gTlVMTDsKPiA+PiArCj4gPj4gIAlzcGluX2xvY2tfaXJxc2F2ZSgmd2JfY29u bmVjdG9yLT5qb2JfbG9jaywgZmxhZ3MpOwo+ID4+ICAJbGlzdF9hZGRfdGFpbCgmam9iLT5saXN0 X2VudHJ5LCAmd2JfY29ubmVjdG9yLT5qb2JfcXVldWUpOwo+ID4+ICAJc3Bpbl91bmxvY2tfaXJx cmVzdG9yZSgmd2JfY29ubmVjdG9yLT5qb2JfbG9jaywgZmxhZ3MpOwo+ID4+IGRpZmYgLS1naXQg YS9kcml2ZXJzL2dwdS9kcm0vdmM0L3ZjNF90eHAuYyBiL2RyaXZlcnMvZ3B1L2RybS92YzQvdmM0 X3R4cC5jCj4gPj4gaW5kZXggYWEyNzliNWIwZGU3Li41ZGFiZDkxZjJkN2UgMTAwNjQ0Cj4gPj4g LS0tIGEvZHJpdmVycy9ncHUvZHJtL3ZjNC92YzRfdHhwLmMKPiA+PiArKysgYi9kcml2ZXJzL2dw dS9kcm0vdmM0L3ZjNF90eHAuYwo+ID4+IEBAIC0zMjcsNyArMzI3LDcgQEAgc3RhdGljIHZvaWQg dmM0X3R4cF9jb25uZWN0b3JfYXRvbWljX2NvbW1pdChzdHJ1Y3QgZHJtX2Nvbm5lY3RvciAqY29u biwKPiA+PiAgCj4gPj4gIAlUWFBfV1JJVEUoVFhQX0RTVF9DVFJMLCBjdHJsKTsKPiA+PiAgCj4g Pj4gLQlkcm1fd3JpdGViYWNrX3F1ZXVlX2pvYigmdHhwLT5jb25uZWN0b3IsIGNvbm5fc3RhdGUt PndyaXRlYmFja19qb2IpOwo+ID4+ICsJZHJtX3dyaXRlYmFja19xdWV1ZV9qb2IoJnR4cC0+Y29u bmVjdG9yLCBjb25uX3N0YXRlKTsKPiA+PiAgfQo+ID4+ICAKPiA+PiAgc3RhdGljIGNvbnN0IHN0 cnVjdCBkcm1fY29ubmVjdG9yX2hlbHBlcl9mdW5jcyB2YzRfdHhwX2Nvbm5lY3Rvcl9oZWxwZXJf ZnVuY3MgPSB7Cj4gPj4gZGlmZiAtLWdpdCBhL2luY2x1ZGUvZHJtL2RybV93cml0ZWJhY2suaCBi L2luY2x1ZGUvZHJtL2RybV93cml0ZWJhY2suaAo+ID4+IGluZGV4IDIzZGY5ZDQ2MzAwMy4uNDc2 NjJjMzYyNzQzIDEwMDY0NAo+ID4+IC0tLSBhL2luY2x1ZGUvZHJtL2RybV93cml0ZWJhY2suaAo+ ID4+ICsrKyBiL2luY2x1ZGUvZHJtL2RybV93cml0ZWJhY2suaAo+ID4+IEBAIC0xMjMsNyArMTIz LDcgQEAgaW50IGRybV93cml0ZWJhY2tfY29ubmVjdG9yX2luaXQoc3RydWN0IGRybV9kZXZpY2Ug KmRldiwKPiA+PiAgCQkJCSBjb25zdCB1MzIgKmZvcm1hdHMsIGludCBuX2Zvcm1hdHMpOwo+ID4+ ICAKPiA+PiAgdm9pZCBkcm1fd3JpdGViYWNrX3F1ZXVlX2pvYihzdHJ1Y3QgZHJtX3dyaXRlYmFj a19jb25uZWN0b3IgKndiX2Nvbm5lY3RvciwKPiA+PiAtCQkJICAgICBzdHJ1Y3QgZHJtX3dyaXRl YmFja19qb2IgKmpvYik7Cj4gPj4gKwkJCSAgICAgc3RydWN0IGRybV9jb25uZWN0b3Jfc3RhdGUg KmNvbm5fc3RhdGUpOwo+ID4+ICAKPiA+PiAgdm9pZCBkcm1fd3JpdGViYWNrX2NsZWFudXBfam9i KHN0cnVjdCBkcm1fd3JpdGViYWNrX2pvYiAqam9iKTsKPiA+PiAgCgotLSAKUmVnYXJkcywKCkxh dXJlbnQgUGluY2hhcnQKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX18KZHJpLWRldmVsIG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Au b3JnCmh0dHBzOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRl dmVs