From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751583AbcAVDGR (ORCPT ); Thu, 21 Jan 2016 22:06:17 -0500 Received: from darkcity.gna.ch ([195.226.6.51]:57387 "EHLO mail.gna.ch" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751007AbcAVDGJ (ORCPT ); Thu, 21 Jan 2016 22:06:09 -0500 Subject: Re: linux-4.4 bisected: kwin5 stuck on kde5 loading screen with radeon To: Mario Kleiner , =?UTF-8?B?VmlsbGUgU3lyasOkbMOk?= References: <20160115122629.GC23290@intel.com> <5699C5E5.90702@gmail.com> <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> From: =?UTF-8?Q?Michel_D=c3=a4nzer?= X-Enigmail-Draft-Status: N1110 Cc: Vlastimil Babka , LKML , dri-devel@lists.freedesktop.org, Alex Deucher , =?UTF-8?Q?Christian_K=c3=b6nig?= Message-ID: <56A19C98.8020208@daenzer.net> Date: Fri, 22 Jan 2016 12:06:00 +0900 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Icedove/38.5.0 MIME-Version: 1.0 In-Reply-To: <20160121100905.GL19130@phenom.ffwll.local> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org [ 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 [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 I suspect this may not be evident with current Intel hardware because dev->max_vblank_count = 0xffffffff, which makes the wraparound code in drm_update_vblank_count a no-op. Maybe you can reproduce it if you artificially set a lower max_vblank_count in the driver. -- Earthling Michel Dänzer | http://www.amd.com Libre software enthusiast | Mesa and X developer From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?Q?Michel_D=c3=a4nzer?= Subject: Re: linux-4.4 bisected: kwin5 stuck on kde5 loading screen with radeon Date: Fri, 22 Jan 2016 12:06:00 +0900 Message-ID: <56A19C98.8020208@daenzer.net> References: <20160115122629.GC23290@intel.com> <5699C5E5.90702@gmail.com> <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> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mail.gna.ch (darkcity.gna.ch [195.226.6.51]) by gabe.freedesktop.org (Postfix) with ESMTP id 5442B6E25F for ; Thu, 21 Jan 2016 19:06:08 -0800 (PST) In-Reply-To: <20160121100905.GL19130@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: Mario Kleiner , =?UTF-8?B?VmlsbGUgU3lyasOkbMOk?= 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 ClsgVHJpbW1pbmcgS0RFIGZvbGtzIGZyb20gQ2MgXQoKT24gMjEuMDEuMjAxNiAxOTowOSwgRGFu aWVsIFZldHRlciB3cm90ZToKPiBPbiBUaHUsIEphbiAyMSwgMjAxNiBhdCAwNTozNjo0NlBNICsw OTAwLCBNaWNoZWwgRMOkbnplciB3cm90ZToKPj4gT24gMjEuMDEuMjAxNiAxNjo1OCwgRGFuaWVs IFZldHRlciB3cm90ZToKPj4+IAo+Pj4gQ2FuIHlvdSBwbGVhc2UgcG9pbnQgbWUgYXQgdGhlIHZi bGFuayBvbi9vZmYganVtcCBidWcgcGxlYXNlPwo+Pgo+PiBBRkFJUiBJIG9yaWdpbmFsbHkgcmVw b3J0ZWQgaXQgaW4gcmVzcG9uc2UgdG8KPj4gaHR0cDovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9h cmNoaXZlcy9kcmktZGV2ZWwvMjAxNS1BdWd1c3QvMDg3ODQxLmh0bWwKPj4gLCBidXQgSSBjYW4n dCBmaW5kIHRoYXQgaW4gdGhlIGFyY2hpdmVzLCBzbyBtYXliZSB0aGF0IHdhcyBqdXN0IG9uIElS Qy4KPj4gU2VlCj4+IGh0dHA6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvYXJjaGl2ZXMvZHJpLWRl dmVsLzIwMTYtSmFudWFyeS8wOTkxMjIuaHRtbAo+PiAuIEJhc2ljYWxseSwgSSByYW4gaW50byB0 aGUgYnVnIGZpeGVkIGJ5IHlvdXIgcGF0Y2ggYmVjYXVzZSB0aGUgY291bnRlcgo+PiBqdW1wZWQg Zm9yd2FyZCBvbiBldmVyeSBEUE1TIG9mZiwgc28gaXQgaGl0IHRoZSAzMi1iaXQgYm91bmRhcnkg YWZ0ZXIKPj4ganVzdCBhIGZldyBkYXlzLgo+IAo+IE9rLCBzbyBqdXN0IHVuY292ZXJlZCB0aGUg b3ZlcmZsb3cgYnVnLgoKTm90IHN1cmUgd2hhdCB5b3UgbWVhbiBieSAianVzdCIsIGJ1dCB0byBi ZSBjbGVhcjogVGhlIGRybV92Ymxhbmtfb24vb2ZmCmNvdW50ZXIganVtcGluZyBidWcgKHNpbWls YXIgdG8gdGhlIGJ1ZyB0aGlzIHRocmVhZCBpcyBhYm91dCksIHdoaWNoCmV4cG9zZWQgdGhlIG92 ZXJmbG93IGJ1ZywgaXMgc3RpbGwgYWxpdmUgYW5kIGtpY2tpbmcgaW4gNC41LiBJdCBzZWVtcwp0 byBoYXBwZW4gd2hlbiB0dXJuaW5nIG9mZiB0aGUgQ1JUQzoKCltkcm06ZHJtX3VwZGF0ZV92Ymxh bmtfY291bnRdIHVwZGF0aW5nIHZibGFuayBjb3VudCBvbiBjcnRjIDA6IGN1cnJlbnQ9MjE4MTA0 Njk0LCBkaWZmPTAsIGh3PTkxNiBod19sYXN0PTkxNgpbZHJtOnJhZGVvbl9nZXRfdmJsYW5rX2Nv dW50ZXJfa21zXSBjcnRjIDA6IGRpc3QgZnJvbSB2Ymxhbmsgc3RhcnQgMwpbZHJtOmRybV9jYWxj X3ZibHRpbWVzdGFtcF9mcm9tX3NjYW5vdXRwb3NdIGNydGMgMCA6IHYgMHg3IHAoMjE5OSwtNDUp QCA3MzA0LjMwNzM1NCAtPiA3MzA0LjMwODAwNiBbZSAwIHVzLCAwIHJlcF0KW2RybTpyYWRlb25f Z2V0X3ZibGFua19jb3VudGVyX2ttc10gY3J0YyAwOiBkaXN0IGZyb20gdmJsYW5rIHN0YXJ0IDMK W2RybTpkcm1fdXBkYXRlX3ZibGFua19jb3VudF0gdXBkYXRpbmcgdmJsYW5rIGNvdW50IG9uIGNy dGMgMDogY3VycmVudD0yMTgxMDQ2OTQsIGRpZmY9MTY3NzYzMDEsIGh3PTEgaHdfbGFzdD05MTYK W2RybTpyYWRlb25fZ2V0X3ZibGFua19jb3VudGVyX2ttc10gUXVlcnkgZmFpbGVkISBzdGF0IDMK W2RybTpyYWRlb25fZ2V0X3ZibGFua19jb3VudGVyX2ttc10gUXVlcnkgZmFpbGVkISBzdGF0IDMK W2RybTpkcm1fdXBkYXRlX3ZibGFua19jb3VudF0gdXBkYXRpbmcgdmJsYW5rIGNvdW50IG9uIGNy dGMgMTogY3VycmVudD0wLCBkaWZmPTAsIGh3PTAgaHdfbGFzdD0wCltkcm06cmFkZW9uX2dldF92 YmxhbmtfY291bnRlcl9rbXNdIFF1ZXJ5IGZhaWxlZCEgc3RhdCAzCltkcm06cmFkZW9uX2dldF92 YmxhbmtfY291bnRlcl9rbXNdIFF1ZXJ5IGZhaWxlZCEgc3RhdCAzCltkcm06ZHJtX3VwZGF0ZV92 YmxhbmtfY291bnRdIHVwZGF0aW5nIHZibGFuayBjb3VudCBvbiBjcnRjIDI6IGN1cnJlbnQ9MCwg ZGlmZj0wLCBodz0wIGh3X2xhc3Q9MApbZHJtOnJhZGVvbl9nZXRfdmJsYW5rX2NvdW50ZXJfa21z XSBRdWVyeSBmYWlsZWQhIHN0YXQgMwpbZHJtOnJhZGVvbl9nZXRfdmJsYW5rX2NvdW50ZXJfa21z XSBRdWVyeSBmYWlsZWQhIHN0YXQgMwpbZHJtOmRybV91cGRhdGVfdmJsYW5rX2NvdW50XSB1cGRh dGluZyB2YmxhbmsgY291bnQgb24gY3J0YyAzOiBjdXJyZW50PTAsIGRpZmY9MCwgaHc9MCBod19s YXN0PTAKW2RybTpyYWRlb25fZ2V0X3ZibGFua19jb3VudGVyX2ttc10gUXVlcnkgZmFpbGVkISBz dGF0IDEKW2RybTpkcm1fY2FsY192Ymx0aW1lc3RhbXBfZnJvbV9zY2Fub3V0cG9zXSBjcnRjIDAg OiB2IDB4MSBwKDAsMClAIDczMDQuMzE3MTQwIC0+IDczMDQuMzE3MTQwIFtlIDAgdXMsIDAgcmVw XQpbZHJtOnJhZGVvbl9nZXRfdmJsYW5rX2NvdW50ZXJfa21zXSBRdWVyeSBmYWlsZWQhIHN0YXQg MQpbZHJtOmRybV91cGRhdGVfdmJsYW5rX2NvdW50XSB1cGRhdGluZyB2YmxhbmsgY291bnQgb24g Y3J0YyAwOiBjdXJyZW50PTIzNDg4MDk5NSwgZGlmZj0xNjc3NzIxNSwgaHc9MCBod19sYXN0PTEK Ckkgc3VzcGVjdCB0aGlzIG1heSBub3QgYmUgZXZpZGVudCB3aXRoIGN1cnJlbnQgSW50ZWwgaGFy ZHdhcmUgYmVjYXVzZQpkZXYtPm1heF92YmxhbmtfY291bnQgPSAweGZmZmZmZmZmLCB3aGljaCBt YWtlcyB0aGUgd3JhcGFyb3VuZCBjb2RlIGluCmRybV91cGRhdGVfdmJsYW5rX2NvdW50IGEgbm8t b3AuIE1heWJlIHlvdSBjYW4gcmVwcm9kdWNlIGl0IGlmIHlvdQphcnRpZmljaWFsbHkgc2V0IGEg bG93ZXIgbWF4X3ZibGFua19jb3VudCBpbiB0aGUgZHJpdmVyLgoKCi0tIApFYXJ0aGxpbmcgTWlj aGVsIETDpG56ZXIgICAgICAgICAgICAgICB8ICAgICAgICAgICAgICAgaHR0cDovL3d3dy5hbWQu Y29tCkxpYnJlIHNvZnR3YXJlIGVudGh1c2lhc3QgICAgICAgICAgICAgfCAgICAgICAgICAgICBN ZXNhIGFuZCBYIGRldmVsb3BlcgpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fXwpkcmktZGV2ZWwgbWFpbGluZyBsaXN0CmRyaS1kZXZlbEBsaXN0cy5mcmVlZGVz a3RvcC5vcmcKaHR0cDovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2Ry aS1kZXZlbAo=