From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefan Lippers-Hollmann Subject: Re: git pull] drm for v4.1-rc1 Date: Mon, 8 Jun 2015 12:28:26 +0200 Message-ID: <20150608122826.4d8a796b@mir> References: <20150605231821.6c740f67@mir> <20150606235248.GZ5176@intel.com> <20150607043253.12f4058e@mir> <1433750815.4364.13.camel@gmail.com> <1433752186.4364.15.camel@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: In-Reply-To: <1433752186.4364.15.camel@gmail.com> Sender: linux-kernel-owner@vger.kernel.org To: Ander Conselvan De Oliveira Cc: Ville =?UTF-8?B?U3lyasOkbMOk?= , Dave Airlie , Nicolas Kalkhof , Jani Nikula , Daniel Vetter , linux-kernel@vger.kernel.org, DRI mailing list , torvalds@linux-foundation.org List-Id: dri-devel@lists.freedesktop.org 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=C3=A4l=C3=A4 wrote: > > > > On Fri, Jun 05, 2015 at 11:18:21PM +0200, Stefan Lippers-Hollma= nn wrote: > > > > > On 2015-04-20, Dave Airlie wrote: [...] > > > > > > Ander Conselvan de Oliveira (28): > > > > > [...] > > > > > > drm/i915: Allocate connector state together with the = connectors > > > > > [...] > > > > >=20 > > > > > This commit introduces a regression relative to v4.0 on an In= tel=20 > > > > > D945GCLF2 mainboard[1] (Atom 330) with Intel 82945G/GZ onboar= d graphics=20 > > > > > using its (only-) VGA connector for me. > > > > >=20 > > > > > v4.1-rc6-52-gff25ea8: > > > > > [ 13.265699] BUG: unable to handle kernel NULL pointer dere= ference at 0000000000000010 > > > > > [ 13.265723] IP: [] intel_modeset_update_= connector_atomic_state+0x61/0x90 [i915] > > > >=20 > > > > 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 immedi= ately > > > > spot any place where we'd forget to set it up though. > > > >=20 > > > > Can you try with something like this so we'd at least find out = which > > > > connector(s) is/are at fault here? > > >=20 > > > 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=20 > > > boot messages: > > >=20 > > > [ 13.492784] connector =3D ffff880079bb8000 > > > [ 13.910439] connector =3D ffff8800795b5800 > > > [ 14.463114] connector =3D ffff8800795b6000 > > > [ 14.700707] connector =3D ffff8800795b6800 > > > [ 14.869418] connector =3D ffff8800795b7000 > > > [ 14.923848] connector =3D ffff8800795b7000 > > >=20 > > > Full, gzipped, bootlog attached - thanks a lot for your efforts. > >=20 > > Could you repeat the process with drm.debug=3D0xe in your kernel co= mmand > > line and send the logs again? >=20 > Never mind, Ville's patch produced all the information necessary. Ple= ase > give the patch I just sent a try. Thanks a lot, as already reported as a response to your patch, your=20 change "drm/i915: Allocate connector state together with the connectors= " fixes the problem for me. Regards Stefan Lippers-Hollmann