From owner-svn-src-stable-9@FreeBSD.ORG Sat Jul 6 20:12:35 2013 Return-Path: Delivered-To: svn-src-stable-9@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by hub.freebsd.org (Postfix) with ESMTP id 6559A3CD; Sat, 6 Jul 2013 20:12:35 +0000 (UTC) (envelope-from kostikbel@gmail.com) Received: from kib.kiev.ua (kib.kiev.ua [IPv6:2001:470:d5e7:1::1]) by mx1.freebsd.org (Postfix) with ESMTP id CBD4112F9; Sat, 6 Jul 2013 20:12:34 +0000 (UTC) Received: from tom.home (kostik@localhost [127.0.0.1]) by kib.kiev.ua (8.14.7/8.14.7) with ESMTP id r66KCTS6018264; Sat, 6 Jul 2013 23:12:29 +0300 (EEST) (envelope-from kostikbel@gmail.com) DKIM-Filter: OpenDKIM Filter v2.8.3 kib.kiev.ua r66KCTS6018264 Received: (from kostik@localhost) by tom.home (8.14.7/8.14.7/Submit) id r66KCTC0018263; Sat, 6 Jul 2013 23:12:29 +0300 (EEST) (envelope-from kostikbel@gmail.com) X-Authentication-Warning: tom.home: kostik set sender to kostikbel@gmail.com using -f Date: Sat, 6 Jul 2013 23:12:29 +0300 From: Konstantin Belousov To: Glen Barber Subject: Re: svn commit: r252835 - stable/9/sys/netinet Message-ID: <20130706201229.GB91021@kib.kiev.ua> References: <201307052011.r65KBRCG063754@svn.freebsd.org> <20130706192146.GL20631@glenbarber.us> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="UXNJ9ANe8MmuJ4dV" Content-Disposition: inline In-Reply-To: <20130706192146.GL20631@glenbarber.us> User-Agent: Mutt/1.5.21 (2010-09-15) X-Spam-Status: No, score=-2.0 required=5.0 tests=ALL_TRUSTED,BAYES_00, DKIM_ADSP_CUSTOM_MED,FREEMAIL_FROM,NML_ADSP_CUSTOM_MED autolearn=no version=3.3.2 X-Spam-Checker-Version: SpamAssassin 3.3.2 (2011-06-06) on tom.home Cc: svn-src-stable@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, Andre Oppermann , svn-src-stable-9@freebsd.org X-BeenThere: svn-src-stable-9@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: SVN commit messages for only the 9-stable src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 06 Jul 2013 20:12:35 -0000 --UXNJ9ANe8MmuJ4dV Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Sat, Jul 06, 2013 at 03:21:46PM -0400, Glen Barber wrote: > On Fri, Jul 05, 2013 at 08:11:27PM +0000, Andre Oppermann wrote: > > Author: andre > > Date: Fri Jul 5 20:11:27 2013 > > New Revision: 252835 > > URL: http://svnweb.freebsd.org/changeset/base/252835 > >=20 > > Log: > > MFC r226433: > > =20 > > Update the comment and description of tcp_sendspace and tcp_recvspace > > to better reflect their purpose. > > =20 > > MFC r226437: > > =20 > > VNET virtualize tcp_sendspace/tcp_recvspace and change the > > type to INT. A long is not necessary as the TCP window is > > limited to 2**30. A larger initial window isn't useful. > > =20 > > MFC r226448: > > =20 > > Move the tcp_sendspace and tcp_recvspace sysctl's from > > the middle of tcp_usrreq.c to the top of tcp_output.c > > and tcp_input.c respectively next to the socket buffer > > autosizing controls. > > =20 > > MFC r227034: > > =20 > > Restore sysctl names for tcp_sendspace/tcp_recvspace. > > =20 > > They seem to be changed unintentionally in r226437, and there were no > > any mentions of renaming in commit log message. > > =20 > > Reported by: Anton Yuzhaninov > >=20 > > Modified: > > stable/9/sys/netinet/tcp_input.c > > stable/9/sys/netinet/tcp_output.c > > stable/9/sys/netinet/tcp_usrreq.c > > stable/9/sys/netinet/tcp_var.h > > Directory Properties: > > stable/9/sys/ (props changed) > >=20 > > Modified: stable/9/sys/netinet/tcp_input.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 > > --- stable/9/sys/netinet/tcp_input.c Fri Jul 5 20:01:07 2013 (r252834) > > +++ stable/9/sys/netinet/tcp_input.c Fri Jul 5 20:11:27 2013 (r252835) > > @@ -194,6 +194,11 @@ SYSCTL_VNET_INT(_net_inet_tcp, OID_AUTO, > > &VNET_NAME(tcp_insecure_rst), 0, > > "Follow the old (insecure) criteria for accepting RST packets"); > > =20 > > +VNET_DEFINE(int, tcp_recvspace) =3D 1024*64 > > +#define V_tcp_recvspace VNET(tcp_recvspace) > > +SYSCTL_VNET_INT(_net_inet_tcp, TCPCTL_RECVSPACE, recvspace, CTLFLAG_RW, > > + &VNET_NAME(tcp_recvspace), 0, "Initial receive socket buffer size"= ); > > + > > VNET_DEFINE(int, tcp_do_autorcvbuf) =3D 1; > > #define V_tcp_do_autorcvbuf VNET(tcp_do_autorcvbuf) > > SYSCTL_VNET_INT(_net_inet_tcp, OID_AUTO, recvbuf_auto, CTLFLAG_RW, >=20 > I think this breaks the LINT build. >=20 > cc -c -O2 -frename-registers -pipe -fno-strict-aliasing -std=3Dc99 -Wall > -Wredundant-decls -Wnested-externs -Wstrict-prototypes > -Wmissing-prototypes -Wpointer-arith -Winline -Wcast-qual -Wundef > -Wno-pointer-sign -fformat-extensions -Wmissing-include-dirs > -fdiagnostics-show-option -nostdinc -I. -I/src/sys > -I/src/sys/contrib/altq -D_KERNEL -DHAVE_KERNEL_OPTION_HEADERS -include > opt_global.h -fno-common -finline-limit=3D8000 --param > inline-unit-growth=3D100 --param large-function-growth=3D1000 -DGPROF > -falign-functions=3D16 -DGPROF4 -DGUPROF -fno-builtin > -fno-omit-frame-pointer -mcmodel=3Dkernel -mno-red-zone -mno-mmx -mno-sse > -msoft-float -fno-asynchronous-unwind-tables -ffreestanding > -fstack-protector -Werror -pg -mprofiler-epilogue > /src/sys/netinet/tcp_input.c > /src/sys/netinet/tcp_input.c:199: error: expected ',' or ';' before 'stat= ic' > /src/sys/netinet/tcp_input.c:199: error: > 'sysctl___net_inet_tcp_recvspace' undeclared here (not in a function) > *** Error code 1 >=20 > Stop in /obj/amd64.amd64/src/sys/LINT-VIMAGE. > *** Error code 1 Hopefully fixed by r252893. --UXNJ9ANe8MmuJ4dV Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.20 (FreeBSD) iQIcBAEBAgAGBQJR2HotAAoJEJDCuSvBvK1B5RsP/3+4hQr5+Wa8fTyg2v1UgY3U DdzKN5utaR/xZuv2TRWOHmGhBqPLwPhkVRV1tPl5+w4pQccsjIQxoKsN/WZNGx9M H3XD2nNzaXUdWZoeAeho0XBTsp/xywTcJkrGoM9D7VLZFCUGSNEiG4y68E2V3MV2 Dm3R6fq9iq/mTAARC3jw0N2VF6XODrtOGmyKHSshBfiXOSfIfnnS2ydd6JFaTt/d wOBLPphlDDWPZK1ef6rH5QMZc38jOtT9nIK1DqReH9X/mQGzB3GZlqG/riMpcLDp ff3TUTjgkFgfT8fuQx8Ho/4INEjcPWTb5zS8OYrTLEum8Q6MRod+vwJ9911ShUl7 LRazLLmcBlRNYhAX0ji1ycvkNkXClIXR3AE2MHLWIjE0Tu8e4pSwcaJ+LqUoqcx2 rHUQUabiJtj64Rr1czqG/Q1QU+2TJJkJSPQ9wDx9p9GUxEl8HL2IQeU3LYijUCew 0w2qcLJHwFLSnCvy9w5vjxaUUqKpHkF4/6gbFqzlnrHVOPBg2Pn/AyxbwoXBWmDG TPI/nCTA5V1YXSCyl1/39G+7Z2s6OGeEHOGMoI1i1KBZRU8Dj3hcyDYjgDYieFa4 cwDVAXlp+mCx2SiSmeXXO+c0yX4XeZdEUQ1FuR8y7xw4mddl1/XKOGoW1EJyJJ/z 070mUWbClTCSv54pID5g =5X+i -----END PGP SIGNATURE----- --UXNJ9ANe8MmuJ4dV--