From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: [PATCH] drm/i915: Postpone plane readout until after encoder readout, v3. Date: Thu, 27 Aug 2015 15:05:52 +0300 Message-ID: <20150827120552.GZ5176@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 mga09.intel.com (mga09.intel.com [134.134.136.24]) by gabe.freedesktop.org (Postfix) with ESMTP id 7E0886E853 for ; Thu, 27 Aug 2015 05:06:33 -0700 (PDT) Content-Disposition: inline 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 Cc: Intel Graphics Development List-Id: intel-gfx@lists.freedesktop.org T24gVGh1LCBBdWcgMjcsIDIwMTUgYXQgMDE6NDc6MjZQTSArMDIwMCwgTWFhcnRlbiBMYW5raG9y c3Qgd3JvdGU6Cj4gV2hlbiByZWFkaW5nIG91dCBodyBzdGF0ZSBmb3IgcGxhbmVzIHdlIGRpc2Fi bGUgaW5hY3RpdmUgcGxhbmVzIHdoaWNoIGluCj4gdHVybiB0cmlnZ2VycyBhbiB1cGRhdGUgb2Yg dGhlIHdhdGVybWFya3MuIFRoZSB1cGRhdGUgZGVwZW5kcyBvbiB0aGUKPiBjcnRjX2Nsb2NrIGJl aW5nIHNldCB3aGljaCBpcyBkb25lIHdoZW4gcmVhZGluZyBvdXQgZW5jb2RlcnMuIFRodXMKPiBw b3N0cG9uZSB0aGUgcGxhbmUgcmVhZG91dCB1bnRpbCBhZnRlciBlbmNvZGVyIHJlYWRvdXQuCj4g Cj4gVGhpcyBwcmV2ZW50cyBhIHdhcm5pbmcgaW4gc2tsX2NvbXB1dGVfbGluZXRpbWVfd20oKSB3 aGVyZSBwaXhlbF9yYXRlCj4gYmVjb21lcyAwIHdoZW4gY3J0Y19jbG9jayBpcyAwLgo+IAo+IFNp Z25lZC1vZmYtYnk6IFBhdHJpayBKYWtvYnNzb24gPHBhdHJpay5qYWtvYnNzb25AbGludXguaW50 ZWwuY29tPgo+IFNpZ25lZC1vZmYtYnk6IE1hYXJ0ZW4gTGFua2hvcnN0IDxtYWFydGVuLmxhbmto b3JzdEBsaW51eC5pbnRlbC5jb20+Cj4gUmVmZXJlbmNlczogaHR0cHM6Ly9idWdzLmZyZWVkZXNr dG9wLm9yZy9zaG93X2J1Zy5jZ2k/aWQ9OTE0MjgKPiAtLS0KPiBkaWZmIC0tZ2l0IGEvZHJpdmVy cy9ncHUvZHJtL2k5MTUvaW50ZWxfZGlzcGxheS5jIGIvZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50 ZWxfZGlzcGxheS5jCj4gaW5kZXggZTM5MjJkOTczZGIwLi4xMThiMjA1ZTdiZDUgMTAwNjQ0Cj4g LS0tIGEvZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfZGlzcGxheS5jCj4gKysrIGIvZHJpdmVy cy9ncHUvZHJtL2k5MTUvaW50ZWxfZGlzcGxheS5jCj4gQEAgLTE1MDMzLDMwICsxNTAzMyw2IEBA IHN0YXRpYyBib29sIHByaW1hcnlfZ2V0X2h3X3N0YXRlKHN0cnVjdCBpbnRlbF9jcnRjICpjcnRj KQo+ICAJcmV0dXJuICEhKEk5MTVfUkVBRChEU1BDTlRSKGNydGMtPnBsYW5lKSkgJiBESVNQTEFZ X1BMQU5FX0VOQUJMRSk7Cj4gIH0KPiAgCj4gLXN0YXRpYyB2b2lkIHJlYWRvdXRfcGxhbmVfc3Rh dGUoc3RydWN0IGludGVsX2NydGMgKmNydGMsCj4gLQkJCQlzdHJ1Y3QgaW50ZWxfY3J0Y19zdGF0 ZSAqY3J0Y19zdGF0ZSkKPiAtewo+IC0Jc3RydWN0IGludGVsX3BsYW5lICpwOwo+IC0Jc3RydWN0 IGludGVsX3BsYW5lX3N0YXRlICpwbGFuZV9zdGF0ZTsKPiAtCWJvb2wgYWN0aXZlID0gY3J0Y19z dGF0ZS0+YmFzZS5hY3RpdmU7Cj4gLQo+IC0JZm9yX2VhY2hfaW50ZWxfcGxhbmUoY3J0Yy0+YmFz ZS5kZXYsIHApIHsKPiAtCQlpZiAoY3J0Yy0+cGlwZSAhPSBwLT5waXBlKQo+IC0JCQljb250aW51 ZTsKPiAtCj4gLQkJcGxhbmVfc3RhdGUgPSB0b19pbnRlbF9wbGFuZV9zdGF0ZShwLT5iYXNlLnN0 YXRlKTsKPiAtCj4gLQkJaWYgKHAtPmJhc2UudHlwZSA9PSBEUk1fUExBTkVfVFlQRV9QUklNQVJZ KQo+IC0JCQlwbGFuZV9zdGF0ZS0+dmlzaWJsZSA9IHByaW1hcnlfZ2V0X2h3X3N0YXRlKGNydGMp Owo+IC0JCWVsc2Ugewo+IC0JCQlpZiAoYWN0aXZlKQo+IC0JCQkJcC0+ZGlzYWJsZV9wbGFuZSgm cC0+YmFzZSwgJmNydGMtPmJhc2UpOwo+IC0KPiAtCQkJcGxhbmVfc3RhdGUtPnZpc2libGUgPSBm YWxzZTsKPiAtCQl9Cj4gLQl9Cj4gLX0KPiAtCj4gIHN0YXRpYyB2b2lkIGludGVsX21vZGVzZXRf cmVhZG91dF9od19zdGF0ZShzdHJ1Y3QgZHJtX2RldmljZSAqZGV2KQo+ICB7Cj4gIAlzdHJ1Y3Qg ZHJtX2k5MTVfcHJpdmF0ZSAqZGV2X3ByaXYgPSBkZXYtPmRldl9wcml2YXRlOwo+IEBAIC0xNTA3 NiwzNSArMTUwNTIsOCBAQCBzdGF0aWMgdm9pZCBpbnRlbF9tb2Rlc2V0X3JlYWRvdXRfaHdfc3Rh dGUoc3RydWN0IGRybV9kZXZpY2UgKmRldikKPiAgCj4gIAkJY3J0Yy0+YmFzZS5zdGF0ZS0+YWN0 aXZlID0gY3J0Yy0+YWN0aXZlOwo+ICAJCWNydGMtPmJhc2UuZW5hYmxlZCA9IGNydGMtPmFjdGl2 ZTsKPiAtCj4gLQkJbWVtc2V0KCZjcnRjLT5iYXNlLm1vZGUsIDAsIHNpemVvZihjcnRjLT5iYXNl Lm1vZGUpKTsKPiAtCQlpZiAoY3J0Yy0+YmFzZS5zdGF0ZS0+YWN0aXZlKSB7Cj4gLQkJCWludGVs X21vZGVfZnJvbV9waXBlX2NvbmZpZygmY3J0Yy0+YmFzZS5tb2RlLCBjcnRjLT5jb25maWcpOwo+ IC0JCQlpbnRlbF9tb2RlX2Zyb21fcGlwZV9jb25maWcoJmNydGMtPmJhc2Uuc3RhdGUtPmFkanVz dGVkX21vZGUsIGNydGMtPmNvbmZpZyk7Cj4gLQkJCVdBUk5fT04oZHJtX2F0b21pY19zZXRfbW9k ZV9mb3JfY3J0YyhjcnRjLT5iYXNlLnN0YXRlLCAmY3J0Yy0+YmFzZS5tb2RlKSk7Cj4gLQo+IC0J CQkvKgo+IC0JCQkgKiBUaGUgaW5pdGlhbCBtb2RlIG5lZWRzIHRvIGJlIHNldCBpbiBvcmRlciB0 byBrZWVwCj4gLQkJCSAqIHRoZSBhdG9taWMgY29yZSBoYXBweS4gSXQgd2FudHMgYSB2YWxpZCBt b2RlIGlmIHRoZQo+IC0JCQkgKiBjcnRjJ3MgZW5hYmxlZCwgc28gd2UgZG8gdGhlIGFib3ZlIGNh bGwuCj4gLQkJCSAqCj4gLQkJCSAqIEF0IHRoaXMgcG9pbnQgc29tZSBzdGF0ZSB1cGRhdGVkIGJ5 IHRoZSBjb25uZWN0b3JzCj4gLQkJCSAqIGluIHRoZWlyIC0+ZGV0ZWN0KCkgY2FsbGJhY2sgaGFz IG5vdCBydW4geWV0LCBzbwo+IC0JCQkgKiBubyByZWNhbGN1bGF0aW9uIGNhbiBiZSBkb25lIHll dC4KPiAtCQkJICoKPiAtCQkJICogRXZlbiBpZiB3ZSBjb3VsZCBkbyBhIHJlY2FsY3VsYXRpb24g YW5kIG1vZGVzZXQKPiAtCQkJICogcmlnaHQgbm93IGl0IHdvdWxkIGNhdXNlIGEgZG91YmxlIG1v ZGVzZXQgaWYKPiAtCQkJICogZmJkZXYgb3IgdXNlcnNwYWNlIGNob29zZXMgYSBkaWZmZXJlbnQg aW5pdGlhbCBtb2RlLgo+IC0JCQkgKgo+IC0JCQkgKiBJZiB0aGF0IGhhcHBlbnMsIHNvbWVvbmUg aW5kaWNhdGVkIHRoZXkgd2FudGVkIGEKPiAtCQkJICogbW9kZSBjaGFuZ2UsIHdoaWNoIG1lYW5z IGl0J3Mgc2FmZSB0byBkbyBhIGZ1bGwKPiAtCQkJICogcmVjYWxjdWxhdGlvbi4KPiAtCQkJICov Cj4gLQkJCWNydGMtPmJhc2Uuc3RhdGUtPm1vZGUucHJpdmF0ZV9mbGFncyA9IEk5MTVfTU9ERV9G TEFHX0lOSEVSSVRFRDsKPiAtCQl9Cj4gLQo+IC0JCWNydGMtPmJhc2UuaHdtb2RlID0gY3J0Yy0+ Y29uZmlnLT5iYXNlLmFkanVzdGVkX21vZGU7Cj4gLQkJcmVhZG91dF9wbGFuZV9zdGF0ZShjcnRj LCB0b19pbnRlbF9jcnRjX3N0YXRlKGNydGMtPmJhc2Uuc3RhdGUpKTsKPiArCQl0b19pbnRlbF9w bGFuZV9zdGF0ZShjcnRjLT5iYXNlLnByaW1hcnktPnN0YXRlKS0+dmlzaWJsZSA9Cj4gKwkJCXBy aW1hcnlfZ2V0X2h3X3N0YXRlKGNydGMpOwoKVGhlIGNvbW1pdCBtZXNzYWdlIGlzIGZvciBhIHRv dGFsbHkgZGlmZmVyZW50IHBhdGNoIGl0IHNlZW1zLgoKSG1tLiBMb29raW5nIGF0IHRoaXMsIEkg c2hvdWxkIGhhdmUganVzdCBtb3ZlZCB0aGUgZW50aXJlIG1vZGUgc2V0dXAKYmxvY2sgaW4gbXkg aHdtb2RlIGZpeCBwYXRjaCB0byB0aGUgZW5kIG9mIHRoZSBmdW5jdGlvbiwgYW5kIHRoaW5ncyB3 b3VsZApiZSBmaXhlZC4KCj4gIAo+ICAJCURSTV9ERUJVR19LTVMoIltDUlRDOiVkXSBodyBzdGF0 ZSByZWFkb3V0OiAlc1xuIiwKPiAgCQkJICAgICAgY3J0Yy0+YmFzZS5iYXNlLmlkLAo+IEBAIC0x NTE4Niw2ICsxNTEzNSwzMyBAQCBpbnRlbF9tb2Rlc2V0X3NldHVwX2h3X3N0YXRlKHN0cnVjdCBk cm1fZGV2aWNlICpkZXYpCj4gIAo+ICAJZm9yX2VhY2hfcGlwZShkZXZfcHJpdiwgcGlwZSkgewo+ ICAJCWNydGMgPSB0b19pbnRlbF9jcnRjKGRldl9wcml2LT5waXBlX3RvX2NydGNfbWFwcGluZ1tw aXBlXSk7Cj4gKwkJY3J0Yy0+YmFzZS5od21vZGUgPSBjcnRjLT5jb25maWctPmJhc2UuYWRqdXN0 ZWRfbW9kZTsKPiArCQltZW1zZXQoJmNydGMtPmJhc2UubW9kZSwgMCwgc2l6ZW9mKGNydGMtPmJh c2UubW9kZSkpOwo+ICsJCWlmIChjcnRjLT5iYXNlLnN0YXRlLT5hY3RpdmUpIHsKPiArCQkJaW50 ZWxfbW9kZV9mcm9tX3BpcGVfY29uZmlnKCZjcnRjLT5iYXNlLm1vZGUsIGNydGMtPmNvbmZpZyk7 Cj4gKwkJCWludGVsX21vZGVfZnJvbV9waXBlX2NvbmZpZygmY3J0Yy0+YmFzZS5zdGF0ZS0+YWRq dXN0ZWRfbW9kZSwgY3J0Yy0+Y29uZmlnKTsKPiArCQkJV0FSTl9PTihkcm1fYXRvbWljX3NldF9t b2RlX2Zvcl9jcnRjKGNydGMtPmJhc2Uuc3RhdGUsICZjcnRjLT5iYXNlLm1vZGUpKTsKPiArCj4g KwkJCS8qCj4gKwkJCSAqIFRoZSBpbml0aWFsIG1vZGUgbmVlZHMgdG8gYmUgc2V0IGluIG9yZGVy IHRvIGtlZXAKPiArCQkJICogdGhlIGF0b21pYyBjb3JlIGhhcHB5LiBJdCB3YW50cyBhIHZhbGlk IG1vZGUgaWYgdGhlCj4gKwkJCSAqIGNydGMncyBlbmFibGVkLCBzbyB3ZSBkbyB0aGUgYWJvdmUg Y2FsbC4KPiArCQkJICoKPiArCQkJICogQXQgdGhpcyBwb2ludCBzb21lIHN0YXRlIHVwZGF0ZWQg YnkgdGhlIGNvbm5lY3RvcnMKPiArCQkJICogaW4gdGhlaXIgLT5kZXRlY3QoKSBjYWxsYmFjayBo YXMgbm90IHJ1biB5ZXQsIHNvCj4gKwkJCSAqIG5vIHJlY2FsY3VsYXRpb24gY2FuIGJlIGRvbmUg eWV0Lgo+ICsJCQkgKgo+ICsJCQkgKiBFdmVuIGlmIHdlIGNvdWxkIGRvIGEgcmVjYWxjdWxhdGlv biBhbmQgbW9kZXNldAo+ICsJCQkgKiByaWdodCBub3cgaXQgd291bGQgY2F1c2UgYSBkb3VibGUg bW9kZXNldCBpZgo+ICsJCQkgKiBmYmRldiBvciB1c2Vyc3BhY2UgY2hvb3NlcyBhIGRpZmZlcmVu dCBpbml0aWFsIG1vZGUuCj4gKwkJCSAqCj4gKwkJCSAqIElmIHRoYXQgaGFwcGVucywgc29tZW9u ZSBpbmRpY2F0ZWQgdGhleSB3YW50ZWQgYQo+ICsJCQkgKiBtb2RlIGNoYW5nZSwgd2hpY2ggbWVh bnMgaXQncyBzYWZlIHRvIGRvIGEgZnVsbAo+ICsJCQkgKiByZWNhbGN1bGF0aW9uLgo+ICsJCQkg Ki8KPiArCQkJY3J0Yy0+YmFzZS5zdGF0ZS0+bW9kZS5wcml2YXRlX2ZsYWdzID0gSTkxNV9NT0RF X0ZMQUdfSU5IRVJJVEVEOwo+ICsJCX0KPiArCgpXaHkgbm90IGtlZXAgaXQgaW4gaW50ZWxfbW9k ZXNldF9yZWFkb3V0X2h3X3N0YXRlKCk/IFRoYXQgd2F5IGFsbCB0aGUKbG93IGxldmVsIGd1bmsg c3RheXMgaW4gaW50ZWxfbW9kZXNldF9yZWFkb3V0X2h3X3N0YXRlKCkgYW5kCmludGVsX21vZGVz ZXRfc2V0dXBfaHdfc3RhdGUoKSBqdXN0IGhhcyB0aGUgaGlnaGVyIGxldmVsIHN0dWZmLgoKPiAg CQlpbnRlbF9zYW5pdGl6ZV9jcnRjKGNydGMpOwo+ICAJCWludGVsX2R1bXBfcGlwZV9jb25maWco Y3J0YywgY3J0Yy0+Y29uZmlnLAo+ICAJCQkJICAgICAgICJbc2V0dXBfaHdfc3RhdGVdIik7Cj4g Cj4gX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KPiBJbnRl bC1nZnggbWFpbGluZyBsaXN0Cj4gSW50ZWwtZ2Z4QGxpc3RzLmZyZWVkZXNrdG9wLm9yZwo+IGh0 dHA6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9pbnRlbC1nZngKCi0t IApWaWxsZSBTeXJqw6Rsw6QKSW50ZWwgT1RDCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fCkludGVsLWdmeCBtYWlsaW5nIGxpc3QKSW50ZWwtZ2Z4QGxpc3Rz LmZyZWVkZXNrdG9wLm9yZwpodHRwOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlz dGluZm8vaW50ZWwtZ2Z4Cg==