From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: [PATCH 1/2] drm: Fix off-by-one in vblank hardware counter wraparound handling Date: Wed, 27 May 2015 10:42:38 +0300 Message-ID: <20150527074238.GI18908@intel.com> References: <1432630419-23490-1-git-send-email-michel@daenzer.net> <20150526114811.GL12971@phenom.ffwll.local> 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 8C34B72105 for ; Wed, 27 May 2015 00:42:50 -0700 (PDT) Content-Disposition: inline In-Reply-To: <20150526114811.GL12971@phenom.ffwll.local> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Daniel Vetter Cc: Michel =?iso-8859-1?Q?D=E4nzer?= , dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org T24gVHVlLCBNYXkgMjYsIDIwMTUgYXQgMDE6NDg6MTFQTSArMDIwMCwgRGFuaWVsIFZldHRlciB3 cm90ZToKPiBPbiBUdWUsIE1heSAyNiwgMjAxNSBhdCAwNTo1MzozOFBNICswOTAwLCBNaWNoZWwg RMOkbnplciB3cm90ZToKPiA+IEZyb206IE1pY2hlbCBEw6RuemVyIDxtaWNoZWwuZGFlbnplckBh bWQuY29tPgo+ID4gCj4gPiBkZXYtPm1heF92YmxhbmtfY291bnQgY29udGFpbnMgdGhlIGxhcmdl c3QgdmFsdWUgdGhhdCBjYW4gYmUgcmVwcmVzZW50ZWQKPiA+IGJ5IHRoZSBoYXJkd2FyZSBjb3Vu dGVyLiBXaGVuIHRoZSBoYXJkd2FyZSBjb3VudGVyIHdyYXBzIGFyb3VuZCwgd2UgaGF2ZQo+ID4g dG8gYWRkIHRoYXQgdmFsdWUgKyAxIHRvIGdldCB0aGUgc2FtZSB2YWx1ZSBhcyBpZiB0aGUgaGFy ZHdhcmUgY291bnRlcgo+ID4gZGlkbid0IHdyYXAgYXJvdW5kLgo+ID4gCj4gPiBTaWduZWQtb2Zm LWJ5OiBNaWNoZWwgRMOkbnplciA8bWljaGVsLmRhZW56ZXJAYW1kLmNvbT4KPiAKPiBXZWxsIHRo ZXJlJ3MgdHdvIHVzZXJzIG9mIHRoaXMgcmVhbGx5LCBvbmUgd2FudHMgdGhlIG1heCAodGhpcyBv bmUgaGVyZSkKPiBhbmQgdGhlIG90aGVyIGEgbWFzay4gQW5kIGFsbCB0aGUgZHJpdmVycyB1c2Ug aXQgYXMgYSBtYXNrLiBNYXliZSByZW5hbWUKPiBpdCB0byB2YmxhbmtfY291bnRlcl9tYXNrIG9y IHNpbWlsYXIgd2hpbGUgYXQgaXQgdG8gcHJldmVudCBmdXJ0aGVyCj4gY29uZnVzaW9uPwoKV2Vs bCwgcmVhbGx5IGl0J3MganVzdCAndmFsdWUgJSAobWF4ICsgMSknLiBKdXN0IGhhcHBlbnMgdG8g d29yayB3aXRoICYKc2luY2UgaXQncyBQT1QtMS4KCj4gLURhbmllbAo+IAo+ID4gLS0tCj4gPiAg ZHJpdmVycy9ncHUvZHJtL2RybV9pcnEuYyB8IDQgKystLQo+ID4gIDEgZmlsZSBjaGFuZ2VkLCAy IGluc2VydGlvbnMoKyksIDIgZGVsZXRpb25zKC0pCj4gPiAKPiA+IGRpZmYgLS1naXQgYS9kcml2 ZXJzL2dwdS9kcm0vZHJtX2lycS5jIGIvZHJpdmVycy9ncHUvZHJtL2RybV9pcnEuYwo+ID4gaW5k ZXggYzhhMzQ0Ny4uZjk2MzRkYSAxMDA2NDQKPiA+IC0tLSBhL2RyaXZlcnMvZ3B1L2RybS9kcm1f aXJxLmMKPiA+ICsrKyBiL2RyaXZlcnMvZ3B1L2RybS9kcm1faXJxLmMKPiA+IEBAIC0xMDAsNyAr MTAwLDcgQEAgc3RhdGljIHZvaWQgZHJtX3VwZGF0ZV92YmxhbmtfY291bnQoc3RydWN0IGRybV9k ZXZpY2UgKmRldiwgaW50IGNydGMpCj4gPiAgCS8qCj4gPiAgCSAqIEludGVycnVwdHMgd2VyZSBk aXNhYmxlZCBwcmlvciB0byB0aGlzIGNhbGwsIHNvIGRlYWwgd2l0aCBjb3VudGVyCj4gPiAgCSAq IHdyYXAgaWYgbmVlZGVkLgo+ID4gLQkgKiBOT1RFISAgSXQncyBwb3NzaWJsZSB3ZSBsb3N0IGEg ZnVsbCBkZXYtPm1heF92YmxhbmtfY291bnQgZXZlbnRzCj4gPiArCSAqIE5PVEUhICBJdCdzIHBv c3NpYmxlIHdlIGxvc3QgYSBmdWxsIGRldi0+bWF4X3ZibGFua19jb3VudCArIDEgZXZlbnRzCj4g PiAgCSAqIGhlcmUgaWYgdGhlIHJlZ2lzdGVyIGlzIHNtYWxsIG9yIHdlIGhhZCB2YmxhbmsgaW50 ZXJydXB0cyBvZmYgZm9yCj4gPiAgCSAqIGEgbG9uZyB0aW1lLgo+ID4gIAkgKgo+ID4gQEAgLTEx Nyw3ICsxMTcsNyBAQCBzdGF0aWMgdm9pZCBkcm1fdXBkYXRlX3ZibGFua19jb3VudChzdHJ1Y3Qg ZHJtX2RldmljZSAqZGV2LCBpbnQgY3J0YykKPiA+ICAJLyogRGVhbCB3aXRoIGNvdW50ZXIgd3Jh cCAqLwo+ID4gIAlkaWZmID0gY3VyX3ZibGFuayAtIHZibGFuay0+bGFzdDsKPiA+ICAJaWYgKGN1 cl92YmxhbmsgPCB2YmxhbmstPmxhc3QpIHsKPiA+IC0JCWRpZmYgKz0gZGV2LT5tYXhfdmJsYW5r X2NvdW50Owo+ID4gKwkJZGlmZiArPSBkZXYtPm1heF92YmxhbmtfY291bnQgKyAxOwo+ID4gIAo+ ID4gIAkJRFJNX0RFQlVHKCJsYXN0X3ZibGFua1slZF09MHgleCwgY3VyX3ZibGFuaz0weCV4ID0+ IGRpZmY9MHgleFxuIiwKPiA+ICAJCQkgIGNydGMsIHZibGFuay0+bGFzdCwgY3VyX3ZibGFuaywg ZGlmZik7Cj4gPiAtLSAKPiA+IDIuMS40Cj4gPiAKPiA+IF9fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fCj4gPiBkcmktZGV2ZWwgbWFpbGluZyBsaXN0Cj4gPiBk cmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCj4gPiBodHRwOi8vbGlzdHMuZnJlZWRlc2t0 b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVsCj4gCj4gLS0gCj4gRGFuaWVsIFZldHRl cgo+IFNvZnR3YXJlIEVuZ2luZWVyLCBJbnRlbCBDb3Jwb3JhdGlvbgo+IGh0dHA6Ly9ibG9nLmZm d2xsLmNoCj4gX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18K PiBkcmktZGV2ZWwgbWFpbGluZyBsaXN0Cj4gZHJpLWRldmVsQGxpc3RzLmZyZWVkZXNrdG9wLm9y Zwo+IGh0dHA6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9kcmktZGV2 ZWwKCi0tIApWaWxsZSBTeXJqw6Rsw6QKSW50ZWwgT1RDCl9fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fCmRyaS1kZXZlbCBtYWlsaW5nIGxpc3QKZHJpLWRldmVs QGxpc3RzLmZyZWVkZXNrdG9wLm9yZwpodHRwOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21haWxt YW4vbGlzdGluZm8vZHJpLWRldmVsCg==