Re: [PATCH 2/4] dma-mapping: add get_required_mask if arch overrides default

2011-09-16 Thread FUJITA Tomonori
Sorry for the late reply, On Thu, 01 Sep 2011 15:35:11 +1000 Benjamin Herrenschmidt wrote: > On Wed, 2011-06-29 at 17:19 +0900, FUJITA Tomonori wrote: > > On Fri, 24 Jun 2011 12:05:23 -0700 > > Nishanth Aravamudan wrote: > > > > > From: Milton Miller > &

Re: [PATCH 2/4] dma-mapping: add get_required_mask if arch overrides default

2011-06-29 Thread FUJITA Tomonori
On Fri, 24 Jun 2011 12:05:23 -0700 Nishanth Aravamudan wrote: > From: Milton Miller > > If an architecture sets ARCH_HAS_DMA_GET_REQUIRED_MASK and has settable > dma_map_ops, the required mask may change by the ops implementation. > For example, a system that always has an mmu inline may only r

[PATCH v3 3/3] tcm ibmvscsis driver

2011-02-14 Thread FUJITA Tomonori
- sends task management response (for now, 'NOT SUPPORTED'). - remove dead code. = From: FUJITA Tomonori Subject: [PATCH v3 3/3] tcm ibmvscsis driver This replaces ibmvstgt driver that uses the old target framework. Signed-off-by: FUJITA Tomonori --- drivers/scsi/ibmvscsi/

[PATCH v2 3/3] tcm ibmvscsis driver

2011-02-13 Thread FUJITA Tomonori
NOT SUPPORTED'). - remove dead code. = From: FUJITA Tomonori Subject: [PATCH v2 3/3] tcm ibmvscsis driver This replaces ibmvstgt driver that uses the old target framework. Signed-off-by: FUJITA Tomonori --- drivers/scsi/ibmvscsi/Makefile|4 +- drivers/scsi/ibmvscsi/

Re: [RFC PATCH 03/11] ppc: Create ops to choose between direct window and iommu based on device mask

2010-10-10 Thread FUJITA Tomonori
On Sat, 09 Oct 2010 10:44:53 +1100 Benjamin Herrenschmidt wrote: > On Fri, 2010-10-08 at 10:33 -0700, Nishanth Aravamudan wrote: > > Also allow the coherent ops to be iommu if only the coherent mask is too > > small, mostly for driver that do not set set the coherent mask but also > > don't use t

Re: [patch 1/1] powerpc: enable ARCH_DMA_ADDR_T_64BIT with ARCH_PHYS_ADDR_T_64BIT

2010-10-02 Thread FUJITA Tomonori
On Sat, 02 Oct 2010 13:20:54 +1000 Benjamin Herrenschmidt wrote: > On Fri, 2010-10-01 at 21:31 -0400, Josh Boyer wrote: > > > +config ARCH_DMA_ADDR_T_64BIT > > > + def_bool ARCH_PHYS_ADDR_T_64BIT > > > + > > > > I seemed to have missed what this is about entirely. Is there some > > place

[PATCH -mm 5/8] powerpc: enable ARCH_DMA_ADDR_T_64BIT with ARCH_PHYS_ADDR_T_64BIT

2010-09-02 Thread FUJITA Tomonori
Signed-off-by: FUJITA Tomonori Cc: linuxppc-dev@lists.ozlabs.org --- arch/powerpc/Kconfig |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig index 631e5a0..b0e58f5 100644 --- a/arch/powerpc/Kconfig +++ b/arch/powerpc/Kconfig

Re: 2.6.35-stable/ppc64/p7: Badness at lib/dma-debug.c:902, Call Trace & Instruction Dump during boot

2010-08-04 Thread FUJITA Tomonori
On Wed, 4 Aug 2010 15:16:34 +0200 "Roedel, Joerg" wrote: > On Mon, Aug 02, 2010 at 07:55:03AM -0400, FUJITA Tomonori wrote: > > I guess that this driver does a partial sync with > > dma_sync_single_for_* API. dma-debug can't handle it properly. It's >

Re: 2.6.35-stable/ppc64/p7: Badness at lib/dma-debug.c:902, Call Trace & Instruction Dump during boot

2010-08-02 Thread FUJITA Tomonori
CC'ed to dma-debug maintainer. On Mon, 02 Aug 2010 16:21:09 +0530 Subrata Modak wrote: > Hi, > > On boot, Badness at lib/dma-debug.c:902, Call Trace & Instruction Dump > are recorded at /var/log/messages: > > > udev: starting ver

[PATCH] powerpc: remove unused 'protect4gb' boot parameter

2010-05-09 Thread FUJITA Tomonori
und was removed (the 383af9525bb27f927511874f6306247ec13f1c28). But somehow I messed the 'protect4gb' boot parameter that was used to enable the workaround. Signed-off-by: FUJITA Tomonori --- arch/powerpc/kernel/iommu.c | 12 1 files changed, 0 insertions(+), 12 deletions(-) di

Re: 2.6.34-rc3 : Badness at lib/dma-debug.c:820 during ibmvscsi init

2010-04-27 Thread FUJITA Tomonori
On Tue, 27 Apr 2010 17:05:21 +1000 Benjamin Herrenschmidt wrote: > On Fri, 2010-04-02 at 15:50 +0900, FUJITA Tomonori wrote: > > > Thanks, here's the patch in the proper format. > > > > = > > From: FUJITA Tomonori > > Subject: [PATCH] ibmvscsi:

Re: 2.6.34-rc3 : Badness at lib/dma-debug.c:820 during ibmvscsi init

2010-04-01 Thread FUJITA Tomonori
On Fri, 02 Apr 2010 12:07:58 +0530 Sachin Sant wrote: > FUJITA Tomonori wrote: > > On Fri, 02 Apr 2010 10:48:55 +0530 > > Sachin Sant wrote: > > > > > >> 2.6.34-rc3 boot on a power5 box produces this badness message > >> during ibmvscsi initia

Re: 2.6.34-rc3 : Badness at lib/dma-debug.c:820 during ibmvscsi init

2010-04-01 Thread FUJITA Tomonori
On Fri, 02 Apr 2010 10:48:55 +0530 Sachin Sant wrote: > 2.6.34-rc3 boot on a power5 box produces this badness message > during ibmvscsi initialization. > > ibmvscsi 3003: Client reserve enabled > ibmvscsi 3003: sent SRP login > ibmvscsi 3003: SRP_LOGIN succeeded > ibmvscsi 3003:

Re: [PATCH v4 05/11] swiotlb: add swiotlb_set_default_size()

2010-03-16 Thread FUJITA Tomonori
On Tue, 16 Mar 2010 14:28:09 -0500 Becky Bruce wrote: > > On Mar 16, 2010, at 5:08 AM, FUJITA Tomonori wrote: > > > On Tue, 16 Mar 2010 06:58:41 +0100 > > Albert Herranz wrote: > > > >> FUJITA Tomonori wrote: > >>> On Fri, 12 Mar

[PATCH] powerpc: fix swiotlb to respect the boot option

2010-03-16 Thread FUJITA Tomonori
t (as x86 does). Signed-off-by: FUJITA Tomonori Tested-by: Becky Bruce --- arch/powerpc/kernel/setup_32.c |6 -- arch/powerpc/kernel/setup_64.c |6 -- arch/powerpc/mm/mem.c |6 ++ 3 files changed, 6 insertions(+), 12 deletions(-) diff --git a/arch/powerpc/k

Re: [PATCH v4 05/11] swiotlb: add swiotlb_set_default_size()

2010-03-16 Thread FUJITA Tomonori
On Tue, 16 Mar 2010 06:58:41 +0100 Albert Herranz wrote: > FUJITA Tomonori wrote: > > On Fri, 12 Mar 2010 20:12:40 +0100 > > Albert Herranz wrote: > > > >> The current SWIOTLB code uses a default of 64MB for the IO TLB area. > >> This size can be

Re: [PATCH v4 04/11] swiotlb: support NOT_COHERENT_CACHE PowerPC platforms

2010-03-15 Thread FUJITA Tomonori
On Tue, 16 Mar 2010 10:54:40 +0900 FUJITA Tomonori wrote: > On Fri, 12 Mar 2010 20:12:39 +0100 > Albert Herranz wrote: > > > The current SWIOTLB code does not support NOT_COHERENT_CACHE platforms. > > This patch adds support for NOT_COHERENT_CACHE platforms to SWIO

Re: [PATCH v4 04/11] swiotlb: support NOT_COHERENT_CACHE PowerPC platforms

2010-03-15 Thread FUJITA Tomonori
On Fri, 12 Mar 2010 20:12:39 +0100 Albert Herranz wrote: > The current SWIOTLB code does not support NOT_COHERENT_CACHE platforms. > This patch adds support for NOT_COHERENT_CACHE platforms to SWIOTLB by > adding two platform specific functions swiotlb_dma_sync_page() and > swiotlb_dma_sync() whi

Re: [PATCH v4 05/11] swiotlb: add swiotlb_set_default_size()

2010-03-15 Thread FUJITA Tomonori
On Fri, 12 Mar 2010 20:12:40 +0100 Albert Herranz wrote: > The current SWIOTLB code uses a default of 64MB for the IO TLB area. > This size can be influenced using a kernel command line parameter "swiotlb". > Unfortunately, the parsing of the kernel command line is done _after_ the > swiotlb is i

Re: [PATCH] powerpc: remove IOMMU_VMERGE config option

2010-03-02 Thread FUJITA Tomonori
On Wed, 03 Mar 2010 11:13:52 +1100 Michael Ellerman wrote: > On Wed, 2010-03-03 at 08:54 +0900, FUJITA Tomonori wrote: > > Seems that IOMMU_VMERGE option description gives the false > > information: > > > > http://marc.info/?l=linux-sparc&m=126753808727669&am

[PATCH] powerpc: remove IOMMU_VMERGE config option

2010-03-02 Thread FUJITA Tomonori
Seems that IOMMU_VMERGE option description gives the false information: http://marc.info/?l=linux-sparc&m=126753808727669&w=2 http://marc.info/?l=linux-sparc&m=126753880528825&w=2 IOMMU_VMERGE is unnecessary nowadays so how about removing it? = From: FUJITA Tomonori Subject:

[PATCH] powerpc: fix pseries/dlpar compile warning without CONFIG_PROC_DEVICETREE

2010-01-05 Thread FUJITA Tomonori
erpc/platforms/pseries/dlpar.c:271: error: unused variable 'prop' arch/powerpc/platforms/pseries/dlpar.c:270: error: unused variable 'parent' make[3]: *** [arch/powerpc/platforms/pseries/dlpar.o] Error 1 Signed-off-by: FUJITA Tomonori --- arch/powerpc/platforms/pseries/dlpar.

[PATCH] powerpc: kill unused swiotlb variable

2009-11-14 Thread FUJITA Tomonori
We can kill unused swiotlb variable. Signed-off-by: FUJITA Tomonori --- arch/powerpc/kernel/dma-swiotlb.c |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/arch/powerpc/kernel/dma-swiotlb.c b/arch/powerpc/kernel/dma-swiotlb.c index e96cbbd..59c9285 100644 --- a/arch

Re: [PATCH 2/8] bitmap: Introduce bitmap_set, bitmap_clear, bitmap_find_next_zero_area

2009-10-17 Thread FUJITA Tomonori
On Sat, 17 Oct 2009 23:43:56 +0900 Akinobu Mita wrote: > 2009/10/17 FUJITA Tomonori : > > On Tue, 13 Oct 2009 18:10:17 +0900 > > Akinobu Mita wrote: > > > >> My user space testing exposed off-by-one error find_next_zero_area > >> in iommu-helper. Som

Re: [PATCH 2/8] bitmap: Introduce bitmap_set, bitmap_clear, bitmap_find_next_zero_area

2009-10-17 Thread FUJITA Tomonori
On Tue, 13 Oct 2009 18:10:17 +0900 Akinobu Mita wrote: > My user space testing exposed off-by-one error find_next_zero_area > in iommu-helper. Some zero area cannot be found by this bug. > > Subject: [PATCH] Fix off-by-one error in find_next_zero_area > > Signed-off-by: Akinobu Mita > --- > l

Re: [PATCH -v2 0/7] powerpc: use asm-generic/dma-mapping-common.h

2009-08-13 Thread FUJITA Tomonori
On Thu, 13 Aug 2009 15:48:42 +1000 Benjamin Herrenschmidt wrote: > On Wed, 2009-08-05 at 14:08 +0900, FUJITA Tomonori wrote: > > > The above swiotlb patchset was merged in -tip so I think that merging > > this patchset via -tip too is the easiest way to handle this patc

[PATCH -v2 1/7] powerpc: remove addr_needs_map in struct dma_mapping_ops

2009-08-04 Thread FUJITA Tomonori
information: http://marc.info/?t=12471906021&r=1&w=2 Signed-off-by: FUJITA Tomonori Acked-by: Becky Bruce --- arch/powerpc/include/asm/device.h |3 ++ arch/powerpc/include/asm/dma-mapping.h |8 +++--- arch/powerpc/include/asm/swiotlb.h |5 +-- arch/powerp

[PATCH -v2 7/7] powerpc: add CONFIG_DMA_API_DEBUG support

2009-08-04 Thread FUJITA Tomonori
Signed-off-by: FUJITA Tomonori --- arch/powerpc/Kconfig |1 + arch/powerpc/include/asm/dma-mapping.h | 11 ++- arch/powerpc/kernel/dma.c | 11 +++ 3 files changed, 22 insertions(+), 1 deletions(-) diff --git a/arch/powerpc/Kconfig b/arch

[PATCH -v2 6/7] powerpc: handle SWIOTLB mapping error properly

2009-08-04 Thread FUJITA Tomonori
Signed-off-by: FUJITA Tomonori --- arch/powerpc/include/asm/dma-mapping.h |5 + arch/powerpc/kernel/dma-swiotlb.c |3 ++- 2 files changed, 7 insertions(+), 1 deletions(-) diff --git a/arch/powerpc/include/asm/dma-mapping.h b/arch/powerpc/include/asm/dma-mapping.h index 91217e4

[PATCH -v2 4/7] powerpc: use dma_map_ops struct

2009-08-04 Thread FUJITA Tomonori
This converts uses dma_map_ops struct (in include/linux/dma-mapping.h) instead of POWERPC homegrown dma_mapping_ops. Signed-off-by: FUJITA Tomonori Acked-by: Becky Bruce --- arch/powerpc/include/asm/device.h |4 +- arch/powerpc/include/asm/dma-mapping.h | 84

[PATCH -v2 3/7] add set_dma_mask hook to struct dma_map_ops

2009-08-04 Thread FUJITA Tomonori
POWERPC needs this hook. SPARC could use it too. Signed-off-by: FUJITA Tomonori Acked-by: Becky Bruce --- include/linux/dma-mapping.h |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/include/linux/dma-mapping.h b/include/linux/dma-mapping.h index c0f6c3c..91b7618 100644

[PATCH -v2 5/7] powerpc: use asm-generic/dma-mapping-common.h

2009-08-04 Thread FUJITA Tomonori
Signed-off-by: FUJITA Tomonori Acked-by: Becky Bruce --- arch/powerpc/Kconfig |6 +- arch/powerpc/include/asm/dma-mapping.h | 242 +--- 2 files changed, 7 insertions(+), 241 deletions(-) diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig

[PATCH -v2 0/7] powerpc: use asm-generic/dma-mapping-common.h

2009-08-04 Thread FUJITA Tomonori
This is the second version of the patchset to convert POWERPC to use asm-generic/dma-mapping-common.h: http://marc.info/?l=linux-kernel&m=124840628128189&w=2 There are some minor changes since the first version are: - removed unused CONFIG_PPC_NEED_DMA_SYNC_OPS [5/7] (Becky pointed out). - ad

[PATCH -v2 2/7] powerpc: remove swiotlb_pci_dma_ops

2009-08-04 Thread FUJITA Tomonori
Now swiotlb_pci_dma_ops is identical to swiotlb_dma_ops; we can use swiotlb_dma_ops with any devices. This removes swiotlb_pci_dma_ops. Signed-off-by: FUJITA Tomonori Acked-by: Becky Bruce --- arch/powerpc/include/asm/swiotlb.h |1 - arch/powerpc/kernel/dma-swiotlb.c

Re: BUG in dma-mapping.h:218 // MESH SCSI driver not working

2009-08-04 Thread FUJITA Tomonori
, patched the tree with your patch, > > rebooted, it worked). > > > > Then, git says: > > f078727b250c2653fc9a564f15547c17ebac3f99 is first bad commit > > commit f078727b250c2653fc9a564f15547c17ebac3f99 > > Author: FUJITA Tomonori > > Date

Re: [PATCH 5/5] powerpc: use asm-generic/dma-mapping-common.h

2009-07-27 Thread FUJITA Tomonori
On Mon, 27 Jul 2009 16:08:46 -0500 Becky Bruce wrote: > > On Jul 23, 2009, at 10:24 PM, FUJITA Tomonori wrote: > > > Signed-off-by: FUJITA Tomonori > > Fujita, > > Since you're removing all the uses of it, you should probably remove > PPC_NEED_DMA

[PATCH 4/5] powerpc: use dma_map_ops struct

2009-07-23 Thread FUJITA Tomonori
This converts uses dma_map_ops struct (in include/linux/dma-mapping.h) instead of POWERPC homegrown dma_mapping_ops. Signed-off-by: FUJITA Tomonori --- arch/powerpc/include/asm/device.h |4 +- arch/powerpc/include/asm/dma-mapping.h | 84 --- arch/powerpc

[PATCH 1/5] powerpc: remove addr_needs_map in struct dma_mapping_ops

2009-07-23 Thread FUJITA Tomonori
information: http://marc.info/?t=12471906021&r=1&w=2 Signed-off-by: FUJITA Tomonori --- arch/powerpc/include/asm/device.h |3 ++ arch/powerpc/include/asm/dma-mapping.h |8 +++--- arch/powerpc/include/asm/swiotlb.h |5 +-- arch/powerpc/kernel/dma-s

[PATCH 0/5] powerpc: use asm-generic/dma-mapping-common.h

2009-07-23 Thread FUJITA Tomonori
This patchset converts POWERPC to use asm-generic/dma-mapping-common.h. The major change is removing addr_needs_map in struct dma_mapping_ops (1/5): http://marc.info/?t=12471906021&r=1&w=2 This is on the top of my swiotlb cleanup patchset: http://marc.info/?l=linuxppc-embedded&m=12471892032

[PATCH 5/5] powerpc: use asm-generic/dma-mapping-common.h

2009-07-23 Thread FUJITA Tomonori
Signed-off-by: FUJITA Tomonori --- arch/powerpc/Kconfig |2 +- arch/powerpc/include/asm/dma-mapping.h | 242 +--- 2 files changed, 7 insertions(+), 237 deletions(-) diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig index d00131c..0603b6c

[PATCH 3/5] add set_dma_mask hook to struct dma_map_ops

2009-07-23 Thread FUJITA Tomonori
POWERPC needs this hook. SPARC could use it too. Signed-off-by: FUJITA Tomonori --- include/linux/dma-mapping.h |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/include/linux/dma-mapping.h b/include/linux/dma-mapping.h index c0f6c3c..91b7618 100644 --- a/include/linux

[PATCH 2/5] powerpc: remove swiotlb_pci_dma_ops

2009-07-23 Thread FUJITA Tomonori
Now swiotlb_pci_dma_ops is identical to swiotlb_dma_ops; we can use swiotlb_dma_ops with any devices. This removes swiotlb_pci_dma_ops. Signed-off-by: FUJITA Tomonori --- arch/powerpc/include/asm/swiotlb.h |1 - arch/powerpc/kernel/dma-swiotlb.c | 14 -- arch

Re: removing addr_needs_map in struct dma_mapping_ops

2009-07-23 Thread FUJITA Tomonori
On Wed, 15 Jul 2009 18:59:03 -0500 Becky Bruce wrote: > > On Jul 13, 2009, at 7:49 PM, FUJITA Tomonori wrote: > > > On Mon, 13 Jul 2009 16:50:43 -0500 > > Becky Bruce wrote: > > > >>> talked about defining something like struct dma_data. Then w

Re: [PATCH 04/15] swiotlb: remove unnecessary swiotlb_bus_to_virt

2009-07-13 Thread FUJITA Tomonori
On Mon, 13 Jul 2009 21:17:21 -0500 Becky Bruce wrote: > > On Jul 9, 2009, at 8:04 PM, FUJITA Tomonori wrote: > > > swiotlb_bus_to_virt is unncessary; we can use swiotlb_bus_to_phys and > > phys_to_virt instead. > > phys_to_virt (also, virt_to_phys) is invalid for h

Re: removing addr_needs_map in struct dma_mapping_ops

2009-07-13 Thread FUJITA Tomonori
On Mon, 13 Jul 2009 16:50:43 -0500 Becky Bruce wrote: > > talked about defining something like struct dma_data. Then we could > > > > struct dev_archdata { > > ... > > > > struct dma_data *ddata; > > }; > > > > or > > > > struct dev_archdata { > > ... > > > > struct dma_data d

Re: [00/15] swiotlb cleanup

2009-07-13 Thread FUJITA Tomonori
On Mon, 13 Jul 2009 10:40:29 +0100 Ian Campbell wrote: > On Mon, 2009-07-13 at 05:20 +0100, FUJITA Tomonori wrote: > > On Fri, 10 Jul 2009 15:02:00 +0100 > > Ian Campbell wrote: > > > > > On Fri, 2009-07-10 at 14:35 +0900, FUJITA Tomonori wrote: > > &g

Re: [00/15] swiotlb cleanup

2009-07-13 Thread FUJITA Tomonori
On Mon, 13 Jul 2009 13:20:22 +0900 FUJITA Tomonori wrote: > On Fri, 10 Jul 2009 16:12:48 +0200 > Ingo Molnar wrote: > > > > functionality and reimplemented the surrounding infrastructure in > > > terms of that (and incorporating our additional requirements). I &

Re: [00/15] swiotlb cleanup

2009-07-12 Thread FUJITA Tomonori
On Fri, 10 Jul 2009 15:02:00 +0100 Ian Campbell wrote: > On Fri, 2009-07-10 at 14:35 +0900, FUJITA Tomonori wrote: > > I don't think that we need to take account of dom0 support; we don't > > have a clear idea about an acceptable dom0 design (it needs to use > >

Re: [00/15] swiotlb cleanup

2009-07-12 Thread FUJITA Tomonori
On Fri, 10 Jul 2009 16:12:48 +0200 Ingo Molnar wrote: > > functionality and reimplemented the surrounding infrastructure in > > terms of that (and incorporating our additional requirements). I > > prototyped this (it is currently unworking, in fact it seems to > > have developed rather a taste

Re: [00/15] swiotlb cleanup

2009-07-09 Thread FUJITA Tomonori
On Fri, 10 Jul 2009 07:12:36 +0200 Ingo Molnar wrote: > > * FUJITA Tomonori wrote: > > > - removes unused (and unnecessary) hooks in swiotlb. > > > > - adds dma_capable() and converts swiotlb to use it. It can be used to > > know if a memory ar

removing addr_needs_map in struct dma_mapping_ops

2009-07-09 Thread FUJITA Tomonori
I'm trying to convert POWERPC to use asm-generic/dma-mapping-common.h. POWERPC needs addr_needs_map() in struct dma_mapping_ops for SWIOTLB support but I want to avoid add addr_needs_map() in struct dma_map_ops. IIRC, you guys think it as a temporary solution and talked about defining something li

[PATCH 04/15] swiotlb: remove unnecessary swiotlb_bus_to_virt

2009-07-09 Thread FUJITA Tomonori
swiotlb_bus_to_virt is unncessary; we can use swiotlb_bus_to_phys and phys_to_virt instead. Signed-off-by: FUJITA Tomonori --- arch/powerpc/kernel/dma-swiotlb.c | 10 -- lib/swiotlb.c | 34 -- 2 files changed, 16 insertions(+), 28

[PATCH 13/15] swiotlb: use phys_to_dma and dma_to_phys

2009-07-09 Thread FUJITA Tomonori
This converts swiotlb to use phys_to_dma and dma_to_phys instead of swiotlb_phys_to_bus() and swiotlb_bus_to_phys(). swiotlb_phys_to_bus() and swiotlb_bus_to_phys() are not necessary so this patch also removes them. Signed-off-by: FUJITA Tomonori --- include/linux/swiotlb.h |5 - lib

[PATCH 03/15] swiotlb: remove swiotlb_arch_range_needs_mapping

2009-07-09 Thread FUJITA Tomonori
Nobody uses swiotlb_arch_range_needs_mapping(). Signed-off-by: FUJITA Tomonori --- arch/x86/kernel/pci-swiotlb.c |5 - include/linux/swiotlb.h |2 -- lib/swiotlb.c | 15 ++- 3 files changed, 2 insertions(+), 20 deletions(-) diff --git a/arch/x86

[PATCH 09/15] swiotlb: use dma_capable()

2009-07-09 Thread FUJITA Tomonori
This converts swiotlb to use dma_capable() instead of swiotlb_arch_address_needs_mapping() and is_buffer_dma_capable(). Signed-off-by: FUJITA Tomonori --- lib/swiotlb.c | 24 +--- 1 files changed, 5 insertions(+), 19 deletions(-) diff --git a/lib/swiotlb.c b/lib/swiotlb.c

[PATCH 12/15] x86, IA64, powerpc: add phys_to_dma() and dma_to_phys()

2009-07-09 Thread FUJITA Tomonori
This adds two functions, phys_to_dma() and dma_to_phys() to x86, IA64 and powerpc. swiotlb uses them. phys_to_dma() converts a physical address to a dma address. dma_to_phys() does the opposite. Signed-off-by: FUJITA Tomonori --- arch/ia64/include/asm/dma-mapping.h| 10 ++ arch

[PATCH 10/15] powerpc: remove unncesary swiotlb_arch_address_needs_mapping

2009-07-09 Thread FUJITA Tomonori
both provide. Signed-off-by: FUJITA Tomonori --- arch/powerpc/kernel/dma-swiotlb.c | 27 +-- 1 files changed, 1 insertions(+), 26 deletions(-) diff --git a/arch/powerpc/kernel/dma-swiotlb.c b/arch/powerpc/kernel/dma-swiotlb.c index 41534ae..a3bbe02 100644 --- a/arch/po

[PATCH 01/15] swiotlb: remove unused swiotlb_alloc_boot()

2009-07-09 Thread FUJITA Tomonori
Nobody uses swiotlb_alloc_boot(). Signed-off-by: FUJITA Tomonori --- arch/x86/kernel/pci-swiotlb.c |5 - include/linux/swiotlb.h |2 -- lib/swiotlb.c |7 +-- 3 files changed, 1 insertions(+), 13 deletions(-) diff --git a/arch/x86/kernel/pci-swiotlb.c b

[PATCH 02/15] swiotlb: remove unused swiotlb_alloc()

2009-07-09 Thread FUJITA Tomonori
Nobody uses swiotlb_alloc(). Signed-off-by: FUJITA Tomonori --- arch/x86/kernel/pci-swiotlb.c |5 - include/linux/swiotlb.h |2 -- lib/swiotlb.c |8 ++-- 3 files changed, 2 insertions(+), 13 deletions(-) diff --git a/arch/x86/kernel/pci-swiotlb.c b

[PATCH 06/15] x86: replace is_buffer_dma_capable() with dma_capable

2009-07-09 Thread FUJITA Tomonori
Signed-off-by: FUJITA Tomonori --- arch/x86/kernel/pci-dma.c |2 +- arch/x86/kernel/pci-gart_64.c |5 ++--- arch/x86/kernel/pci-nommu.c |2 +- 3 files changed, 4 insertions(+), 5 deletions(-) diff --git a/arch/x86/kernel/pci-dma.c b/arch/x86/kernel/pci-dma.c index 1a041bc

[PATCH 08/15] powerpc: add dma_capable() to replace is_buffer_dma_capable()

2009-07-09 Thread FUJITA Tomonori
dma_capable() eventually replaces is_buffer_dma_capable(), which tells if a memory area is dma-capable or not. The problem of is_buffer_dma_capable() is that it doesn't take a pointer to struct device so it doesn't work for POWERPC. Signed-off-by: FUJITA Tomonori --- arch/powerpc/i

[PATCH 05/15] x86: add dma_capable() to replace is_buffer_dma_capable()

2009-07-09 Thread FUJITA Tomonori
dma_capable() eventually replaces is_buffer_dma_capable(), which tells if a memory area is dma-capable or not. The problem of is_buffer_dma_capable() is that it doesn't take a pointer to struct device so it doesn't work for POWERPC. Signed-off-by: FUJITA Tomonori --- arch/x86/inclu

[00/15] swiotlb cleanup

2009-07-09 Thread FUJITA Tomonori
- removes unused (and unnecessary) hooks in swiotlb. - adds dma_capable() and converts swiotlb to use it. It can be used to know if a memory area is dma capable or not. I added is_buffer_dma_capable() for the same purpose long ago but it turned out that the function doesn't work on POWERPC. This

[PATCH 07/15] ia64: add dma_capable() to replace is_buffer_dma_capable()

2009-07-09 Thread FUJITA Tomonori
dma_capable() eventually replaces is_buffer_dma_capable(), which tells if a memory area is dma-capable or not. The problem of is_buffer_dma_capable() is that it doesn't take a pointer to struct device so it doesn't work for POWERPC. Signed-off-by: FUJITA Tomonori --- arch/ia64/inclu

[PATCH 14/15] powerpc: remove unused swiotlb_phys_to_bus() and swiotlb_bus_to_phys()

2009-07-09 Thread FUJITA Tomonori
phys_to_dma() and dma_to_phys() are used instead of swiotlb_phys_to_bus() and swiotlb_bus_to_phys(). Signed-off-by: FUJITA Tomonori --- arch/powerpc/kernel/dma-swiotlb.c | 11 --- 1 files changed, 0 insertions(+), 11 deletions(-) diff --git a/arch/powerpc/kernel/dma-swiotlb.c b/arch

[PATCH 11/15] remove is_buffer_dma_capable()

2009-07-09 Thread FUJITA Tomonori
is_buffer_dma_capable() was replaced with dma_capable(). is_buffer_dma_capable() tells if a buffer is dma-capable or not. However, it doesn't take a pointer to struct device so it doesn't work for POWERPC. Signed-off-by: FUJITA Tomonori --- include/linux/dma-mapping.h |5

[PATCH 15/15] x86: remove unused swiotlb_phys_to_bus() and swiotlb_bus_to_phys()

2009-07-09 Thread FUJITA Tomonori
phys_to_dma() and dma_to_phys() are used instead of swiotlb_phys_to_bus() and swiotlb_bus_to_phys(). Signed-off-by: FUJITA Tomonori --- arch/x86/kernel/pci-swiotlb.c | 10 -- 1 files changed, 0 insertions(+), 10 deletions(-) diff --git a/arch/x86/kernel/pci-swiotlb.c b/arch/x86

Re: PowerPC PCI DMA issues (prefetch/coherency?)

2009-06-16 Thread FUJITA Tomonori
On Wed, 17 Jun 2009 10:18:45 +1000 Benjamin Herrenschmidt wrote: > On Tue, 2009-06-16 at 20:02 +0200, Arnd Bergmann wrote: > > On Tuesday 16 June 2009, Scott Wood wrote: > > > > If the > > > > device is the only one, you can also use dma_alloc_noncoherent() and > > > > flush explicitly with dma_c

Re: [PATCH V2 2/3] powerpc: Add support for swiotlb on 32-bit

2009-05-22 Thread FUJITA Tomonori
On Thu, 21 May 2009 13:18:54 -0700 Jeremy Fitzhardinge wrote: > Becky Bruce wrote: > > I can work with that, but it's going to be a bit inefficient, as I > > actually need the dma_addr_t, not the phys_addr_t, so I'll have to > > convert. In every case, this is a conversion I've already done an

Re: [PATCH V2 2/3] powerpc: Add support for swiotlb on 32-bit

2009-05-22 Thread FUJITA Tomonori
On Thu, 21 May 2009 20:01:37 +0100 Ian Campbell wrote: > > It's not actually clear to me that we need that check, though. Can > > someone explain what case that was designed to catch? > > If map_single fails and returns NULL then we try to use > io_tlb_overflow_buffer, if that is somehow not D

Re: [PATCH V2 2/3] powerpc: Add support for swiotlb on 32-bit

2009-05-18 Thread FUJITA Tomonori
CC'ed linux-kernel On Thu, 14 May 2009 17:42:28 -0500 Becky Bruce wrote: > This patch includes the basic infrastructure to use swiotlb > bounce buffering on 32-bit powerpc. It is not yet enabled on > any platforms. Probably the most interesting bit is the > addition of addr_needs_map to dma_op

Re: [PATCH 4/5] powerpc: Add support for swiotlb on 32-bit

2009-04-20 Thread FUJITA Tomonori
On Mon, 20 Apr 2009 13:03:20 -0500 Becky Bruce wrote: > > On Apr 20, 2009, at 11:57 AM, Kumar Gala wrote: > > > > > On Apr 20, 2009, at 11:26 AM, Becky Bruce wrote: > > > >> This patch includes the basic infrastructure to use swiotlb > >> bounce buffering on 32-bit powerpc. It is not yet enabl

Re: [PATCH 1/5] powerpc: Use sg->dma_length in sg_dma_len() macro on 32-bit

2009-04-20 Thread FUJITA Tomonori
On Mon, 20 Apr 2009 15:06:16 -0500 Kumar Gala wrote: > > On Apr 20, 2009, at 11:26 AM, Becky Bruce wrote: > > > Currently, the 32-bit code uses sg->length instead of sg->dma_lentgh > > to report sg_dma_len. However, since the default dma code for 32-bit > > (the dma_direct case) sets dma_lengt

[RESEND] [PATCH] powerpc: remove dead BIO_VMERGE_BOUNDARY definition

2008-12-13 Thread FUJITA Tomonori
This is a resend of: http://marc.info/?l=linux-kernel&m=122482703616607&w=2 = From: FUJITA Tomonori Subject: [PATCH] powerpc: remove dead BIO_VMERGE_BOUNDARY definition The block layer dropped the virtual merge feature (b8b3e16cfe6435d961f6aaebcfd52a1ff2a988c5). BIO_VMERGE_BOUNDARY de

[PATCH] powerpc: remove dead BIO_VMERGE_BOUNDARY definition

2008-10-23 Thread FUJITA Tomonori
The block layer dropped the virtual merge feature (b8b3e16cfe6435d961f6aaebcfd52a1ff2a988c5). BIO_VMERGE_BOUNDARY definition is meaningless now (For POWER, BIO_VMERGE_BOUNDARY has been meaningless for a long time since POWER disables the virtual merge feature). Signed-off-by: FUJITA Tomonori

Re: [PATCH] Revert "Merge branch 'x86/iommu' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/linux-2.6-tip into for-linus"

2008-08-01 Thread FUJITA Tomonori
On Fri, 01 Aug 2008 00:04:17 -0700 (PDT) David Miller <[EMAIL PROTECTED]> wrote: > From: Joerg Roedel <[EMAIL PROTECTED]> > Date: Fri, 1 Aug 2008 09:03:28 +0200 > > > That may be the reason that your fix is not yet upstream. > > The reason is more-so because Linus simply hasn't merged more > tha

Re: [PATCH] Revert "Merge branch 'x86/iommu' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/linux-2.6-tip into for-linus"

2008-07-31 Thread FUJITA Tomonori
On Fri, 1 Aug 2008 09:43:23 +1000 Stephen Rothwell <[EMAIL PROTECTED]> wrote: > This reverts commit 29111f579f4f3f2a07385f931854ab0527ae7ea5. > > This undoes the hasty addition of a global version of iommu_num_pages() > that broke both the powerpc and sparc builds. This function can be > revisit

Re: ide pmac breakage

2008-07-29 Thread FUJITA Tomonori
On Tue, 29 Jul 2008 21:26:11 +0200 Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]> wrote: > On Tuesday 29 July 2008, Bartlomiej Zolnierkiewicz wrote: > > On Tuesday 29 July 2008, Bartlomiej Zolnierkiewicz wrote: > > > On Tuesday 29 July 2008, Benjamin Herrenschmidt wrote: > > > > On Tue, 2008-07-29

Re: ide pmac breakage

2008-07-28 Thread FUJITA Tomonori
On Mon, 28 Jul 2008 16:31:56 +0200 Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]> wrote: > > > However, the machine crashes when removing the media-bay CD-ROM drive. > > > > > > Crash appears to be a NULL deref, possibly in elv_may_queue() though > > > I don't have a clean backtrace yet, working o

[PATCH] spufs: fix compile error

2008-05-13 Thread FUJITA Tomonori
]: *** [arch/powerpc/platforms/cell/spufs/file.o] Error 1 Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]> --- arch/powerpc/platforms/cell/spufs/file.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/arch/powerpc/platforms/cell/spufs/file.c b/arch/powerpc/platfor

Re: [patch 2/5] PS3: Add time include to lpm

2008-04-30 Thread FUJITA Tomonori
On Wed, 30 Apr 2008 15:25:09 -0700 Geoff Levand <[EMAIL PROTECTED]> wrote: > From: FUJITA Tomonori <[EMAIL PROTECTED]> > > Add an include statement for get_tb(). > > Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]> > Signed-off-by: Geoff Levand <[EM

Re: SCSI errors on powerpc with 2.6.24-rc6-mm1

2007-12-31 Thread FUJITA Tomonori
On Mon, 31 Dec 2007 17:57:35 +0530 Balbir Singh <[EMAIL PROTECTED]> wrote: > FUJITA Tomonori wrote: > > > > Oops, it's for -mm. > > > > Hi, > > I just tested this patch and it works fine for me so far. > > Tested-by: Balbir Singh <

Re: + iommu-sg-add-iommu-helper-functions-for-the-free-area-management.patch added to -mm tree

2007-12-30 Thread FUJITA Tomonori
add IOMMU helper functions for the free area management > From: FUJITA Tomonori <[EMAIL PROTECTED]> > > This adds IOMMU helper functions for the free area management. These > functions take care of LLD's segment boundary limit for IOMMUs. They would be > useful for IOMMUs

Re: SCSI errors on powerpc with 2.6.24-rc6-mm1

2007-12-28 Thread FUJITA Tomonori
On Fri, 28 Dec 2007 13:05:39 +0530 Balbir Singh <[EMAIL PROTECTED]> wrote: > FUJITA Tomonori wrote: > [snip] > > Thanks, > > > > Can you try this? > > > > > > diff --git a/lib/iommu-helper.c b/lib/iommu-helper.c > > index e7d8544..495575a 1

Re: SCSI errors on powerpc with 2.6.24-rc6-mm1

2007-12-27 Thread FUJITA Tomonori
On Thu, 27 Dec 2007 11:15:37 +0530 Balbir Singh <[EMAIL PROTECTED]> wrote: > FUJITA Tomonori wrote: > > On Thu, 27 Dec 2007 10:08:25 +0530 > > Balbir Singh <[EMAIL PROTECTED]> wrote: > > > >> FUJITA Tomonori wrote: > >>> On Mon, 24 Dec 20

Re: SCSI errors on powerpc with 2.6.24-rc6-mm1

2007-12-26 Thread FUJITA Tomonori
On Thu, 27 Dec 2007 10:08:25 +0530 Balbir Singh <[EMAIL PROTECTED]> wrote: > FUJITA Tomonori wrote: > > On Mon, 24 Dec 2007 10:18:50 +0530 > > Balbir Singh <[EMAIL PROTECTED]> wrote: > > > [snip] > > > > > I might break the IOMMU code. Can you

Re: SCSI errors on powerpc with 2.6.24-rc6-mm1

2007-12-26 Thread FUJITA Tomonori
On Mon, 24 Dec 2007 10:18:50 +0530 Balbir Singh <[EMAIL PROTECTED]> wrote: > Hi, > > I've just seen this on my dmesg, this is new, never seen this before on > this box and it happens only with this version of the kernel. > > In this configuration, the page size is set to 64K and I've enabled fak