From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Zanoni, Paulo R" Subject: Re: Regression of v4.6-rc vs. v4.5 bisected: a98ee79317b4 "drm/i915/fbc: enable FBC by default on HSW and BDW" Date: Thu, 5 May 2016 23:55:56 +0000 Message-ID: <1462492552.14511.10.camel@intel.com> References: <20160426210008.2f79fcdf@kant> <20160429100741.6be95385@kant> <20160430155154.597829ca@kant> <20160505194506.63b9c113@kant> <1462474211.29701.21.camel@intel.com> <20160506005457.1fa4b4e3@kant> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <20160506005457.1fa4b4e3@kant> Content-Language: en-US Content-ID: <60F7BFFBD226FD459ADEB361F9AC32E4@intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: "stefanr@s5r6.in-berlin.de" Cc: "airlied@redhat.com" , "intel-gfx@lists.freedesktop.org" , "linux-kernel@vger.kernel.org" , "dri-devel@lists.freedesktop.org" , "Vetter, Daniel" List-Id: dri-devel@lists.freedesktop.org RW0gU2V4LCAyMDE2LTA1LTA2IMOgcyAwMDo1NCArMDIwMCwgU3RlZmFuIFJpY2h0ZXIgZXNjcmV2 ZXU6DQo+IE9uIE1heSAwNSBaYW5vbmksIFBhdWxvIFIgd3JvdGU6DQo+ID4gDQo+ID4gRW0gUXVp LCAyMDE2LTA1LTA1IMOgcyAxOTo0NSArMDIwMCwgU3RlZmFuIFJpY2h0ZXIgZXNjcmV2ZXU6DQo+ ID4gPiANCj4gPiA+IMKgwqDCoMKgT2gsIGFuZCBpbiBjYXNlIHlvdSAtIHRoZSBwZXJzb24gcmVh ZGluZyB0aGlzIGNvbW1pdCBtZXNzYWdlDQo+ID4gPiAtIGZvdW5kDQo+ID4gPiDCoMKgwqDCoHRo aXMgY29tbWl0IHRocm91Z2ggZ2l0IGJpc2VjdCwgcGxlYXNlIGRvIHRoZSBmb2xsb3dpbmc6DQo+ ID4gPiDCoMKgwqDCoMKgLSBDaGVjayB5b3VyIGRtZXNnIGFuZCBzZWUgaWYgdGhlcmUgYXJlIGVy cm9yIG1lc3NhZ2VzDQo+ID4gPiBtZW50aW9uaW5nDQo+ID4gPiDCoMKgwqDCoMKgwqDCoHVuZGVy cnVucyBhcm91bmQgdGhlIHRpbWUgeW91ciBwcm9ibGVtIHN0YXJ0ZWQgaGFwcGVuaW5nLg0KPiA+ ID4gDQo+ID4gPiBXZWxsLCBJIGFsd2F5cyBoYWQgdGhlIGZvbGxvd2luZ3MgbGluZXMgaW4gZG1l c2c6DQo+ID4gPiBbZHJtOmludGVsX3NldF9jcHVfZmlmb191bmRlcnJ1bl9yZXBvcnRpbmddICpF UlJPUiogdW5jbGVhcmVkDQo+ID4gPiBmaWZvIHVuZGVycnVuIG9uIHBpcGUgQQ0KPiA+ID4gW2Ry bTppbnRlbF9jcHVfZmlmb191bmRlcnJ1bl9pcnFfaGFuZGxlcl0gKkVSUk9SKiBDUFUgcGlwZSBB IEZJRk8NCj4gPiA+IHVuZGVycnVuwqDCoA0KPiA+IE9oLCB3ZWxsLi4uIEkgaGFkIGEgcGF0Y2gg dGhhdCB3b3VsZCBqdXN0IGRpc2FibGUgRkJDIGluIGNhc2Ugd2UNCj4gPiBzYXcgYQ0KPiA+IEZJ Rk8gdW5kZXJydW4sIGJ1dCBpdCB3YXMgcmVqZWN0ZWQuIE1heWJlIHRoaXMgaXMgdGhlIHRpbWUg dG8gdGhpbmsNCj4gPiBhYm91dCBpdCBhZ2Fpbj8gT3RoZXJ3aXNlLCBJIGNhbid0IHRoaW5rIG9m IG11Y2ggYmVzaWRlcyBkaXNhYmxpbmcNCj4gPiBGQkMNCj4gPiBvbiBIU1cgdW50aWwgYWxsIHRo ZSB1bmRlcnJ1bnMgYW5kIHdhdGVybWFya3MgcmVncmVzc2lvbnMgYXJlIGZpeGVkDQo+ID4gZm9y ZXZlci4NCj4gSnVzdCB0byBiZSBjbGVhciB0aG91Z2gsIEkga25vdyB0aGF0IHRoZXNlIG1lc3Nh Z2VzIGFyZSBlbWl0dGVkIHdoZW4NCj4gdGhlDQo+IG1vbml0b3IgaXMgc3dpdGNoZWQgb24sIGFu ZCB3aGVuIHNkZG0gaXMgYmVpbmcgc2h1dCBkb3duIC0tLSBidXQgSSBkbw0KPiBub3QNCj4ga25v dyB3aGV0aGVyIHRoZXJlIGlzIGFueSBzb3J0IG9mIHVuZGVycnVuIHdoZW4gSSBnZXQgdGhlIEZC QyByZWxhdGVkDQo+IGZyZWV6ZSAoc2luY2UgSSBqdXN0IGRvbid0IGdldCBhbnkga2VybmVsIG1l c3NhZ2VzIGF0IHRoYXQgcG9pbnQpLg0KDQpUaGUgZmFjdCB0aGF0IHVuZGVycnVucyBoYXZlIG9j Y3VycmVkIGVhcmxpZXIgaXMgZW5vdWdoIHRvIGtub3cgdGhhdA0Kc29tZXRoaW5nIGlzIHdyb25n IChtb3N0IHByb2JhYmx5LCBiYWQgd2F0ZXJtYXJrcyk6IHdlIHN0b3AgcmVwb3J0aW5nDQp1bmRl cnJ1bnMgb25jZSB3ZSBnZXQgdGhlIGZpcnN0IG9uZS4gSW4gYWRkaXRpb24sIHdlIGFscmVhZHkg a25vdyB0aGF0DQpGQkMgaGFzIHRoZSB0ZW5kZW5jeSB0byBhbXBsaWZ5IGFwcGFyZW50bHktaGFy bWxlc3MgRklGTyB1bmRlcnJ1bnMgaW50bw0KYmxhY2sgc2NyZWVucywgYW5kIEkgd291bGRuJ3Qg YmUgc3VycHJpc2VkIHRvIGxlYXJuIHRoYXQgaXQgY291bGQgYWxzbw0KY2F1c2UgZnVsbCBtYWNo aW5lIGxvY2t1cHMuDQoNCj4gDQo+IElzIHRoZXJlIGEgY2hhbmNlIHRoYXQgYSBzZXJpYWwgY29u c29sZSB3b3VsZCBmYXJlIGJldHRlciB0aGFuDQo+IG5ldGNvbnNvbGU/wqDCoFRoaXMgYm9hcmQg YW5kIGFub3RoZXIgUEMgaW4gaXRzIHZpY2luaXR5IGhhdmUgZ290DQo+IG9uYm9hcmQNCj4gc2Vy aWFsIHBvcnRzIGJ1dCBJIGRvbid0IGhhdmUgY2FibGVzIGF0IHRoZSBtb21lbnQuDQoNCkluIHRo ZSBwYXN0LCBmb3Igc29tZSBzcGVjaWZpYyBjYXNlcyBub3QgcmVsYXRlZCB0byBGQkMsIEkgaGFk IG1vcmUNCmx1Y2sgd2l0aCBzZXJpYWwgY29uc29sZSB0aGFuIHdpdGggbmV0Y29uc29sZS4gQnV0 IGlmIHRoaXMgaXMgcmVhbGx5DQpjYXVzZWQgYnkgRkJDIGFuZCB3YXRlcm1hcmtzLCBJIGRvbid0 IHRoaW5rIHlvdSdsbCBiZSBhYmxlIHRvIGdyYWIgYW55DQpzcGVjaWZpYyBtZXNzYWdlIGF0IHRo ZSB0aW1lIG9mIHRoZSBtYWNoaW5lIGhhbmcuIE9UT0gsIGlmIHNvbWV0aGluZw0KYWN0dWFsbHkg c2hvd3MgdXAsIGl0IGNvdWxkIGhlbHAgaW52YWxpZGF0ZSBvdXIgY3VycmVudCBhc3N1bXB0aW9u IG9mDQp0aGUgcmVsYXRpb25zaGlwIGJldHdlZW4gdGhlIHByb2JsZW0gYW5kIEZCQyBhbmQgdW5k ZXJydW5zLg0KDQo+IA0KPiA+IA0KPiA+ID4gDQo+ID4gPiDCoMKgwqDCoMKgLSBEb3dubG9hZCBp bnRlbC1ncHUtdG9vbHMsIGNvbXBpbGUgaXQsIGFuZCBydW46DQo+ID4gPiDCoMKgwqDCoMKgwqDC oCQgc3VkbyAuL3Rlc3RzL2ttc19mcm9udGJ1ZmZlcl90cmFja2luZyAtLXJ1bi1zdWJ0ZXN0DQo+ ID4gPiAnKmZiYy0qJyAyPiYxIHwgdGVlIGZiYy50eHTCoMKgDQo+ID4gPiDCoMKgwqDCoMKgwqDC oFRoZW4gc2VuZCB1cyB0aGUgZmJjLnR4dCBmaWxlLCBlc3BlY2lhbGx5IGlmIHlvdSBnZXQgYQ0K PiA+ID4gZmFpbHVyZS4NCj4gPiA+IMKgwqDCoMKgwqDCoMKgVGhpcyB3aWxsIHJlYWxseSBtYXhp bWl6ZSB5b3VyIGNoYW5jZXMgb2YgZ2V0dGluZyB0aGUgYnVnDQo+ID4gPiBmaXhlZA0KPiA+ID4g wqDCoMKgwqDCoMKgwqBxdWlja2x5Lg0KPiA+ID4gDQo+ID4gPiBEbyB5b3UgbmVlZCB0aGlzIHdo aWxlIEZCQyBpcyBlbmFibGVkLCBvciBjYW4gSSBydW4gaXQgd2hpbGUgRkJDDQo+ID4gPiBpcw0K PiA+ID4gZGlzYWJsZWQ/wqDCoA0KPiA+IEZCQyBlbmFibGVkLiBDb25zaWRlcmluZyB5b3VyIGRl c2NyaXB0aW9uLCBteSBob3BlIGlzIHRoYXQgbWF5YmUNCj4gPiBzb21lDQo+ID4gc3BlY2lmaWMg c3VidGVzdCB3aWxsIGJlIGFibGUgdG8gaGFuZyB5b3VyIG1hY2hpbmUsIHNvIHRlc3RpbmcgdGhp cw0KPiA+IGFnYWluIHdpbGwgcmVxdWlyZSBvbmx5IHJ1bm5pbmcgdGhlIHNwZWNpZmljIHN1YnRl c3QgaW5zdGVhZCBvZg0KPiA+IHdhaXRpbmcNCj4gPiAxOCBob3Vycy4NCj4gVGhlIGttc19mcm9u dGJ1ZmZlcl90cmFja2luZyBydW5zIGZyb20gd2hpY2ggSSBwb3N0ZWQgb3V0cHV0IHR3bw0KPiBo b3Vycw0KPiBhZ28gZGlkIG5vdCB0cmlnZ2VyIGEgbG9ja3VwLg0KPiANCj4gKEkgcmFuIHRoZW0g d2hpbGUgWDExIHdhcyBzaHV0IGRvd24gYmVjYXVzZSBvdGhlcndpc2UNCj4ga21zX2Zyb250YnVm ZmVyX3RyYWNraW5nIHdvdWxkIHNraXAgYWxsIHRlc3RzIHdpdGggIkNhbid0IGJlY29tZSBEUk0N Cj4gbWFzdGVyLCBwbGVhc2UgY2hlY2sgaWYgbm8gb3RoZXIgRFJNIGNsaWVudCBpcyBydW5uaW5n LiIpDQoNClllcywgdGhpcyBpcyB0aGUgY29ycmVjdCB3YXkuDQoNCj4gDQo+ID4gDQo+ID4gPiAN Cj4gPiA+IFBTOg0KPiA+ID4gSSBhbSBtZW50aW9uaW5nIHRoZSBmb2xsb3dpbmcganVzdCBpbiBj YXNlIHRoYXQgaXQgaGFzIGFueQ0KPiA+ID4gcmVsYXRpb25zaGlwDQo+ID4gPiB3aXRoIHRoZSBG QkMgcmVsYXRlZCBrZXJuZWwgZnJlZXplcy7CoMKgTWF5YmUgaXQgZG9lc24ndC4uLsKgwqBUaGVy ZQ0KPiA+ID4gaXMNCj4gPiA+IGFub3RoZXIgcmVjZW50IHJlZ3Jlc3Npb24gb24gdGhpcyBQQywg YnV0IEkgaGF2ZSBub3QgeWV0IGZpZ3VyZWQNCj4gPiA+IG91dA0KPiA+ID4gd2hldGhlciBpdCB3 YXMgaW50cm9kdWNlZCBieSBhbnkgcGFydGljdWxhciBrZXJuZWwgdmVyc2lvbi7CoMKgVGhlDQo+ ID4gPiByZWdyZXNzaW9uIGlzOsKgwqBXaGVuIHN3aXRjaGluZyBmcm9tIFgxMSB0byB0ZXh0IGNv bnNvbGUgYnkNCj4gPiA+IFtDdHJsXVtBbHRdW0Z4XQ0KPiA+ID4gb3IgYnkgc2h1dHRpbmcgZG93 biBzZGRtLCBJIG9mdGVuIG9ubHkgZ2V0IGEgYmxhbmsgc2NyZWVuLsKgwqBJDQo+ID4gPiBzdXNw ZWN0DQo+ID4gPiB0aGF0IHRoaXMgcmVncmVzc2lvbiB3YXMgaW50cm9kdWNlZCB3aGVuIEkgcmVw bGFjZWQga2RtIGJ5IHNkZG0sDQo+ID4gPiBidXQNCj4gPiA+IEkgYW0gbm90IHN1cmUgYWJvdXQg dGhhdC7CoMKgDQo+ID4gTWF5YmUgdGhlcmUgaXMgc29tZSByZWxhdGlvbnNoaXAsIHNpbmNlIHRo aXMgb3BlcmF0aW9uIGludm9sdmVzIGENCj4gPiBtb2RlDQo+ID4gY2hhbmdlLiBZb3UgY2FuIGFs c28gdHJ5IGNoZWNraW5nIGRtZXNnIHRvIHNlZSBpZiB0aGVyZSBhcmUNCj4gPiB1bmRlcnJ1bnMN Cj4gPiByaWdodCB3aGVuIHlvdSBkbyB0aGUgY2hhbmdlLg0KPiBZZXMsIHRoaXMgaXMgYWNjb21w YW5pZWQgYnkNCj4gW2RybTppbnRlbF9zZXRfY3B1X2ZpZm9fdW5kZXJydW5fcmVwb3J0aW5nXSAq RVJST1IqIHVuY2xlYXJlZCBmaWZvDQo+IHVuZGVycnVuIG9uIHBpcGUgQQ0KPiBbZHJtOmludGVs X2NwdV9maWZvX3VuZGVycnVuX2lycV9oYW5kbGVyXSAqRVJST1IqIENQVSBwaXBlIEEgRklGTw0K PiB1bmRlcnJ1bsKgwqAKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX18KSW50ZWwtZ2Z4IG1haWxpbmcgbGlzdApJbnRlbC1nZnhAbGlzdHMuZnJlZWRlc2t0b3Au b3JnCmh0dHBzOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vaW50ZWwt Z2Z4Cg== From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757207AbcEEXz7 (ORCPT ); Thu, 5 May 2016 19:55:59 -0400 Received: from mga11.intel.com ([192.55.52.93]:56125 "EHLO mga11.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755704AbcEEXz6 (ORCPT ); Thu, 5 May 2016 19:55:58 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.24,584,1455004800"; d="scan'208";a="969730814" From: "Zanoni, Paulo R" To: "stefanr@s5r6.in-berlin.de" CC: "airlied@redhat.com" , "intel-gfx@lists.freedesktop.org" , "linux-kernel@vger.kernel.org" , "dri-devel@lists.freedesktop.org" , "Vetter, Daniel" Subject: Re: Regression of v4.6-rc vs. v4.5 bisected: a98ee79317b4 "drm/i915/fbc: enable FBC by default on HSW and BDW" Thread-Topic: Regression of v4.6-rc vs. v4.5 bisected: a98ee79317b4 "drm/i915/fbc: enable FBC by default on HSW and BDW" Thread-Index: AQHRpvX9lrDl4AzzVUu8HLsBjaDOhp+rJVSAgABEY4CAABEFAA== Date: Thu, 5 May 2016 23:55:56 +0000 Message-ID: <1462492552.14511.10.camel@intel.com> References: <20160426210008.2f79fcdf@kant> <20160429100741.6be95385@kant> <20160430155154.597829ca@kant> <20160505194506.63b9c113@kant> <1462474211.29701.21.camel@intel.com> <20160506005457.1fa4b4e3@kant> In-Reply-To: <20160506005457.1fa4b4e3@kant> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.252.202.11] Content-Type: text/plain; charset="utf-8" Content-ID: <60F7BFFBD226FD459ADEB361F9AC32E4@intel.com> MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from base64 to 8bit by mail.home.local id u45Nu9is013993 Em Sex, 2016-05-06 às 00:54 +0200, Stefan Richter escreveu: > On May 05 Zanoni, Paulo R wrote: > > > > Em Qui, 2016-05-05 às 19:45 +0200, Stefan Richter escreveu: > > > > > >     Oh, and in case you - the person reading this commit message > > > - found > > >     this commit through git bisect, please do the following: > > >      - Check your dmesg and see if there are error messages > > > mentioning > > >        underruns around the time your problem started happening. > > > > > > Well, I always had the followings lines in dmesg: > > > [drm:intel_set_cpu_fifo_underrun_reporting] *ERROR* uncleared > > > fifo underrun on pipe A > > > [drm:intel_cpu_fifo_underrun_irq_handler] *ERROR* CPU pipe A FIFO > > > underrun   > > Oh, well... I had a patch that would just disable FBC in case we > > saw a > > FIFO underrun, but it was rejected. Maybe this is the time to think > > about it again? Otherwise, I can't think of much besides disabling > > FBC > > on HSW until all the underruns and watermarks regressions are fixed > > forever. > Just to be clear though, I know that these messages are emitted when > the > monitor is switched on, and when sddm is being shut down --- but I do > not > know whether there is any sort of underrun when I get the FBC related > freeze (since I just don't get any kernel messages at that point). The fact that underruns have occurred earlier is enough to know that something is wrong (most probably, bad watermarks): we stop reporting underruns once we get the first one. In addition, we already know that FBC has the tendency to amplify apparently-harmless FIFO underruns into black screens, and I wouldn't be surprised to learn that it could also cause full machine lockups. > > Is there a chance that a serial console would fare better than > netconsole?  This board and another PC in its vicinity have got > onboard > serial ports but I don't have cables at the moment. In the past, for some specific cases not related to FBC, I had more luck with serial console than with netconsole. But if this is really caused by FBC and watermarks, I don't think you'll be able to grab any specific message at the time of the machine hang. OTOH, if something actually shows up, it could help invalidate our current assumption of the relationship between the problem and FBC and underruns. > > > > > > > > >      - Download intel-gpu-tools, compile it, and run: > > >        $ sudo ./tests/kms_frontbuffer_tracking --run-subtest > > > '*fbc-*' 2>&1 | tee fbc.txt   > > >        Then send us the fbc.txt file, especially if you get a > > > failure. > > >        This will really maximize your chances of getting the bug > > > fixed > > >        quickly. > > > > > > Do you need this while FBC is enabled, or can I run it while FBC > > > is > > > disabled?   > > FBC enabled. Considering your description, my hope is that maybe > > some > > specific subtest will be able to hang your machine, so testing this > > again will require only running the specific subtest instead of > > waiting > > 18 hours. > The kms_frontbuffer_tracking runs from which I posted output two > hours > ago did not trigger a lockup. > > (I ran them while X11 was shut down because otherwise > kms_frontbuffer_tracking would skip all tests with "Can't become DRM > master, please check if no other DRM client is running.") Yes, this is the correct way. > > > > > > > > > PS: > > > I am mentioning the following just in case that it has any > > > relationship > > > with the FBC related kernel freezes.  Maybe it doesn't...  There > > > is > > > another recent regression on this PC, but I have not yet figured > > > out > > > whether it was introduced by any particular kernel version.  The > > > regression is:  When switching from X11 to text console by > > > [Ctrl][Alt][Fx] > > > or by shutting down sddm, I often only get a blank screen.  I > > > suspect > > > that this regression was introduced when I replaced kdm by sddm, > > > but > > > I am not sure about that.   > > Maybe there is some relationship, since this operation involves a > > mode > > change. You can also try checking dmesg to see if there are > > underruns > > right when you do the change. > Yes, this is accompanied by > [drm:intel_set_cpu_fifo_underrun_reporting] *ERROR* uncleared fifo > underrun on pipe A > [drm:intel_cpu_fifo_underrun_irq_handler] *ERROR* CPU pipe A FIFO > underrun