From mboxrd@z Thu Jan 1 00:00:00 1970 From: James Bottomley Subject: Re: [PATCH 1/3] iscsi iser: remove DMA restrictions Date: Thu, 14 Feb 2008 12:10:33 -0600 Message-ID: <1203012634.3158.5.camel@localhost.localdomain> References: <20080212205252.GB13643@osc.edu> <20080212205403.GC13643@osc.edu> <1202850645.3137.132.camel@localhost.localdomain> <47B480D9.6080206@cs.wisc.edu> Mime-Version: 1.0 Content-Type: text/plain Content-Transfer-Encoding: 7bit Return-path: Received: from accolon.hansenpartnership.com ([76.243.235.52]:51436 "EHLO accolon.hansenpartnership.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752119AbYBNSKp (ORCPT ); Thu, 14 Feb 2008 13:10:45 -0500 In-Reply-To: <47B480D9.6080206@cs.wisc.edu> Sender: linux-scsi-owner@vger.kernel.org List-Id: linux-scsi@vger.kernel.org To: Mike Christie Cc: Pete Wyckoff , Erez Zilber , Roland Dreier , linux-scsi On Thu, 2008-02-14 at 11:56 -0600, Mike Christie wrote: > > You really don't want to do this. That signals to the block layer that > > we have an iommu, although it's practically the same thing as a 64 bit > > DMA mask ... but I'd just leave it to the DMA mask to set this up > > correctly. Anything else is asking for a subtle bug to turn up years > > from now when something causes the mask and the limit to be mismatched. > > > > I thought BLK_BOUNCE_ANY just meant "don't bounce anything" (that was > from the blkdev.h comments). It does ... that's why it's used in the IOMMU case ... and why it's practically the same as a 64 bit mask. > We used it for iscsi_tcp because the network layer can take any type > of page and will do the right thing for the hardware it eventually > gets sent to. Right, to you it means never bounce because net wants to do it instead. However, I don't think that's the case for iSER, is it? ... as in, if I've got the pathway tracing correct, it just goes to the infiniband device and gets mapped there. If there's a mask mismatch (very unlikely, I know) we get a very subtle and hard to trace error. James