Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
	id S1752646AbbFHK3R (ORCPT <rfc822;w@1wt.eu>);
	Mon, 8 Jun 2015 06:29:17 -0400
Received: from mout.gmx.net ([212.227.15.18]:64344 "EHLO mout.gmx.net"
	rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP
	id S1751235AbbFHK3H convert rfc822-to-8bit (ORCPT
	<rfc822;linux-kernel@vger.kernel.org>);
	Mon, 8 Jun 2015 06:29:07 -0400
Date: Mon, 8 Jun 2015 12:28:26 +0200
From: Stefan Lippers-Hollmann <s.l-h@gmx.de>
To: Ander Conselvan De Oliveira <conselvan2@gmail.com>
Cc: Ville =?UTF-8?B?U3lyasOkbMOk?= <ville.syrjala@linux.intel.com>,
        Dave Airlie <airlied@linux.ie>, Nicolas Kalkhof <nkalkhof@web.de>,
        Jani Nikula <jani.nikula@intel.com>,
        Daniel Vetter <daniel.vetter@ffwll.ch>, linux-kernel@vger.kernel.org,
        DRI mailing list <dri-devel@lists.freedesktop.org>,
        torvalds@linux-foundation.org
Subject: Re: git pull] drm for v4.1-rc1
Message-ID: <20150608122826.4d8a796b@mir>
In-Reply-To: <1433752186.4364.15.camel@gmail.com>
References: <alpine.DEB.2.00.1504200432000.1232@skynet.skynet.ie>
	<20150605231821.6c740f67@mir>
	<20150606235248.GZ5176@intel.com>
	<20150607043253.12f4058e@mir>
	<1433750815.4364.13.camel@gmail.com>
	<1433752186.4364.15.camel@gmail.com>
X-Mailer: Claws Mail 3.11.1 (GTK+ 2.24.28; x86_64-pc-linux-gnu)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8BIT
X-Provags-ID: V03:K0:6xAqqGs6iazN8T5tJe65EiOW4TE3a6FmPZDJXtKA7b/4GzJLwPW
 gLImnEHEhOTpKCxtlNgRPKFQXVg6BMcn8cTuq8smJ8NJqf+wa1SKq1LciMzCSo4CH0AeJx1
 0i4nnEX/rW8AQo5PRcHK9BNhiayHsg7QgkYxI0ECu1IrtRnbfj5sQTq+/BrifgpZ+YfPRc4
 e6MXW6HEvSktD2q8fSK/w==
X-UI-Out-Filterresults: notjunk:1;
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
Content-Length: 2601
Lines: 59

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/