From owner-cvs-all Thu Jul 22 23:46:49 1999 Delivered-To: cvs-all@freebsd.org Received: from alpo.whistle.com (alpo.whistle.com [207.76.204.38]) by hub.freebsd.org (Postfix) with ESMTP id 10439156B4; Thu, 22 Jul 1999 23:46:45 -0700 (PDT) (envelope-from julian@whistle.com) Received: from current1.whistle.com (current1.whistle.com [207.76.205.22]) by alpo.whistle.com (8.9.1a/8.9.1) with SMTP id XAA32019; Thu, 22 Jul 1999 23:45:04 -0700 (PDT) Date: Thu, 22 Jul 1999 23:45:03 -0700 (PDT) From: Julian Elischer To: Greg Lehey Cc: Dag-Erling Smorgrav , cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/pci ide_pci.c In-Reply-To: <19990723105940.D84734@freebie.lemis.com> Message-ID: MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk yes On Fri, 23 Jul 1999, Greg Lehey wrote: > On Thursday, 22 July 1999 at 12:41:43 -0700, Julian Elischer wrote: > > On Thu, 22 Jul 1999, Dag-Erling Smorgrav wrote: > > > >> des 1999/07/22 04:34:13 PDT > >> > >> Modified files: (Branch: RELENG_3) > >> sys/pci ide_pci.c > >> Log: > >> Back out previous commit so IDE works again. > >> Whatever happened to testing before MFC? > >> > >> Revision Changes Path > >> 1.28.2.2 +99 -124 src/sys/pci/ide_pci.c > > > > ERK! > > I found it.. > > - firstpage = DMA_PG_SZ - ((uintptr_t)vaddr & (DMA_PG_SZ)); > > + firstpage = DMA_PG_SZ - ((uintptr_t)vaddr & (DMA_PG_SZ - 1)); > > > > gives the same result for page alligned transfers, but is definitly wrong > > for unalligned transfers.. > > we must be doing all alligned transfers! > > > > (I have a rack of 4 machines doing load testing with the bad code and all > > working fine!) at a guess, the failure must have been on an access to the > > raw device. and.... AHA! the Cx5530 can not do accesses not alligned to > > 16 byte boundaries, so we are not testing them.. they get done by PIO, and > > not this code. this explains why we can do heavy testing on it and not > > see this bug > > Was this the thing that caused UDMA to fall into a heap on the ground? > > Greg > -- > See complete headers for address, home page and phone numbers > finger grog@lemis.com for PGP public key > > To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message