From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chris Wilson Subject: Re: has the i915 "black screen" boot issue returned? Date: Fri, 28 Jan 2011 08:59:17 +0000 Message-ID: <849307$bbem1h@azsmga001.ch.intel.com> References: <1bdc18$jdgfni@fmsmga002.fm.intel.com> <0d30dc$ksovh8@orsmga001.jf.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mga14.intel.com (mga14.intel.com [143.182.124.37]) by gabe.freedesktop.org (Postfix) with ESMTP id 1ADB99E779 for ; Fri, 28 Jan 2011 00:59:21 -0800 (PST) In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org Errors-To: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org To: "Robert P. J. Day" Cc: dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org On Thu, 27 Jan 2011 17:33:04 -0500 (EST), "Robert P. J. Day" wrote: > > one more observation which should nail things down: > > On Thu, 27 Jan 2011, Robert P. J. Day wrote: > > > ok, i'm getting different results from this morning, not sure why, > > but i'm fairly confident i've isolated it to this extent. here's the > > salient part of the git log i'm looking at: > > > > ===== git log ===== > > > > commit abb72c828878a2c69b2cfb33ac30007c8ecd735e > > Merge: 58bbf01 8e934db > > Author: Dave Airlie > > Date: Tue Jan 25 08:41:58 2011 +1000 > > > > Merge branch 'drm-intel-fixes-2' of ssh://master.kernel.org/pub/scm/linux/kernel/g > > > > * 'drm-intel-fixes-2' of ssh://master.kernel.org/pub/scm/linux/kernel/git/ickle/dr > > drm/i915: Prevent uninitialised reads during error state capture > > drm/i915: Use consistent mappings for OpRegion between ACPI and i915 > > drm/i915: Handle the no-interrupts case for UMS by polling > > drm/i915: Disable high-precision vblank timestamping for UMS > > drm/i915: Increase the amount of defense before computing vblank timestamps > > drm/i915,agp/intel: Do not clear stolen entries > > Remove MAYBE_BUILD_BUG_ON > > BUILD_BUG_ON: make it handle more cases > > module: fix missing semicolons in MODULE macro usage > > param: add null statement to compiled-in module params > > module: fix linker error for MODULE_VERSION when !MODULE and CONFIG_SYSFS=n > > module: show version information for built-in modules in sysfs > > selinux: return -ENOMEM when memory allocation fails > > tpm: fix panic caused by "tpm: Autodetect itpm devices" > > TPM: Long default timeout fix > > trusted keys: Fix a memory leak in trusted_update(). > > keys: add trusted and encrypted maintainers > > encrypted-keys: rename encrypted_defined files to encrypted > > trusted-keys: rename trusted_defined files to trusted > > drm/i915: Recognise non-VGA display devices > > ... > > > > commit 58bbf018a70c562437eeae121a5d021ba7fe56a5 > > Author: Alex Deucher > > Date: Mon Jan 24 17:14:26 2011 -0500 > > > > drm/radeon/kms: add new radeon_info ioctl query for clock crystal freq > > > > Needed for timer queries in the 3D driver. > > > > Signed-off-by: Alex Deucher > > Signed-off-by: Dave Airlie > > > > ===== git log ===== > > > > now: > > > > * commit 58bbf018 appears to be fine, boots reliably > > * commit 8e934dbf causes black screen issue > > > > as you can see, rather than test the *merge* above, i decided to back > > up one level and test the commit that was part of that merge, and it > > failed. is this helpful? here's the log entry: > > > > commit 8e934dbf264418afe4d1dff34ce074ecc14280db > > Author: Chris Wilson > > Date: Mon Jan 24 12:34:00 2011 +0000 > > > > drm/i915: Prevent uninitialised reads during error state capture > > > > error_bo and pinned_bo could be used uninitialised if there were no > > active buffers. > > > > Caught by kmemcheck. > > > > Signed-off-by: Chris Wilson > > > > > > i'm going to go on and test the merge itself, abb72c82, since i > > thought that was working fine earlier today. but you can see which > > commit now seems to fail. does this make any sense? > > i did indeed test commit abb72c82 (the merge of the above), and it > failed. not sure why i posted earlier today that it worked, i must > have built something incorrectly. so, in conclusion: > > * commit 58bbf018 appears to be fine, boots reliably > * commit 8e934dbf causes black screen issue > * commit abb72c82 also causes black screen issue > > and on that note, signing off for the evening. > > rday > > -- > > ======================================================================== > Robert P. J. Day Waterloo, Ontario, CANADA > http://crashcourse.ca > > Twitter: http://twitter.com/rpjday > LinkedIn: http://ca.linkedin.com/in/rpjday > ======================================================================== From: Chris Wilson Subject: Re: has the i915 "black screen" boot issue returned? To: "Robert P. J. Day" Cc: dri-devel@lists.freedesktop.org Bcc: chris@chris-wilson.co.uk In-Reply-To: References: <1bdc18$jdgfni@fmsmga002.fm.intel.com> <0d30dc$ksovh8@orsmga001.jf.intel.com> On Thu, 27 Jan 2011 16:39:20 -0500 (EST), "Robert P. J. Day" wrote: > ok, i'm getting different results from this morning, not sure why, > but i'm fairly confident i've isolated it to this extent. here's the > salient part of the git log i'm looking at: Well, we have two endpoints, so let git attack: $ git bisect start $ git bisect good 58bbf018a70c562437eeae121a5d021ba7fe56a5 $ git bisect bad abb72c828878a2c69b2cfb33ac30007c8ecd735e That shouldn't take more than a few recompiles to identify the troublemaker. -Chris -- Chris Wilson, Intel Open Source Technology Centre