Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 26 Oct 2009 21:09:35 +0100
From:      Ulrich =?utf-8?B?U3DDtnJsZWlu?= <uqs@spoerlein.net>
To:        hackers@freebsd.org
Subject:   in_cksum.h for sparc64 missing ifdef KERNEL?
Message-ID:  <20091026200935.GA13602@roadrunner.spoerlein.net>

next in thread | raw e-mail | index | archive | help

--uAKRQypu60I7Lcqm
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline

Hi,

while trying to cleanup some WARNS issues under sbin/, I noticed that
natd(8) fails to compile for sparc64 only, due to missing "struct
mbuf" declaration in in_cksum.h. Comparing that header to other arch's
headers leads me to believe an #ifdef is missing.

See attached patch, but please note that I do not have the resources to
run this through a make universe right now.

Regards,
Uli

--uAKRQypu60I7Lcqm
Content-Type: text/x-diff; charset=us-ascii
Content-Disposition: attachment; filename="cksum.diff"

1618[tip]   350410f79434   2009-10-26 21:02 +0100   uqs
  Put this under KERNEL just like all other arch do.

diff -r 5558195fb93d -r 350410f79434 sys/sparc64/include/in_cksum.h
--- a/sys/sparc64/include/in_cksum.h	Mon Oct 26 19:57:23 2009 +0100
+++ b/sys/sparc64/include/in_cksum.h	Mon Oct 26 21:02:06 2009 +0100
@@ -164,6 +164,8 @@
 	return (__ret);
 }
 
+#ifdef _KERNEL
 u_short	in_cksum_skip(struct mbuf *m, int len, int skip);
+#endif
 
 #endif /* _MACHINE_IN_CKSUM_H_ */


--uAKRQypu60I7Lcqm--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20091026200935.GA13602>