Date: Wed, 16 Jun 2004 22:42:57 +0100
From: Christoph Hellwig <hch@infradead.org>
To: Alan Cox <alan@redhat.com>
Cc: linux-kernel@vger.kernel.org, linux-scsi@vger.kernel.orgy
Subject: Re: PATCH: Further aacraid work
Message-ID: <20040616214257.GA20787@infradead.org>
Mail-Followup-To: Christoph Hellwig <hch@infradead.org>,
	Alan Cox <alan@redhat.com>, linux-kernel@vger.kernel.org,
	linux-scsi@vger.kernel.orgy
References: <20040616210455.GA13385@devserv.devel.redhat.com> <20040616213343.GA20488@infradead.org> <20040616214048.GA27169@devserv.devel.redhat.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20040616214048.GA27169@devserv.devel.redhat.com>
User-Agent: Mutt/1.4.1i
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1024
Lines: 21

On Wed, Jun 16, 2004 at 05:40:48PM -0400, Alan Cox wrote:
> > Looks mostly good except for the GART iommu ifdef.  That code is bogus for
> > almost everything but a plain PC and should just be killed.
> 
> There are lots of problems with the PC centric view of the world some
> aacraid hardware has.  At the moment I'm still working on trying to understand
> the rules and I'll need to talk to Mark some more. I've also got a third
> party trace suggesting a request for low DMA memory came in through the
> gart which is above the address in the mask to look at.
> 
> Its something I hope to get rid of eventually. In the meantime the GART
> define is needed to make it work on AMD64.

Well, the code is completely wrong.  It must not only go away for AMD64 but
for all arches.

-
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/