From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754354AbcAWSYy (ORCPT ); Sat, 23 Jan 2016 13:24:54 -0500 Received: from mail-wm0-f52.google.com ([74.125.82.52]:33512 "EHLO mail-wm0-f52.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754045AbcAWSYC (ORCPT ); Sat, 23 Jan 2016 13:24:02 -0500 Subject: Re: linux-4.4 bisected: kwin5 stuck on kde5 loading screen with radeon To: =?UTF-8?B?VmlsbGUgU3lyasOkbMOk?= , =?UTF-8?Q?Michel_D=c3=a4nzer?= References: <569CC357.8030302@suse.cz> <569FEEDE.4060409@gmail.com> <56A053CE.7000500@daenzer.net> <56A06D2E.4000008@gmail.com> <56A07CF9.5060506@daenzer.net> <56A07D97.6030606@daenzer.net> <20160121075849.GH19130@phenom.ffwll.local> <56A0989E.30006@daenzer.net> <20160121100905.GL19130@phenom.ffwll.local> <56A19C98.8020208@daenzer.net> <20160122151835.GM23290@intel.com> <56A27508.4050701@gmail.com> Cc: Vlastimil Babka , LKML , dri-devel@lists.freedesktop.org, Alex Deucher , =?UTF-8?Q?Christian_K=c3=b6nig?= From: Mario Kleiner Message-ID: <56A3C53E.9010908@gmail.com> Date: Sat, 23 Jan 2016 19:23:58 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.5.1 MIME-Version: 1.0 In-Reply-To: <56A27508.4050701@gmail.com> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 01/22/2016 07:29 PM, Mario Kleiner wrote: > > > On 01/22/2016 04:18 PM, Ville Syrjälä wrote: >> On Fri, Jan 22, 2016 at 12:06:00PM +0900, Michel Dänzer wrote: >>> >>> [ Trimming KDE folks from Cc ] >>> >>> On 21.01.2016 19:09, Daniel Vetter wrote: >>>> On Thu, Jan 21, 2016 at 05:36:46PM +0900, Michel Dänzer wrote: >>>>> On 21.01.2016 16:58, Daniel Vetter wrote: >>>>>> >>>>>> Can you please point me at the vblank on/off jump bug please? >>>>> >>>>> AFAIR I originally reported it in response to >>>>> http://lists.freedesktop.org/archives/dri-devel/2015-August/087841.html >>>>> >>>>> , but I can't find that in the archives, so maybe that was just on >>>>> IRC. >>>>> See >>>>> http://lists.freedesktop.org/archives/dri-devel/2016-January/099122.html >>>>> >>>>> . Basically, I ran into the bug fixed by your patch because the >>>>> counter >>>>> jumped forward on every DPMS off, so it hit the 32-bit boundary after >>>>> just a few days. >>>> >>>> Ok, so just uncovered the overflow bug. >>> >>> Not sure what you mean by "just", but to be clear: The drm_vblank_on/off >>> counter jumping bug (similar to the bug this thread is about), which >>> exposed the overflow bug, is still alive and kicking in 4.5. It seems >>> to happen when turning off the CRTC: >>> >>> [drm:drm_update_vblank_count] updating vblank count on crtc 0: >>> current=218104694, diff=0, hw=916 hw_last=916 >>> [drm:radeon_get_vblank_counter_kms] crtc 0: dist from vblank start 3 >>> [drm:drm_calc_vbltimestamp_from_scanoutpos] crtc 0 : v 0x7 >>> p(2199,-45)@ 7304.307354 -> 7304.308006 [e 0 us, 0 rep] >>> [drm:radeon_get_vblank_counter_kms] crtc 0: dist from vblank start 3 >>> [drm:drm_update_vblank_count] updating vblank count on crtc 0: >>> current=218104694, diff=16776301, hw=1 hw_last=916 >> >> Not sure what bug we're talking about here, but here the hw counter >> clearly jumps backwards. >> >>> [drm:radeon_get_vblank_counter_kms] Query failed! stat 3 >>> [drm:radeon_get_vblank_counter_kms] Query failed! stat 3 >>> [drm:drm_update_vblank_count] updating vblank count on crtc 1: >>> current=0, diff=0, hw=0 hw_last=0 >>> [drm:radeon_get_vblank_counter_kms] Query failed! stat 3 >>> [drm:radeon_get_vblank_counter_kms] Query failed! stat 3 >>> [drm:drm_update_vblank_count] updating vblank count on crtc 2: >>> current=0, diff=0, hw=0 hw_last=0 >>> [drm:radeon_get_vblank_counter_kms] Query failed! stat 3 >>> [drm:radeon_get_vblank_counter_kms] Query failed! stat 3 >>> [drm:drm_update_vblank_count] updating vblank count on crtc 3: >>> current=0, diff=0, hw=0 hw_last=0 >>> [drm:radeon_get_vblank_counter_kms] Query failed! stat 1 >>> [drm:drm_calc_vbltimestamp_from_scanoutpos] crtc 0 : v 0x1 p(0,0)@ >>> 7304.317140 -> 7304.317140 [e 0 us, 0 rep] >>> [drm:radeon_get_vblank_counter_kms] Query failed! stat 1 >>> [drm:drm_update_vblank_count] updating vblank count on crtc 0: >>> current=234880995, diff=16777215, hw=0 hw_last=1 >> >> Same here. >> >> These things just don't happen on i915 because drm_vblank_off() and >> drm_vblank_on() are always called around the times when the hw counter >> might get reset. Or at least that's how it should be. >> > > Fwiw, testing the HD-57570 single display with my patch that uses > drm_vblank_off/on() in the DPMS OFF/ON path of radeon-kms does show > hardware counter reset to zero as expected, but no jumps of software > vblank counter. So with that vblank_off/on placement it seems to work > nicely here. > > -mario > I spoke too early. The jump doesn't happen when i change video modes - video resolution / refresh rate etc, despite hw counter reset. But if i just disable and then reenable a display, the software counter jumps. -mario From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mario Kleiner Subject: Re: linux-4.4 bisected: kwin5 stuck on kde5 loading screen with radeon Date: Sat, 23 Jan 2016 19:23:58 +0100 Message-ID: <56A3C53E.9010908@gmail.com> References: <569CC357.8030302@suse.cz> <569FEEDE.4060409@gmail.com> <56A053CE.7000500@daenzer.net> <56A06D2E.4000008@gmail.com> <56A07CF9.5060506@daenzer.net> <56A07D97.6030606@daenzer.net> <20160121075849.GH19130@phenom.ffwll.local> <56A0989E.30006@daenzer.net> <20160121100905.GL19130@phenom.ffwll.local> <56A19C98.8020208@daenzer.net> <20160122151835.GM23290@intel.com> <56A27508.4050701@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: Received: from mail-wm0-f50.google.com (mail-wm0-f50.google.com [74.125.82.50]) by gabe.freedesktop.org (Postfix) with ESMTPS id 329B86E0D5 for ; Sat, 23 Jan 2016 10:24:02 -0800 (PST) Received: by mail-wm0-f50.google.com with SMTP id u188so20850123wmu.1 for ; Sat, 23 Jan 2016 10:24:02 -0800 (PST) In-Reply-To: <56A27508.4050701@gmail.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: =?UTF-8?B?VmlsbGUgU3lyasOkbMOk?= , =?UTF-8?Q?Michel_D=c3=a4nzer?= Cc: Alex Deucher , dri-devel@lists.freedesktop.org, LKML , Vlastimil Babka , =?UTF-8?Q?Christian_K=c3=b6nig?= List-Id: dri-devel@lists.freedesktop.org T24gMDEvMjIvMjAxNiAwNzoyOSBQTSwgTWFyaW8gS2xlaW5lciB3cm90ZToKPgo+Cj4gT24gMDEv MjIvMjAxNiAwNDoxOCBQTSwgVmlsbGUgU3lyasOkbMOkIHdyb3RlOgo+PiBPbiBGcmksIEphbiAy MiwgMjAxNiBhdCAxMjowNjowMFBNICswOTAwLCBNaWNoZWwgRMOkbnplciB3cm90ZToKPj4+Cj4+ PiBbIFRyaW1taW5nIEtERSBmb2xrcyBmcm9tIENjIF0KPj4+Cj4+PiBPbiAyMS4wMS4yMDE2IDE5 OjA5LCBEYW5pZWwgVmV0dGVyIHdyb3RlOgo+Pj4+IE9uIFRodSwgSmFuIDIxLCAyMDE2IGF0IDA1 OjM2OjQ2UE0gKzA5MDAsIE1pY2hlbCBEw6RuemVyIHdyb3RlOgo+Pj4+PiBPbiAyMS4wMS4yMDE2 IDE2OjU4LCBEYW5pZWwgVmV0dGVyIHdyb3RlOgo+Pj4+Pj4KPj4+Pj4+IENhbiB5b3UgcGxlYXNl IHBvaW50IG1lIGF0IHRoZSB2Ymxhbmsgb24vb2ZmIGp1bXAgYnVnIHBsZWFzZT8KPj4+Pj4KPj4+ Pj4gQUZBSVIgSSBvcmlnaW5hbGx5IHJlcG9ydGVkIGl0IGluIHJlc3BvbnNlIHRvCj4+Pj4+IGh0 dHA6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvYXJjaGl2ZXMvZHJpLWRldmVsLzIwMTUtQXVndXN0 LzA4Nzg0MS5odG1sCj4+Pj4+Cj4+Pj4+ICwgYnV0IEkgY2FuJ3QgZmluZCB0aGF0IGluIHRoZSBh cmNoaXZlcywgc28gbWF5YmUgdGhhdCB3YXMganVzdCBvbgo+Pj4+PiBJUkMuCj4+Pj4+IFNlZQo+ Pj4+PiBodHRwOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL2FyY2hpdmVzL2RyaS1kZXZlbC8yMDE2 LUphbnVhcnkvMDk5MTIyLmh0bWwKPj4+Pj4KPj4+Pj4gLiBCYXNpY2FsbHksIEkgcmFuIGludG8g dGhlIGJ1ZyBmaXhlZCBieSB5b3VyIHBhdGNoIGJlY2F1c2UgdGhlCj4+Pj4+IGNvdW50ZXIKPj4+ Pj4ganVtcGVkIGZvcndhcmQgb24gZXZlcnkgRFBNUyBvZmYsIHNvIGl0IGhpdCB0aGUgMzItYml0 IGJvdW5kYXJ5IGFmdGVyCj4+Pj4+IGp1c3QgYSBmZXcgZGF5cy4KPj4+Pgo+Pj4+IE9rLCBzbyBq dXN0IHVuY292ZXJlZCB0aGUgb3ZlcmZsb3cgYnVnLgo+Pj4KPj4+IE5vdCBzdXJlIHdoYXQgeW91 IG1lYW4gYnkgImp1c3QiLCBidXQgdG8gYmUgY2xlYXI6IFRoZSBkcm1fdmJsYW5rX29uL29mZgo+ Pj4gY291bnRlciBqdW1waW5nIGJ1ZyAoc2ltaWxhciB0byB0aGUgYnVnIHRoaXMgdGhyZWFkIGlz IGFib3V0KSwgd2hpY2gKPj4+IGV4cG9zZWQgdGhlIG92ZXJmbG93IGJ1ZywgaXMgc3RpbGwgYWxp dmUgYW5kIGtpY2tpbmcgaW4gNC41LiBJdCBzZWVtcwo+Pj4gdG8gaGFwcGVuIHdoZW4gdHVybmlu ZyBvZmYgdGhlIENSVEM6Cj4+Pgo+Pj4gW2RybTpkcm1fdXBkYXRlX3ZibGFua19jb3VudF0gdXBk YXRpbmcgdmJsYW5rIGNvdW50IG9uIGNydGMgMDoKPj4+IGN1cnJlbnQ9MjE4MTA0Njk0LCBkaWZm PTAsIGh3PTkxNiBod19sYXN0PTkxNgo+Pj4gW2RybTpyYWRlb25fZ2V0X3ZibGFua19jb3VudGVy X2ttc10gY3J0YyAwOiBkaXN0IGZyb20gdmJsYW5rIHN0YXJ0IDMKPj4+IFtkcm06ZHJtX2NhbGNf dmJsdGltZXN0YW1wX2Zyb21fc2Nhbm91dHBvc10gY3J0YyAwIDogdiAweDcKPj4+IHAoMjE5OSwt NDUpQCA3MzA0LjMwNzM1NCAtPiA3MzA0LjMwODAwNiBbZSAwIHVzLCAwIHJlcF0KPj4+IFtkcm06 cmFkZW9uX2dldF92YmxhbmtfY291bnRlcl9rbXNdIGNydGMgMDogZGlzdCBmcm9tIHZibGFuayBz dGFydCAzCj4+PiBbZHJtOmRybV91cGRhdGVfdmJsYW5rX2NvdW50XSB1cGRhdGluZyB2Ymxhbmsg Y291bnQgb24gY3J0YyAwOgo+Pj4gY3VycmVudD0yMTgxMDQ2OTQsIGRpZmY9MTY3NzYzMDEsIGh3 PTEgaHdfbGFzdD05MTYKPj4KPj4gTm90IHN1cmUgd2hhdCBidWcgd2UncmUgdGFsa2luZyBhYm91 dCBoZXJlLCBidXQgaGVyZSB0aGUgaHcgY291bnRlcgo+PiBjbGVhcmx5IGp1bXBzIGJhY2t3YXJk cy4KPj4KPj4+IFtkcm06cmFkZW9uX2dldF92YmxhbmtfY291bnRlcl9rbXNdIFF1ZXJ5IGZhaWxl ZCEgc3RhdCAzCj4+PiBbZHJtOnJhZGVvbl9nZXRfdmJsYW5rX2NvdW50ZXJfa21zXSBRdWVyeSBm YWlsZWQhIHN0YXQgMwo+Pj4gW2RybTpkcm1fdXBkYXRlX3ZibGFua19jb3VudF0gdXBkYXRpbmcg dmJsYW5rIGNvdW50IG9uIGNydGMgMToKPj4+IGN1cnJlbnQ9MCwgZGlmZj0wLCBodz0wIGh3X2xh c3Q9MAo+Pj4gW2RybTpyYWRlb25fZ2V0X3ZibGFua19jb3VudGVyX2ttc10gUXVlcnkgZmFpbGVk ISBzdGF0IDMKPj4+IFtkcm06cmFkZW9uX2dldF92YmxhbmtfY291bnRlcl9rbXNdIFF1ZXJ5IGZh aWxlZCEgc3RhdCAzCj4+PiBbZHJtOmRybV91cGRhdGVfdmJsYW5rX2NvdW50XSB1cGRhdGluZyB2 YmxhbmsgY291bnQgb24gY3J0YyAyOgo+Pj4gY3VycmVudD0wLCBkaWZmPTAsIGh3PTAgaHdfbGFz dD0wCj4+PiBbZHJtOnJhZGVvbl9nZXRfdmJsYW5rX2NvdW50ZXJfa21zXSBRdWVyeSBmYWlsZWQh IHN0YXQgMwo+Pj4gW2RybTpyYWRlb25fZ2V0X3ZibGFua19jb3VudGVyX2ttc10gUXVlcnkgZmFp bGVkISBzdGF0IDMKPj4+IFtkcm06ZHJtX3VwZGF0ZV92YmxhbmtfY291bnRdIHVwZGF0aW5nIHZi bGFuayBjb3VudCBvbiBjcnRjIDM6Cj4+PiBjdXJyZW50PTAsIGRpZmY9MCwgaHc9MCBod19sYXN0 PTAKPj4+IFtkcm06cmFkZW9uX2dldF92YmxhbmtfY291bnRlcl9rbXNdIFF1ZXJ5IGZhaWxlZCEg c3RhdCAxCj4+PiBbZHJtOmRybV9jYWxjX3ZibHRpbWVzdGFtcF9mcm9tX3NjYW5vdXRwb3NdIGNy dGMgMCA6IHYgMHgxIHAoMCwwKUAKPj4+IDczMDQuMzE3MTQwIC0+IDczMDQuMzE3MTQwIFtlIDAg dXMsIDAgcmVwXQo+Pj4gW2RybTpyYWRlb25fZ2V0X3ZibGFua19jb3VudGVyX2ttc10gUXVlcnkg ZmFpbGVkISBzdGF0IDEKPj4+IFtkcm06ZHJtX3VwZGF0ZV92YmxhbmtfY291bnRdIHVwZGF0aW5n IHZibGFuayBjb3VudCBvbiBjcnRjIDA6Cj4+PiBjdXJyZW50PTIzNDg4MDk5NSwgZGlmZj0xNjc3 NzIxNSwgaHc9MCBod19sYXN0PTEKPj4KPj4gU2FtZSBoZXJlLgo+Pgo+PiBUaGVzZSB0aGluZ3Mg anVzdCBkb24ndCBoYXBwZW4gb24gaTkxNSBiZWNhdXNlIGRybV92Ymxhbmtfb2ZmKCkgYW5kCj4+ IGRybV92Ymxhbmtfb24oKSBhcmUgYWx3YXlzIGNhbGxlZCBhcm91bmQgdGhlIHRpbWVzIHdoZW4g dGhlIGh3IGNvdW50ZXIKPj4gbWlnaHQgZ2V0IHJlc2V0LiBPciBhdCBsZWFzdCB0aGF0J3MgaG93 IGl0IHNob3VsZCBiZS4KPj4KPgo+IEZ3aXcsIHRlc3RpbmcgdGhlIEhELTU3NTcwIHNpbmdsZSBk aXNwbGF5IHdpdGggbXkgcGF0Y2ggdGhhdCB1c2VzCj4gZHJtX3ZibGFua19vZmYvb24oKSBpbiB0 aGUgRFBNUyBPRkYvT04gcGF0aCBvZiByYWRlb24ta21zIGRvZXMgc2hvdwo+IGhhcmR3YXJlIGNv dW50ZXIgcmVzZXQgdG8gemVybyBhcyBleHBlY3RlZCwgYnV0IG5vIGp1bXBzIG9mIHNvZnR3YXJl Cj4gdmJsYW5rIGNvdW50ZXIuIFNvIHdpdGggdGhhdCB2Ymxhbmtfb2ZmL29uIHBsYWNlbWVudCBp dCBzZWVtcyB0byB3b3JrCj4gbmljZWx5IGhlcmUuCj4KPiAtbWFyaW8KPgoKSSBzcG9rZSB0b28g ZWFybHkuIFRoZSBqdW1wIGRvZXNuJ3QgaGFwcGVuIHdoZW4gaSBjaGFuZ2UgdmlkZW8gbW9kZXMg LSAKdmlkZW8gcmVzb2x1dGlvbiAvIHJlZnJlc2ggcmF0ZSBldGMsIGRlc3BpdGUgaHcgY291bnRl ciByZXNldC4gQnV0IGlmIGkgCmp1c3QgZGlzYWJsZSBhbmQgdGhlbiByZWVuYWJsZSBhIGRpc3Bs YXksIHRoZSBzb2Z0d2FyZSBjb3VudGVyIGp1bXBzLgoKLW1hcmlvCl9fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCmRyaS1kZXZlbCBtYWlsaW5nIGxpc3QKZHJp LWRldmVsQGxpc3RzLmZyZWVkZXNrdG9wLm9yZwpodHRwOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3Jn L21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVsCg==