On Monday 18 May 2009, Geert Uytterhoeven wrote:
+#ifndef dma_alloc_coherent
+static inline void *
+dma_alloc_coherent(struct device *dev, size_t size, dma_addr_t *dma_handle,
+ gfp_t flag)
+{
+ void *virt = kmalloc(size, flag);
kmalloc() may fail.
+ *dma_handle = virt_to_phys(virt);
Not all variants of virt_to_phys() may handle the NULL case very well.
I took a statistically invalid sample: some just cast to unsigned
long, other subtract PAGE_OFFSET.
Right. It should not really hurt, because all users need to check virt
before using dma_handle, but I can fix it just to be sure.
I'll wait for comments from Fujita Tomonori and others before I repost.
Arnd <><
--
To unsubscribe from this list: send the line "unsubscribe linux-m68k" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
