diff options
author | FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp> | 2008-09-08 05:10:12 -0400 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2008-09-08 09:50:06 -0400 |
commit | 8a53ad675f86ee003482b557da944e070d3c4859 (patch) | |
tree | f20f205bb29fec54405a60b706d9b444f79cd8e9 | |
parent | de9f521fb72dd091aa4989fe2e004ecf4785a850 (diff) |
x86: fix nommu_alloc_coherent allocation with NULL device argument
We need to use __GFP_DMA for NULL device argument (fallback_dev) with
pci-nommu. It's a hack for ISA (and some old code) so we need to use
GFP_DMA.
Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
Acked-by: Joerg Roedel <joerg.roedel@amd.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
-rw-r--r-- | arch/x86/kernel/pci-nommu.c | 3 | ||||
-rw-r--r-- | include/asm-x86/dma-mapping.h | 2 |
2 files changed, 3 insertions, 2 deletions
diff --git a/arch/x86/kernel/pci-nommu.c b/arch/x86/kernel/pci-nommu.c index 0f51883cc6a8..ada1c87cafc2 100644 --- a/arch/x86/kernel/pci-nommu.c +++ b/arch/x86/kernel/pci-nommu.c | |||
@@ -80,7 +80,6 @@ nommu_alloc_coherent(struct device *hwdev, size_t size, | |||
80 | int node; | 80 | int node; |
81 | struct page *page; | 81 | struct page *page; |
82 | 82 | ||
83 | gfp &= ~(__GFP_DMA | __GFP_HIGHMEM | __GFP_DMA32); | ||
84 | gfp |= __GFP_ZERO; | 83 | gfp |= __GFP_ZERO; |
85 | 84 | ||
86 | dma_mask = hwdev->coherent_dma_mask; | 85 | dma_mask = hwdev->coherent_dma_mask; |
@@ -93,7 +92,7 @@ nommu_alloc_coherent(struct device *hwdev, size_t size, | |||
93 | node = dev_to_node(hwdev); | 92 | node = dev_to_node(hwdev); |
94 | 93 | ||
95 | #ifdef CONFIG_X86_64 | 94 | #ifdef CONFIG_X86_64 |
96 | if (dma_mask <= DMA_32BIT_MASK) | 95 | if (dma_mask <= DMA_32BIT_MASK && !(gfp & GFP_DMA)) |
97 | gfp |= GFP_DMA32; | 96 | gfp |= GFP_DMA32; |
98 | #endif | 97 | #endif |
99 | 98 | ||
diff --git a/include/asm-x86/dma-mapping.h b/include/asm-x86/dma-mapping.h index 088c56814aa3..ad8b49032d18 100644 --- a/include/asm-x86/dma-mapping.h +++ b/include/asm-x86/dma-mapping.h | |||
@@ -246,6 +246,8 @@ dma_alloc_coherent(struct device *dev, size_t size, dma_addr_t *dma_handle, | |||
246 | struct dma_mapping_ops *ops = get_dma_ops(dev); | 246 | struct dma_mapping_ops *ops = get_dma_ops(dev); |
247 | void *memory; | 247 | void *memory; |
248 | 248 | ||
249 | gfp &= ~(__GFP_DMA | __GFP_HIGHMEM | __GFP_DMA32); | ||
250 | |||
249 | if (dma_alloc_from_coherent(dev, size, dma_handle, &memory)) | 251 | if (dma_alloc_from_coherent(dev, size, dma_handle, &memory)) |
250 | return memory; | 252 | return memory; |
251 | 253 | ||