From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: [PATCH 1/4] drm/atomic: integrate modeset lock with private objects Date: Wed, 21 Feb 2018 17:27:20 +0200 Message-ID: <20180221152720.GP5453@intel.com> References: <20180221143730.30285-1-robdclark@gmail.com> <20180221143730.30285-2-robdclark@gmail.com> <20180221144919.GN5453@intel.com> <20180221150727.GO5453@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Content-Disposition: inline In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Rob Clark Cc: David Airlie , linux-arm-msm , Linux Kernel Mailing List , dri-devel List-Id: linux-arm-msm@vger.kernel.org T24gV2VkLCBGZWIgMjEsIDIwMTggYXQgMTA6MjA6MDNBTSAtMDUwMCwgUm9iIENsYXJrIHdyb3Rl Ogo+IE9uIFdlZCwgRmViIDIxLCAyMDE4IGF0IDEwOjA3IEFNLCBWaWxsZSBTeXJqw6Rsw6QKPiA8 dmlsbGUuc3lyamFsYUBsaW51eC5pbnRlbC5jb20+IHdyb3RlOgo+ID4gT24gV2VkLCBGZWIgMjEs IDIwMTggYXQgMDk6NTQ6NDlBTSAtMDUwMCwgUm9iIENsYXJrIHdyb3RlOgo+ID4+IE9uIFdlZCwg RmViIDIxLCAyMDE4IGF0IDk6NDkgQU0sIFZpbGxlIFN5cmrDpGzDpAo+ID4+IDx2aWxsZS5zeXJq YWxhQGxpbnV4LmludGVsLmNvbT4gd3JvdGU6Cj4gPj4gPiBPbiBXZWQsIEZlYiAyMSwgMjAxOCBh dCAwOTozNzoyMUFNIC0wNTAwLCBSb2IgQ2xhcmsgd3JvdGU6Cj4gPj4gPj4gRm9sbG93IHRoZSBz YW1lIHBhdHRlcm4gb2YgbG9ja2luZyBhcyB3aXRoIG90aGVyIHN0YXRlIG9iamVjdHMuICBUaGlz Cj4gPj4gPj4gYXZvaWRzIGJvaWxlcnBsYXRlIGluIHRoZSBkcml2ZXIuCj4gPj4gPgo+ID4+ID4g SSdtIG5vdCBzdXJlIHdlIHJlYWxseSB3YW50IHRvIGRvIHRoaXMuIFdoYXQgaWYgdGhlIGRyaXZl ciB3YW50cyBhCj4gPj4gPiBjdXN0b20gbG9ja2luZyBzY2hlbWUgZm9yIHRoaXMgc3RhdGU/Cj4g Pj4KPiA+PiBUaGF0IHNlZW1zIGxpa2Ugc29tZXRoaW5nIHdlIHdhbnQgdG8gZGlzY291cmFnZSwg aWUuIGFsbCB0aGUgbW9yZQo+ID4+IHJlYXNvbiBmb3IgdGhpcyBwYXRjaC4KPiA+Pgo+ID4+IFRo ZXJlIGlzIG5vIHJlYXNvbiBkcml2ZXJzIGNvdWxkIG5vdCBzcGxpdCB0aGVpciBnbG9iYWwgc3Rh dGUgaW50bwo+ID4+IG11bHRpcGxlIHByaXZhdGUgb2JqcydzLCBlYWNoIHdpdGggdGhlaXIgb3du IGxvY2ssIGZvciBtb3JlIGZpbmUKPiA+PiBncmFpbmVkIGxvY2tpbmcuICBUaGF0IGlzIGJhc2lj YWxseSB0aGUgb25seSB2YWxpZCByZWFzb24gSSBjYW4gdGhpbmsKPiA+PiBvZiBmb3IgImN1c3Rv bSBsb2NraW5nIi4KPiA+Cj4gPiBJbiBpOTE1IHdlIGhhdmUgYXQgbGVhc3Qgb25lIGNhc2UgdGhh dCB3b3VsZCB3YW50IHNvbWV0aGluZyBjbG9zZSB0byBhbgo+ID4gcndsb2NrLiBBbnkgY3J0YyBs b2NrIGlzIGVub3VnaCBmb3IgcmVhZCwgbmVlZCBhbGwgb2YgdGhlbSBmb3Igd3JpdGUuCj4gPiBU aG91Z2ggaWYgd2Ugd2FudGVkIHRvIHVzZSBwcml2YXRlIG9ianMgZm9yIHRoYXQgd2UgbWlnaHQg bmVlZCB0bwo+ID4gYWN0dWFsbHkgbWFrZSB0aGUgc3RhdGVzIHJlZmNvdW50ZWQgYXMgd2VsbCwg b3RoZXJ3aXNlIEkgY2FuIGltYWdpbmUKPiA+IHdlIG1pZ2h0IGxhbmQgaW4gc29tZSB1c2UtYWZ0 ZXItZnJlZSBpc3N1ZXMgb25jZSBhZ2Fpbi4KPiA+Cj4gPiBNYXliZSB3ZSBjb3VsZCBkdXBsaWNh dGUgdGhlIHN0YXRlIGludG8gcGVyLWNydGMgYW5kIGdsb2JhbCBjb3BpZXMsIGJ1dAo+ID4gdGhl biB3ZSBoYXZlIHRvIGtlZXAgYWxsIG9mIHRob3NlIGluIHN5bmMgc29tZWhvdyB3aGljaCBkb2Vz bid0IHNvdW5kCj4gPiBwYXJ0aWN1bGFybHkgcGxlYXNhbnQuCj4gCj4gT3IganVzdCBrZWVwIHlv dXIgb3duIGRyaXZlciBsb2NrIGZvciByZWFkLCBhbmQgdXNlIHRoYXQgcGx1cyB0aGUgY29yZQo+ IG1vZGVzZXQgbG9jayBmb3Igd3JpdGU/CgpJZiB3ZSBjYW4ndCBhZGQgdGhlIHByaXZhdGUgb2Jq IHRvIHRoZSBzdGF0ZSB3ZSBjYW4ndCByZWFsbHkgdXNlIGl0LgoKPiAKPiBCUiwKPiAtUgo+IAo+ ID4KPiA+Pgo+ID4+IChBbmQgb2ZjIGRyaXZlcnMgY291bGQgYWRkIHRoZXJlIG93biBsb2NrcyBp biBhZGRpdGlvbiB0byB3aGF0IGlzIGRvbmUKPiA+PiBieSBjb3JlLCBidXQgSSdkIHJhdGhlciBs b29rIGF0IHRoYXQgb24gYSBjYXNlIGJ5IGNhc2UgYmFzaXMsIHJhdGhlcgo+ID4+IHRoYW4gaXQg YmVpbmcgcGFydCBvZiB0aGUgYm9pbGVycGxhdGUgaW4gZWFjaCBkcml2ZXIuKQo+ID4+Cj4gPj4g QlIsCj4gPj4gLVIKPiA+Pgo+ID4+ID4+Cj4gPj4gPj4gU2lnbmVkLW9mZi1ieTogUm9iIENsYXJr IDxyb2JkY2xhcmtAZ21haWwuY29tPgo+ID4+ID4+IC0tLQo+ID4+ID4+ICBkcml2ZXJzL2dwdS9k cm0vZHJtX2F0b21pYy5jIHwgOSArKysrKysrKy0KPiA+PiA+PiAgaW5jbHVkZS9kcm0vZHJtX2F0 b21pYy5oICAgICB8IDUgKysrKysKPiA+PiA+PiAgMiBmaWxlcyBjaGFuZ2VkLCAxMyBpbnNlcnRp b25zKCspLCAxIGRlbGV0aW9uKC0pCj4gPj4gPj4KPiA+PiA+PiBkaWZmIC0tZ2l0IGEvZHJpdmVy cy9ncHUvZHJtL2RybV9hdG9taWMuYyBiL2RyaXZlcnMvZ3B1L2RybS9kcm1fYXRvbWljLmMKPiA+ PiA+PiBpbmRleCBmYzhjNGRhNDA5ZmYuLjAwNGU2MjFhYjMwNyAxMDA2NDQKPiA+PiA+PiAtLS0g YS9kcml2ZXJzL2dwdS9kcm0vZHJtX2F0b21pYy5jCj4gPj4gPj4gKysrIGIvZHJpdmVycy9ncHUv ZHJtL2RybV9hdG9taWMuYwo+ID4+ID4+IEBAIC0xMDc4LDYgKzEwNzgsOCBAQCBkcm1fYXRvbWlj X3ByaXZhdGVfb2JqX2luaXQoc3RydWN0IGRybV9wcml2YXRlX29iaiAqb2JqLAo+ID4+ID4+ICB7 Cj4gPj4gPj4gICAgICAgbWVtc2V0KG9iaiwgMCwgc2l6ZW9mKCpvYmopKTsKPiA+PiA+Pgo+ID4+ ID4+ICsgICAgIGRybV9tb2Rlc2V0X2xvY2tfaW5pdCgmb2JqLT5sb2NrKTsKPiA+PiA+PiArCj4g Pj4gPj4gICAgICAgb2JqLT5zdGF0ZSA9IHN0YXRlOwo+ID4+ID4+ICAgICAgIG9iai0+ZnVuY3Mg PSBmdW5jczsKPiA+PiA+PiAgfQo+ID4+ID4+IEBAIC0xMDkzLDYgKzEwOTUsNyBAQCB2b2lkCj4g Pj4gPj4gIGRybV9hdG9taWNfcHJpdmF0ZV9vYmpfZmluaShzdHJ1Y3QgZHJtX3ByaXZhdGVfb2Jq ICpvYmopCj4gPj4gPj4gIHsKPiA+PiA+PiAgICAgICBvYmotPmZ1bmNzLT5hdG9taWNfZGVzdHJv eV9zdGF0ZShvYmosIG9iai0+c3RhdGUpOwo+ID4+ID4+ICsgICAgIGRybV9tb2Rlc2V0X2xvY2tf ZmluaSgmb2JqLT5sb2NrKTsKPiA+PiA+PiAgfQo+ID4+ID4+ICBFWFBPUlRfU1lNQk9MKGRybV9h dG9taWNfcHJpdmF0ZV9vYmpfZmluaSk7Cj4gPj4gPj4KPiA+PiA+PiBAQCAtMTExMyw3ICsxMTE2 LDcgQEAgc3RydWN0IGRybV9wcml2YXRlX3N0YXRlICoKPiA+PiA+PiAgZHJtX2F0b21pY19nZXRf cHJpdmF0ZV9vYmpfc3RhdGUoc3RydWN0IGRybV9hdG9taWNfc3RhdGUgKnN0YXRlLAo+ID4+ID4+ ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICBzdHJ1Y3QgZHJtX3ByaXZhdGVfb2JqICpv YmopCj4gPj4gPj4gIHsKPiA+PiA+PiAtICAgICBpbnQgaW5kZXgsIG51bV9vYmpzLCBpOwo+ID4+ ID4+ICsgICAgIGludCBpbmRleCwgbnVtX29ianMsIGksIHJldDsKPiA+PiA+PiAgICAgICBzaXpl X3Qgc2l6ZTsKPiA+PiA+PiAgICAgICBzdHJ1Y3QgX19kcm1fcHJpdmF0ZV9vYmpzX3N0YXRlICph cnI7Cj4gPj4gPj4gICAgICAgc3RydWN0IGRybV9wcml2YXRlX3N0YXRlICpvYmpfc3RhdGU7Cj4g Pj4gPj4gQEAgLTExMjIsNiArMTEyNSwxMCBAQCBkcm1fYXRvbWljX2dldF9wcml2YXRlX29ial9z dGF0ZShzdHJ1Y3QgZHJtX2F0b21pY19zdGF0ZSAqc3RhdGUsCj4gPj4gPj4gICAgICAgICAgICAg ICBpZiAob2JqID09IHN0YXRlLT5wcml2YXRlX29ianNbaV0ucHRyKQo+ID4+ID4+ICAgICAgICAg ICAgICAgICAgICAgICByZXR1cm4gc3RhdGUtPnByaXZhdGVfb2Jqc1tpXS5zdGF0ZTsKPiA+PiA+ Pgo+ID4+ID4+ICsgICAgIHJldCA9IGRybV9tb2Rlc2V0X2xvY2soJm9iai0+bG9jaywgc3RhdGUt PmFjcXVpcmVfY3R4KTsKPiA+PiA+PiArICAgICBpZiAocmV0KQo+ID4+ID4+ICsgICAgICAgICAg ICAgcmV0dXJuIEVSUl9QVFIocmV0KTsKPiA+PiA+PiArCj4gPj4gPj4gICAgICAgbnVtX29ianMg PSBzdGF0ZS0+bnVtX3ByaXZhdGVfb2JqcyArIDE7Cj4gPj4gPj4gICAgICAgc2l6ZSA9IHNpemVv Zigqc3RhdGUtPnByaXZhdGVfb2JqcykgKiBudW1fb2JqczsKPiA+PiA+PiAgICAgICBhcnIgPSBr cmVhbGxvYyhzdGF0ZS0+cHJpdmF0ZV9vYmpzLCBzaXplLCBHRlBfS0VSTkVMKTsKPiA+PiA+PiBk aWZmIC0tZ2l0IGEvaW5jbHVkZS9kcm0vZHJtX2F0b21pYy5oIGIvaW5jbHVkZS9kcm0vZHJtX2F0 b21pYy5oCj4gPj4gPj4gaW5kZXggMDkwNzZhNjI1NjM3Li45YWU1M2I3M2M5ZDIgMTAwNjQ0Cj4g Pj4gPj4gLS0tIGEvaW5jbHVkZS9kcm0vZHJtX2F0b21pYy5oCj4gPj4gPj4gKysrIGIvaW5jbHVk ZS9kcm0vZHJtX2F0b21pYy5oCj4gPj4gPj4gQEAgLTIxOCw2ICsyMTgsMTEgQEAgc3RydWN0IGRy bV9wcml2YXRlX3N0YXRlX2Z1bmNzIHsKPiA+PiA+PiAgICogJmRybV9tb2Rlc2V0X2xvY2sgaXMg cmVxdWlyZWQgdG8gZHVwbGljYXRlIGFuZCB1cGRhdGUgdGhpcyBvYmplY3QncyBzdGF0ZS4KPiA+ PiA+PiAgICovCj4gPj4gPj4gIHN0cnVjdCBkcm1fcHJpdmF0ZV9vYmogewo+ID4+ID4+ICsgICAg IC8qKgo+ID4+ID4+ICsgICAgICAqIEBsb2NrOiBNb2Rlc2V0IGxvY2sgdG8gcHJvdGVjdCB0aGUg c3RhdGUgb2JqZWN0Lgo+ID4+ID4+ICsgICAgICAqLwo+ID4+ID4+ICsgICAgIHN0cnVjdCBkcm1f bW9kZXNldF9sb2NrIGxvY2s7Cj4gPj4gPj4gKwo+ID4+ID4+ICAgICAgIC8qKgo+ID4+ID4+ICAg ICAgICAqIEBzdGF0ZTogQ3VycmVudCBhdG9taWMgc3RhdGUgZm9yIHRoaXMgZHJpdmVyIHByaXZh dGUgb2JqZWN0Lgo+ID4+ID4+ICAgICAgICAqLwo+ID4+ID4+IC0tCj4gPj4gPj4gMi4xNC4zCj4g Pj4gPj4KPiA+PiA+PiBfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fXwo+ID4+ID4+IGRyaS1kZXZlbCBtYWlsaW5nIGxpc3QKPiA+PiA+PiBkcmktZGV2ZWxAbGlz dHMuZnJlZWRlc2t0b3Aub3JnCj4gPj4gPj4gaHR0cHM6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcv bWFpbG1hbi9saXN0aW5mby9kcmktZGV2ZWwKPiA+PiA+Cj4gPj4gPiAtLQo+ID4+ID4gVmlsbGUg U3lyasOkbMOkCj4gPj4gPiBJbnRlbCBPVEMKPiA+Cj4gPiAtLQo+ID4gVmlsbGUgU3lyasOkbMOk Cj4gPiBJbnRlbCBPVEMKCi0tIApWaWxsZSBTeXJqw6Rsw6QKSW50ZWwgT1RDCl9fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCmRyaS1kZXZlbCBtYWlsaW5nIGxp c3QKZHJpLWRldmVsQGxpc3RzLmZyZWVkZXNrdG9wLm9yZwpodHRwczovL2xpc3RzLmZyZWVkZXNr dG9wLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2RyaS1kZXZlbAo=