From chi@bd.mbn.or.jp  Fri Dec  4 01:32:57 1998
Received: from bd.mbn.or.jp (bd.mbn.or.jp [202.217.0.43])
          by hub.freebsd.org (8.8.8/8.8.8) with ESMTP id BAA12466
          for <FreeBSD-gnats-submit@freebsd.org>; Fri, 4 Dec 1998 01:32:56 -0800 (PST)
          (envelope-from chi@bd.mbn.or.jp)
Received: from chino.localhost (cse5-34.sapporo.mbn.or.jp [202.217.111.58])
	by bd.mbn.or.jp (8.9.1/bd.mbn.or.jp-2.0) with ESMTP id SAA07552
	for <FreeBSD-gnats-submit@freebsd.org>; Fri, 4 Dec 1998 18:32:30 +0900 (JST)
Message-Id: <199812040932.SAA07552@bd.mbn.or.jp>
Date: Fri, 4 Dec 1998 18:32:30 +0900 (JST)
From: chi@bd.mbn.or.jp
Reply-To: chi@bd.mbn.or.jp
To: FreeBSD-gnats-submit@freebsd.org
Subject: lnc driver does not set IFF_MULTICAST 
X-Send-Pr-Version: 3.2

>Number:         8960
>Category:       kern
>Synopsis:       lnc driver does not set IFF_MULTICAST
>Confidential:   no
>Severity:       non-critical
>Priority:       medium
>Responsible:    freebsd-bugs
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Fri Dec  4 01:40:01 PST 1998
>Closed-Date:    Fri Oct 8 19:45:56 PDT 1999
>Last-Modified:  Fri Oct  8 19:48:00 PDT 1999
>Originator:     Chiharu Shibata
>Release:        FreeBSD 2.2.6-RELEASE i386
>Organization:
Japan FreeBSD Users Group
>Environment:

	all AMD PCnet based LAN board(lnc driver)

>Description:

	The lnc driver does not set IFF_MULTICAST flag
	even if "LNC_MULTICAST" is defined in /sys/i386/isa/if_lnc.c.

>How-To-Repeat:

	(1) Uncomment "#define LNC_MULTICAST" in /sys/i386/isa/if_lnc.c
	    or add "options LNC_MULTICAST" in kernel conf file.
	(2) Re-configure new kernel and boot.
	(3) Type "ifconfig -a".
	    >lnc0: flags=843<UP,BROADCAST,RUNNING,SIMPLEX> mtu 1500

	    This should be the following.
	    >lnc0: flags=8843<UP,BROADCAST,RUNNING,SIMPLEX,MULTICAST> mtu 1500

>Fix:
	
--- if_lnc.c.orig	Thu Dec  3 15:50:12 1998
+++ if_lnc.c	Thu Dec  3 15:50:41 1998
@@ -69,6 +69,11 @@
 /* Some defines that should really be in generic locations */
 #define FCS_LEN 4
 #define MULTICAST_FILTER_LEN 8
+#ifdef LNC_MULTICAST
+#define LNC_IFF_DEFAULT (IFF_BROADCAST | IFF_SIMPLEX | IFF_MULTICAST)
+#else
+#define LNC_IFF_DEFAULT (IFF_BROADCAST | IFF_SIMPLEX)
+#endif
 
 #include <sys/param.h>
 #include <sys/systm.h>
@@ -1272,7 +1277,7 @@
 	sc->arpcom.ac_if.if_name = lncdriver.name;
 	sc->arpcom.ac_if.if_unit = unit;
 	sc->arpcom.ac_if.if_mtu = ETHERMTU;
-	sc->arpcom.ac_if.if_flags = IFF_BROADCAST | IFF_SIMPLEX;
+	sc->arpcom.ac_if.if_flags = LNC_IFF_DEFAULT;
 	sc->arpcom.ac_if.if_timer = 0;
 	sc->arpcom.ac_if.if_output = ether_output;
 	sc->arpcom.ac_if.if_start = lnc_start;
@@ -1383,7 +1388,7 @@
 
 	s = splimp();
 	lnc_stop(sc);
-	sc->arpcom.ac_if.if_flags |= IFF_BROADCAST | IFF_SIMPLEX; /* XXX??? */
+	sc->arpcom.ac_if.if_flags |= LNC_IFF_DEFAULT; /* XXX??? */
 
 	/*
 	 * This sets up the memory area for the controller. Memory is set up for
>Release-Note:
>Audit-Trail:

From: chi@bd.mbn.or.jp (Chiharu Shibata)
To: FreeBSD-gnats-submit@FreeBSD.ORG
Cc:  
Subject: Re: kern/8960: request: remove this PR
Date: Sat, 9 Oct 1999 11:12:36 +0900 (JST)

 This is Chiharu Shibata, who sent this PR the other day.
 
 I'll request to cancel this PR.
 Because i386/if_lnc.c committed as rev1.53 has same effect.
 
 >Category:       kern
 >Responsible:    freebsd-bugs
 >Synopsis:       lnc driver does not set IFF_MULTICAST
 >Arrival-Date:   Fri Dec  4 01:40:01 PST 1998
 -- 
 Chiharu Shibata    chi@bd.mbn.or.jp    <http://plaza17.mbn.or.jp/~chi/>
                                                     Powered by FreeBSD.
 
State-Changed-From-To: open->closed 
State-Changed-By: chris 
State-Changed-When: Fri Oct 8 19:45:56 PDT 1999 
State-Changed-Why:  
This was fixed in revision 1.53 of sys/i386/isa/if_lnc.c. 
>Unformatted:
