Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752235AbcDZN3Z (ORCPT ); Tue, 26 Apr 2016 09:29:25 -0400 Received: from mail-lf0-f47.google.com ([209.85.215.47]:35688 "EHLO mail-lf0-f47.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752195AbcDZN3V (ORCPT ); Tue, 26 Apr 2016 09:29:21 -0400 Date: Tue, 26 Apr 2016 15:29:18 +0200 From: Niklas =?iso-8859-1?Q?S=F6derlund?= To: Christoph Hellwig Cc: Laurent Pinchart , Dan Williams , Vinod Koul , linux-renesas-soc@vger.kernel.org, "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" , "dmaengine@vger.kernel.org" , iommu@lists.linux-foundation.org, robin.murphy@arm.com, geert+renesas@glider.be, Linus Walleij , Arnd Bergmann , linux-arch@vger.kernel.org Subject: Re: [PATCH v5 3/9] dma-mapping: add dma_{map,unmap}_resource Message-ID: <20160426132918.GN28777@bigcity.dyn.berto.se> Mail-Followup-To: Christoph Hellwig , Laurent Pinchart , Dan Williams , Vinod Koul , linux-renesas-soc@vger.kernel.org, "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" , "dmaengine@vger.kernel.org" , iommu@lists.linux-foundation.org, robin.murphy@arm.com, geert+renesas@glider.be, Linus Walleij , Arnd Bergmann , linux-arch@vger.kernel.org References: <1457404974-1800-1-git-send-email-niklas.soderlund+renesas@ragnatech.se> <20160311125846.GF1111@bigcity.dyn.berto.se> <20160315082254.GE9136@infradead.org> <3286525.zPAGiD4Xk2@avalon> <20160321152601.GA11674@infradead.org> <20160413132916.GC19650@bigcity.dyn.berto.se> <20160421134942.GA3325@infradead.org> <20160425142619.GJ28777@bigcity.dyn.berto.se> <20160425191004.GA13906@infradead.org> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20160425191004.GA13906@infradead.org> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 763 Lines: 22 Hi Christoph, On 2016-04-25 12:10:04 -0700, Christoph Hellwig wrote: > On Mon, Apr 25, 2016 at 04:26:19PM +0200, Niklas S?derlund wrote: > > I have followed the call path from the usage in > > drivers/dma/sh/rcar-dmac.c and made sure the dma_addr_t is not used in a > > bad way. > > The dma-debug routines are called from the generic code in > include/linux/dma-mapping.h, and from my reading of the other patches > in your series you are calling it for these as well. You are correct I have not consider that dma_mapping_error() call in to lib/dma-debug.c. I will see if I can make the dma_mapping_error() safe to use with a dma_addr_t obtained from dma_map_resource() and post a new series. Thanks for pointing this out! -- Regards, Niklas S?derlund