From mboxrd@z Thu Jan 1 00:00:00 1970 From: Laurent Pinchart Subject: Re: [PATCH 01/15] drm: omapdrm: Fix plane state free in plane reset handler Date: Sun, 13 Dec 2015 22:39:02 +0200 Message-ID: <103576498.oQSQlJ3ayB@avalon> References: <1449268039-24682-1-git-send-email-laurent.pinchart@ideasonboard.com> <1449268039-24682-2-git-send-email-laurent.pinchart@ideasonboard.com> <566821E7.8030202@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from galahad.ideasonboard.com (galahad.ideasonboard.com [185.26.127.97]) by gabe.freedesktop.org (Postfix) with ESMTPS id CB60F6E414 for ; Sun, 13 Dec 2015 17:36:14 -0800 (PST) In-Reply-To: <566821E7.8030202@ti.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Tomi Valkeinen Cc: dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org SGkgVG9taSwKCk9uIFdlZG5lc2RheSAwOSBEZWNlbWJlciAyMDE1IDE0OjQzOjE5IFRvbWkgVmFs a2VpbmVuIHdyb3RlOgo+IE9uIDA1LzEyLzE1IDAwOjI3LCBMYXVyZW50IFBpbmNoYXJ0IHdyb3Rl Ogo+ID4gVGhlIHBsYW5lIHJlc2V0IGhhbmRsZXIgZnJlZXMgdGhlIHBsYW5lIHN0YXRlIGFuZCBh bGxvY2F0ZXMgYSBuZXcKPiA+IGRlZmF1bHQgc3RhdGUsIGJ1dCB3aGVuIGRvaW5nIHNvIGF0dGVt cHQgdG8gZnJlZSB0aGUgcGxhbmUgc3RhdGUgdXNpbmcKPiA+IHRoZSBiYXNlIHBsYW5lIHN0YXRl IHBvaW50ZXIgaW5zdGVhZCBvZiBjYXN0aW5nIGl0IHRvIHRoZQo+ID4gZHJpdmVyLXNwZWNpZmlj IHN0YXRlIG9iamVjdCB0aGF0IGhhcyBiZWVuIGFsbG9jYXRlZC4gRml4IGl0IGJ5IHVzaW5nCj4g PiB0aGUgb21hcF9wbGFuZV9hdG9taWNfZGVzdHJveV9zdGF0ZSgpIGZ1bmN0aW9uIHRvIGRlc3Ry b3kgdGhlIHBsYW5lCj4gPiBzdGF0ZSBpbnN0ZWFkIG9mIGR1cGxpY2F0aW5nIHRoZSBjb2RlLgo+ ID4gCj4gPiBTaWduZWQtb2ZmLWJ5OiBMYXVyZW50IFBpbmNoYXJ0IDxsYXVyZW50LnBpbmNoYXJ0 QGlkZWFzb25ib2FyZC5jb20+Cj4gPiAtLS0KPiA+IAo+ID4gIGRyaXZlcnMvZ3B1L2RybS9vbWFw ZHJtL29tYXBfcGxhbmUuYyB8IDUzICsrKysrKysrKysrKysrKystLS0tLS0tLS0tLS0tLS0KPiA+ ICAxIGZpbGUgY2hhbmdlZCwgMjYgaW5zZXJ0aW9ucygrKSwgMjcgZGVsZXRpb25zKC0pCj4gPiAK PiA+IGRpZmYgLS1naXQgYS9kcml2ZXJzL2dwdS9kcm0vb21hcGRybS9vbWFwX3BsYW5lLmMKPiA+ IGIvZHJpdmVycy9ncHUvZHJtL29tYXBkcm0vb21hcF9wbGFuZS5jIGluZGV4IDMwNTRiZGE3MjY4 OC4uMTFkNDA2YjE2MGUxCj4gPiAxMDA2NDQKPiA+IC0tLSBhL2RyaXZlcnMvZ3B1L2RybS9vbWFw ZHJtL29tYXBfcGxhbmUuYwo+ID4gKysrIGIvZHJpdmVycy9ncHUvZHJtL29tYXBkcm0vb21hcF9w bGFuZS5jCj4gPiBAQCAtMTg4LDMzICsxODgsNiBAQCBzdGF0aWMgY29uc3Qgc3RydWN0IGRybV9w bGFuZV9oZWxwZXJfZnVuY3MKPiA+IG9tYXBfcGxhbmVfaGVscGVyX2Z1bmNzID0gewo+ID4gIAku YXRvbWljX2Rpc2FibGUgPSBvbWFwX3BsYW5lX2F0b21pY19kaXNhYmxlLAo+ID4gIH07Cj4gPiAK PiA+IC1zdGF0aWMgdm9pZCBvbWFwX3BsYW5lX3Jlc2V0KHN0cnVjdCBkcm1fcGxhbmUgKnBsYW5l KQo+ID4gLXsKPiA+IC0Jc3RydWN0IG9tYXBfcGxhbmUgKm9tYXBfcGxhbmUgPSB0b19vbWFwX3Bs YW5lKHBsYW5lKTsKPiA+IC0Jc3RydWN0IG9tYXBfcGxhbmVfc3RhdGUgKm9tYXBfc3RhdGU7Cj4g PiAtCj4gPiAtCWlmIChwbGFuZS0+c3RhdGUgJiYgcGxhbmUtPnN0YXRlLT5mYikKPiA+IC0JCWRy bV9mcmFtZWJ1ZmZlcl91bnJlZmVyZW5jZShwbGFuZS0+c3RhdGUtPmZiKTsKPiA+IC0KPiA+IC0J a2ZyZWUocGxhbmUtPnN0YXRlKTsKPiA+IC0JcGxhbmUtPnN0YXRlID0gTlVMTDsKPiA+IC0KPiA+ IC0Jb21hcF9zdGF0ZSA9IGt6YWxsb2Moc2l6ZW9mKCpvbWFwX3N0YXRlKSwgR0ZQX0tFUk5FTCk7 Cj4gPiAtCWlmIChvbWFwX3N0YXRlID09IE5VTEwpCj4gPiAtCQlyZXR1cm47Cj4gPiAtCj4gPiAt CS8qCj4gPiAtCSAqIFNldCBkZWZhdWx0cyBkZXBlbmRpbmcgb24gd2hldGhlciB3ZSBhcmUgYSBw cmltYXJ5IG9yIG92ZXJsYXkKPiA+IC0JICogcGxhbmUuCj4gPiAtCSAqLwo+ID4gLQlvbWFwX3N0 YXRlLT56b3JkZXIgPSBwbGFuZS0+dHlwZSA9PSBEUk1fUExBTkVfVFlQRV9QUklNQVJZCj4gPiAt CQkJICAgPyAwIDogb21hcF9wbGFuZS0+aWQ7Cj4gPiAtCW9tYXBfc3RhdGUtPmJhc2Uucm90YXRp b24gPSBCSVQoRFJNX1JPVEFURV8wKTsKPiA+IC0KPiA+IC0JcGxhbmUtPnN0YXRlID0gJm9tYXBf c3RhdGUtPmJhc2U7Cj4gPiAtCXBsYW5lLT5zdGF0ZS0+cGxhbmUgPSBwbGFuZTsKPiA+IC19Cj4g PiAtCj4gPiAgc3RhdGljIHZvaWQgb21hcF9wbGFuZV9kZXN0cm95KHN0cnVjdCBkcm1fcGxhbmUg KnBsYW5lKQo+ID4gIHsKPiA+ICAJc3RydWN0IG9tYXBfcGxhbmUgKm9tYXBfcGxhbmUgPSB0b19v bWFwX3BsYW5lKHBsYW5lKTsKPiA+IEBAIC0yNzAsNiArMjQzLDMyIEBAIHN0YXRpYyB2b2lkIG9t YXBfcGxhbmVfYXRvbWljX2Rlc3Ryb3lfc3RhdGUoc3RydWN0Cj4gPiBkcm1fcGxhbmUgKnBsYW5l LAo+ID4gIAlrZnJlZSh0b19vbWFwX3BsYW5lX3N0YXRlKHN0YXRlKSk7Cj4gPiAgfQo+ID4gCj4g PiArc3RhdGljIHZvaWQgb21hcF9wbGFuZV9yZXNldChzdHJ1Y3QgZHJtX3BsYW5lICpwbGFuZSkK PiA+ICt7Cj4gPiArCXN0cnVjdCBvbWFwX3BsYW5lICpvbWFwX3BsYW5lID0gdG9fb21hcF9wbGFu ZShwbGFuZSk7Cj4gPiArCXN0cnVjdCBvbWFwX3BsYW5lX3N0YXRlICpvbWFwX3N0YXRlOwo+ID4g Kwo+ID4gKwlpZiAocGxhbmUtPnN0YXRlKSB7Cj4gPiArCQlvbWFwX3BsYW5lX2F0b21pY19kZXN0 cm95X3N0YXRlKHBsYW5lLCBwbGFuZS0+c3RhdGUpOwo+ID4gKwkJcGxhbmUtPnN0YXRlID0gTlVM TDsKPiA+ICsJfQo+ID4gKwo+ID4gKwlvbWFwX3N0YXRlID0ga3phbGxvYyhzaXplb2YoKm9tYXBf c3RhdGUpLCBHRlBfS0VSTkVMKTsKPiA+ICsJaWYgKG9tYXBfc3RhdGUgPT0gTlVMTCkKPiA+ICsJ CXJldHVybjsKPiA+ICsKPiA+ICsJLyoKPiA+ICsJICogU2V0IGRlZmF1bHRzIGRlcGVuZGluZyBv biB3aGV0aGVyIHdlIGFyZSBhIHByaW1hcnkgb3Igb3ZlcmxheQo+ID4gKwkgKiBwbGFuZS4KPiA+ ICsJICovCj4gPiArCW9tYXBfc3RhdGUtPnpvcmRlciA9IHBsYW5lLT50eXBlID09IERSTV9QTEFO RV9UWVBFX1BSSU1BUlkKPiA+ICsJCQkgICA/IDAgOiBvbWFwX3BsYW5lLT5pZDsKPiA+ICsJb21h cF9zdGF0ZS0+YmFzZS5yb3RhdGlvbiA9IEJJVChEUk1fUk9UQVRFXzApOwo+ID4gKwo+ID4gKwlw bGFuZS0+c3RhdGUgPSAmb21hcF9zdGF0ZS0+YmFzZTsKPiA+ICsJcGxhbmUtPnN0YXRlLT5wbGFu ZSA9IHBsYW5lOwo+ID4gK30KPiA+ICsKPiA+ICBzdGF0aWMgaW50IG9tYXBfcGxhbmVfYXRvbWlj X3NldF9wcm9wZXJ0eShzdHJ1Y3QgZHJtX3BsYW5lICpwbGFuZSwKPiA+ICAJCQkJCSAgc3RydWN0 IGRybV9wbGFuZV9zdGF0ZSAqc3RhdGUsCj4gPiAgCQkJCQkgIHN0cnVjdCBkcm1fcHJvcGVydHkg KnByb3BlcnR5LAo+IAo+IFRoaXMgb25lIG1vdmVzIHRoZSBmdW5jdGlvbiwgc28gaXQncyBwcmV0 dHkgaGFyZCB0byBzZWUgd2hhdCBhY3R1YWxseQo+IHdhcyBjaGFuZ2VkLgoKSXQncyB1bmZvcnR1 bmF0ZWx5IG5lZWRlZCBpZiB3ZSB3YW50IHRvIGF2b2lkIGZvcndhcmQgZGVjbGFyYXRpb25zLiBJ IGNvdWxkIApzcGxpdCB0aGUgcGF0Y2ggaW4gdHdvLCBidXQgZ2l2ZW4gdGhlIHNpemUgb2YgdGhl IGZ1bmN0aW9uIGFuZCB0aGUgZXh0ZW50IG9mIAp0aGUgY2hhbmdlIEkgdGhvdWdodCBpdCB3b3Vs ZG4ndCBiZSB3b3J0aCBpdC4KCi0tIApSZWdhcmRzLAoKTGF1cmVudCBQaW5jaGFydAoKX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVsIG1haWxp bmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHA6Ly9saXN0cy5mcmVl ZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9kcmktZGV2ZWwK