From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH 2/6] drm: Prevent vblank counter bumps > 1 with active vblank clients. Date: Tue, 9 Feb 2016 11:23:02 +0100 Message-ID: <20160209102302.GT11240@phenom.ffwll.local> References: <1454894009-15466-1-git-send-email-mario.kleiner.de@gmail.com> <1454894009-15466-3-git-send-email-mario.kleiner.de@gmail.com> <20160209095638.GM11240@phenom.ffwll.local> <20160209100727.GG23290@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mail-wm0-f52.google.com (mail-wm0-f52.google.com [74.125.82.52]) by gabe.freedesktop.org (Postfix) with ESMTPS id A633C6E542 for ; Tue, 9 Feb 2016 02:22:40 -0800 (PST) Received: by mail-wm0-f52.google.com with SMTP id c200so53355073wme.0 for ; Tue, 09 Feb 2016 02:22:40 -0800 (PST) Content-Disposition: inline In-Reply-To: <20160209100727.GG23290@intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Cc: daniel.vetter@ffwll.ch, michel@daenzer.net, linux@bernd-steinhauser.de, stable@vger.kernel.org, dri-devel@lists.freedesktop.org, alexander.deucher@amd.com, christian.koenig@amd.com, vbabka@suse.cz List-Id: dri-devel@lists.freedesktop.org T24gVHVlLCBGZWIgMDksIDIwMTYgYXQgMTI6MDc6MjdQTSArMDIwMCwgVmlsbGUgU3lyasOkbMOk IHdyb3RlOgo+IE9uIFR1ZSwgRmViIDA5LCAyMDE2IGF0IDEwOjU2OjM4QU0gKzAxMDAsIERhbmll bCBWZXR0ZXIgd3JvdGU6Cj4gPiBPbiBNb24sIEZlYiAwOCwgMjAxNiBhdCAwMjoxMzoyNUFNICsw MTAwLCBNYXJpbyBLbGVpbmVyIHdyb3RlOgo+ID4gPiBUaGlzIGZpeGVzIGEgcmVncmVzc2lvbiBp bnRyb2R1Y2VkIGJ5IHRoZSBuZXcgZHJtX3VwZGF0ZV92YmxhbmtfY291bnQoKQo+ID4gPiBpbXBs ZW1lbnRhdGlvbiBpbiBMaW51eCA0LjQ6Cj4gPiA+IAo+ID4gPiBSZXN0cmljdCB0aGUgYnVtcCBv ZiB0aGUgc29mdHdhcmUgdmJsYW5rIGNvdW50ZXIgaW4gZHJtX3VwZGF0ZV92YmxhbmtfY291bnQo KQo+ID4gPiB0byBhIHNhZmUgbWF4aW11bSB2YWx1ZSBvZiArMSB3aGVuZXZlciB0aGVyZSBpcyB0 aGUgcG9zc2liaWxpdHkgdGhhdAo+ID4gPiBjb25jdXJyZW50IHJlYWRlcnMgb2YgdmJsYW5rIHRp bWVzdGFtcHMgY291bGQgYmUgYWN0aXZlIGF0IHRoZSBtb21lbnQsCj4gPiA+IGFzIHRoZSBjdXJy ZW50IGltcGxlbWVudGF0aW9uIG9mIHRoZSB0aW1lc3RhbXAgY2FjaGluZyBhbmQgdXBkYXRpbmcg aXMKPiA+ID4gbm90IHNhZmUgYWdhaW5zdCBjb25jdXJyZW50IHJlYWRlcnMgZm9yIGNhbGxzIHRv IHN0b3JlX3ZibGFuaygpIHdpdGggYQo+ID4gPiBidW1wIG9mIGFueXRoaW5nIGJ1dCArMS4gQSBi dW1wICE9IDEgd291bGQgdmVyeSBsaWtlbHkgcmV0dXJuIGNvcnJ1cHRlZAo+ID4gPiB0aW1lc3Rh bXBzIHRvIHVzZXJzcGFjZSwgYmVjYXVzZSB0aGUgc2FtZSBzbG90IGluIHRoZSBjYWNoZSBjb3Vs ZAo+ID4gPiBiZSBjb25jdXJyZW50bHkgd3JpdHRlbiBieSBzdG9yZV92YmxhbmsoKSBhbmQgcmVh ZCBieSBvbmUgb2YgdGhvc2UKPiA+ID4gcmVhZGVycyBpbiBhIG5vbi1hdG9taWMgZmFzaGlvbiBh bmQgd2l0aG91dCB0aGUgcmVhZC1yZXRyeSBsb2dpYwo+ID4gPiBkZXRlY3RpbmcgdGhpcyBjb2xs aXNpb24uCj4gPiA+IAo+ID4gPiBDb25jdXJyZW50IHJlYWRlcnMgY2FuIGV4aXN0IHdoaWxlIGRy bV91cGRhdGVfdmJsYW5rX2NvdW50KCkgaXMgY2FsbGVkCj4gPiA+IGZyb20gdGhlIGRybV92Ymxh bmtfb2ZmKCkgb3IgZHJtX3ZibGFua19vbigpIGZ1bmN0aW9ucyBvciBvdGhlciBub24tdmJsYW5r LQo+ID4gPiBpcnEgY2FsbGVycy4gSG93ZXZlciwgYWxsIHRob3NlIGNhbGxzIGFyZSBoYXBwZW5p bmcgd2l0aCB0aGUgdmJsX2xvY2sKPiA+ID4gbG9ja2VkIHRoZXJlYnkgcHJldmVudGluZyBhIGRy bV92YmxhbmtfZ2V0KCksIHNvIHRoZSB2YmxhbmsgcmVmY291bnQKPiA+ID4gY2FuJ3QgaW5jcmVh c2Ugd2hpbGUgZHJtX3VwZGF0ZV92YmxhbmtfY291bnQoKSBpcyBleGVjdXRpbmcuIFRoZXJlZm9y ZQo+ID4gPiBhIHplcm8gdmJsYW5rIHJlZmNvdW50IGR1cmluZyBleGVjdXRpb24gb2YgdGhhdCBm dW5jdGlvbiBzaWduYWxzIHRoYXQKPiA+ID4gaXMgc2FmZSBmb3IgYXJiaXRyYXJ5IGNvdW50ZXIg YnVtcHMgaWYgY2FsbGVkIGZyb20gb3V0c2lkZSB2YmxhbmsgaXJxLAo+ID4gPiB3aGVyZWFzIGEg bm9uLXplcm8gY291bnQgaXMgbm90IHNhZmUuCj4gPiA+IAo+ID4gPiBXaGVuZXZlciB0aGUgZnVu Y3Rpb24gaXMgY2FsbGVkIGZyb20gdmJsYW5rIGlycSwgd2UgaGF2ZSB0byBhc3N1bWUgY29uY3Vy cmVudAo+ID4gPiByZWFkZXJzIGNvdWxkIHNob3cgdXAgYW55IHRpbWUgZHVyaW5nIGl0cyBleGVj dXRpb24sIGV2ZW4gaWYgdGhlIHJlZmNvdW50Cj4gPiA+IGlzIGN1cnJlbnRseSB6ZXJvLCBhcyB2 YmxhbmsgaXJxcyBhcmUgdXN1YWxseSBvbmx5IGVuYWJsZWQgZHVlIHRvIHRoZQo+ID4gPiBwcmVz ZW5jZSBvZiByZWFkZXJzLCBhbmQgYmVjYXVzZSB3aGVuIGl0IGlzIGNhbGxlZCBmcm9tIHZibGFu ayBpcnEgaXQKPiA+ID4gY2FuJ3QgaG9sZCB0aGUgdmJsX2xvY2sgdG8gcHJvdGVjdCBpdCBmcm9t IHN1ZGRlbiBidW1wcyBpbiB2YmxhbmsgcmVmY291bnQuCj4gPiA+IFRoZXJlZm9yZSBhbHNvIHJl c3RyaWN0IGJ1bXBzIHRvICsxIHdoZW4gdGhlIGZ1bmN0aW9uIGlzIGNhbGxlZCBmcm9tIHZibGFu awo+ID4gPiBpcnEuCj4gPiA+IAo+ID4gPiBTdWNoIGJ1bXBzIG9mIG1vcmUgdGhhbiArMSBjYW4g aGFwcGVuIGF0IG90aGVyIHRpbWVzIHRoYW4gcmVlbmFibGluZwo+ID4gPiB2YmxhbmsgaXJxcywg ZS5nLiwgd2hlbiByZWd1bGFyIHZibGFuayBpbnRlcnJ1cHRzIGdldCBkZWxheWVkIGJ5IG1vcmUK PiA+ID4gdGhhbiAxIGZyYW1lIGR1ZSB0byBsb25nIGhlbGQgbG9ja3MsIGxvbmcgaXJxIG9mZiBw ZXJpb2RzLCByZWFsdGltZQo+ID4gPiBwcmVlbXB0aW9uIG9uIFJUIGtlcm5lbHMsIG9yIHN5c3Rl bSBtYW5hZ2VtZW50IGludGVycnVwdHMuCj4gPiA+IAo+ID4gPiBTaWduZWQtb2ZmLWJ5OiBNYXJp byBLbGVpbmVyIDxtYXJpby5rbGVpbmVyLmRlQGdtYWlsLmNvbT4KPiA+ID4gQ2M6IDxzdGFibGVA dmdlci5rZXJuZWwub3JnPiAjIDQuNCsKPiA+ID4gQ2M6IG1pY2hlbEBkYWVuemVyLm5ldAo+ID4g PiBDYzogdmJhYmthQHN1c2UuY3oKPiA+ID4gQ2M6IHZpbGxlLnN5cmphbGFAbGludXguaW50ZWwu Y29tCj4gPiA+IENjOiBkYW5pZWwudmV0dGVyQGZmd2xsLmNoCj4gPiA+IENjOiBkcmktZGV2ZWxA bGlzdHMuZnJlZWRlc2t0b3Aub3JnCj4gPiA+IENjOiBhbGV4YW5kZXIuZGV1Y2hlckBhbWQuY29t Cj4gPiA+IENjOiBjaHJpc3RpYW4ua29lbmlnQGFtZC5jb20KPiA+IAo+ID4gSW1vIHRoaXMgaXMg ZHVjdC10YXBlLiBJZiB3ZSB3YW50IHRvIGZpeCB0aGlzIHVwIHByb3Blcmx5IEkgdGhpbmsgd2UK PiA+IHNob3VsZCBqdXN0IHVzZSBhIGZ1bGwtYmxvd24gc2VxbG9jayBpbnN0ZWFkIG9mIG91ciBo YW5kLXJvbGxlZCBvbmUuIEFuZAo+ID4gdGhhdCBjb3VsZCBoYW5kbGUgYW55IGluY3JlbWVudCBh dCBhbGwuCj4gCj4gQW5kIEkgZXZlbiBmaXhlZCB0aGlzIFsxXSBhbG1vc3QgYSBoYWxmIGEgeWVh ciBhZ28gd2hlbiBJIHNlbnQgdGhlCj4gb3JpZ2luYWwgc2VyaWVzLCBidXQgdGhhdCBwYXJ0IGdv dCBoZWxkIGhvc3RhZ2UgdG8gdGhlIHNhbWUgc2VxbG9jawo+IGFyZ3VtZW50LiBQZXJmZWN0IGlz IHRoZSBlbmVteSBvZiBnb29kLgo+IAo+IFsxXSBodHRwczovL2xpc3RzLmZyZWVkZXNrdG9wLm9y Zy9hcmNoaXZlcy9pbnRlbC1nZngvMjAxNS1TZXB0ZW1iZXIvMDc1ODc5Lmh0bWwKCkhtIHllYWgs IHRoYXQgZG9lcyBzdWZmZXIgZnJvbSByZWludmVudGluZyBzZXFsb2Nrcy4gQnV0IEknZCBwcmVm ZXIgeW91cgpwYXRjaCBvdmVyIE1hcmlvJ3MgaGFjayBoZXJlIHRiaC4gWW91ciBwYXRjaCB3aXRo IHNlcWxvY2sgd291bGQgYmUgZXZlbgptb3JlIGF3ZXNvbWUuCi1EYW5pZWwKCj4gCj4gPiAtRGFu aWVsCj4gPiAKPiA+ID4gLS0tCj4gPiA+ICBkcml2ZXJzL2dwdS9kcm0vZHJtX2lycS5jIHwgNDEg KysrKysrKysrKysrKysrKysrKysrKysrKysrKysrKysrKysrKysrKysKPiA+ID4gIDEgZmlsZSBj aGFuZ2VkLCA0MSBpbnNlcnRpb25zKCspCj4gPiA+IAo+ID4gPiBkaWZmIC0tZ2l0IGEvZHJpdmVy cy9ncHUvZHJtL2RybV9pcnEuYyBiL2RyaXZlcnMvZ3B1L2RybS9kcm1faXJxLmMKPiA+ID4gaW5k ZXggYmNiODUyOC4uYWEyYzc0YiAxMDA2NDQKPiA+ID4gLS0tIGEvZHJpdmVycy9ncHUvZHJtL2Ry bV9pcnEuYwo+ID4gPiArKysgYi9kcml2ZXJzL2dwdS9kcm0vZHJtX2lycS5jCj4gPiA+IEBAIC0y MjEsNiArMjIxLDQ3IEBAIHN0YXRpYyB2b2lkIGRybV91cGRhdGVfdmJsYW5rX2NvdW50KHN0cnVj dCBkcm1fZGV2aWNlICpkZXYsIHVuc2lnbmVkIGludCBwaXBlLAo+ID4gPiAgCQlkaWZmID0gKGZs YWdzICYgRFJNX0NBTExFRF9GUk9NX1ZCTElSUSkgIT0gMDsKPiA+ID4gIAl9Cj4gPiA+ICAKPiA+ ID4gKwkvKgo+ID4gPiArCSAqIFJlc3RyaWN0IHRoZSBidW1wIG9mIHRoZSBzb2Z0d2FyZSB2Ymxh bmsgY291bnRlciB0byBhIHNhZmUgbWF4aW11bQo+ID4gPiArCSAqIHZhbHVlIG9mICsxIHdoZW5l dmVyIHRoZXJlIGlzIHRoZSBwb3NzaWJpbGl0eSB0aGF0IGNvbmN1cnJlbnQgcmVhZGVycwo+ID4g PiArCSAqIG9mIHZibGFuayB0aW1lc3RhbXBzIGNvdWxkIGJlIGFjdGl2ZSBhdCB0aGUgbW9tZW50 LCBhcyB0aGUgY3VycmVudAo+ID4gPiArCSAqIGltcGxlbWVudGF0aW9uIG9mIHRoZSB0aW1lc3Rh bXAgY2FjaGluZyBhbmQgdXBkYXRpbmcgaXMgbm90IHNhZmUKPiA+ID4gKwkgKiBhZ2FpbnN0IGNv bmN1cnJlbnQgcmVhZGVycyBmb3IgY2FsbHMgdG8gc3RvcmVfdmJsYW5rKCkgd2l0aCBhIGJ1bXAK PiA+ID4gKwkgKiBvZiBhbnl0aGluZyBidXQgKzEuIEEgYnVtcCAhPSAxIHdvdWxkIHZlcnkgbGlr ZWx5IHJldHVybiBjb3JydXB0ZWQKPiA+ID4gKwkgKiB0aW1lc3RhbXBzIHRvIHVzZXJzcGFjZSwg YmVjYXVzZSB0aGUgc2FtZSBzbG90IGluIHRoZSBjYWNoZSBjb3VsZAo+ID4gPiArCSAqIGJlIGNv bmN1cnJlbnRseSB3cml0dGVuIGJ5IHN0b3JlX3ZibGFuaygpIGFuZCByZWFkIGJ5IG9uZSBvZiB0 aG9zZQo+ID4gPiArCSAqIHJlYWRlcnMgd2l0aG91dCB0aGUgcmVhZC1yZXRyeSBsb2dpYyBkZXRl Y3RpbmcgdGhlIGNvbGxpc2lvbi4KPiA+ID4gKwkgKgo+ID4gPiArCSAqIENvbmN1cnJlbnQgcmVh ZGVycyBjYW4gZXhpc3Qgd2hlbiB3ZSBhcmUgY2FsbGVkIGZyb20gdGhlCj4gPiA+ICsJICogZHJt X3ZibGFua19vZmYoKSBvciBkcm1fdmJsYW5rX29uKCkgZnVuY3Rpb25zIGFuZCBvdGhlciBub24t dmJsYW5rLQo+ID4gPiArCSAqIGlycSBjYWxsZXJzLiBIb3dldmVyLCBhbGwgdGhvc2UgY2FsbHMg dG8gdXMgYXJlIGhhcHBlbmluZyB3aXRoIHRoZQo+ID4gPiArCSAqIHZibF9sb2NrIGxvY2tlZCB0 byBwcmV2ZW50IGRybV92YmxhbmtfZ2V0KCksIHNvIHRoZSB2YmxhbmsgcmVmY291bnQKPiA+ID4g KwkgKiBjYW4ndCBpbmNyZWFzZSB3aGlsZSB3ZSBhcmUgZXhlY3V0aW5nLiBUaGVyZWZvcmUgYSB6 ZXJvIHJlZmNvdW50IGF0Cj4gPiA+ICsJICogdGhpcyBwb2ludCBpcyBzYWZlIGZvciBhcmJpdHJh cnkgY291bnRlciBidW1wcyBpZiB3ZSBhcmUgY2FsbGVkCj4gPiA+ICsJICogb3V0c2lkZSB2Ymxh bmsgaXJxLCBhIG5vbi16ZXJvIGNvdW50IGlzIG5vdCAxMDAlIHNhZmUuIFVuZm9ydHVuYXRlbHkK PiA+ID4gKwkgKiB3ZSBtdXN0IGFsc28gYWNjZXB0IGEgcmVmY291bnQgb2YgMSwgYXMgd2hlbmV2 ZXIgd2UgYXJlIGNhbGxlZCBmcm9tCj4gPiA+ICsJICogZHJtX3ZibGFua19nZXQoKSAtPiBkcm1f dmJsYW5rX2VuYWJsZSgpIHRoZSByZWZjb3VudCB3aWxsIGJlIDEgYW5kCj4gPiA+ICsJICogd2Ug bXVzdCBsZXQgdGhhdCBvbmUgcGFzcyB0aHJvdWdoIGluIG9yZGVyIHRvIG5vdCBsb3NlIHZibGFu ayBjb3VudHMKPiA+ID4gKwkgKiBkdXJpbmcgdmJsYW5rIGlycSBvZmYgLSB3aGljaCB3b3VsZCBj b21wbGV0ZWx5IGRlZmVhdCB0aGUgd2hvbGUKPiA+ID4gKwkgKiBwb2ludCBvZiB0aGlzIHJvdXRp bmUuCj4gPiA+ICsJICoKPiA+ID4gKwkgKiBXaGVuZXZlciB3ZSBhcmUgY2FsbGVkIGZyb20gdmJs YW5rIGlycSwgd2UgaGF2ZSB0byBhc3N1bWUgY29uY3VycmVudAo+ID4gPiArCSAqIHJlYWRlcnMg ZXhpc3Qgb3IgY2FuIHNob3cgdXAgYW55IHRpbWUgZHVyaW5nIG91ciBleGVjdXRpb24sIGV2ZW4g aWYKPiA+ID4gKwkgKiB0aGUgcmVmY291bnQgaXMgY3VycmVudGx5IHplcm8sIGFzIHZibGFuayBp cnFzIGFyZSB1c3VhbGx5IG9ubHkKPiA+ID4gKwkgKiBlbmFibGVkIGR1ZSB0byB0aGUgcHJlc2Vu Y2Ugb2YgcmVhZGVycywgYW5kIGJlY2F1c2Ugd2hlbiB3ZSBhcmUgY2FsbGVkCj4gPiA+ICsJICog ZnJvbSB2YmxhbmsgaXJxIHdlIGNhbid0IGhvbGQgdGhlIHZibF9sb2NrIHRvIHByb3RlY3QgdXMg ZnJvbSBzdWRkZW4KPiA+ID4gKwkgKiBidW1wcyBpbiB2YmxhbmsgcmVmY291bnQuIFRoZXJlZm9y ZSBhbHNvIHJlc3RyaWN0IGJ1bXBzIHRvICsxIHdoZW4KPiA+ID4gKwkgKiBjYWxsZWQgZnJvbSB2 YmxhbmsgaXJxLgo+ID4gPiArCSAqLwo+ID4gPiArCWlmICgoZGlmZiA+IDEpICYmIChhdG9taWNf cmVhZCgmdmJsYW5rLT5yZWZjb3VudCkgPiAxIHx8Cj4gPiA+ICsJICAgIChmbGFncyAmIERSTV9D QUxMRURfRlJPTV9WQkxJUlEpKSkgewo+ID4gPiArCQlEUk1fREVCVUdfVkJMKCJjbGFtcGluZyB2 YmxhbmsgYnVtcCB0byAxIG9uIGNydGMgJXU6IGRpZmZyPSV1ICIKPiA+ID4gKwkJCSAgICAgICJy ZWZjb3VudCAldSwgdmJsaXJxICV1XG4iLCBwaXBlLCBkaWZmLAo+ID4gPiArCQkJICAgICAgYXRv bWljX3JlYWQoJnZibGFuay0+cmVmY291bnQpLAo+ID4gPiArCQkJICAgICAgKGZsYWdzICYgRFJN X0NBTExFRF9GUk9NX1ZCTElSUSkgIT0gMCk7Cj4gPiA+ICsJCWRpZmYgPSAxOwo+ID4gPiArCX0K PiA+ID4gKwo+ID4gPiAgCURSTV9ERUJVR19WQkwoInVwZGF0aW5nIHZibGFuayBjb3VudCBvbiBj cnRjICV1OiIKPiA+ID4gIAkJICAgICAgIiBjdXJyZW50PSV1LCBkaWZmPSV1LCBodz0ldSBod19s YXN0PSV1XG4iLAo+ID4gPiAgCQkgICAgICBwaXBlLCB2YmxhbmstPmNvdW50LCBkaWZmLCBjdXJf dmJsYW5rLCB2YmxhbmstPmxhc3QpOwo+ID4gPiAtLSAKPiA+ID4gMS45LjEKPiA+ID4gCj4gPiAK PiA+IC0tIAo+ID4gRGFuaWVsIFZldHRlcgo+ID4gU29mdHdhcmUgRW5naW5lZXIsIEludGVsIENv cnBvcmF0aW9uCj4gPiBodHRwOi8vYmxvZy5mZndsbC5jaAo+IAo+IC0tIAo+IFZpbGxlIFN5cmrD pGzDpAo+IEludGVsIE9UQwoKLS0gCkRhbmllbCBWZXR0ZXIKU29mdHdhcmUgRW5naW5lZXIsIElu dGVsIENvcnBvcmF0aW9uCmh0dHA6Ly9ibG9nLmZmd2xsLmNoCl9fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fCmRyaS1kZXZlbCBtYWlsaW5nIGxpc3QKZHJpLWRl dmVsQGxpc3RzLmZyZWVkZXNrdG9wLm9yZwpodHRwczovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9t YWlsbWFuL2xpc3RpbmZvL2RyaS1kZXZlbAo=