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 b...@kernel.crashing.org wrote: On Wed, 2011-06-29 at 17:19 +0900, FUJITA Tomonori wrote: On Fri, 24 Jun 2011 12:05:23 -0700 Nishanth Aravamudan n...@us.ibm.com wrote: From: Milton Miller milt

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 n...@us.ibm.com wrote: From: Milton Miller milt...@bga.com 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

[PATCH v3 3/3] tcm ibmvscsis driver

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

[PATCH v2 3/3] tcm ibmvscsis driver

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

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 b...@kernel.crashing.org 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

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 b...@kernel.crashing.org 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

[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 fujita.tomon...@lab.ntt.co.jp 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

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 joerg.roe...@amd.com 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 likely that this is a false

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 subr...@linux.vnet.ibm.com wrote: Hi, On boot, Badness at lib/dma-debug.c:902, Call Trace Instruction Dump are recorded at /var/log/messages:

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

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

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 b...@kernel.crashing.org 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 fujita.tomon...@lab.ntt.co.jp Subject: [PATCH] ibmvscsi: fix DMA

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

2010-04-02 Thread FUJITA Tomonori
On Fri, 02 Apr 2010 10:48:55 +0530 Sachin Sant sach...@in.ibm.com 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

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

2010-04-02 Thread FUJITA Tomonori
On Fri, 02 Apr 2010 12:07:58 +0530 Sachin Sant sach...@in.ibm.com wrote: FUJITA Tomonori wrote: On Fri, 02 Apr 2010 10:48:55 +0530 Sachin Sant sach...@in.ibm.com wrote: 2.6.34-rc3 boot on a power5 box produces this badness message during ibmvscsi initialization. ibmvscsi

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 albert_herr...@yahoo.es wrote: FUJITA Tomonori wrote: On Fri, 12 Mar 2010 20:12:40 +0100 Albert Herranz albert_herr...@yahoo.es wrote: The current SWIOTLB code uses a default of 64MB for the IO TLB area. This size can be influenced

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

2010-03-16 Thread FUJITA Tomonori
(as x86 does). Signed-off-by: FUJITA Tomonori fujita.tomon...@lab.ntt.co.jp Tested-by: Becky Bruce bec...@kernel.crashing.org --- 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

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 bec...@kernel.crashing.org wrote: On Mar 16, 2010, at 5:08 AM, FUJITA Tomonori wrote: On Tue, 16 Mar 2010 06:58:41 +0100 Albert Herranz albert_herr...@yahoo.es wrote: FUJITA Tomonori wrote: On Fri, 12 Mar 2010 20:12:40 +0100 Albert

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 albert_herr...@yahoo.es 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_

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 albert_herr...@yahoo.es 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

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 fujita.tomon...@lab.ntt.co.jp wrote: On Fri, 12 Mar 2010 20:12:39 +0100 Albert Herranz albert_herr...@yahoo.es wrote: The current SWIOTLB code does not support NOT_COHERENT_CACHE platforms. This patch adds support for NOT_COHERENT_CACHE

[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-sparcm=126753808727669w=2 http://marc.info/?l=linux-sparcm=126753880528825w=2 IOMMU_VMERGE is unnecessary nowadays so how about removing it? = From: FUJITA Tomonori fujita.tomon...@lab.ntt.co.jp

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 mich...@ellerman.id.au 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-sparcm=126753808727669w=2 http://marc.info/?l

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

2010-01-05 Thread FUJITA Tomonori
: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 fujita.tomon...@lab.ntt.co.jp --- arch/powerpc/platforms/pseries/dlpar.c |4 +++- 1 files

[PATCH] powerpc: kill unused swiotlb variable

2009-11-14 Thread FUJITA Tomonori
We can kill unused swiotlb variable. Signed-off-by: FUJITA Tomonori fujita.tomon...@lab.ntt.co.jp --- 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

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 akinobu.m...@gmail.com 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

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 akinobu.m...@gmail.com wrote: 2009/10/17 FUJITA Tomonori fujita.tomon...@lab.ntt.co.jp: On Tue, 13 Oct 2009 18:10:17 +0900 Akinobu Mita akinobu.m...@gmail.com wrote: My user space testing exposed off-by-one error find_next_zero_area

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 b...@kernel.crashing.org 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 patchset

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

2009-08-04 Thread FUJITA Tomonori
, at a certain moment I needed your patch (I booted, got a problem, patched the tree with your patch, rebooted, it worked). Then, git says: f078727b250c2653fc9a564f15547c17ebac3f99 is first bad commit commit f078727b250c2653fc9a564f15547c17ebac3f99 Author: FUJITA Tomonori fujita.tomon

[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 fujita.tomon...@lab.ntt.co.jp Acked-by: Becky Bruce bec...@kernel.crashing.org --- arch/powerpc/include/asm/swiotlb.h

[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-kernelm=124840628128189w=2 There are some minor changes since the first version are: - removed unused CONFIG_PPC_NEED_DMA_SYNC_OPS [5/7] (Becky pointed out). -

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

2009-08-04 Thread FUJITA Tomonori
Signed-off-by: FUJITA Tomonori fujita.tomon...@lab.ntt.co.jp Acked-by: Becky Bruce bec...@kernel.crashing.org --- arch/powerpc/Kconfig |6 +- arch/powerpc/include/asm/dma-mapping.h | 242 +--- 2 files changed, 7 insertions(+), 241 deletions

[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 fujita.tomon...@lab.ntt.co.jp Acked-by: Becky Bruce bec...@kernel.crashing.org --- include/linux/dma-mapping.h |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/include/linux/dma-mapping.h b

[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 fujita.tomon...@lab.ntt.co.jp Acked-by: Becky Bruce bec...@kernel.crashing.org --- arch/powerpc/include/asm/device.h |4 +- arch/powerpc

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

2009-08-04 Thread FUJITA Tomonori
Signed-off-by: FUJITA Tomonori fujita.tomon...@lab.ntt.co.jp --- 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

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

2009-08-04 Thread FUJITA Tomonori
Signed-off-by: FUJITA Tomonori fujita.tomon...@lab.ntt.co.jp --- 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

[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=12471906021r=1w=2 Signed-off-by: FUJITA Tomonori fujita.tomon...@lab.ntt.co.jp Acked-by: Becky Bruce bec...@kernel.crashing.org --- arch/powerpc/include/asm/device.h |3 ++ arch/powerpc/include/asm/dma-mapping.h |8 +++--- arch/powerpc/include

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 bec...@kernel.crashing.org wrote: On Jul 23, 2009, at 10:24 PM, FUJITA Tomonori wrote: Signed-off-by: FUJITA Tomonori fujita.tomon...@lab.ntt.co.jp Fujita, Since you're removing all the uses of it, you should probably remove

[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 fujita.tomon...@lab.ntt.co.jp --- 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

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

2009-07-23 Thread FUJITA Tomonori
Signed-off-by: FUJITA Tomonori fujita.tomon...@lab.ntt.co.jp --- 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

[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=12471906021r=1w=2 This is on the top of my swiotlb cleanup patchset:

[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 fujita.tomon...@lab.ntt.co.jp --- arch/powerpc/include/asm/device.h |4 +- arch/powerpc/include/asm/dma-mapping.h | 84

Re: [00/15] swiotlb cleanup

2009-07-13 Thread FUJITA Tomonori
On Mon, 13 Jul 2009 13:20:22 +0900 FUJITA Tomonori fujita.tomon...@lab.ntt.co.jp wrote: On Fri, 10 Jul 2009 16:12:48 +0200 Ingo Molnar mi...@elte.hu wrote: functionality and reimplemented the surrounding infrastructure in terms of that (and incorporating our additional requirements). I

Re: [00/15] swiotlb cleanup

2009-07-13 Thread FUJITA Tomonori
On Mon, 13 Jul 2009 10:40:29 +0100 Ian Campbell ian.campb...@eu.citrix.com wrote: On Mon, 2009-07-13 at 05:20 +0100, FUJITA Tomonori wrote: On Fri, 10 Jul 2009 15:02:00 +0100 Ian Campbell ian.campb...@citrix.com wrote: On Fri, 2009-07-10 at 14:35 +0900, FUJITA Tomonori wrote: I

Re: [00/15] swiotlb cleanup

2009-07-12 Thread FUJITA Tomonori
On Fri, 10 Jul 2009 16:12:48 +0200 Ingo Molnar mi...@elte.hu 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

Re: [00/15] swiotlb cleanup

2009-07-12 Thread FUJITA Tomonori
On Fri, 10 Jul 2009 15:02:00 +0100 Ian Campbell ian.campb...@citrix.com 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 swiotlb code? I

[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 fujita.tomon...@lab.ntt.co.jp --- arch/x86/kernel/pci-swiotlb.c | 10 -- 1 files changed, 0 insertions(+), 10 deletions(-) diff --git a/arch/x86/kernel

[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 fujita.tomon...@lab.ntt.co.jp --- include/linux/dma

[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 fujita.tomon...@lab.ntt.co.jp --- arch/powerpc/kernel/dma-swiotlb.c | 11 --- 1 files changed, 0 insertions(+), 11 deletions(-) diff --git a/arch/powerpc

[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 fujita.tomon...@lab.ntt.co.jp

[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 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 fujita.tomon...@lab.ntt.co.jp

[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 fujita.tomon...@lab.ntt.co.jp

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

2009-07-09 Thread FUJITA Tomonori
Signed-off-by: FUJITA Tomonori fujita.tomon...@lab.ntt.co.jp --- 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

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

2009-07-09 Thread FUJITA Tomonori
Nobody uses swiotlb_alloc(). Signed-off-by: FUJITA Tomonori fujita.tomon...@lab.ntt.co.jp --- 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

[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 fujita.tomon...@lab.ntt.co.jp --- 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

[PATCH 10/15] powerpc: remove unncesary swiotlb_arch_address_needs_mapping

2009-07-09 Thread FUJITA Tomonori
provide. Signed-off-by: FUJITA Tomonori fujita.tomon...@lab.ntt.co.jp --- 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

[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 fujita.tomon...@lab.ntt.co.jp --- arch/ia64/include/asm/dma-mapping.h

[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 fujita.tomon...@lab.ntt.co.jp --- lib/swiotlb.c | 24 +--- 1 files changed, 5 insertions(+), 19 deletions(-) diff --git

[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 fujita.tomon...@lab.ntt.co.jp --- arch/x86/kernel/pci-swiotlb.c |5 - include/linux/swiotlb.h |2 -- lib/swiotlb.c | 15 ++- 3 files changed, 2 insertions(+), 20

[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 fujita.tomon...@lab.ntt.co.jp --- include/linux

[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 fujita.tomon...@lab.ntt.co.jp --- arch/powerpc/kernel/dma-swiotlb.c | 10 -- lib/swiotlb.c | 34 -- 2 files

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

Re: [00/15] swiotlb cleanup

2009-07-09 Thread FUJITA Tomonori
On Fri, 10 Jul 2009 07:12:36 +0200 Ingo Molnar mi...@elte.hu wrote: * FUJITA Tomonori fujita.tomon...@lab.ntt.co.jp 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 area is dma capable

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

2009-06-16 Thread FUJITA Tomonori
On Wed, 17 Jun 2009 10:18:45 +1000 Benjamin Herrenschmidt b...@kernel.crashing.org 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

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 ian.campb...@eu.citrix.com 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

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 jer...@goop.org 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

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 bec...@kernel.crashing.org 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

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 ga...@kernel.crashing.org 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)

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 bec...@kernel.crashing.org 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

[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-kernelm=122482703616607w=2 = From: FUJITA Tomonori fujita.tomon...@lab.ntt.co.jp Subject: [PATCH] powerpc: remove dead BIO_VMERGE_BOUNDARY definition The block layer dropped the virtual merge feature (b8b3e16cfe6435d961f6aaebcfd52a1ff2a988c5

[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 than a couple

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 revisited

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 at 13:41

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 on it... I

[PATCH] spufs: fix compile error

2008-05-13 Thread FUJITA Tomonori
/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/platforms/cell/spufs/file.c index 80911a3

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 asm/time.h statement for get_tb(). Signed-off-by: FUJITA Tomonori [EMAIL PROTECTED] Signed-off-by: Geoff Levand [EMAIL PROTECTED] --- drivers/ps3/ps3-lpm.c:192

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

2007-12-30 Thread FUJITA Tomonori
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 that use bitmap for the free area management. Signed-off-by: FUJITA Tomonori

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 100644 --- a/lib/iommu-helper.c +++ b/lib/iommu-helper.c @@ -8,15

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 2007 10:18:50 +0530 Balbir Singh [EMAIL PROTECTED] wrote: [snip] I

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 fake NUMA

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 reproduce it easily? If so, reverting my IOMMU patches (I've