From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: [PATCH 09/12] drm/i915: wait for a vblank instead of 50ms when enabling FBC Date: Fri, 13 Nov 2015 23:20:19 +0200 Message-ID: <20151113212019.GG4437@intel.com> References: <1447444424-17168-1-git-send-email-paulo.r.zanoni@intel.com> <1447444424-17168-10-git-send-email-paulo.r.zanoni@intel.com> <20151113210343.GB569@nuc-i3427.alporthouse.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by gabe.freedesktop.org (Postfix) with ESMTP id AF3E16E6BA for ; Fri, 13 Nov 2015 13:20:40 -0800 (PST) Content-Disposition: inline In-Reply-To: <20151113210343.GB569@nuc-i3427.alporthouse.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Chris Wilson , Paulo Zanoni , intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org T24gRnJpLCBOb3YgMTMsIDIwMTUgYXQgMDk6MDM6NDNQTSArMDAwMCwgQ2hyaXMgV2lsc29uIHdy b3RlOgo+IE9uIEZyaSwgTm92IDEzLCAyMDE1IGF0IDA1OjUzOjQxUE0gLTAyMDAsIFBhdWxvIFph bm9uaSB3cm90ZToKPiA+IEluc3RlYWQgb2Ygd2FpdGluZyBmb3IgNTBtcywganVzdCB3YWl0IHVu dGlsIHRoZSBuZXh0IHZibGFuaywgc2luY2UKPiA+IGl0J3MgdGhlIG1pbmltdW0gcmVxdWlyZW1l bnQuCj4gPiAKPiA+IFRoaXMgbW92ZXMgUEM3IHJlc2lkZW5jeSBvbiBteSBzcGVjaWZpYyBCRFcg bWFjaGluZSBydW5uaW5nIENpbm5hbW9uCj4gPiBmcm9tIDYwLTcwJSB0byA4NC04OSUuIFdpdGhv dXQgRkJDLCBJIGdldCAyMC0yNSUuIEknbSB1c2luZyBhCj4gPiAzMjAweDE4MDAgZURQIHBhbmVs LiBOb3RpY2U6IHRoaXMgd2FzIHRoZSBjYXNlIHdoZW4gdGhlIHBhdGNoIHdhcwo+ID4gb3JpZ2lu YWxseSBwcm9wb3NlZCwgdGhlIG9yZGVyIG9mIHRoZSBGQkMgcGF0Y2hlcyBjaGFuZ2VkIHNpbmNl IHRoZW4sCj4gPiBzbyB0aGUgYWN0dWFsIG51bWJlcnMgbWlnaHQgYmUgc2xpZ2h0bHkgZGlmZmVy ZW50IG5vdy4KPiA+IAo+ID4gdjI6Cj4gPiAgIC0gUmViYXNlIGFmdGVyIGNoYW5naW5nIHRoZSBw YXRjaCBvcmRlci4KPiA+ICAgLSBVcGRhdGUgdGhlIGNvbW1pdCBtZXNzYWdlLgo+ID4gCj4gPiBT aWduZWQtb2ZmLWJ5OiBQYXVsbyBaYW5vbmkgPHBhdWxvLnIuemFub25pQGludGVsLmNvbT4KPiA+ IC0tLQo+ID4gIGRyaXZlcnMvZ3B1L2RybS9pOTE1L2k5MTVfZHJ2LmggIHwgIDIgKy0KPiA+ICBk cml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9mYmMuYyB8IDEyICsrKy0tLS0tLS0tLQo+ID4gIDIg ZmlsZXMgY2hhbmdlZCwgNCBpbnNlcnRpb25zKCspLCAxMCBkZWxldGlvbnMoLSkKPiA+IAo+ID4g ZGlmZiAtLWdpdCBhL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2k5MTVfZHJ2LmggYi9kcml2ZXJzL2dw dS9kcm0vaTkxNS9pOTE1X2Rydi5oCj4gPiBpbmRleCA5NDE4YmQ1Li5lYTA4NzE0IDEwMDY0NAo+ ID4gLS0tIGEvZHJpdmVycy9ncHUvZHJtL2k5MTUvaTkxNV9kcnYuaAo+ID4gKysrIGIvZHJpdmVy cy9ncHUvZHJtL2k5MTUvaTkxNV9kcnYuaAo+ID4gQEAgLTkxOSw5ICs5MTksOSBAQCBzdHJ1Y3Qg aTkxNV9mYmMgewo+ID4gIAo+ID4gIAlzdHJ1Y3QgaW50ZWxfZmJjX3dvcmsgewo+ID4gIAkJYm9v bCBzY2hlZHVsZWQ7Cj4gPiArCQl1MzIgc2NoZWR1bGVkX3ZibGFuazsKPiA+ICAJCXN0cnVjdCB3 b3JrX3N0cnVjdCB3b3JrOwo+ID4gIAkJc3RydWN0IGRybV9mcmFtZWJ1ZmZlciAqZmI7Cj4gPiAt CQl1bnNpZ25lZCBsb25nIGVuYWJsZV9qaWZmaWVzOwo+ID4gIAl9IHdvcms7Cj4gPiAgCj4gPiAg CWNvbnN0IGNoYXIgKm5vX2ZiY19yZWFzb247Cj4gPiBkaWZmIC0tZ2l0IGEvZHJpdmVycy9ncHUv ZHJtL2k5MTUvaW50ZWxfZmJjLmMgYi9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9mYmMuYwo+ ID4gaW5kZXggYWE4MjA3NS4uNzJkZThhMSAxMDA2NDQKPiA+IC0tLSBhL2RyaXZlcnMvZ3B1L2Ry bS9pOTE1L2ludGVsX2ZiYy5jCj4gPiArKysgYi9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9m YmMuYwo+ID4gQEAgLTM5MSw3ICszOTEsNiBAQCBzdGF0aWMgdm9pZCBpbnRlbF9mYmNfd29ya19m bihzdHJ1Y3Qgd29ya19zdHJ1Y3QgKl9fd29yaykKPiA+ICAJCWNvbnRhaW5lcl9vZihfX3dvcmss IHN0cnVjdCBkcm1faTkxNV9wcml2YXRlLCBmYmMud29yay53b3JrKTsKPiA+ICAJc3RydWN0IGlu dGVsX2ZiY193b3JrICp3b3JrID0gJmRldl9wcml2LT5mYmMud29yazsKPiA+ICAJc3RydWN0IGlu dGVsX2NydGMgKmNydGMgPSBkZXZfcHJpdi0+ZmJjLmNydGM7Cj4gPiAtCXVuc2lnbmVkIGxvbmcg ZGVsYXlfamlmZmllcyA9IG1zZWNzX3RvX2ppZmZpZXMoNTApOwo+ID4gIAo+ID4gIHJldHJ5Ogo+ ID4gIAkvKiBEZWxheSB0aGUgYWN0dWFsIGVuYWJsaW5nIHRvIGxldCBwYWdlZmxpcHBpbmcgY2Vh c2UgYW5kIHRoZQo+ID4gQEAgLTQwMCwxNCArMzk5LDkgQEAgcmV0cnk6Cj4gPiAgCSAqIHZibGFu ayB0byBwYXNzIGFmdGVyIGRpc2FibGluZyB0aGUgRkJDIGJlZm9yZSB3ZSBhdHRlbXB0Cj4gPiAg CSAqIHRvIG1vZGlmeSB0aGUgY29udHJvbCByZWdpc3RlcnMuCj4gPiAgCSAqCj4gPiAtCSAqIEEg bW9yZSBjb21wbGljYXRlZCBzb2x1dGlvbiB3b3VsZCBpbnZvbHZlIHRyYWNraW5nIHZibGFua3MK PiA+IC0JICogZm9sbG93aW5nIHRoZSB0ZXJtaW5hdGlvbiBvZiB0aGUgcGFnZS1mbGlwcGluZyBz ZXF1ZW5jZQo+ID4gLQkgKiBhbmQgaW5kZWVkIHBlcmZvcm1pbmcgdGhlIGVuYWJsZSBhcyBhIGNv LXJvdXRpbmUgYW5kIG5vdAo+ID4gLQkgKiB3YWl0aW5nIHN5bmNocm9ub3VzbHkgdXBvbiB0aGUg dmJsYW5rLgo+ID4gLQkgKgo+ID4gIAkgKiBXYUZiY1dhaXRGb3JWQmxhbmtCZWZvcmVFbmFibGU6 aWxrLHNuYgo+ID4gIAkgKi8KPiA+IC0Jd2FpdF9yZW1haW5pbmdfbXNfZnJvbV9qaWZmaWVzKHdv cmstPmVuYWJsZV9qaWZmaWVzLCBkZWxheV9qaWZmaWVzKTsKPiA+ICsJaW50ZWxfd2FpdF9mb3Jf dmJsYW5rKGRldl9wcml2LT5kZXYsIGNydGMtPnBpcGUpOwo+ID4gIAo+ID4gIAltdXRleF9sb2Nr KCZkZXZfcHJpdi0+ZmJjLmxvY2spOwo+ID4gIAo+ID4gQEAgLTQxNiw3ICs0MTAsNyBAQCByZXRy eToKPiA+ICAJCWdvdG8gb3V0Owo+ID4gIAo+ID4gIAkvKiBXZXJlIHdlIGRlbGF5ZWQgYWdhaW4g d2hpbGUgdGhpcyBmdW5jdGlvbiB3YXMgc2xlZXBpbmc/ICovCj4gPiAtCWlmICh0aW1lX2FmdGVy KHdvcmstPmVuYWJsZV9qaWZmaWVzICsgZGVsYXlfamlmZmllcywgamlmZmllcykpIHsKPiA+ICsJ aWYgKGRybV9jcnRjX3ZibGFua19nZXQoJmNydGMtPmJhc2UpID09IHdvcmstPnNjaGVkdWxlZF92 YmxhbmspIHsKPiA+ICAJCW11dGV4X3VubG9jaygmZGV2X3ByaXYtPmZiYy5sb2NrKTsKPiA+ICAJ CWdvdG8gcmV0cnk7Cj4gPiAgCX0KPiA+IEBAIC00NDksNyArNDQzLDcgQEAgc3RhdGljIHZvaWQg aW50ZWxfZmJjX3NjaGVkdWxlX2FjdGl2YXRpb24oc3RydWN0IGludGVsX2NydGMgKmNydGMpCj4g PiAgCSAqIGppZmZ5IGNvdW50LiAqLwo+ID4gIAl3b3JrLT5mYiA9IGNydGMtPmJhc2UucHJpbWFy eS0+ZmI7Cj4gPiAgCXdvcmstPnNjaGVkdWxlZCA9IHRydWU7Cj4gPiAtCXdvcmstPmVuYWJsZV9q aWZmaWVzID0gamlmZmllczsKPiA+ICsJd29yay0+c2NoZWR1bGVkX3ZibGFuayA9IGRybV9jcnRj X3ZibGFua19jb3VudCgmY3J0Yy0+YmFzZSk7Cj4gCj4gSXNuJ3QgdGhlIGZyYW1lIGNvdW50ZXIg b25seSBpbmNyZW1lbnRpbmcgd2hpbHN0IHRoZSB2YmxhbmsgSVJRIGlzCj4gZW5hYmxlZD8gVmls bGU/CgpJIHNlZSBhICIrIGlmIChkcm1fY3J0Y192YmxhbmtfZ2V0KCIgZWFybGllci4KClRoYXQg c2FpZCwgZHJtX2NydGNfdmJsYW5rX2NvdW50KCkgaXMgcmFjeS4gVGhlIHJlYWRlciBtYXkgcmFj ZSB3aXRoCnRoZSB2YmxhbmsgaXJxIGhhbmRsZXIgYWZ0ZXIgdGhlIHN0YXJ0IG9mIHZibGFuayBo YXMgYWxyZWFkeSBiZWVuCnBhc3NlZCwgdGh1cyBzYW1wbGluZyBhIHN0YWxlIHZhbHVlLCBhbmQg dGhlbiBhY3R1YWxseSBub3Qgd2FpdGluZwp1bnRpbCB0aGUgbmV4dCB2YmxhbmsuCgpJdCdzIG1v cmUgb2YgYSBwcm9ibGVtIG9uIGdlbjIvMyB3aGljaCBkaWRuJ3QgaGF2ZSB0aGUgInN0YXJ0IG9m CnZibGFuayIgaW50ZXJydXB0IGFuZCBpbnN0ZWFkIHJlbHkgb24gdGhlICJmcmFtZSBzdGFydCIg aW50ZXJydXB0LgpUaGVyZSdzIGF0IGxlYXN0IG9uZSAod2VsbCwgYWxtb3N0KSBzY2FubGluZSBi ZXR3ZWVuIHRoZSB0d28gZXZlbnRzLgoKSSd2ZSBiZWVuIG1lYW5pbmcgdG8gYWRkIGFub3RoZXIg ZnVuY3Rpb24gdG8gdGhlIHZibGFuayBjb2RlIHRoYXQKY291bGQgYmUgY2FsbGVkIGJldHdlZW4g dGhlIGRybV92YmxhbmtfZ2V0KCkgYW5kIGRybV9jcnRjX3ZibGFua19jb3VudCgpCnRvIG1ha2Ug c3VyZSB0aGUgc2FtcGxlZCBjb3VudCBpcyByZWFsbHkgdXAgdG8gZGF0ZS4gSSdkIHVzZSB0aGF0 IG9uCmdlbjIgYXQgbGVhc3Qgc2luY2UgaXQgbGFja3MgdGhlIGh3IGNvdW50ZXIuIEZvciB0aGUg b3RoZXIgcGxhdGZvcm1zLAppdCBvdWdodCB0byBiZSBlYXNpZXIgdG8ganVzdCB1c2UgdGhlIGh3 IGNvdW50ZXIgZXZlcnl3aGVyZSBzaW5jZQp0aGF0IGluY3JlbWVudHMgYXRvbWljYWxseSB3aXRo IHRoZSBzdGFydCBvZiB2YmxhbmsgYW5kIGRvZXNuJ3Qgc3VmZmVyCmZyb20gdGhpcyByYWNlLgoK LS0gClZpbGxlIFN5cmrDpGzDpApJbnRlbCBPVEMKX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX18KSW50ZWwtZ2Z4IG1haWxpbmcgbGlzdApJbnRlbC1nZnhAbGlz dHMuZnJlZWRlc2t0b3Aub3JnCmh0dHA6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9s aXN0aW5mby9pbnRlbC1nZngK