Date: Fri, 28 Oct 2016 09:51:15 -0700
From: Matt Roper <matthew.d.roper@intel.com>
To: Greg KH <gregkh@linuxfoundation.org>
Cc: Gaston Gonzalez <gascoar@gmail.com>, linux-kernel@vger.kernel.org,
        stable@vger.kernel.org, intel-gfx@lists.freedesktop.org
Subject: Re: xrandr fails after resume from hibernation in kernels 4.7.4
 and 4.8.0
Message-ID: <20161028165115.GL5924@intel.com>
References: <20161004234303.GA2670@debianG>
 <20161005052323.GA8915@kroah.com>
 <20161007133817.GA2166@debianG>
 <20161028155340.GC19549@kroah.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20161028155340.GC19549@kroah.com>
User-Agent: Mutt/1.5.20 (2009-06-14)
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 4767
Lines: 105

On Fri, Oct 28, 2016 at 11:53:40AM -0400, Greg KH wrote:
> On Fri, Oct 07, 2016 at 10:38:17AM -0300, Gaston Gonzalez wrote:
> > On Wed, Oct 05, 2016 at 07:23:23AM +0200, Greg KH wrote:
> > > On Tue, Oct 04, 2016 at 08:43:03PM -0300, Gaston Gonzalez wrote:
> > > > Hi,
> > > > 
> > > > After hibernation I get the following error when I tried to connect to monitor
> > > > through hdmi:
> > > > 
> > > > $ xrandr --output LVDS1 --off  --output  HDMI1 --auto
> > > > xrandr: Configure crtc 1 failed
> > > > 
> > > > This does not happen in kernel in kernel v4.6.7 but do happen in kernels v4.7.4
> > > > and v4.8.0
> > > 
> > > Ah, can you use 'git bisect' to track down the offending patch?
> > > 
> > > Also, if you let the graphics driver authors know, they are probably the
> > > best ones to help out with this, not the "generic" stable mailing list.
> > > 
> > > thanks,
> > > 
> > > greg k-h
> > 
> > Hello,
> > 
> > The culprit commit seems to be:
> > 
> > commit: ed4a6a7
> > drm/i915: Add two-stage ILK-style watermark programming
> > 
> > Attached the bisection log.
> > 
> > Let me know if additional test or bisect is needed.
> > 
> > Best regards,
> > 
> > Gaston
> 
> > git bisect start '--' 'drivers/gpu/drm/'
> > # good: [096998b11906dd79df5af4d688c9974342dd09f2] Linux 4.6.7
> > git bisect good 096998b11906dd79df5af4d688c9974342dd09f2
> > # bad: [c8d2bc9bc39ebea8437fd974fdbc21847bb897a3] Linux 4.8
> > git bisect bad c8d2bc9bc39ebea8437fd974fdbc21847bb897a3
> > # good: [2dcd0af568b0cf583645c8a317dd12e344b1c72a] Linux 4.6
> > git bisect good 2dcd0af568b0cf583645c8a317dd12e344b1c72a
> > # bad: [3c85f20a289d044f303f473ee6ab7502303fc3b0] Merge tag 'omapdrm-4.8' of git://git.kernel.org/pub/scm/linux/kernel/git/tomba/linux into drm-next
> > git bisect bad 3c85f20a289d044f303f473ee6ab7502303fc3b0
> > # bad: [a64424d722504926f3375bc4887976e3bfe3a01d] Merge branch 'drm-next-4.7' of git://people.freedesktop.org/~agd5f/linux into drm-next
> > git bisect bad a64424d722504926f3375bc4887976e3bfe3a01d
> > # bad: [b5bf0f1ea3658254bd72ef64abc97786e8a32255] drm/exynos: clean up register definions for fimd and decon
> > git bisect bad b5bf0f1ea3658254bd72ef64abc97786e8a32255
> > # bad: [27878ede4fec7b929c3010710ba4d55c617c621d] drm/i915: Throw out BUGs from DPLL/PCH functions
> > git bisect bad 27878ede4fec7b929c3010710ba4d55c617c621d
> > # bad: [72341af4285ae1337c0dfdfa3e68318b52b8757c] drm/i915: hide away VBT private data in a separate header
> > git bisect bad 72341af4285ae1337c0dfdfa3e68318b52b8757c
> > # bad: [71f0a626143368b8aead361ffaff7e36d043fd8e] drm/i915: Only use sanitized values for ILK watermarks
> > git bisect bad 71f0a626143368b8aead361ffaff7e36d043fd8e
> > # bad: [66e2c4c39cc37beaccc24c9d14c75d627fce9cf4] drm/i915/gen9: Disable DC states if power well support is disabled
> > git bisect bad 66e2c4c39cc37beaccc24c9d14c75d627fce9cf4
> > # bad: [4f2d9934bd6ac73950832c96b385822846670668] drm/i915: Pass drm_frambuffer to intel_compute_page_offset()
> > git bisect bad 4f2d9934bd6ac73950832c96b385822846670668
> > # bad: [1d5bf5d9d9ef0c1e639d36178a224d83888c5a29] drm/i915: Add missing NULL check before calling initial_watermarks
> > git bisect bad 1d5bf5d9d9ef0c1e639d36178a224d83888c5a29
> > # bad: [d9f8e52b22454a30aaaf26b7ef029598b30abf8e] drm/i915: remove dead code
> > git bisect bad d9f8e52b22454a30aaaf26b7ef029598b30abf8e
> > # bad: [c3454d575da162cd310d9b83696baefb29d10a70] drm/i915: remove left over dead code
> > git bisect bad c3454d575da162cd310d9b83696baefb29d10a70
> > # bad: [ed4a6a7ca853253f9b86f3005d76345482a71283] drm/i915: Add two-stage ILK-style watermark programming (v11)
> > git bisect bad ed4a6a7ca853253f9b86f3005d76345482a71283
> > # first bad commit: [ed4a6a7ca853253f9b86f3005d76345482a71283] drm/i915: Add two-stage ILK-style watermark programming (v11)
> 
> 
> Nothing from the graphics developers?  Come on, here is someone who
> found a specific patch that causes a problem, and no one responds?
> 
> Why isn't this fixed by reverting the above mentioned patch in Linus's
> tree already?  Or, is there something else that needs to be done?
> 
> thanks,
> 
> greg k-h

I was away for a couple months so I think I missed this thread when
going through my backlog of emails, sorry.  Is there a bugzilla filed
for this already with logs and more details?  If not, please add one at
[1] to help make sure this doesn't slip through the cracks.  If you boot
with "drm.debug=0xf" on the kernel command line, the output of your
dmesg log would also help us identify what's happening here.

Thanks.


Matt

[1] https://bugs.freedesktop.org/enter_bug.cgi?product=DRI&component=DRM/Intel


-- 
Matt Roper
Graphics Software Engineer
IoTG Platform Enabling & Development
Intel Corporation
(916) 356-2795