From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: [PATCH] drm/i915: Move vblank wait determination to 'check' phase Date: Thu, 19 Mar 2015 21:36:28 +0200 Message-ID: <20150319193628.GV17419@intel.com> References: <20150318220122.GG31220@intel.com> <1426716287-13092-1-git-send-email-matthew.d.roper@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by gabe.freedesktop.org (Postfix) with ESMTP id BC8186E49A for ; Thu, 19 Mar 2015 12:36:42 -0700 (PDT) Content-Disposition: inline In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Paulo Zanoni Cc: Intel Graphics Development , Paulo Zanoni List-Id: intel-gfx@lists.freedesktop.org T24gVGh1LCBNYXIgMTksIDIwMTUgYXQgMDQ6MTY6NDFQTSAtMDMwMCwgUGF1bG8gWmFub25pIHdy b3RlOgo+IDIwMTUtMDMtMTggMTk6MDQgR01ULTAzOjAwIE1hdHQgUm9wZXIgPG1hdHRoZXcuZC5y b3BlckBpbnRlbC5jb20+Ogo+ID4gRGV0ZXJtaW5pbmcgd2hldGhlciB3ZSdsbCBuZWVkIHRvIHdh aXQgZm9yIHZibGFua3MgaXMgc29tZXRoaW5nIHdlCj4gPiBzaG91bGQgZGV0ZXJtaW5lIGR1cmlu ZyB0aGUgYXRvbWljICdjaGVjaycgcGhhc2UsIG5vdCB0aGUgJ2NvbW1pdCcKPiA+IHBoYXNlLiAg Tm90ZSB0aGF0IHdlIG9ubHkgc2V0IHRoZXNlIGJpdHMgaW4gdGhlIGJyYW5jaCBvZiAnY2hlY2sn IHdoZXJlCj4gPiBpbnRlbF9jcnRjLT5hY3RpdmUgaXMgdHJ1ZSBzbyB0aGF0IHdlIGRvbid0IHRy eSB0byB3YWl0IG9uIGEgZGlzYWJsZWQKPiA+IENSVEMuCj4gPgo+ID4gVGhlIHdob2xlICd3YWl0 IGZvciB2YmxhbmsgYWZ0ZXIgdXBkYXRlJyBmbGFnIHNob3VsZCBnbyBhd2F5IGluIHRoZQo+ID4g ZnV0dXJlLCBvbmNlIHdlIHN0YXJ0IGhhbmRsaW5nIHdhdGVybWFya3MgaW4gYSBwcm9wZXIgYXRv bWljIG1hbm5lci4KPiAKPiBBZGQgdGhpcyB0byB0aGUgY29tbWl0IG1lc3NhZ2U6Cj4gCj4gUmVn cmVzc2lvbiBpbnRyb2R1Y2VkIGJ5Ogo+IAo+IGNvbW1pdCAyZmRkN2RlZjE2ZGQ3NTgwZjI5Nzgy NzkzMDEyNmMxNmIxNTJlYzExCj4gQXV0aG9yOiBNYXR0IFJvcGVyIDxtYXR0aGV3LmQucm9wZXJA aW50ZWwuY29tPgo+IERhdGU6ICAgV2VkIE1hciA0IDEwOjQ5OjA0IDIwMTUgLTA4MDAKPiAgICAg ZHJtL2k5MTU6IERvbid0IGNsb2JiZXIgcGxhbmUgc3RhdGUgb24gaW50ZXJuYWwgZGlzYWJsZXMK PiAKPiAoYXQgbGVhc3QgYWNjb3JkaW5nIHRvIFFBJ3MgYmlzZWN0KQo+IAo+ID4KPiA+IENjOiBQ YXVsbyBaYW5vbmkgPHBhdWxvLnIuemFub25pQGludGVsLmNvbT4KPiA+IFJvb3QtY2F1c2UtYW5h bHlzaXMtYnk6IFBhdWxvIFphbm9uaSA8cGF1bG8uci56YW5vbmlAaW50ZWwuY29tPgo+ID4gQnVn emlsbGE6IGh0dHBzOi8vYnVncy5mcmVlZGVza3RvcC5vcmcvc2hvd19idWcuY2dpP2lkPTg5NTUw Cj4gPiBUZXN0Y2FzZTogaWd0L3BtX3JwbS9sZWdhY3ktcGxhbmVzCj4gPiBUZXN0Y2FzZTogaWd0 L3BtX3JwbS9sZWdhY3ktcGxhbmVzLWRwbXMKPiA+IFRlc3RjYXNlOiBpZ3QvcG1fcnBtL3VuaXZl cnNhbC1wbGFuZXMKPiA+IFRlc3RjYXNlOiBpZ3QvcG1fcnBtL3VuaXZlcnNhbC1wbGFuZXMtZHBt cwo+ID4gU2lnbmVkLW9mZi1ieTogTWF0dCBSb3BlciA8bWF0dGhldy5kLnJvcGVyQGludGVsLmNv bT4KPiA+IC0tLQo+ID4gUGF1bG8sIGNhbiB5b3UgdGVzdCB0aGlzIHBhdGNoIGFuZCBzZWUgaWYg aXQgc29sdmVzIHRoZSBwcm9ibGVtPyAgVGhlIG9ubHkKPiA+IHBsYXRmb3JtIEkgaGF2ZSBhY2Nl c3MgdG8gKElWQikgZG9lc24ndCBoYXZlIHJ1bnRpbWUgcG93ZXIgbWFuYWdlbWVudCwgc28gSQo+ ID4gY2FuJ3QgYWN0dWFsbHkgcnVuIHRoZSByZWxldmFudCB0ZXN0Y2FzZXMgbXlzZWxmLgo+IAo+ IFllcywgdGhpcyBwYXRjaCBtYWtlcyB0aGUgV0FSTiBnbyBhd2F5IG9uIG15IEJEVyBtYWNoaW5l IDopCj4gVGVzdGVkLWJ5OiBQYXVsbyBaYW5vbmkgPHBhdWxvLnIuemFub25pQGludGVsLmNvbT4K PiAKPiBPbmUgb2YgdGhlIHBvc3NpYmxlIHByb2JsZW1zIHdpdGggdGhpcyBwYXRjaCBpcyB0aGF0 LCBiZWZvcmUgaXQsIHRoZQo+IHdhaXRfdmJsYW5rIHdhcyBvbmx5IHNldCBmb3IgSUxLLUJEVywg YnV0IG5vdyB3ZSBzZXQgaXQgZm9yIGFsbAo+IHBsYXRmb3Jtcy4gRG8gd2UgcmVhbGx5IHdhbnQg dGhpcz8gT3RoZXJ3aXNlLCBpdCBsb29rcyBnb29kLCBhbHRob3VnaAo+IEkgaGF2ZW4ndCBiZWVu IGNsb3NlbHkgZm9sbG93aW5nIHRoZSBhdG9taWMgcmV3b3JrIHRvIGJlIGFibGUgdG8KPiBhc3Nl cnQgdGhpcyBpcyByZWFsbHkgYWNjb3JkaW5nIHRvIHRoZSBncmFuZCBwbGFucy4gRGFuaWVsL1Zp bGxlIGNvdWxkCj4gY29tbWVudCBoZXJlIDopCgpTb21ldGhpbnMgc2VlbXMgYSBiaXQgb2ZmIHRv IG1lIGlmIHdlIGVuZCB1cCBjYWxsaW5nIC5kaXNhYmxlX3BsYW5lKCkKd2l0aCBhIGRpc2FibGVk IHBpcGUuIFNvIGZpeGluZyB0aGluZ3Mgc28gdGhhdCB3b24ndCBoYXBwZW4KbWlnaHQgYmUgYSBi ZXR0ZXIgYXBwcm9hY2guCgo+IAo+ID4KPiA+ICBkcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9z cHJpdGUuYyB8IDI0ICsrKysrKysrKystLS0tLS0tLS0tLS0tLQo+ID4gIDEgZmlsZSBjaGFuZ2Vk LCAxMCBpbnNlcnRpb25zKCspLCAxNCBkZWxldGlvbnMoLSkKPiA+Cj4gPiBkaWZmIC0tZ2l0IGEv ZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfc3ByaXRlLmMgYi9kcml2ZXJzL2dwdS9kcm0vaTkx NS9pbnRlbF9zcHJpdGUuYwo+ID4gaW5kZXggYTgyODczNi4uZmFkMWQ5ZiAxMDA2NDQKPiA+IC0t LSBhL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX3Nwcml0ZS5jCj4gPiArKysgYi9kcml2ZXJz L2dwdS9kcm0vaTkxNS9pbnRlbF9zcHJpdGUuYwo+ID4gQEAgLTc0NywxMyArNzQ3LDYgQEAgaXZi X2Rpc2FibGVfcGxhbmUoc3RydWN0IGRybV9wbGFuZSAqcGxhbmUsIHN0cnVjdCBkcm1fY3J0YyAq Y3J0YykKPiA+ICAgICAgICAgSTkxNV9XUklURShTUFJTVVJGKHBpcGUpLCAwKTsKPiA+Cj4gPiAg ICAgICAgIGludGVsX2ZsdXNoX3ByaW1hcnlfcGxhbmUoZGV2X3ByaXYsIGludGVsX2NydGMtPnBs YW5lKTsKPiA+IC0KPiA+IC0gICAgICAgLyoKPiA+IC0gICAgICAgICogQXZvaWQgdW5kZXJydW5z IHdoZW4gZGlzYWJsaW5nIHRoZSBzcHJpdGUuCj4gPiAtICAgICAgICAqIEZJWE1FIHJlbW92ZSBv bmNlIHdhdGVybWFyayB1cGRhdGVzIGFyZSBkb25lIHByb3Blcmx5Lgo+ID4gLSAgICAgICAgKi8K PiA+IC0gICAgICAgaW50ZWxfY3J0Yy0+YXRvbWljLndhaXRfdmJsYW5rID0gdHJ1ZTsKPiA+IC0g ICAgICAgaW50ZWxfY3J0Yy0+YXRvbWljLnVwZGF0ZV9zcHJpdGVfd2F0ZXJtYXJrcyB8PSAoMSA8 PCBkcm1fcGxhbmVfaW5kZXgocGxhbmUpKTsKPiA+ICB9Cj4gPgo+ID4gIHN0YXRpYyBpbnQKPiA+ IEBAIC05MzcsMTMgKzkzMCw2IEBAIGlsa19kaXNhYmxlX3BsYW5lKHN0cnVjdCBkcm1fcGxhbmUg KnBsYW5lLCBzdHJ1Y3QgZHJtX2NydGMgKmNydGMpCj4gPiAgICAgICAgIEk5MTVfV1JJVEUoRFZT U1VSRihwaXBlKSwgMCk7Cj4gPgo+ID4gICAgICAgICBpbnRlbF9mbHVzaF9wcmltYXJ5X3BsYW5l KGRldl9wcml2LCBpbnRlbF9jcnRjLT5wbGFuZSk7Cj4gPiAtCj4gPiAtICAgICAgIC8qCj4gPiAt ICAgICAgICAqIEF2b2lkIHVuZGVycnVucyB3aGVuIGRpc2FibGluZyB0aGUgc3ByaXRlLgo+ID4g LSAgICAgICAgKiBGSVhNRSByZW1vdmUgb25jZSB3YXRlcm1hcmsgdXBkYXRlcyBhcmUgZG9uZSBw cm9wZXJseS4KPiA+IC0gICAgICAgICovCj4gPiAtICAgICAgIGludGVsX2NydGMtPmF0b21pYy53 YWl0X3ZibGFuayA9IHRydWU7Cj4gPiAtICAgICAgIGludGVsX2NydGMtPmF0b21pYy51cGRhdGVf c3ByaXRlX3dhdGVybWFya3MgfD0gKDEgPDwgZHJtX3BsYW5lX2luZGV4KHBsYW5lKSk7Cj4gPiAg fQo+ID4KPiA+ICAvKioKPiA+IEBAIC0xMjYyLDYgKzEyNDgsMTYgQEAgZmluaXNoOgo+ID4gICAg ICAgICAgICAgICAgICAgICBwbGFuZS0+c3RhdGUtPmZiLT5tb2RpZmllclswXSAhPQo+ID4gICAg ICAgICAgICAgICAgICAgICBzdGF0ZS0+YmFzZS5mYi0+bW9kaWZpZXJbMF0pCj4gPiAgICAgICAg ICAgICAgICAgICAgICAgICBpbnRlbF9jcnRjLT5hdG9taWMudXBkYXRlX3dtID0gdHJ1ZTsKPiA+ ICsKPiA+ICsgICAgICAgICAgICAgICBpZiAoIXN0YXRlLT52aXNpYmxlKSB7Cj4gPiArICAgICAg ICAgICAgICAgICAgICAgICAvKgo+ID4gKyAgICAgICAgICAgICAgICAgICAgICAgICogQXZvaWQg dW5kZXJydW5zIHdoZW4gZGlzYWJsaW5nIHRoZSBzcHJpdGUuCj4gPiArICAgICAgICAgICAgICAg ICAgICAgICAgKiBGSVhNRSByZW1vdmUgb25jZSB3YXRlcm1hcmsgdXBkYXRlcyBhcmUgZG9uZSBw cm9wZXJseS4KPiA+ICsgICAgICAgICAgICAgICAgICAgICAgICAqLwo+ID4gKyAgICAgICAgICAg ICAgICAgICAgICAgaW50ZWxfY3J0Yy0+YXRvbWljLndhaXRfdmJsYW5rID0gdHJ1ZTsKPiA+ICsg ICAgICAgICAgICAgICAgICAgICAgIGludGVsX2NydGMtPmF0b21pYy51cGRhdGVfc3ByaXRlX3dh dGVybWFya3MgfD0KPiA+ICsgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgKDEgPDwgZHJt X3BsYW5lX2luZGV4KHBsYW5lKSk7Cj4gPiArICAgICAgICAgICAgICAgfQo+ID4gICAgICAgICB9 Cj4gPgo+ID4gICAgICAgICByZXR1cm4gMDsKPiA+IC0tCj4gPiAxLjguNS4xCj4gPgo+ID4gX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KPiA+IEludGVsLWdm eCBtYWlsaW5nIGxpc3QKPiA+IEludGVsLWdmeEBsaXN0cy5mcmVlZGVza3RvcC5vcmcKPiA+IGh0 dHA6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9pbnRlbC1nZngKPiAK PiAKPiAKPiAtLSAKPiBQYXVsbyBaYW5vbmkKPiBfX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fXwo+IEludGVsLWdmeCBtYWlsaW5nIGxpc3QKPiBJbnRlbC1nZnhA bGlzdHMuZnJlZWRlc2t0b3Aub3JnCj4gaHR0cDovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWls bWFuL2xpc3RpbmZvL2ludGVsLWdmeAoKLS0gClZpbGxlIFN5cmrDpGzDpApJbnRlbCBPVEMKX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KSW50ZWwtZ2Z4IG1h aWxpbmcgbGlzdApJbnRlbC1nZnhAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHA6Ly9saXN0cy5m cmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9pbnRlbC1nZngK