From owner-svn-src-projects@FreeBSD.ORG Sun Aug 8 16:41:24 2010 Return-Path: Delivered-To: svn-src-projects@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 287041065677 for ; Sun, 8 Aug 2010 16:41:24 +0000 (UTC) (envelope-from kostikbel@gmail.com) Received: from mail.zoral.com.ua (mx0.zoral.com.ua [91.193.166.200]) by mx1.freebsd.org (Postfix) with ESMTP id B5CA38FC1A for ; Sun, 8 Aug 2010 16:41:23 +0000 (UTC) Received: from deviant.kiev.zoral.com.ua (root@deviant.kiev.zoral.com.ua [10.1.1.148]) by mail.zoral.com.ua (8.14.2/8.14.2) with ESMTP id o78GDh90068531 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Sun, 8 Aug 2010 19:13:43 +0300 (EEST) (envelope-from kostikbel@gmail.com) Received: from deviant.kiev.zoral.com.ua (kostik@localhost [127.0.0.1]) by deviant.kiev.zoral.com.ua (8.14.4/8.14.4) with ESMTP id o78GDhF2053256; Sun, 8 Aug 2010 19:13:43 +0300 (EEST) (envelope-from kostikbel@gmail.com) Received: (from kostik@localhost) by deviant.kiev.zoral.com.ua (8.14.4/8.14.4/Submit) id o78GDg21053255; Sun, 8 Aug 2010 19:13:42 +0300 (EEST) (envelope-from kostikbel@gmail.com) X-Authentication-Warning: deviant.kiev.zoral.com.ua: kostik set sender to kostikbel@gmail.com using -f Date: Sun, 8 Aug 2010 19:13:42 +0300 From: Kostik Belousov To: Attilio Rao Message-ID: <20100808161342.GF22295@deviant.kiev.zoral.com.ua> References: <201008081544.o78FihwX008357@svn.freebsd.org> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="LOZHMhvLB6lqiwkv" Content-Disposition: inline In-Reply-To: <201008081544.o78FihwX008357@svn.freebsd.org> User-Agent: Mutt/1.4.2.3i X-Virus-Scanned: clamav-milter 0.95.2 at skuns.kiev.zoral.com.ua X-Virus-Status: Clean X-Spam-Status: No, score=-2.3 required=5.0 tests=ALL_TRUSTED,AWL,BAYES_40, DNS_FROM_OPENWHOIS autolearn=no version=3.2.5 X-Spam-Checker-Version: SpamAssassin 3.2.5 (2008-06-10) on skuns.kiev.zoral.com.ua Cc: svn-src-projects@freebsd.org, src-committers@freebsd.org Subject: Re: svn commit: r211076 - projects/sv/sys/net X-BeenThere: svn-src-projects@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "SVN commit messages for the src " projects" tree" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 08 Aug 2010 16:41:24 -0000 --LOZHMhvLB6lqiwkv Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Sun, Aug 08, 2010 at 03:44:43PM +0000, Attilio Rao wrote: > Author: attilio > Date: Sun Aug 8 15:44:43 2010 > New Revision: 211076 > URL: http://svn.freebsd.org/changeset/base/211076 >=20 > Log: > - Compile netdump.debug.force just when NETDUMP_DEBUG in included > - Fix the usage of nd_force_crash which is not necessarilly to have > a static storage >=20 > Modified: > projects/sv/sys/net/netdump_client.c >=20 > Modified: projects/sv/sys/net/netdump_client.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 > --- projects/sv/sys/net/netdump_client.c Sun Aug 8 15:35:17 2010 (r21107= 5) > +++ projects/sv/sys/net/netdump_client.c Sun Aug 8 15:44:43 2010 (r21107= 6) > @@ -146,7 +146,9 @@ static int netdump_send_arp(void); > static void netdump_trigger(void *arg, int howto); > static int netdump_udp_output(struct mbuf *m); > =20 > +#ifdef NETDUMP_DEBUG > static int sysctl_force_crash(SYSCTL_HANDLER_ARGS); > +#endif > static int sysctl_ip(SYSCTL_HANDLER_ARGS); > static int sysctl_nic(SYSCTL_HANDLER_ARGS); > =20 > @@ -166,7 +168,6 @@ static int nd_enable =3D 0; /* if we shou > static struct in_addr nd_server =3D {INADDR_ANY}; /* server address */ > static struct in_addr nd_client =3D {INADDR_ANY}; /* client (our) addres= s */ > struct ifnet *nd_nic =3D NULL; > -static int nd_force_crash=3D0; > static int nd_polls=3D10000; /* Times to poll the NIC (0.5ms each poll) = before > * assuming packetloss occurred: 5s by default */ > static int nd_retries=3D10; /* Times to retransmit lost packets */ > @@ -292,16 +293,18 @@ sysctl_nic(SYSCTL_HANDLER_ARGS) > return error; > } > =20 > +#ifdef NETDUMP_DEBUG > static int > sysctl_force_crash(SYSCTL_HANDLER_ARGS)=20 > { > - int error; > + int error, force_crash; > =20 > - error =3D sysctl_handle_int(oidp, &nd_force_crash, nd_force_crash, req); > + force_crash =3D 0; > + error =3D sysctl_handle_int(oidp, &force_crash, force_crash, req); > if (error || req->newptr =3D=3D NULL) > return error; > =20 > - switch (nd_force_crash) { > + switch (force_crash) { > case 1: > printf("\nLivelocking system...\n"); > for (;;); > @@ -329,6 +332,7 @@ sysctl_force_crash(SYSCTL_HANDLER_ARGS)=20 > } > return 0; > } > +#endif > =20 > SYSCTL_NODE(_net, OID_AUTO, dump, CTLFLAG_RW, 0, "netdump"); > SYSCTL_PROC(_net_dump, OID_AUTO, server, CTLTYPE_STRING|CTLFLAG_RW, &nd_= server, > @@ -337,8 +341,6 @@ SYSCTL_PROC(_net_dump, OID_AUTO, client, > 0, sysctl_ip, "A", "dump client"); > SYSCTL_PROC(_net_dump, OID_AUTO, nic, CTLTYPE_STRING|CTLFLAG_RW, &nd_nic, > IFNAMSIZ, sysctl_nic, "A", "NIC to dump on"); > -SYSCTL_PROC(_net_dump, OID_AUTO, crash, CTLTYPE_INT|CTLFLAG_RW, > - 0, sizeof(nd_force_crash), sysctl_force_crash, "I", "force crashing"); > SYSCTL_INT(_net_dump, OID_AUTO, polls, CTLTYPE_INT|CTLFLAG_RW, &nd_polls= , 0, > "times to poll NIC per retry"); > SYSCTL_INT(_net_dump, OID_AUTO, retries, CTLTYPE_INT|CTLFLAG_RW, &nd_ret= ries, 0, > @@ -346,6 +348,10 @@ SYSCTL_INT(_net_dump, OID_AUTO, retries, > SYSCTL_INT(_net_dump, OID_AUTO, enable, CTLTYPE_INT|CTLFLAG_RW, &nd_enab= le, > 0, "enable network dump"); > TUNABLE_INT("net.dump.enable", &nd_enable); > +#ifdef NETDUMP_DEBUG > +SYSCTL_PROC(_net_dump, OID_AUTO, crash, CTLTYPE_INT|CTLFLAG_RW, 0, sizeo= f(int), > + sysctl_force_crash, "I", "force crashing"); > +#endif We have debug.kdb.panic and friends. Might be, it makes sense to move net.dump.crash there ? debug. looks as a more logical choice for most sysctls too. --LOZHMhvLB6lqiwkv Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.10 (FreeBSD) iEYEARECAAYFAkxe17YACgkQC3+MBN1Mb4j08ACgr9G78ZnXJGBXIEhVgGSS8vS4 J/wAnii3ugkydHeJn5el85/vVOIlQ/XR =Lgyc -----END PGP SIGNATURE----- --LOZHMhvLB6lqiwkv--