* [patch for 2.6.27? 4/6] drivers/net/mlx4/alloc.c needs mm.h
@ 2008-09-05 21:04 akpm
2008-09-05 23:44 ` Roland Dreier
2008-09-13 19:13 ` Jeff Garzik
0 siblings, 2 replies; 3+ messages in thread
From: akpm @ 2008-09-05 21:04 UTC (permalink / raw)
To: jeff; +Cc: netdev, akpm, davem
From: Andrew Morton <akpm@linux-foundation.org>
sparc32 allmodconfig with linux-next:
drivers/net/mlx4/alloc.c: In function 'mlx4_buf_alloc':
drivers/net/mlx4/alloc.c:164: error: 'PAGE_KERNEL' undeclared (first use in this function)
drivers/net/mlx4/alloc.c:164: error: (Each undeclared identifier is reported only once
drivers/net/mlx4/alloc.c:164: error: for each function it appears in.)
this is due to some header shuffle in linux-next. I didn't look to see what
it was. I'd sugges that this patch be merged ahead of a linux-next merge to
avoid bisection breaks.
We strictly only need asm/pgtable.h, but going direct to asm includes always
seems grubby.
Cc: Jeff Garzik <jeff@garzik.org>
Cc: "David S. Miller" <davem@davemloft.net>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
---
drivers/net/mlx4/alloc.c | 1 +
1 file changed, 1 insertion(+)
diff -puN drivers/net/mlx4/alloc.c~drivers-net-mlx4-allocc-needs-mmh drivers/net/mlx4/alloc.c
--- a/drivers/net/mlx4/alloc.c~drivers-net-mlx4-allocc-needs-mmh
+++ a/drivers/net/mlx4/alloc.c
@@ -33,6 +33,7 @@
#include <linux/errno.h>
#include <linux/slab.h>
+#include <linux/mm.h>
#include <linux/bitmap.h>
#include <linux/dma-mapping.h>
#include <linux/vmalloc.h>
_
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [patch for 2.6.27? 4/6] drivers/net/mlx4/alloc.c needs mm.h
2008-09-05 21:04 [patch for 2.6.27? 4/6] drivers/net/mlx4/alloc.c needs mm.h akpm
@ 2008-09-05 23:44 ` Roland Dreier
2008-09-13 19:13 ` Jeff Garzik
1 sibling, 0 replies; 3+ messages in thread
From: Roland Dreier @ 2008-09-05 23:44 UTC (permalink / raw)
To: akpm; +Cc: jeff, netdev, davem
> drivers/net/mlx4/alloc.c:164: error: 'PAGE_KERNEL' undeclared (first use in this function)
> +#include <linux/mm.h>
Looks like a good fix to me.
- R.
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [patch for 2.6.27? 4/6] drivers/net/mlx4/alloc.c needs mm.h
2008-09-05 21:04 [patch for 2.6.27? 4/6] drivers/net/mlx4/alloc.c needs mm.h akpm
2008-09-05 23:44 ` Roland Dreier
@ 2008-09-13 19:13 ` Jeff Garzik
1 sibling, 0 replies; 3+ messages in thread
From: Jeff Garzik @ 2008-09-13 19:13 UTC (permalink / raw)
To: akpm; +Cc: netdev, davem
akpm@linux-foundation.org wrote:
> From: Andrew Morton <akpm@linux-foundation.org>
>
> sparc32 allmodconfig with linux-next:
>
> drivers/net/mlx4/alloc.c: In function 'mlx4_buf_alloc':
> drivers/net/mlx4/alloc.c:164: error: 'PAGE_KERNEL' undeclared (first use in this function)
> drivers/net/mlx4/alloc.c:164: error: (Each undeclared identifier is reported only once
> drivers/net/mlx4/alloc.c:164: error: for each function it appears in.)
>
> this is due to some header shuffle in linux-next. I didn't look to see what
> it was. I'd sugges that this patch be merged ahead of a linux-next merge to
> avoid bisection breaks.
>
> We strictly only need asm/pgtable.h, but going direct to asm includes always
> seems grubby.
>
> Cc: Jeff Garzik <jeff@garzik.org>
> Cc: "David S. Miller" <davem@davemloft.net>
> Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
> ---
>
> drivers/net/mlx4/alloc.c | 1 +
> 1 file changed, 1 insertion(+)
applied
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2008-09-13 19:13 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-09-05 21:04 [patch for 2.6.27? 4/6] drivers/net/mlx4/alloc.c needs mm.h akpm
2008-09-05 23:44 ` Roland Dreier
2008-09-13 19:13 ` Jeff Garzik
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).