diff mbox

[1/1] Add missing include into include/linux/netdevice.h

Message ID 1238150213-30416-1-git-send-email-dmitri.vorobiev@movial.com
State Accepted, archived
Delegated to: David Miller
Headers show

Commit Message

Dmitri Vorobiev March 27, 2009, 10:36 a.m. UTC
The inline function skb_gro_mac_header defined in include/linux/netdevice.h
makes use of page_address(). Depending on configuration options, the latter
is either defined as a macro or is declared as a function in another header
file, namely include/linux/mm.h. However, include/linux/netdevice.h does not
include include/linux/mm.h.

On MIPS, this has produced the following build error:

  CC      kernel/sysctl_check.o
In file included from include/linux/icmpv6.h:173,
                 from include/linux/ipv6.h:208,
                 from include/net/ip_vs.h:26,
                 from kernel/sysctl_check.c:6:
include/linux/netdevice.h: In function 'skb_gro_mac_header':
include/linux/netdevice.h:1132: error: implicit declaration of function
'page_address'
include/linux/netdevice.h:1133: warning: pointer/integer type mismatch
in conditional expression
make[1]: *** [kernel/sysctl_check.o] Error 1
make: *** [kernel] Error 2

The patch adds the missing include and fixes the build error.

Signed-off-by: Dmitri Vorobiev <dmitri.vorobiev@movial.com>
---
 include/linux/netdevice.h |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

Comments

David Miller March 27, 2009, 10:55 p.m. UTC | #1
From: Dmitri Vorobiev <dmitri.vorobiev@movial.com>
Date: Fri, 27 Mar 2009 12:36:53 +0200

> The inline function skb_gro_mac_header defined in include/linux/netdevice.h
> makes use of page_address(). Depending on configuration options, the latter
> is either defined as a macro or is declared as a function in another header
> file, namely include/linux/mm.h. However, include/linux/netdevice.h does not
> include include/linux/mm.h.
> 
> On MIPS, this has produced the following build error:

Applied, thanks.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/include/linux/netdevice.h b/include/linux/netdevice.h
index be3ebd7..1b55952 100644
--- a/include/linux/netdevice.h
+++ b/include/linux/netdevice.h
@@ -32,6 +32,7 @@ 
 #ifdef __KERNEL__
 #include <linux/timer.h>
 #include <linux/delay.h>
+#include <linux/mm.h>
 #include <asm/atomic.h>
 #include <asm/cache.h>
 #include <asm/byteorder.h>