From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S268070AbUBRUqm (ORCPT ); Wed, 18 Feb 2004 15:46:42 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S268072AbUBRUqm (ORCPT ); Wed, 18 Feb 2004 15:46:42 -0500 Received: from delerium.kernelslacker.org ([81.187.208.145]:53177 "EHLO delerium.codemonkey.org.uk") by vger.kernel.org with ESMTP id S268070AbUBRUqk (ORCPT ); Wed, 18 Feb 2004 15:46:40 -0500 Date: Wed, 18 Feb 2004 20:44:06 +0000 From: Dave Jones To: Andi Kleen Cc: linux-kernel@vger.kernel.org Subject: Re: [PATCH] Enable Intel AGP on x86-64 Message-ID: <20040218204406.GB6242@redhat.com> Mail-Followup-To: Dave Jones , Andi Kleen , linux-kernel@vger.kernel.org References: <200402182006.i1IK6bL7022634@hera.kernel.org> <20040218202325.GZ6242@redhat.com> <20040219021149.519d0754.ak@suse.de> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20040219021149.519d0754.ak@suse.de> User-Agent: Mutt/1.4.1i Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Feb 19, 2004 at 02:11:49AM +0100, Andi Kleen wrote: > > Please don't do this. At least copy intel-agp.c to > > something new and throw out all the dozens of chipsets > > that will never appear on ia32e. > > > > Splitting agpgart up to seperate drivers allowed us > > to stop adding cruft upon cruft with each generation > > of chipsets. I don't want to have to spend half of > > 2.7 decrufting agpgart again. > > Huh? Did you actually read the patch? Yes, did you actually read my mail? > It doesn't change the AGP driver at all, just enables it in Kconfig because > Intel chipsets can be now used on the x86-64 kernel too. You *really* think you're going to see a 440BX GART on ia32e ? i810 ? i820 ? i830 ? etc. etc. I'd be *very* surprised if anything but the current generation of ia32 chipsets gets used on ia32e. It just doesn't make sense. Without even looking at the code I'll bet you can shrink it by at least 75%. Dave