From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jani Nikula Subject: Re: [PATCH] drm/i915: Postpone plane readout until after encoder readout, v3. Date: Tue, 13 Oct 2015 15:40:27 +0300 Message-ID: <87d1wj54ck.fsf@intel.com> References: <1438347848-11293-1-git-send-email-patrik.jakobsson@linux.intel.com> <55BF7C5D.6030405@linux.intel.com> <55C1E95C.6030101@linux.intel.com> <20150826144343.GV1367@phenom.ffwll.local> <55DEF8CE.3060600@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by gabe.freedesktop.org (Postfix) with ESMTP id 5FB236E35B for ; Tue, 13 Oct 2015 05:37:01 -0700 (PDT) In-Reply-To: <55DEF8CE.3060600@linux.intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Maarten Lankhorst , Daniel Vetter Cc: Intel Graphics Development List-Id: intel-gfx@lists.freedesktop.org T24gVGh1LCAyNyBBdWcgMjAxNSwgTWFhcnRlbiBMYW5raG9yc3QgPG1hYXJ0ZW4ubGFua2hvcnN0 QGxpbnV4LmludGVsLmNvbT4gd3JvdGU6Cj4gV2hlbiByZWFkaW5nIG91dCBodyBzdGF0ZSBmb3Ig cGxhbmVzIHdlIGRpc2FibGUgaW5hY3RpdmUgcGxhbmVzIHdoaWNoIGluCj4gdHVybiB0cmlnZ2Vy cyBhbiB1cGRhdGUgb2YgdGhlIHdhdGVybWFya3MuIFRoZSB1cGRhdGUgZGVwZW5kcyBvbiB0aGUK PiBjcnRjX2Nsb2NrIGJlaW5nIHNldCB3aGljaCBpcyBkb25lIHdoZW4gcmVhZGluZyBvdXQgZW5j b2RlcnMuIFRodXMKPiBwb3N0cG9uZSB0aGUgcGxhbmUgcmVhZG91dCB1bnRpbCBhZnRlciBlbmNv ZGVyIHJlYWRvdXQuCj4KPiBUaGlzIHByZXZlbnRzIGEgd2FybmluZyBpbiBza2xfY29tcHV0ZV9s aW5ldGltZV93bSgpIHdoZXJlIHBpeGVsX3JhdGUKPiBiZWNvbWVzIDAgd2hlbiBjcnRjX2Nsb2Nr IGlzIDAuCj4KPiBTaWduZWQtb2ZmLWJ5OiBQYXRyaWsgSmFrb2Jzc29uIDxwYXRyaWsuamFrb2Jz c29uQGxpbnV4LmludGVsLmNvbT4KPiBTaWduZWQtb2ZmLWJ5OiBNYWFydGVuIExhbmtob3JzdCA8 bWFhcnRlbi5sYW5raG9yc3RAbGludXguaW50ZWwuY29tPgo+IFJlZmVyZW5jZXM6IGh0dHBzOi8v YnVncy5mcmVlZGVza3RvcC5vcmcvc2hvd19idWcuY2dpP2lkPTkxNDI4CgpNYWFydGVuLCBJIHBy ZXN1bWUgdGhpcyBwYXRjaCBpcyBzdXBlcnNlZGVkIGJ5IHRoZSBjb21taXRzIHJlZmVyZW5jZWQg aW4KdGhhdCBidWcuIElzIHRoYXQgY29ycmVjdD8KCkJSLApKYW5pLgoKCgo+IC0tLQo+IGRpZmYg LS1naXQgYS9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9kaXNwbGF5LmMgYi9kcml2ZXJzL2dw dS9kcm0vaTkxNS9pbnRlbF9kaXNwbGF5LmMKPiBpbmRleCBlMzkyMmQ5NzNkYjAuLjExOGIyMDVl N2JkNSAxMDA2NDQKPiAtLS0gYS9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9kaXNwbGF5LmMK PiArKysgYi9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9kaXNwbGF5LmMKPiBAQCAtMTUwMzMs MzAgKzE1MDMzLDYgQEAgc3RhdGljIGJvb2wgcHJpbWFyeV9nZXRfaHdfc3RhdGUoc3RydWN0IGlu dGVsX2NydGMgKmNydGMpCj4gIAlyZXR1cm4gISEoSTkxNV9SRUFEKERTUENOVFIoY3J0Yy0+cGxh bmUpKSAmIERJU1BMQVlfUExBTkVfRU5BQkxFKTsKPiAgfQo+ICAKPiAtc3RhdGljIHZvaWQgcmVh ZG91dF9wbGFuZV9zdGF0ZShzdHJ1Y3QgaW50ZWxfY3J0YyAqY3J0YywKPiAtCQkJCXN0cnVjdCBp bnRlbF9jcnRjX3N0YXRlICpjcnRjX3N0YXRlKQo+IC17Cj4gLQlzdHJ1Y3QgaW50ZWxfcGxhbmUg KnA7Cj4gLQlzdHJ1Y3QgaW50ZWxfcGxhbmVfc3RhdGUgKnBsYW5lX3N0YXRlOwo+IC0JYm9vbCBh Y3RpdmUgPSBjcnRjX3N0YXRlLT5iYXNlLmFjdGl2ZTsKPiAtCj4gLQlmb3JfZWFjaF9pbnRlbF9w bGFuZShjcnRjLT5iYXNlLmRldiwgcCkgewo+IC0JCWlmIChjcnRjLT5waXBlICE9IHAtPnBpcGUp Cj4gLQkJCWNvbnRpbnVlOwo+IC0KPiAtCQlwbGFuZV9zdGF0ZSA9IHRvX2ludGVsX3BsYW5lX3N0 YXRlKHAtPmJhc2Uuc3RhdGUpOwo+IC0KPiAtCQlpZiAocC0+YmFzZS50eXBlID09IERSTV9QTEFO RV9UWVBFX1BSSU1BUlkpCj4gLQkJCXBsYW5lX3N0YXRlLT52aXNpYmxlID0gcHJpbWFyeV9nZXRf aHdfc3RhdGUoY3J0Yyk7Cj4gLQkJZWxzZSB7Cj4gLQkJCWlmIChhY3RpdmUpCj4gLQkJCQlwLT5k aXNhYmxlX3BsYW5lKCZwLT5iYXNlLCAmY3J0Yy0+YmFzZSk7Cj4gLQo+IC0JCQlwbGFuZV9zdGF0 ZS0+dmlzaWJsZSA9IGZhbHNlOwo+IC0JCX0KPiAtCX0KPiAtfQo+IC0KPiAgc3RhdGljIHZvaWQg aW50ZWxfbW9kZXNldF9yZWFkb3V0X2h3X3N0YXRlKHN0cnVjdCBkcm1fZGV2aWNlICpkZXYpCj4g IHsKPiAgCXN0cnVjdCBkcm1faTkxNV9wcml2YXRlICpkZXZfcHJpdiA9IGRldi0+ZGV2X3ByaXZh dGU7Cj4gQEAgLTE1MDc2LDM1ICsxNTA1Miw4IEBAIHN0YXRpYyB2b2lkIGludGVsX21vZGVzZXRf cmVhZG91dF9od19zdGF0ZShzdHJ1Y3QgZHJtX2RldmljZSAqZGV2KQo+ICAKPiAgCQljcnRjLT5i YXNlLnN0YXRlLT5hY3RpdmUgPSBjcnRjLT5hY3RpdmU7Cj4gIAkJY3J0Yy0+YmFzZS5lbmFibGVk ID0gY3J0Yy0+YWN0aXZlOwo+IC0KPiAtCQltZW1zZXQoJmNydGMtPmJhc2UubW9kZSwgMCwgc2l6 ZW9mKGNydGMtPmJhc2UubW9kZSkpOwo+IC0JCWlmIChjcnRjLT5iYXNlLnN0YXRlLT5hY3RpdmUp IHsKPiAtCQkJaW50ZWxfbW9kZV9mcm9tX3BpcGVfY29uZmlnKCZjcnRjLT5iYXNlLm1vZGUsIGNy dGMtPmNvbmZpZyk7Cj4gLQkJCWludGVsX21vZGVfZnJvbV9waXBlX2NvbmZpZygmY3J0Yy0+YmFz ZS5zdGF0ZS0+YWRqdXN0ZWRfbW9kZSwgY3J0Yy0+Y29uZmlnKTsKPiAtCQkJV0FSTl9PTihkcm1f YXRvbWljX3NldF9tb2RlX2Zvcl9jcnRjKGNydGMtPmJhc2Uuc3RhdGUsICZjcnRjLT5iYXNlLm1v ZGUpKTsKPiAtCj4gLQkJCS8qCj4gLQkJCSAqIFRoZSBpbml0aWFsIG1vZGUgbmVlZHMgdG8gYmUg c2V0IGluIG9yZGVyIHRvIGtlZXAKPiAtCQkJICogdGhlIGF0b21pYyBjb3JlIGhhcHB5LiBJdCB3 YW50cyBhIHZhbGlkIG1vZGUgaWYgdGhlCj4gLQkJCSAqIGNydGMncyBlbmFibGVkLCBzbyB3ZSBk byB0aGUgYWJvdmUgY2FsbC4KPiAtCQkJICoKPiAtCQkJICogQXQgdGhpcyBwb2ludCBzb21lIHN0 YXRlIHVwZGF0ZWQgYnkgdGhlIGNvbm5lY3RvcnMKPiAtCQkJICogaW4gdGhlaXIgLT5kZXRlY3Qo KSBjYWxsYmFjayBoYXMgbm90IHJ1biB5ZXQsIHNvCj4gLQkJCSAqIG5vIHJlY2FsY3VsYXRpb24g Y2FuIGJlIGRvbmUgeWV0Lgo+IC0JCQkgKgo+IC0JCQkgKiBFdmVuIGlmIHdlIGNvdWxkIGRvIGEg cmVjYWxjdWxhdGlvbiBhbmQgbW9kZXNldAo+IC0JCQkgKiByaWdodCBub3cgaXQgd291bGQgY2F1 c2UgYSBkb3VibGUgbW9kZXNldCBpZgo+IC0JCQkgKiBmYmRldiBvciB1c2Vyc3BhY2UgY2hvb3Nl cyBhIGRpZmZlcmVudCBpbml0aWFsIG1vZGUuCj4gLQkJCSAqCj4gLQkJCSAqIElmIHRoYXQgaGFw cGVucywgc29tZW9uZSBpbmRpY2F0ZWQgdGhleSB3YW50ZWQgYQo+IC0JCQkgKiBtb2RlIGNoYW5n ZSwgd2hpY2ggbWVhbnMgaXQncyBzYWZlIHRvIGRvIGEgZnVsbAo+IC0JCQkgKiByZWNhbGN1bGF0 aW9uLgo+IC0JCQkgKi8KPiAtCQkJY3J0Yy0+YmFzZS5zdGF0ZS0+bW9kZS5wcml2YXRlX2ZsYWdz ID0gSTkxNV9NT0RFX0ZMQUdfSU5IRVJJVEVEOwo+IC0JCX0KPiAtCj4gLQkJY3J0Yy0+YmFzZS5o d21vZGUgPSBjcnRjLT5jb25maWctPmJhc2UuYWRqdXN0ZWRfbW9kZTsKPiAtCQlyZWFkb3V0X3Bs YW5lX3N0YXRlKGNydGMsIHRvX2ludGVsX2NydGNfc3RhdGUoY3J0Yy0+YmFzZS5zdGF0ZSkpOwo+ ICsJCXRvX2ludGVsX3BsYW5lX3N0YXRlKGNydGMtPmJhc2UucHJpbWFyeS0+c3RhdGUpLT52aXNp YmxlID0KPiArCQkJcHJpbWFyeV9nZXRfaHdfc3RhdGUoY3J0Yyk7Cj4gIAo+ICAJCURSTV9ERUJV R19LTVMoIltDUlRDOiVkXSBodyBzdGF0ZSByZWFkb3V0OiAlc1xuIiwKPiAgCQkJICAgICAgY3J0 Yy0+YmFzZS5iYXNlLmlkLAo+IEBAIC0xNTE4Niw2ICsxNTEzNSwzMyBAQCBpbnRlbF9tb2Rlc2V0 X3NldHVwX2h3X3N0YXRlKHN0cnVjdCBkcm1fZGV2aWNlICpkZXYpCj4gIAo+ICAJZm9yX2VhY2hf cGlwZShkZXZfcHJpdiwgcGlwZSkgewo+ICAJCWNydGMgPSB0b19pbnRlbF9jcnRjKGRldl9wcml2 LT5waXBlX3RvX2NydGNfbWFwcGluZ1twaXBlXSk7Cj4gKwkJY3J0Yy0+YmFzZS5od21vZGUgPSBj cnRjLT5jb25maWctPmJhc2UuYWRqdXN0ZWRfbW9kZTsKPiArCQltZW1zZXQoJmNydGMtPmJhc2Uu bW9kZSwgMCwgc2l6ZW9mKGNydGMtPmJhc2UubW9kZSkpOwo+ICsJCWlmIChjcnRjLT5iYXNlLnN0 YXRlLT5hY3RpdmUpIHsKPiArCQkJaW50ZWxfbW9kZV9mcm9tX3BpcGVfY29uZmlnKCZjcnRjLT5i YXNlLm1vZGUsIGNydGMtPmNvbmZpZyk7Cj4gKwkJCWludGVsX21vZGVfZnJvbV9waXBlX2NvbmZp ZygmY3J0Yy0+YmFzZS5zdGF0ZS0+YWRqdXN0ZWRfbW9kZSwgY3J0Yy0+Y29uZmlnKTsKPiArCQkJ V0FSTl9PTihkcm1fYXRvbWljX3NldF9tb2RlX2Zvcl9jcnRjKGNydGMtPmJhc2Uuc3RhdGUsICZj cnRjLT5iYXNlLm1vZGUpKTsKPiArCj4gKwkJCS8qCj4gKwkJCSAqIFRoZSBpbml0aWFsIG1vZGUg bmVlZHMgdG8gYmUgc2V0IGluIG9yZGVyIHRvIGtlZXAKPiArCQkJICogdGhlIGF0b21pYyBjb3Jl IGhhcHB5LiBJdCB3YW50cyBhIHZhbGlkIG1vZGUgaWYgdGhlCj4gKwkJCSAqIGNydGMncyBlbmFi bGVkLCBzbyB3ZSBkbyB0aGUgYWJvdmUgY2FsbC4KPiArCQkJICoKPiArCQkJICogQXQgdGhpcyBw b2ludCBzb21lIHN0YXRlIHVwZGF0ZWQgYnkgdGhlIGNvbm5lY3RvcnMKPiArCQkJICogaW4gdGhl aXIgLT5kZXRlY3QoKSBjYWxsYmFjayBoYXMgbm90IHJ1biB5ZXQsIHNvCj4gKwkJCSAqIG5vIHJl Y2FsY3VsYXRpb24gY2FuIGJlIGRvbmUgeWV0Lgo+ICsJCQkgKgo+ICsJCQkgKiBFdmVuIGlmIHdl IGNvdWxkIGRvIGEgcmVjYWxjdWxhdGlvbiBhbmQgbW9kZXNldAo+ICsJCQkgKiByaWdodCBub3cg aXQgd291bGQgY2F1c2UgYSBkb3VibGUgbW9kZXNldCBpZgo+ICsJCQkgKiBmYmRldiBvciB1c2Vy c3BhY2UgY2hvb3NlcyBhIGRpZmZlcmVudCBpbml0aWFsIG1vZGUuCj4gKwkJCSAqCj4gKwkJCSAq IElmIHRoYXQgaGFwcGVucywgc29tZW9uZSBpbmRpY2F0ZWQgdGhleSB3YW50ZWQgYQo+ICsJCQkg KiBtb2RlIGNoYW5nZSwgd2hpY2ggbWVhbnMgaXQncyBzYWZlIHRvIGRvIGEgZnVsbAo+ICsJCQkg KiByZWNhbGN1bGF0aW9uLgo+ICsJCQkgKi8KPiArCQkJY3J0Yy0+YmFzZS5zdGF0ZS0+bW9kZS5w cml2YXRlX2ZsYWdzID0gSTkxNV9NT0RFX0ZMQUdfSU5IRVJJVEVEOwo+ICsJCX0KPiArCj4gIAkJ aW50ZWxfc2FuaXRpemVfY3J0YyhjcnRjKTsKPiAgCQlpbnRlbF9kdW1wX3BpcGVfY29uZmlnKGNy dGMsIGNydGMtPmNvbmZpZywKPiAgCQkJCSAgICAgICAiW3NldHVwX2h3X3N0YXRlXSIpOwo+Cj4g X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KPiBJbnRlbC1n ZnggbWFpbGluZyBsaXN0Cj4gSW50ZWwtZ2Z4QGxpc3RzLmZyZWVkZXNrdG9wLm9yZwo+IGh0dHA6 Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9pbnRlbC1nZngKCi0tIApK YW5pIE5pa3VsYSwgSW50ZWwgT3BlbiBTb3VyY2UgVGVjaG5vbG9neSBDZW50ZXIKX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KSW50ZWwtZ2Z4IG1haWxpbmcg bGlzdApJbnRlbC1nZnhAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHA6Ly9saXN0cy5mcmVlZGVz a3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9pbnRlbC1nZngK