Re: git pull] drm for v4.1-rc1

From	Stefan Lippers-Hollmann <s.l-h@gmx.de>
Newsgroups	linux.kernel
Subject	Re: git pull] drm for v4.1-rc1
Date	2015-06-08 12:30 +0200
Message-ID	<pz2tz-48L-5@gated-at.bofh.it> (permalink)
References	(1 earlier) <py7bZ-3VJ-27@gated-at.bofh.it> <pywam-7bL-7@gated-at.bofh.it> <pyyFb-2Ah-5@gated-at.bofh.it> <pz0i5-Ys-1@gated-at.bofh.it> <pz0L9-1xJ-29@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

Hi

On 2015-06-08, Ander Conselvan De Oliveira wrote:
> On Mon, 2015-06-08 at 11:06 +0300, Ander Conselvan De Oliveira wrote:
> > On Sun, 2015-06-07 at 04:32 +0200, Stefan Lippers-Hollmann wrote:
> > > On 2015-06-07, Ville Syrjälä wrote:
> > > > On Fri, Jun 05, 2015 at 11:18:21PM +0200, Stefan Lippers-Hollmann wrote:
> > > > > On 2015-04-20, Dave Airlie wrote:
[...]
> > > > > > Ander Conselvan de Oliveira (28):
> > > > > [...]
> > > > > >       drm/i915: Allocate connector state together with the connectors
> > > > > [...]
> > > > > 
> > > > > This commit introduces a regression relative to v4.0 on an Intel 
> > > > > D945GCLF2 mainboard[1] (Atom 330) with Intel 82945G/GZ onboard graphics 
> > > > > using its (only-) VGA connector for me.
> > > > > 
> > > > > v4.1-rc6-52-gff25ea8:
> > > > > [   13.265699] BUG: unable to handle kernel NULL pointer dereference at 0000000000000010
> > > > > [   13.265723] IP: [<ffffffffa0556f01>] intel_modeset_update_connector_atomic_state+0x61/0x90 [i915]
> > > > 
> > > > Hmm. Smells like a connector with a NULL state pointer, and the bad
> > > > commit touched exactly the part that sets it up. I can't immediately
> > > > spot any place where we'd forget to set it up though.
> > > > 
> > > > Can you try with something like this so we'd at least find out which
> > > > connector(s) is/are at fault here?
> > > 
> > > With the patch applied, the kernel (v4.1-rc6-104-g4b17069) locks up even
> > > harder, so I had to switch to a serial console in order to fetch the 
> > > boot messages:
> > > 
> > > [   13.492784] connector = ffff880079bb8000
> > > [   13.910439] connector = ffff8800795b5800
> > > [   14.463114] connector = ffff8800795b6000
> > > [   14.700707] connector = ffff8800795b6800
> > > [   14.869418] connector = ffff8800795b7000
> > > [   14.923848] connector = ffff8800795b7000
> > > 
> > > Full, gzipped, bootlog attached - thanks a lot for your efforts.
> > 
> > Could you repeat the process with drm.debug=0xe in your kernel command
> > line and send the logs again?
> 
> Never mind, Ville's patch produced all the information necessary. Please
> give the patch I just sent a try.

Thanks a lot, as already reported as a response to your patch, your 
change "drm/i915: Allocate connector state together with the connectors"
fixes the problem for me.

Regards
	Stefan Lippers-Hollmann
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Thread

Re: git pull] drm for v4.1-rc1 Ville Syrjälä <ville.syrjala@linux.intel.com> - 2015-06-07 02:00 +0200
  Re: git pull] drm for v4.1-rc1 Stefan Lippers-Hollmann <s.l-h@gmx.de> - 2015-06-07 04:40 +0200
    Re: git pull] drm for v4.1-rc1 Stefan Lippers-Hollmann <s.l-h@gmx.de> - 2015-06-08 12:30 +0200
    [Fwd: Re: [PATCH] drm/i915: Properly initialize SDVO analog  connectors] Ander Conselvan De Oliveira <conselvan2@gmail.com> - 2015-06-08 12:40 +0200
    Re: [PATCH] drm/i915: Properly initialize SDVO analog connectors Jani Nikula <jani.nikula@intel.com> - 2015-06-08 13:00 +0200

csiph-web