From owner-freebsd-hackers Mon Apr 15 7:36:53 2002 Delivered-To: freebsd-hackers@freebsd.org Received: from one.net (ip-216-23-55-100.adsl.one.net [216.23.55.100]) by hub.freebsd.org (Postfix) with SMTP id 8D1DE37B427 for ; Mon, 15 Apr 2002 07:36:45 -0700 (PDT) Received: (qmail 18418 invoked by uid 1001); 15 Apr 2002 14:37:54 -0000 Date: Mon, 15 Apr 2002 10:37:54 -0400 From: Coleman Kane To: Frank Mayhar Cc: hackers@freebsd.org Subject: Re: Minor weirdness in pci/agp_amd.c. Message-ID: <20020415103754.A18397@freebsd.org> References: <200204142357.g3ENvHFT091843@realtime.exit.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <200204142357.g3ENvHFT091843@realtime.exit.com> User-Agent: Mutt/1.3.22.1i Sender: owner-freebsd-hackers@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG This was the case in older revisions, it has been fixed in RELENG_4 and -CURRENT. The driver shipped with 4.5-RELEASE was broken. -- coleman On Sun, Apr 14, 2002 at 04:57:17PM -0700, Frank Mayhar wrote: > I'm working on writing a driver for the ServerWorks AGP support from the > Linux driver (sans documentation, SIGH :-(). I've been using the various > other drivers as models, particularly the AMD driver, since it seems to be > closest in many ways to the ServerWorks architecture. > > Anyway, I've run into a minor oddity in agp_amd_alloc_gatt(), in pci/agp_amd.c. > At lines 120-121: > gatt->ag_pdir = vtophys((vm_offset_t) gatt->ag_vdir); > gatt->ag_pdir = vtophys(gatt->ag_virtual); > > Looks like one or the other is redundant? Probably the first, I would guess, > if the code actually works as it is. > -- > Frank Mayhar frank@exit.com http://www.exit.com/ > Exit Consulting http://www.gpsclock.com/ > > To Unsubscribe: send mail to majordomo@FreeBSD.org > with "unsubscribe freebsd-hackers" in the body of the message > To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-hackers" in the body of the message