linux-kernel - Re: [PATCH RFT linux-next] frv: dma-mapping: support debug_dma_mapping

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <1353003455.2532.50.camel@lorien2>
Date:	Thu, 15 Nov 2012 11:17:35 -0700
From:	Shuah Khan <shuah.khan@...com>
To:	dhowells@...hat.com, Marek Szyprowski <m.szyprowski@...sung.com>
Cc:	LKML <linux-kernel@...r.kernel.org>, shuahkhan@...il.com
Subject: Re: [PATCH RFT linux-next] frv: dma-mapping: support
 debug_dma_mapping_error

On Fri, 2012-10-26 at 11:15 -0600, Shuah Khan wrote:
> On Fri, 2012-10-26 at 10:48 -0600, Shuah Khan wrote:
> > Add support for debug_dma_mapping_error() call to avoid warning from
> > debug_dma_unmap() interface when it checks for mapping error checked
> > status. Without this patch, device driver failed to check map error
> > warning is generated.
> > 
> > Signed-off-by: Shuah Khan <shuah.khan@...com>
> 
> Please ignore this patch if frv doesn't use dma_map_page() and
> dma_map_single() from arch/mips/include/asm/dma-mapping.
> 
> frv defines dma_map_page() and dma_map_single() in
> arch/frv/mb93090-mb00/pci-dma.c and
> arch/frv/mb93090-mb00/pci-dma-nommu.c
> 
> -- Shuah
> > ---
> >  arch/frv/include/asm/dma-mapping.h |    2 ++
> >  1 file changed, 2 insertions(+)
> > 
> > diff --git a/arch/frv/include/asm/dma-mapping.h b/arch/frv/include/asm/dma-mapping.h
> > index dfb8110..d8a8aac 100644
> > --- a/arch/frv/include/asm/dma-mapping.h
> > +++ b/arch/frv/include/asm/dma-mapping.h
> > @@ -6,6 +6,7 @@
> >  #include <asm/cacheflush.h>
> >  #include <asm/scatterlist.h>
> >  #include <asm/io.h>
> > +#include <linux/dma-debug.h>
> >  
> >  /*
> >   * See Documentation/DMA-API.txt for the description of how the
> > @@ -97,6 +98,7 @@ void dma_sync_sg_for_device(struct device *dev, struct scatterlist *sg, int nele
> >  static inline
> >  int dma_mapping_error(struct device *dev, dma_addr_t dma_addr)
> >  {
> > +	debug_dma_mapping_error(dev, dma_addr);
> >  	return 0;
> >  }
> >  
> 

Marek,

This one is for frv to go through your tree with the other arch patches
for debug_dmap_mapping_error(). However, I am not very sure if this
change is necessary for frv if it doesn't use generic dma interfaces. I
will defer to you and David on deciding if this patch is needed.

frv defines dma_map_page() and dma_map_single() in
arch/frv/mb93090-mb00/pci-dma.c and
 arch/frv/mb93090-mb00/pci-dma-nommu.c

Thanks,
-- Shuah


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/