Re: [Intel-gfx] [git pull] drm fixes

From: Takashi Iwai <tiwai@suse.de>
To: Daniel Vetter <daniel@ffwll.ch>
Cc: Josh Boyer <jwboyer@fedoraproject.org>,
	Intel Graphics Development <intel-gfx@lists.freedesktop.org>,
	"Linux-Kernel@Vger. Kernel. Org" <linux-kernel@vger.kernel.org>,
	DRI mailing list <dri-devel@lists.freedesktop.org>,
	Xi Ruoyao <xry111@outlook.com>,
	Linus Torvalds <torvalds@linux-foundation.org>
Subject: Re: [Intel-gfx] [git pull] drm fixes
Date: Wed, 25 Mar 2015 16:26:59 +0100	[thread overview]
Message-ID: <s5hpp7xdrng.wl-tiwai@suse.de> (raw)
In-Reply-To: <20150325140008.GJ1349@phenom.ffwll.local>

At Wed, 25 Mar 2015 15:00:08 +0100,
Daniel Vetter wrote:
> 
> On Wed, Mar 25, 2015 at 09:11:17AM -0400, Josh Boyer wrote:
> > On Wed, Mar 25, 2015 at 4:54 AM, Daniel Vetter <daniel@ffwll.ch> wrote:
> > >> >> commit f55548b5af87ebfc586ca75748947f1c1b1a4a52
> > >> >> Author: Damien Lespiau <damien.lespiau@intel.com>
> > >> >> Date:   Thu Feb 5 18:30:20 2015 +0000
> > >> >>
> > >> >>     drm/i915: Don't try to reference the fb in get_initial_plane_config()
> > >> >>
> > >> >> From linux-next?
> > >> >
> > >> > Yes, building now.  Will let you know as soon as I test it on both machines.
> > >>
> > >> OK, with that commit applied I no longer get the kref.h splat and the
> > >> NUC machine boots headless.  I still see the backtrace below on both
> > >> the NUC and the macbook.  I have a copy of it with drm.debug=0xff from
> > >> the NUC here:
> > >>
> > >> https://jwboyer.fedorapeople.org/pub/nuc-drm-debug-ff-with-fixes.txt
> > >>
> > >> Getting better at least :).
> > >
> > > On top of what you currently have please also cherry-pick
> > >
> > > commit fb9981aa675eb7b398849915364916fd98833cfa
> > > Author: Damien Lespiau <damien.lespiau@intel.com>
> > > Date:   Thu Feb 5 19:24:25 2015 +0000
> > >
> > >     drm/i915: Fix atomic state when reusing the firmware fb
> > >
> > > from -next. Let's hope this terminates eventually ;-)
> > 
> > Hm.  That one doesn't apply cleanly.  I think because it needs:
> > 
> > From 2d14030b1a9d0e89cfdca6f16851e2eac8cb4de0 Mon Sep 17 00:00:00 2001
> > From: Damien Lespiau <damien.lespiau@intel.com>
> > Date: Thu, 5 Feb 2015 17:22:18 +0000
> > Subject: drm/i915: Store the initial framebuffer in initial_plane_config
> > 
> > first.  Do you want me to grab both, or should I try and figure out
> > how to backport fb9981aa67 without it?
> 
> Oops missed that. The active ingredient is setting crtc->primary->state->crtc like this:

As I wrote in another mail, I can confirm that cherry-picking of three
commits (and manual compile fixes) fixed the problem on my machine.

Takashi
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel