Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757168Ab2HXI3r (ORCPT ); Fri, 24 Aug 2012 04:29:47 -0400 Received: from hqemgate04.nvidia.com ([216.228.121.35]:13386 "EHLO hqemgate04.nvidia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756771Ab2HXI3m (ORCPT ); Fri, 24 Aug 2012 04:29:42 -0400 X-PGP-Universal: processed; by hqnvupgp05.nvidia.com on Fri, 24 Aug 2012 01:29:40 -0700 From: Hiroshi Doyu To: CC: Hiroshi Doyu , , , , , , , , , , , , , Subject: [v3 0/4] ARM: dma-mapping: IOMMU atomic allocation Date: Fri, 24 Aug 2012 11:29:01 +0300 Message-ID: <1345796945-21115-1-git-send-email-hdoyu@nvidia.com> X-Mailer: git-send-email 1.7.5.4 MIME-Version: 1.0 Content-Type: text/plain Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 5642 Lines: 91 Hi, The commit e9da6e9 "ARM: dma-mapping: remove custom consistent dma region" breaks the compatibility with existing drivers. This causes the following kernel oops(*1). That driver has called dma_pool_alloc() to allocate memory from the interrupt context, and it hits BUG_ON(in_interrpt()) in "get_vm_area_caller()". This patch seris fixes this problem with making use of the pre-allocate atomic memory pool which DMA is using in the same way as DMA does now. Any comment would be really appreciated. v3: Provide a different path for IOMMU for more clean code. (Marek) atomic_pool is backed with struct page *pages[]. v2: Don't modify attrs(DMA_ATTR_NO_KERNEL_MAPPING) for atomic allocation. (Marek) Modify vzalloc (KyongHo, Minchan) v1: http://lists.linaro.org/pipermail/linaro-mm-sig/2012-August/002398.html *1: [ 8.321343] ------------[ cut here ]------------ [ 8.325971] kernel BUG at kernel/mm/vmalloc.c:1322! [ 8.333615] Internal error: Oops - BUG: 0 [#1] PREEMPT SMP ARM [ 8.339436] Modules linked in: [ 8.342496] CPU: 0 Tainted: G W (3.4.6-00067-g5d485f7 #67) [ 8.349192] PC is at __get_vm_area_node.isra.29+0x164/0x16c [ 8.354758] LR is at get_vm_area_caller+0x4c/0x54 [ 8.359454] pc : [] lr : [] psr: 20000193 [ 8.359458] sp : c09edca0 ip : c09ec000 fp : ae278000 [ 8.370922] r10: f0000000 r9 : c011aa54 r8 : c0a26cb8 [ 8.376136] r7 : 00000001 r6 : 000000d0 r5 : 20000008 r4 : c09edca0 [ 8.382651] r3 : 00010000 r2 : 20000008 r1 : 00000001 r0 : 00001000 [ 8.389166] Flags: nzCv IRQs off FIQs on Mode SVC_32 ISA ARM Segment kernel [ 8.396549] Control: 10c5387d Table: ad98c04a DAC: 00000015 .... [ 9.169162] dfa0: 412fc099 c09ec000 00000000 c000fdd8 c06df1e4 c0a1b080 00000000 00000000 [ 9.177329] dfc0: c0a235cc 8000406a 00000000 c0986818 ffffffff ffffffff c0986404 00000000 [ 9.185497] dfe0: 00000000 c09bb070 10c5387d c0a19c58 c09bb064 80008044 00000000 00000000 [ 9.193673] [] (__get_vm_area_node.isra.29+0x164/0x16c) from [] (get_vm_area_caller+0x4c/0x54) [ 9.204022] [] (get_vm_area_caller+0x4c/0x54) from [] (__iommu_alloc_remap.isra.14+0x2c/0xfc) [ 9.214276] [] (__iommu_alloc_remap.isra.14+0x2c/0xfc) from [] (arm_iommu_alloc_attrs+0xc4/0xf8) [ 9.224795] [] (arm_iommu_alloc_attrs+0xc4/0xf8) from [] (pool_alloc_page.constprop.5+0x6c/0xf8) [ 9.235309] [] (pool_alloc_page.constprop.5+0x6c/0xf8) from [] (dma_pool_alloc+0x80/0x170) [ 9.245304] [] (dma_pool_alloc+0x80/0x170) from [] (tegra_build_dtd+0x48/0x14c) [ 9.254344] [] (tegra_build_dtd+0x48/0x14c) from [] (tegra_req_to_dtd+0x7c/0xa8) [ 9.263467] [] (tegra_req_to_dtd+0x7c/0xa8) from [] (tegra_ep_queue+0x154/0x33c) [ 9.272592] [] (tegra_ep_queue+0x154/0x33c) from [] (composite_setup+0x364/0x6d4) [ 9.281804] [] (composite_setup+0x364/0x6d4) from [] (android_setup+0xb8/0x14c) [ 9.290843] [] (android_setup+0xb8/0x14c) from [] (setup_received_irq+0xbc/0x270) [ 9.300053] [] (setup_received_irq+0xbc/0x270) from [] (tegra_udc_irq+0x2ac/0x2c4) [ 9.309353] [] (tegra_udc_irq+0x2ac/0x2c4) from [] (handle_irq_event_percpu+0x78/0x2e0) [ 9.319087] [] (handle_irq_event_percpu+0x78/0x2e0) from [] (handle_irq_event+0x44/0x64) [ 9.328907] [] (handle_irq_event+0x44/0x64) from [] (handle_fasteoi_irq+0xc4/0x16c) [ 9.338294] [] (handle_fasteoi_irq+0xc4/0x16c) from [] (generic_handle_irq+0x34/0x48) [ 9.347858] [] (generic_handle_irq+0x34/0x48) from [] (handle_IRQ+0x54/0xb4) [ 9.356637] [] (handle_IRQ+0x54/0xb4) from [] (gic_handle_irq+0x2c/0x60) [ 9.365068] [] (gic_handle_irq+0x2c/0x60) from [] (__irq_svc+0x40/0x70) [ 9.373405] Exception stack(0xc09edf10 to 0xc09edf58) [ 9.378447] df00: 00000000 000f4240 00000003 00000000 [ 9.386615] df20: 00000000 e55bbc00 ef66f3ca 00000001 00000000 412fc099 c0abb9c8 00000000 [ 9.394781] df40: 3b9ac9ff c09edf58 c027a9bc c0042880 20000113 ffffffff [ 9.401396] [] (__irq_svc+0x40/0x70) from [] (tegra_idle_enter_lp3+0x68/0x78) [ 9.410272] [] (tegra_idle_enter_lp3+0x68/0x78) from [] (cpuidle_idle_call+0xdc/0x3a4) [ 9.419922] [] (cpuidle_idle_call+0xdc/0x3a4) from [] (cpu_idle+0xd8/0x134) [ 9.428612] [] (cpu_idle+0xd8/0x134) from [] (start_kernel+0x27c/0x2cc) [ 9.436952] Code: e1a00004 e3a04000 eb002265 eaffffe0 (e7f001f2) [ 9.443038] ---[ end trace 1b75b31a2719ed24 ]--- [ 9.447645] Kernel panic - not syncing: Fatal exception in interrupt Hiroshi Doyu (4): ARM: dma-mapping: atomic_pool with struct page **pages ARM: dma-mapping: Refactor out to introduce __in_atomic_pool ARM: dma-mapping: Introduce __atomic_get_pages() for __iommu_get_pages() ARM: dma-mapping: IOMMU allocates pages from atomic_pool with GFP_ATOMIC arch/arm/mm/dma-mapping.c | 90 ++++++++++++++++++++++++++++++++++++++++----- 1 files changed, 80 insertions(+), 10 deletions(-) -- 1.7.5.4 -- 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/