From mboxrd@z Thu Jan 1 00:00:00 1970 From: davem@davemloft.net (David Miller) Date: Sat, 11 Jun 2011 16:25:53 -0700 (PDT) Subject: [PATCH v3 4/5] net: ep93xx_eth: drop GFP_DMA from call to dma_alloc_coherent() In-Reply-To: <6f1c0ce5bbc0c36ce509d6503363c5e9ee3f626b.1307816881.git.mika.westerberg@iki.fi> References: <6f1c0ce5bbc0c36ce509d6503363c5e9ee3f626b.1307816881.git.mika.westerberg@iki.fi> Message-ID: <20110611.162553.618147586345938739.davem@davemloft.net> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org From: Mika Westerberg Date: Sat, 11 Jun 2011 21:39:57 +0300 > Commit a197b59ae6e8 (mm: fail GFP_DMA allocations when ZONE_DMA is not > configured) made page allocator to return NULL if GFP_DMA is set but > CONFIG_ZONE_DMA is disabled. > > This causes ep93xx_eth to fail: > > WARNING: at mm/page_alloc.c:2251 __alloc_pages_nodemask+0x11c/0x638() > Modules linked in: > [] (unwind_backtrace+0x0/0xf4) from [] (warn_slowpath_common+0x48/0x60) > [] (warn_slowpath_common+0x48/0x60) from [] (warn_slowpath_null+0x1c/0x24) > [] (warn_slowpath_null+0x1c/0x24) from [] (__alloc_pages_nodemask+0x11c/0x638) > [] (__alloc_pages_nodemask+0x11c/0x638) from [] (__dma_alloc+0x8c/0x3ec) > [] (__dma_alloc+0x8c/0x3ec) from [] (dma_alloc_coherent+0x54/0x60) > [] (dma_alloc_coherent+0x54/0x60) from [] (ep93xx_open+0x20/0x864) > [] (ep93xx_open+0x20/0x864) from [] (__dev_open+0xb8/0x108) > [] (__dev_open+0xb8/0x108) from [] (__dev_change_flags+0x70/0x128) > [] (__dev_change_flags+0x70/0x128) from [] (dev_change_flags+0x10/0x48) > [] (dev_change_flags+0x10/0x48) from [] (ip_auto_config+0x190/0xf68) > [] (ip_auto_config+0x190/0xf68) from [] (do_one_initcall+0x34/0x18c) > [] (do_one_initcall+0x34/0x18c) from [] (kernel_init+0x94/0x134) > [] (kernel_init+0x94/0x134) from [] (kernel_thread_exit+0x0/0x8) > > Since there is no restrictions for DMA on ep93xx, we can fix this by just > removing the GFP_DMA flag from the call. > > Signed-off-by: Mika Westerberg > Acked-by: Russell King > Acked-by: H Hartley Sweeten > Tested-by: Petr Stetiar Applied.