netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 1/2] enic: Fix build without _HAVE_ARCH_IPV6_CSUM
@ 2008-10-13 20:35 Roland Dreier
  2008-10-14  1:51 ` David Miller
  0 siblings, 1 reply; 2+ messages in thread
From: Roland Dreier @ 2008-10-13 20:35 UTC (permalink / raw)
  To: jeff; +Cc: scofeldm, netdev

From: Scott Feldman <scofeldm@cisco.com>

On architectures (such as powerpc) that don't define
_HAVE_ARCH_IPV6_CSUM, the enic compile fails with

    drivers/net/enic/enic_main.c: In function 'enic_queue_wq_skb_tso':
    drivers/net/enic/enic_main.c:575: error: implicit declaration of function 'csum_ipv6_magic'

Fix this by including <net/ip6_checksum.h>

Signed-off-by: Roland Dreier <rolandd@cisco.com>
---
Jeff, this is part of a patch Scott sent a while ago
(https://kerneltrap.org/mailarchive/linux-netdev/2008/9/20/3350614/thread)
that seems to have gotten dropped.  Please apply.

 drivers/net/enic/enic_main.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/drivers/net/enic/enic_main.c b/drivers/net/enic/enic_main.c
index f3a47a8..80119c9 100644
--- a/drivers/net/enic/enic_main.c
+++ b/drivers/net/enic/enic_main.c
@@ -35,6 +35,8 @@
 #include <linux/ipv6.h>
 #include <linux/tcp.h>
 
+#include <net/ip6_checksum.h>
+
 #include "cq_enet_desc.h"
 #include "vnic_dev.h"
 #include "vnic_intr.h"

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH 1/2] enic: Fix build without _HAVE_ARCH_IPV6_CSUM
  2008-10-13 20:35 [PATCH 1/2] enic: Fix build without _HAVE_ARCH_IPV6_CSUM Roland Dreier
@ 2008-10-14  1:51 ` David Miller
  0 siblings, 0 replies; 2+ messages in thread
From: David Miller @ 2008-10-14  1:51 UTC (permalink / raw)
  To: rdreier; +Cc: jeff, scofeldm, netdev

From: Roland Dreier <rdreier@cisco.com>
Date: Mon, 13 Oct 2008 13:35:22 -0700

> From: Scott Feldman <scofeldm@cisco.com>
> 
> On architectures (such as powerpc) that don't define
> _HAVE_ARCH_IPV6_CSUM, the enic compile fails with
> 
>     drivers/net/enic/enic_main.c: In function 'enic_queue_wq_skb_tso':
>     drivers/net/enic/enic_main.c:575: error: implicit declaration of function 'csum_ipv6_magic'
> 
> Fix this by including <net/ip6_checksum.h>
> 
> Signed-off-by: Roland Dreier <rolandd@cisco.com>

I already received a fix for this seperately in my tree from
Kamalesh Babulal.

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2008-10-14  1:52 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-10-13 20:35 [PATCH 1/2] enic: Fix build without _HAVE_ARCH_IPV6_CSUM Roland Dreier
2008-10-14  1:51 ` David Miller

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).