From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mario Kleiner Subject: Re: [PATCH 1/2] drm/radeon: Use drm_vblank_off/on to fix vblank counter trouble. Date: Fri, 22 Jan 2016 18:08:29 +0100 Message-ID: <56A2620D.7050909@gmail.com> References: <1453364174-11248-1-git-send-email-mario.kleiner.de@gmail.com> <56A095E7.8030303@daenzer.net> <56A0A1D1.3000905@gmail.com> <56A19F3C.5030208@daenzer.net> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: Received: from mail-wm0-f68.google.com (mail-wm0-f68.google.com [74.125.82.68]) by gabe.freedesktop.org (Postfix) with ESMTPS id E3ED07214C for ; Fri, 22 Jan 2016 09:08:34 -0800 (PST) Received: by mail-wm0-f68.google.com with SMTP id b14so18472574wmb.1 for ; Fri, 22 Jan 2016 09:08:34 -0800 (PST) In-Reply-To: <56A19F3C.5030208@daenzer.net> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: =?UTF-8?Q?Michel_D=c3=a4nzer?= Cc: alexander.deucher@amd.com, daniel.vetter@ffwll.ch, dri-devel@lists.freedesktop.org, vbabka@suse.cz, christian.koenig@amd.com List-Id: dri-devel@lists.freedesktop.org T24gMDEvMjIvMjAxNiAwNDoxNyBBTSwgTWljaGVsIETDpG56ZXIgd3JvdGU6Cj4gT24gMjEuMDEu MjAxNiAxODoxNiwgTWFyaW8gS2xlaW5lciB3cm90ZToKPj4gT24gMDEvMjEvMjAxNiAwOToyNSBB TSwgTWljaGVsIETDpG56ZXIgd3JvdGU6Cj4+PiBPbiAyMS4wMS4yMDE2IDE3OjE2LCBNYXJpbyBL bGVpbmVyIHdyb3RlOgo+Pj4+Cj4+Pj4gVGhpcyBwYXRjaCByZXBsYWNlcyBjYWxscyB0byBkcm1f dmJsYW5rX3ByZS9wb3N0X21vZGVzZXQgaW4gdGhlCj4+Pj4gZHJpdmVycyBkcG1zIGNvZGUgd2l0 aCBjYWxscyB0byBkcm1fdmJsYW5rX29mZi9vbiwgYXMgcmVjb21tZW5kZWQKPj4+PiBmb3IgZHJp dmVycyB3aXRoIGh3IGNvdW50ZXJzIHRoYXQgcmVzZXQgdG8gemVybyBkdXJpbmcgbW9kZXNldC4K Pj4+Cj4+PiBTb3VuZHMgbGlrZSB5b3UgZmVsbCBmb3IgdGhlIGRybV92Ymxhbmtfb24vb2ZmIHBy b3BhZ2FuZGEuIDooCj4+Pgo+Pj4gVGhpcyB3YXMgd29ya2luZyBmaW5lIHdpdGggZHJtX3ZibGFu a19wcmUvcG9zdF9tb2Rlc2V0LCB0aGF0IGl0IGJyb2tlCj4+PiBpcyBzaW1wbHkgYSByZWdyZXNz aW9uLgo+Pgo+PiBJIGFncmVlIHdpdGggeW91IHRoYXQgcHJlL3Bvc3QgbW9kZXNldCBicmVha2Fn ZSBpcyBhIHJlZ3Jlc3Npb24uIEl0J3MKPj4ganVzdCB0aGF0IGkgc3R1bWJsZWQgb3ZlciB0aGUg b24vb2ZmIHN0dWZmIHdoaWxlIHNlYXJjaGluZyBmb3IgYQo+PiBzb2x1dGlvbiBhbmQgdGhlIG90 aGVyIHNvcnQgb2YgaGFja3MgaSBjb3VsZCB0aGluayBvZiBsb29rZWQgc2ltaWxhciBvcgo+PiBt b3JlIGNvbnZvbHV0ZWQvaGFja3kvZnJhZ2lsZSB0byBtZS4KPgo+IEZpbmRpbmcgYW5kIGZpeGlu ZyB0aGUgY2F1c2Ugb2YgYSByZWdyZXNzaW9uIGlzbid0IGEgaGFjaywgaXQncwo+IGVzdGFibGlz aGVkIHByb2NlZHVyZS4KPgoKVGhhdCdzIG5vdCB3aGF0IGkgbWVhbnQuIEkgbWVhbnQgaSBjb3Vs ZG4ndCBmaW5kIHNvbWV0aGluZyBsZXNzIApjb21wbGljYXRlZC9yaXNreS93aXRob3V0IG5ldyBy ZWdyZXNzaW9uIHBvdGVudGlhbCwgc28gdGhpcyBsb29rZWQgbGlrZSAKYSBiZXR0ZXIgc29sdXRp b24uIE9mIGNvdXJzZSBpIHdvdWxkIGhhdmUgdGVzdGVkIG15IG93biBwYXRjaGVzIGFnYWluc3Qg CmF0IGxlYXN0IGEgY291cGxlIGJpdHMgb2YgdXNlcnNwYWNlIChhdGkgZGR4LCBtb2Rlc2V0dGlu ZyBkZHgsIHdlc3RvbiksIAppIGp1c3QgZGlkbid0IGhhdmUgYWNjZXNzIHRvIHRoZSBtYWNoaW5l IHllc3RlcmRheS4KCkFueXdheSwgc29tZSBtb3JlIGhvdXJzIG9mIHRoaW5raW5nIGFuZCBjb2Rl IGJyb3dzaW5nIGxhdGVyLCBub3cgaSB0aGluayAKaSBoYXZlIGEgc2ltcGxlIGFuZCBzYWZlIHNv bHV0aW9uIHdoaWNoIHNob3VsZCBob3BlZnVsbHkgcmVzdG9yZSB0aGUgCmRybV92YmxhbmtfcHJl L3Bvc3RfbW9kZXNldCBiZWhhdmlvdXIgd2l0aCBvbmx5IGEgZmV3IGxpbmVzIG9mIGNvcmUgCmNv ZGUuIEF0IHRoZSBzYW1lIHRpbWUgaXQgc2hvdWxkIGZpeCB1cCBhbm90aGVyIGJ1ZyBpbiB0aGF0 IG5ldyAKZHJtX3VwZGF0ZV92YmxhbmtfY291bnQgY29kZSB0aGF0IGkganVzdCByZWFsaXplZCwg aW4gYSB3YXkgc2ltcGxlIAplbm91Z2ggZm9yIGEgc3RhYmxlIGZpeC4KCk5vdyBpIGp1c3QgbmVl ZCB0byBhY3R1YWxseSBjb2RlIGFuZCB0ZXN0IGl0IGZpcnN0LgoKPgo+PiBBbmQgdGhleSBwcm9i YWJseSB3b3VsZG4ndCBzb2x2ZSB0aGF0IG90aGVyIHNtYWxsIHJhY2UgaSBmb3VuZCBhcyBlYXNp bHkKPj4gLSBJIGRvbid0IHRoaW5rIGl0J3MgbGlrZWx5IHRvIGhhcHBlbiAob2Z0ZW4vYXQgYWxs PykgaW4gcHJhY3RpY2UsIGJ1dCBpCj4+IGhhdmUgdHJvdWJsZSAiZm9yZ2V0dGluZyIgYWJvdXQg aXRzIGV4aXN0ZW5jZSBub3cuCj4KPiBUaGF0J3Mgc29tZXRoaW5nIHdoaWNoIHNob3VsZCBiZSBh ZGRyZXNzZWQgaW5kZXBlbmRlbnRseSBmcm9tIHRoZQo+IHJlZ3Jlc3Npb24gZml4Lgo+Cj4gUGxl YXNlIHNwbGl0IHVwIHRoZSBQTSBmaXhlcyBmcm9tIHlvdXIgcGF0Y2ggaW50byBvbmUgb3IgdHdv IHNlcGFyYXRlCj4gcGF0Y2hlcyAod2hpY2ggbWF5IGJlIGFwcHJvcHJpYXRlIGZvciA0LjUgLyBz dGFibGUgdHJlZXMpLCBhbmQgbGVhdmUgdGhlCj4gc3dpdGNoIHRvIGRybV92Ymxhbmtfb24vb2Zm IHRvIERhbmllbCdzIHBhdGNoIGZvciA0LjYuCj4KCkkgd2lsbCBsb29rIGludG8gdGhhdCBvbmNl IGknbSBkb25lIHdpdGggdGhlIGFib3ZlLCBhbmQgcHJvYmFibHkgZ290IApzb21lIHNsZWVwIGFn YWluLgoKRml4aW5nIHRoaXMgcmFjZSByZWdyZXNzaW9uIHdpdGhvdXQgc3dpdGNoaW5nIHRvIHZi bGFua19vZmYvb24gbWlnaHQgCm5lZWQgYSBzbWFsbCBiaXQgb2YgZXh0cmEgYmFuZCBhaWQgdGhl cmUuCgpCdHcuIHdydC4gdGhlIHJhZGVvbl9wbS5jIGZpeDogSXQncyBjZXJ0YWlubHkgZ29vZCB0 byBmaXggdGhhdCBwb3RlbnRpYWwgCmRybV92YmxhbmtfZ2V0L3B1dCBpbWJhbGFuY2UuIEkgd29u ZGVyIGlmIHRoYXQgIm1pZ2h0IGdsaXRjaCIgREVCVUcgCm1lc3NhZ2UgbWFrZXMgbXVjaCBzZW5z ZSB0aG91Z2guIENhbiB0aGF0IGNvZGUgcnVuIGR1cmluZyBhIG1vZGVzZXQgYXQgCmFsbD8gQW5k IGlmIHNvLCBpJ2QgYWxtb3N0IGV4cGVjdCB0aGF0IHRoZXJlIHdvbid0IGJlIGFueSB2Ymxhbmsg aXJxcyAKYXZhaWxhYmxlIGF0IHRoYXQgcG9pbnQgYW55d2F5IC0gb25jZSB0aGUgY3J0YydzIGFy ZSBvZmYgdGhleSBkb24ndCAKdHJpZ2dlciB2YmxhbmsgaXJxcyBhbnltb3JlIC0gc28gdGhhdCBj b2RlIG1pZ2h0IGdsaXRjaCBkdWUgdG8gbGFjayBvZiAKdmJsYW5rIHN5bmMgcmVnYXJkbGVzcyBp ZiBkcm1fdmJsYW5rX2dldCBpcyBzdWNjZXNzZnVsIG9yIG5vdD8KClRoZSBvdGhlciB0aGluZyBp cyBteSBwbGFjZW1lbnQgb2YgdGhlIHJhZGVvbl9wbV9jb21wdXRlX2Nsb2NrcygpIGluIHRoZSAK RFBNU19PTiBwYXRoLiBJIG1vdmVkIGl0IHRvIGZpeCB0aGUgcG90ZW50aWFsIGV4dHJhIHJhY2Ug aSBkZXNjcmliZWQuIApCdXQgdGhpbmtpbmcgYWJvdXQgaXQsIHdvdWxkbid0IGJlIHRoZSBiZXR0 ZXIgcGxhY2UgYXQgdGhlIGJlZ2lubmluZyBvZiAKdGhlIERQTVMgb24gcGF0aCwgYmVmb3JlIHRo ZSBhdG9tYmlvcyBjYWxscyByZWVuYWJsZSB0aGUgY3J0Y3M/IEkgZG9uJ3QgCmtub3cgdGhlIGRy aXZlciB3ZWxsIGVub3VnaCwgYnV0IGl0IGxvb2tlZCBhIGJpdCBzdXNwaWNpb3VzIHRvIG1lIHRo YXQgCnRoZSBtZW1vcnkgY2xvY2tzLCBsaW5lYnVmZmVyIHdhdGVybWFya3MgZXRjLiBnZXQgdXBk YXRlZCBmb3IgdGhldyBuZXcgCnZpZGVvIG1vZGUgYWZ0ZXIgdGhlIGNydGMgaGFzIGJlZW4gZW5h YmxlZC4gV29uJ3QgaXQgdGhlbiBwb3RlbnRpYWxseSAKc3RhcnQgcnVubmluZyBmb3IgYSBtb21l bnQgd2l0aCB3cm9uZyBtZW1vcnkgYmFuZHdpZHRoIGV0Yy4/IFRoYXQncyAKcHJvYmFibHkgc29t ZXRoaW5nIGZvciB5b3UgdG8gY2hlY2sgLSBubyBpZGVhLCBqdXN0IHNvbWV0aGluZyBpIG5vdGlj ZWQgCmFzIHNsaWdodGx5IG9kZCB0byBtZS4KCkFsc28gbW92aW5nIGl0IHVwIG1pZ2h0IGF2b2lk IGNvbGxpc2lvbnMgd2l0aCBEYW5pZWwncyBwYXRjaCwgaWYgdGhhdCAKbW92ZSBkb2Vzbid0IGh1 cnQuCgo+Cj4+PiBJJ20gbm90IGFnYWluc3Qgc3dpdGNoaW5nIHRvIGRybV92Ymxhbmtfb24vb2Zm IGZvciA0LjYsIGJ1dCBpdCdzIG5vdCBhCj4+PiBzb2x1dGlvbiBmb3Igb2xkZXIga2VybmVscy4K Pj4KPj4gTGludXggNC40IGlzIGFuIGVzcGVjaWFsbHkgaW1wb3J0YW50IHN0YWJsZSBrZXJuZWwg Zm9yIG1lIGJlY2F1c2UgaXQncwo+PiBzdXBwb3NlZCB0byBiZSB0aGUgc3RhbmRhcmQgZGlzdHJv IGtlcm5lbCBmb3IgVWJ1bnR1IDE2LjA0LUxUUyBhbmQKPj4gc2libGluZ3MvZGVyaXZhdGl2ZXMg KExpbnV4IE1pbnQpIGZvciB1cCB0byB0aGUgbmV4dCA1IHllYXJzLiBIYXZpbmcKPj4gbWFueSBv ZiBteSBuZXVyb3NjaWVuY2UgdXNlcnMgZW5kaW5nIG9uIHRoYXQga2VybmVsIGFzIHRoZWlyIHZl cnkgZmlyc3QKPj4gaW1wcmVzc2lvbiBvZiBMaW51eCB3aXRoIHNvbWV0aGluZyBwb3RlbnRpYWxs eSBicm9rZW4gaW4gdmJsYW5rIGxhbmQKPj4gc2NhcmVzIG1lLiBUaGUgcmVsaWFiaWxpdHkgb2Yg dGltaW5nL3RpbWVzdGFtcGluZyBzdHVmZiBpcwo+PiBzdXBlci1pbXBvcnRhbnQgZm9yIHRoZW0s IGF0IHRoZSBzYW1lIHRpbWUgaGFuZC1ob2xkaW5nIG1hbnkgb2YgdGhlbQo+PiB0aHJvdWdoIG5v bi1zdGFuZGFyZCBrZXJuZWwgdXBncmFkZXMgd291bGQgYmUgc28gbXVjaCBub3QgZnVuLgo+Cj4g QnV0IGZhc3QtdHJhY2tpbmcgdGhlIHN3aXRjaCB0byBkcm1fdmJsYW5rX29uL29mZiwgd2hpY2gg aGF2ZW4ndCBiZWVuCj4gd2lkZWx5IHRlc3RlZCB3aXRoIHRoaXMgZHJpdmVyLCBhbGwgdGhlIHdh eSB0byA0LjQgc2VlbXMgbGVzcyByaXNreSB0bwo+IHlvdT8gU2VyaW91c2x5Pwo+CgpJdCBtYWRl IGEgbG90IG9mIHNlbnNlIGFmdGVyIDEyIGhvdXJzIG9mIGJyb3dzaW5nIGNvZGUsIHRoaW5raW5n IGFib3V0IAphbGwga2luZCBvZiByYWNlIGNvbmRpdGlvbnMgYW5kIG90aGVyIG5ldyBwZXJzb25h bCBob3Jyb3JzIDspIC0gTHVja2lseSAKaSBkb24ndCBoYXZlIHRvIGRlY2lkZS4KCj4+IEp1c3Qg dG8gc2F5IGknbSBwcm9iYWJseSB3YXkgdG9vIGJpYXNlZCB3cnQuIHdoYXQgc29sdXRpb24gZm9y IHRoaXMKPj4gc2hvdWxkIGdldCBiYWNrcG9ydGVkIGludG8gYW4gb2xkZXIga2VybmVsLgo+Cj4g QXQgdGhlIHJpc2sgb2Ygc291bmRpbmcgbGlrZSBhIGJyb2tlbiByZWNvcmQ6IFRoZXJlJ3MgYSBi dWcgaW4gNC40IChhbmQKPiBvbGRlci9uZXdlciBrZXJuZWxzKSBjYXVzaW5nIHRoZSB2Ymxhbmsg Y291bnRlciB0byBqdW1wIGZvcndhcmQgYWNyb3NzCj4gRFBNUyBvZmYgd2l0aCBkcm1fdmJsYW5r X29uL29mZi4gU28gaXQgc291bmRzIGxpa2UgeW91IGRvbid0IHdhbnQgdG8gdXNlCj4gdGhvc2Ug YXQgbGVhc3QgdW50aWwgd2UndmUgZm91bmQgYW5kIGZpeGVkIHRoYXQuCj4KPgoKTW9kZXNldHRp bmcgImp1bXBzIiBhcmUgbm90IGEgcHJvYmxlbSBmb3IgbXkgYXBwbGljYXRpb24sIGFzIHByb2R1 Y3Rpb24gCm9yIHRlc3Qgc2Vzc2lvbnMgd29uJ3QgZXZlciBydW4gZHVyaW5nIG1vZGVzZXRzIG9y IG90aGVyICJkaXNydXB0aXZlIiAKZXZlbnRzIGxpa2UgZHBtcyBvbi9vZmYuIFRoYXQncyBhbHNv IHdoeSBteSBvd24gdGVzdCBzdWl0ZSBkaWRuJ3QgZGV0ZWN0IApzdWNoIHRyb3VibGUgZXZlciBk ZXNwaXRlIGV4dGVuc2l2ZSB0ZXN0aW5nLiBPYnZpb3VzbHkgbm90IGhhdmluZyBhIHdlbGwgCndv cmtpbmcgZGVza3RvcCBvciBub24td29ya2luZyBsb2dpbnMgZXRjLiB3b3VsZCBiZSBhIGJhZCBw cm9ibGVtIGZvciAKbXlzZWxmIGFuZCBteSB1c2Vycy4gSXQgc3RpbGwgc3VycHJpc2VzIG1lIHRo YXQgdGhlIGJ1ZyBWbGFzdGltaWwgCnJlcG9ydGVkIG5ldmVyIGhhcHBlbmVkIHRvIG15c2VsZiBp biBkYXlzLCBnaXZlbiB0aGF0IGkgbG9nZ2Vkb3V0L2luIGEgCmxvdCBmb3IgdGVzdGluZyBkaWZm ZXJlbnQgWC1TZXJ2ZXIgeG9yZy5jb25mIGNvbmZpZ3Mgb24gS0RFIFBsYXNtYSA1LiAKRXZlbiBu b3cgYXMgaSBzZWUgdGhlIGp1bXBzIGR1cmluZyBtb2Rlc2V0cyBpbiBteSBkbWVzZyBpIGhhdmVu J3QgCm1hbmFnZWQgdG8gZ2V0IFBsYXNtYSBvciBnbHhnZWFycyBldGMuIHRvIGxvY2t1cCBkdXJp bmcgZG96ZW5zIG9mIG1vZGVzZXRzLgoKLW1hcmlvCl9fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fCmRyaS1kZXZlbCBtYWlsaW5nIGxpc3QKZHJpLWRldmVsQGxp c3RzLmZyZWVkZXNrdG9wLm9yZwpodHRwOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4v bGlzdGluZm8vZHJpLWRldmVsCg==