From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932128AbeDZPZS (ORCPT ); Thu, 26 Apr 2018 11:25:18 -0400 Received: from mga14.intel.com ([192.55.52.115]:25954 "EHLO mga14.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754556AbeDZPZR (ORCPT ); Thu, 26 Apr 2018 11:25:17 -0400 X-Amp-Result: UNKNOWN X-Amp-Original-Verdict: FILE UNKNOWN X-Amp-File-Uploaded: False X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.49,330,1520924400"; d="scan'208";a="35476489" Date: Thu, 26 Apr 2018 18:25:13 +0300 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= To: Dave Jones , Linux Kernel , Jani Nikula , Joonas Lahtinen , Rodrigo Vivi , intel-gfx@lists.freedesktop.org Subject: Re: [Intel-gfx] 4.17-rc2: Could not determine valid watermarks for inherited state Message-ID: <20180426152513.GR23723@intel.com> References: <20180423152713.2rrwwye3wfstbv5v@codemonkey.org.uk> <20180426131045.GH23723@intel.com> <20180426142719.fzivt2e6kialcbp4@codemonkey.org.uk> <20180426145614.GP23723@intel.com> <20180426151641.GQ23723@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20180426151641.GQ23723@intel.com> User-Agent: Mutt/1.9.4 (2018-02-28) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Apr 26, 2018 at 06:16:41PM +0300, Ville Syrjälä wrote: > On Thu, Apr 26, 2018 at 05:56:14PM +0300, Ville Syrjälä wrote: > > On Thu, Apr 26, 2018 at 10:27:19AM -0400, Dave Jones wrote: > > > [ 1.176131] [drm:i9xx_get_initial_plane_config] pipe A/primary A with fb: size=800x600@32, offset=0, pitch 3200, size 0x1d4c00 > > > [ 1.176161] [drm:i915_gem_object_create_stolen_for_preallocated] creating preallocated stolen object: stolen_offset=0x0000000000000000, gtt_offset=0x0000000000000000, size=0x00000000001d5000 > > > [ 1.176312] [drm:intel_alloc_initial_plane_obj.isra.127] initial plane fb obj (ptrval) > > > [ 1.176351] [drm:intel_modeset_init] pipe A active planes 0x1 > > > [ 1.176456] [drm:drm_atomic_helper_check_plane_state] Plane must cover entire CRTC > > > [ 1.176481] [drm:drm_rect_debug_print] dst: 800x600+0+0 > > > [ 1.176494] [drm:drm_rect_debug_print] clip: 1366x768+0+0 > > > > OK, so that's the problem right there. The fb we took over from the > > BIOS was 800x600, but now we're trying to set up a 1366x768 mode. > > > > We seem to be missing checks to make sure the initial fb is actually > > big enough for the mode we're currently using :( > > Actually we do read out the pipe src size as 800x600 initially, which > make sense. And we even stuff that into the mode.h/vdisplay, so up to > that point everything is pretty much correct. It goes wrong is when > intel_modeset_readout_hw_state() calls intel_mode_from_pipe_config() > as that will override the h/vdisplay with the actual crtc timings > instead of the pipe src size. > > So I suppose we should be able to just add the sanity checks for the > fb vs. h/vdisplay, and at least we should get past this error. A > slightly bigger mystery is what will happen later when our pipe src > size doesn't actually agree with the h/vdisplay. The first modeset > will correct it, but we might want some kind of extra sanitize step > for fastboot type of stuff. Hmm. Or maybe we should just stick to the pipe src size. I'm curious whether this fixes the problem? diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c index 0f8c7389e87d..30824beedef7 100644 --- a/drivers/gpu/drm/i915/intel_display.c +++ b/drivers/gpu/drm/i915/intel_display.c @@ -15284,6 +15284,8 @@ static void intel_modeset_readout_hw_state(struct drm_device *dev) memset(&crtc->base.mode, 0, sizeof(crtc->base.mode)); if (crtc_state->base.active) { intel_mode_from_pipe_config(&crtc->base.mode, crtc_state); + crtc->base.mode.hdisplay = crtc_state->pipe_src_w; + crtc->base.mode.vdisplay = crtc_state->pipe_src_h; intel_mode_from_pipe_config(&crtc_state->base.adjusted_mode, crtc_state); WARN_ON(drm_atomic_set_mode_for_crtc(crtc->base.state, &crtc->base.mode)); -- Ville Syrjälä Intel From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: 4.17-rc2: Could not determine valid watermarks for inherited state Date: Thu, 26 Apr 2018 18:25:13 +0300 Message-ID: <20180426152513.GR23723@intel.com> References: <20180423152713.2rrwwye3wfstbv5v@codemonkey.org.uk> <20180426131045.GH23723@intel.com> <20180426142719.fzivt2e6kialcbp4@codemonkey.org.uk> <20180426145614.GP23723@intel.com> <20180426151641.GQ23723@intel.com> 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 ESMTPS id C00FA6E725 for ; Thu, 26 Apr 2018 15:25:17 +0000 (UTC) Content-Disposition: inline In-Reply-To: <20180426151641.GQ23723@intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Dave Jones , Linux Kernel , Jani Nikula , Joonas Lahtinen , Rodrigo Vivi , intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org T24gVGh1LCBBcHIgMjYsIDIwMTggYXQgMDY6MTY6NDFQTSArMDMwMCwgVmlsbGUgU3lyasOkbMOk IHdyb3RlOgo+IE9uIFRodSwgQXByIDI2LCAyMDE4IGF0IDA1OjU2OjE0UE0gKzAzMDAsIFZpbGxl IFN5cmrDpGzDpCB3cm90ZToKPiA+IE9uIFRodSwgQXByIDI2LCAyMDE4IGF0IDEwOjI3OjE5QU0g LTA0MDAsIERhdmUgSm9uZXMgd3JvdGU6Cj4gPiA+IFsgICAgMS4xNzYxMzFdIFtkcm06aTl4eF9n ZXRfaW5pdGlhbF9wbGFuZV9jb25maWddIHBpcGUgQS9wcmltYXJ5IEEgd2l0aCBmYjogc2l6ZT04 MDB4NjAwQDMyLCBvZmZzZXQ9MCwgcGl0Y2ggMzIwMCwgc2l6ZSAweDFkNGMwMAo+ID4gPiBbICAg IDEuMTc2MTYxXSBbZHJtOmk5MTVfZ2VtX29iamVjdF9jcmVhdGVfc3RvbGVuX2Zvcl9wcmVhbGxv Y2F0ZWRdIGNyZWF0aW5nIHByZWFsbG9jYXRlZCBzdG9sZW4gb2JqZWN0OiBzdG9sZW5fb2Zmc2V0 PTB4MDAwMDAwMDAwMDAwMDAwMCwgZ3R0X29mZnNldD0weDAwMDAwMDAwMDAwMDAwMDAsIHNpemU9 MHgwMDAwMDAwMDAwMWQ1MDAwCj4gPiA+IFsgICAgMS4xNzYzMTJdIFtkcm06aW50ZWxfYWxsb2Nf aW5pdGlhbF9wbGFuZV9vYmouaXNyYS4xMjddIGluaXRpYWwgcGxhbmUgZmIgb2JqICAgICAgICAg KHB0cnZhbCkKPiA+ID4gWyAgICAxLjE3NjM1MV0gW2RybTppbnRlbF9tb2Rlc2V0X2luaXRdIHBp cGUgQSBhY3RpdmUgcGxhbmVzIDB4MQo+ID4gPiBbICAgIDEuMTc2NDU2XSBbZHJtOmRybV9hdG9t aWNfaGVscGVyX2NoZWNrX3BsYW5lX3N0YXRlXSBQbGFuZSBtdXN0IGNvdmVyIGVudGlyZSBDUlRD Cj4gPiA+IFsgICAgMS4xNzY0ODFdIFtkcm06ZHJtX3JlY3RfZGVidWdfcHJpbnRdIGRzdDogODAw eDYwMCswKzAKPiA+ID4gWyAgICAxLjE3NjQ5NF0gW2RybTpkcm1fcmVjdF9kZWJ1Z19wcmludF0g Y2xpcDogMTM2Nng3NjgrMCswCj4gPiAKPiA+IE9LLCBzbyB0aGF0J3MgdGhlIHByb2JsZW0gcmln aHQgdGhlcmUuIFRoZSBmYiB3ZSB0b29rIG92ZXIgZnJvbSB0aGUKPiA+IEJJT1Mgd2FzIDgwMHg2 MDAsIGJ1dCBub3cgd2UncmUgdHJ5aW5nIHRvIHNldCB1cCBhIDEzNjZ4NzY4IG1vZGUuCj4gPiAK PiA+IFdlIHNlZW0gdG8gYmUgbWlzc2luZyBjaGVja3MgdG8gbWFrZSBzdXJlIHRoZSBpbml0aWFs IGZiIGlzIGFjdHVhbGx5Cj4gPiBiaWcgZW5vdWdoIGZvciB0aGUgbW9kZSB3ZSdyZSBjdXJyZW50 bHkgdXNpbmcgOigKPiAKPiBBY3R1YWxseSB3ZSBkbyByZWFkIG91dCB0aGUgcGlwZSBzcmMgc2l6 ZSBhcyA4MDB4NjAwIGluaXRpYWxseSwgd2hpY2gKPiBtYWtlIHNlbnNlLiBBbmQgd2UgZXZlbiBz dHVmZiB0aGF0IGludG8gdGhlIG1vZGUuaC92ZGlzcGxheSwgc28gdXAgdG8KPiB0aGF0IHBvaW50 IGV2ZXJ5dGhpbmcgaXMgcHJldHR5IG11Y2ggY29ycmVjdC4gSXQgZ29lcyB3cm9uZyBpcyB3aGVu Cj4gaW50ZWxfbW9kZXNldF9yZWFkb3V0X2h3X3N0YXRlKCkgY2FsbHMgaW50ZWxfbW9kZV9mcm9t X3BpcGVfY29uZmlnKCkKPiBhcyB0aGF0IHdpbGwgb3ZlcnJpZGUgdGhlIGgvdmRpc3BsYXkgd2l0 aCB0aGUgYWN0dWFsIGNydGMgdGltaW5ncwo+IGluc3RlYWQgb2YgdGhlIHBpcGUgc3JjIHNpemUu Cj4gCj4gU28gSSBzdXBwb3NlIHdlIHNob3VsZCBiZSBhYmxlIHRvIGp1c3QgYWRkIHRoZSBzYW5p dHkgY2hlY2tzIGZvciB0aGUKPiBmYiB2cy4gaC92ZGlzcGxheSwgYW5kIGF0IGxlYXN0IHdlIHNo b3VsZCBnZXQgcGFzdCB0aGlzIGVycm9yLiBBCj4gc2xpZ2h0bHkgYmlnZ2VyIG15c3RlcnkgaXMg d2hhdCB3aWxsIGhhcHBlbiBsYXRlciB3aGVuIG91ciBwaXBlIHNyYwo+IHNpemUgZG9lc24ndCBh Y3R1YWxseSBhZ3JlZSB3aXRoIHRoZSBoL3ZkaXNwbGF5LiBUaGUgZmlyc3QgbW9kZXNldAo+IHdp bGwgY29ycmVjdCBpdCwgYnV0IHdlIG1pZ2h0IHdhbnQgc29tZSBraW5kIG9mIGV4dHJhIHNhbml0 aXplIHN0ZXAKPiBmb3IgZmFzdGJvb3QgdHlwZSBvZiBzdHVmZi4KCkhtbS4gT3IgbWF5YmUgd2Ug c2hvdWxkIGp1c3Qgc3RpY2sgdG8gdGhlIHBpcGUgc3JjIHNpemUuCgpJJ20gY3VyaW91cyB3aGV0 aGVyIHRoaXMgZml4ZXMgdGhlIHByb2JsZW0/CgpkaWZmIC0tZ2l0IGEvZHJpdmVycy9ncHUvZHJt L2k5MTUvaW50ZWxfZGlzcGxheS5jIGIvZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfZGlzcGxh eS5jCmluZGV4IDBmOGM3Mzg5ZTg3ZC4uMzA4MjRiZWVkZWY3IDEwMDY0NAotLS0gYS9kcml2ZXJz L2dwdS9kcm0vaTkxNS9pbnRlbF9kaXNwbGF5LmMKKysrIGIvZHJpdmVycy9ncHUvZHJtL2k5MTUv aW50ZWxfZGlzcGxheS5jCkBAIC0xNTI4NCw2ICsxNTI4NCw4IEBAIHN0YXRpYyB2b2lkIGludGVs X21vZGVzZXRfcmVhZG91dF9od19zdGF0ZShzdHJ1Y3QgZHJtX2RldmljZSAqZGV2KQogCQltZW1z ZXQoJmNydGMtPmJhc2UubW9kZSwgMCwgc2l6ZW9mKGNydGMtPmJhc2UubW9kZSkpOwogCQlpZiAo Y3J0Y19zdGF0ZS0+YmFzZS5hY3RpdmUpIHsKIAkJCWludGVsX21vZGVfZnJvbV9waXBlX2NvbmZp ZygmY3J0Yy0+YmFzZS5tb2RlLCBjcnRjX3N0YXRlKTsKKwkJCWNydGMtPmJhc2UubW9kZS5oZGlz cGxheSA9IGNydGNfc3RhdGUtPnBpcGVfc3JjX3c7CisJCQljcnRjLT5iYXNlLm1vZGUudmRpc3Bs YXkgPSBjcnRjX3N0YXRlLT5waXBlX3NyY19oOwogCQkJaW50ZWxfbW9kZV9mcm9tX3BpcGVfY29u ZmlnKCZjcnRjX3N0YXRlLT5iYXNlLmFkanVzdGVkX21vZGUsIGNydGNfc3RhdGUpOwogCQkJV0FS Tl9PTihkcm1fYXRvbWljX3NldF9tb2RlX2Zvcl9jcnRjKGNydGMtPmJhc2Uuc3RhdGUsICZjcnRj LT5iYXNlLm1vZGUpKTsKCi0tIApWaWxsZSBTeXJqw6Rsw6QKSW50ZWwKX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KSW50ZWwtZ2Z4IG1haWxpbmcgbGlzdApJ bnRlbC1nZnhAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlzdHMuZnJlZWRlc2t0b3Au b3JnL21haWxtYW4vbGlzdGluZm8vaW50ZWwtZ2Z4Cg==