netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [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).