From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754574AbXFWSmm (ORCPT ); Sat, 23 Jun 2007 14:42:42 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752296AbXFWSme (ORCPT ); Sat, 23 Jun 2007 14:42:34 -0400 Received: from mx1.redhat.com ([66.187.233.31]:33699 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752136AbXFWSme (ORCPT ); Sat, 23 Jun 2007 14:42:34 -0400 Date: Sat, 23 Jun 2007 14:42:21 -0400 From: Dave Jones To: Andrew Morton Cc: Wang Zhenyu , carlo@alinoe.com, linux-kernel@vger.kernel.org, eric@anholt.net Subject: Re: [PATCH][AGPGART] intel_agp: don't load if no IGD and AGP port Message-ID: <20070623184221.GB18605@redhat.com> Mail-Followup-To: Dave Jones , Andrew Morton , Wang Zhenyu , carlo@alinoe.com, linux-kernel@vger.kernel.org, eric@anholt.net References: <20070618225009.GE13538@alinoe.com> <20070619233716.GA5779@alinoe.com> <20070620011521.GB2856@zhen-devel.sh.intel.com> <20070620014227.GA4090@zhen-devel.sh.intel.com> <20070620140237.GB5524@alinoe.com> <20070620154610.GA2802@localhost.localdomain> <20070621054318.GA31381@zhen-devel.sh.intel.com> <20070623095203.66bb8572.akpm@linux-foundation.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20070623095203.66bb8572.akpm@linux-foundation.org> User-Agent: Mutt/1.5.14 (2007-02-12) Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org On Sat, Jun 23, 2007 at 09:52:03AM -0700, Andrew Morton wrote: > > On Thu, 21 Jun 2007 13:43:18 +0800 Wang Zhenyu wrote: > > Thanks Carlo to report this problem. The following patch should fix > > his and potential issue. > > > > [AGPGART] intel_agp: don't load if no IGD detected and no AGP port > > > > After i915 chip, GMCH has no AGP port. Origin bridge driver in device > > table will try to access illegal regs like APBASE, APSIZE, etc. This > > may cause problem. > > > > So mark them as NULL in the table, we won't load if no IGD got detect > > and bridge has no AGP port. > > Looking at the above, I have no way of telling what the actual bug is, nor > have I any way of telling what the consequences would be of not having this > patch in 2.6.22. Nor can I tell whether we want it in 2.6.21.x. > > I probably _could_ work this out, and kinda did with a bit of list-trolling > (verdict: needed in 2.6.22) but please, take care to describe the > importance of a patch in the changelog? This got merged a day or two ago. "will try to access illegal regs.." being the key part of the changelog above. Without this diff, it goes bang, and stops booting on certain chipsets. Dave -- http://www.codemonkey.org.uk