Subject: kern/6793: Kernel does not link with recent ipip changes.
To: None <gnats-bugs@gnats.netbsd.org>
From: None <christos@zoulas.com>
List: netbsd-bugs
Date: 01/12/1999 08:27:40
>Number:         6793
>Category:       kern
>Synopsis:       -DMROUTING -UIPIP and kernel does not link
>Confidential:   no
>Severity:       non-critical
>Priority:       medium
>Responsible:    kern-bug-people (Kernel Bug People)
>State:          open
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Tue Jan 12 05:35:00 1999
>Last-Modified:
>Originator:     Christos Zoulas
>Organization:
	Astron Software
>Release:        1999012
>Environment:
System: NetBSD hrothgar.gw.com 1.3H NetBSD 1.3H (TAC-GENERIC) #2: Sun Nov 1 14:11:13 EST 1998 kim@hrothgar.gw.com:/net/hrothgar/src-1/NetBSD/cvsroot/src/sys/arch/i386/compile/TAC-GENERIC i386


>Description:
	Kernel does not build because in /usr/src/sys/conf/files:

		file netinet/ip_ipip.c ipip needs-count

	But we need ip_ipip.c for mrouting. The easy change:

		file netinet/ip_ipip.c ipip | mrouting needs-count

	does not work because mrouting is a flag, not a device.

>How-To-Repeat:
	define MROUTING and don't define IPIP in your kernel.

>Fix:
	I think that having needs-count lines should not belong in the files
	lines, but in the device and pseudo-device declarations. This way
	one can avoid repeating them ad-nauseum. Otherwise the syntax can
	be extended to accommodate (fopts flgs) and apply flgs to the
	appropriate fopts. Otherwise one can easily remove the error check
	from config. But that is ugly.
>Audit-Trail:
>Unformatted: