From: Andi Kleen <ak@suse.de>
To: discuss@x86-64.org
Subject: Re: [discuss] Re: [PATCH 2/4] x86-64: Calgary IOMMU - move valid_dma_direction into the callers
Date: Fri, 26 May 2006 09:57:01 +0200
User-Agent: KMail/1.9.1
Cc: Jeff Garzik <jeff@garzik.org>, Muli Ben-Yehuda <mulix@mulix.org>,
       Jon Mason <jdmason@us.ibm.com>, Muli Ben-Yehuda <muli@il.ibm.com>,
       Linux-Kernel <linux-kernel@vger.kernel.org>,
       Andrew Morton <akpm@osdl.org>
References: <20060525033550.GD7720@us.ibm.com> <20060525094236.GB22495@granada.merseine.nu> <44757FD3.3070805@garzik.org>
In-Reply-To: <44757FD3.3070805@garzik.org>
MIME-Version: 1.0
Content-Disposition: inline
Message-Id: <200605260957.02163.ak@suse.de>
Content-Type: text/plain;
  charset="iso-8859-1"
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1310
Lines: 31

On Thursday 25 May 2006 11:58, Jeff Garzik wrote:
> Muli Ben-Yehuda wrote:
> > On Thu, May 25, 2006 at 12:35:07AM -0400, Jeff Garzik wrote:
> >> Jon Mason wrote:
> >>> >From Andi Kleen's comments on the original Calgary patch, move
> >>> valid_dma_direction into the calling functions.
> >>>
> >>> Signed-off-by: Muli Ben-Yehuda <muli@il.ibm.com>
> >>> Signed-off-by: Jon Mason <jdmason@us.ibm.com>
> >> Even though BUG_ON() includes unlikely(), this introduces additional 
> >> tests in very hot paths.
> > 
> > Are they really very hot? I mean if you're calling the DMA API, you're
> > about to frob the hardware or have already frobbed it - does this
> > check really matter?
> 
> When you are adding a check that will _never_ be hit in production, to 
> the _hottest_ paths in the kernel, you can be assured it matters...

pci_dma_* shouldn't be that hot. Or at least IO usually has so much
overhead that some more bugging shouldn't matter.

On the other hand if the problem of passing wrong parameters here
isn't common I would be ok with dropping them.

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