From owner-svn-src-all@FreeBSD.ORG Sat Dec 17 15:27:57 2011 Return-Path: Delivered-To: svn-src-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 2BEA0106564A; Sat, 17 Dec 2011 15:27:57 +0000 (UTC) (envelope-from bzeeb-lists@lists.zabbadoz.net) Received: from mx1.sbone.de (mx1.sbone.de [IPv6:2a01:4f8:130:3ffc::401:25]) by mx1.freebsd.org (Postfix) with ESMTP id 6885F8FC0A; Sat, 17 Dec 2011 15:27:56 +0000 (UTC) Received: from mail.sbone.de (mail.sbone.de [IPv6:fde9:577b:c1a9:31::2013:587]) (using TLSv1 with cipher ADH-CAMELLIA256-SHA (256/256 bits)) (No client certificate requested) by mx1.sbone.de (Postfix) with ESMTPS id A24D725D37C7; Sat, 17 Dec 2011 15:27:54 +0000 (UTC) Received: from content-filter.sbone.de (content-filter.sbone.de [IPv6:fde9:577b:c1a9:31::2013:2742]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by mail.sbone.de (Postfix) with ESMTPS id BA9B4BD732F; Sat, 17 Dec 2011 15:27:53 +0000 (UTC) X-Virus-Scanned: amavisd-new at sbone.de Received: from mail.sbone.de ([IPv6:fde9:577b:c1a9:31::2013:587]) by content-filter.sbone.de (content-filter.sbone.de [fde9:577b:c1a9:31::2013:2742]) (amavisd-new, port 10024) with ESMTP id 7y2QYu5HKdZG; Sat, 17 Dec 2011 15:27:51 +0000 (UTC) Received: from orange-en1.sbone.de (orange-en1.sbone.de [IPv6:fde9:577b:c1a9:31:cabc:c8ff:fecf:e8e3]) (using TLSv1 with cipher AES128-SHA (128/128 bits)) (No client certificate requested) by mail.sbone.de (Postfix) with ESMTPSA id 99B16BD732E; Sat, 17 Dec 2011 15:27:51 +0000 (UTC) Mime-Version: 1.0 (Apple Message framework v1084) Content-Type: text/plain; charset=us-ascii From: "Bjoern A. Zeeb" In-Reply-To: <20111212130849.GA24560@onelab2.iet.unipi.it> Date: Sat, 17 Dec 2011 15:27:51 +0000 Content-Transfer-Encoding: quoted-printable Message-Id: <2C809F6E-4D03-4744-966B-E67E991A18C7@lists.zabbadoz.net> References: <201112111846.pBBIkE65064248@svn.freebsd.org> <20111212130849.GA24560@onelab2.iet.unipi.it> To: Luigi Rizzo X-Mailer: Apple Mail (2.1084) Cc: Jack F Vogel , svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org Subject: Re: svn commit: r228415 - head/sys/dev/e1000 X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 17 Dec 2011 15:27:57 -0000 On 12. Dec 2011, at 13:08 , Luigi Rizzo wrote: > On Sun, Dec 11, 2011 at 06:46:14PM +0000, Jack F Vogel wrote: >> Author: jfv >> Date: Sun Dec 11 18:46:14 2011 >> New Revision: 228415 >> URL: http://svn.freebsd.org/changeset/base/228415 >>=20 >> Log: >> Last change still had an issue, one more time... >=20 > Jack, > the previous commit (rev. 228387) removed netmap support from if_igb.c > I believe it was unintentional. Can you put it back, or do you want > me to handle it ? >=20 I am not sure why yet (and it could be someone broke the "clean" parts = of the build system) but I am seeing this and cannot remember reports = the last days: In file included from = /scratch/tmp/bz/head.universe/sys/modules/em/../../dev/e1000/if_em.c:400: @/dev/netmap/if_em_netmap.h: In function 'em_netmap_rxsync': @/dev/netmap/if_em_netmap.h:332: warning: dereferencing 'void *' pointer @/dev/netmap/if_em_netmap.h:332: error: request for member 'dt_mt' in = something not a structure or union for a LINT build on sparc64. > cheers > luigi >=20 >> Modified: >> head/sys/dev/e1000/if_em.c >> head/sys/dev/e1000/if_igb.c >>=20 >> Modified: head/sys/dev/e1000/if_em.c >> = =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D >> --- head/sys/dev/e1000/if_em.c Sun Dec 11 17:32:37 2011 = (r228414) >> +++ head/sys/dev/e1000/if_em.c Sun Dec 11 18:46:14 2011 = (r228415) >> @@ -1045,8 +1045,8 @@ em_ioctl(struct ifnet *ifp, u_long comma >> struct ifreq *ifr =3D (struct ifreq *)data; >> #if defined(INET) || defined(INET6) >> struct ifaddr *ifa =3D (struct ifaddr *)data; >> - bool avoid_reset =3D FALSE; >> #endif >> + bool avoid_reset =3D FALSE; >> int error =3D 0; >>=20 >> if (adapter->in_detach) >> @@ -1062,7 +1062,6 @@ em_ioctl(struct ifnet *ifp, u_long comma >> if (ifa->ifa_addr->sa_family =3D=3D AF_INET6) >> avoid_reset =3D TRUE; >> #endif >> -#if defined(INET) || defined(INET6) >> /* >> ** Calling init results in link renegotiation, >> ** so we avoid doing it when possible. >> @@ -1071,11 +1070,12 @@ em_ioctl(struct ifnet *ifp, u_long comma >> ifp->if_flags |=3D IFF_UP; >> if (!(ifp->if_drv_flags & IFF_DRV_RUNNING)) >> em_init(adapter); >> +#ifdef INET >> if (!(ifp->if_flags & IFF_NOARP)) >> arp_ifinit(ifp, ifa); >> +#endif >> } else >> error =3D ether_ioctl(ifp, command, data); >> -#endif >> break; >> case SIOCSIFMTU: >> { >>=20 >> Modified: head/sys/dev/e1000/if_igb.c >> = =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D >> --- head/sys/dev/e1000/if_igb.c Sun Dec 11 17:32:37 2011 = (r228414) >> +++ head/sys/dev/e1000/if_igb.c Sun Dec 11 18:46:14 2011 = (r228415) >> @@ -992,8 +992,8 @@ igb_ioctl(struct ifnet *ifp, u_long comm >> struct ifreq *ifr =3D (struct ifreq *)data; >> #if defined(INET) || defined(INET6) >> struct ifaddr *ifa =3D (struct ifaddr *)data; >> - bool avoid_reset =3D FALSE; >> #endif >> + bool avoid_reset =3D FALSE; >> int error =3D 0; >>=20 >> if (adapter->in_detach) >> @@ -1009,7 +1009,6 @@ igb_ioctl(struct ifnet *ifp, u_long comm >> if (ifa->ifa_addr->sa_family =3D=3D AF_INET6) >> avoid_reset =3D TRUE; >> #endif >> -#if defined(INET) || defined(INET6) >> /* >> ** Calling init results in link renegotiation, >> ** so we avoid doing it when possible. >> @@ -1018,11 +1017,12 @@ igb_ioctl(struct ifnet *ifp, u_long comm >> ifp->if_flags |=3D IFF_UP; >> if (!(ifp->if_drv_flags & IFF_DRV_RUNNING)) >> igb_init(adapter); >> +#ifdef INET >> if (!(ifp->if_flags & IFF_NOARP)) >> arp_ifinit(ifp, ifa); >> +#endif >> } else >> error =3D ether_ioctl(ifp, command, data); >> -#endif >> break; >> case SIOCSIFMTU: >> { --=20 Bjoern A. Zeeb You have to have visions! Stop bit received. Insert coin for new address family.