From owner-svn-src-head@freebsd.org Thu Jan 7 09:24:34 2016 Return-Path: Delivered-To: svn-src-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 9F86CA65B16; Thu, 7 Jan 2016 09:24:34 +0000 (UTC) (envelope-from fidaj@ukr.net) Received: from frv158.fwdcdn.com (frv158.fwdcdn.com [212.42.77.158]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 6296E1E56; Thu, 7 Jan 2016 09:24:34 +0000 (UTC) (envelope-from fidaj@ukr.net) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=ukr.net; s=fsm; h=Content-Transfer-Encoding:Content-Type:MIME-Version:References:In-Reply-To:Message-ID:Subject:Cc:To:From:Date; bh=mv9ztdeWCPLuAFidRQUD7lWJSY3vAUxL6XguTKJoIe8=; b=gt5OvezXZTJlb1zSKILe6iB6IguV2RYgYr8cG/XciryC/dNU79wuKvgyYMPONUtrnazBSc/CPIFAK9OP/Z3TFzTi9MKMn94g1AkvrY75/HZttTUqsBbrWtaj7WzIo978aFMt2dEJSpHGOZRzzciVn4ptiSs6GfseIMcLBwQkZOc=; Received: from [178.137.139.189] (helo=nonamehost.local) by frv158.fwdcdn.com with esmtpsa ID 1aH6o1-000LZL-Sn ; Thu, 07 Jan 2016 11:24:29 +0200 Date: Thu, 7 Jan 2016 11:24:24 +0200 From: Ivan Klymenko To: NGie Cooper Cc: Gleb Smirnoff , "svn-src-head@freebsd.org" , "svn-src-all@freebsd.org" , "src-committers@freebsd.org" Subject: Re: svn commit: r293284 - in head/sys: dev/cxgb/ulp/tom dev/cxgbe/tom netinet Message-ID: <20160107112424.701146a9@nonamehost.local> In-Reply-To: References: <201601070014.u070EgW1059880@repo.freebsd.org> X-Mailer: Claws Mail 3.13.1 (GTK+ 2.24.29; amd64-portbld-freebsd11.0) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Authentication-Result: IP=178.137.139.189; mail.from=fidaj@ukr.net; dkim=pass; header.d=ukr.net X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 07 Jan 2016 09:24:34 -0000 On Wed, 6 Jan 2016 18:20:50 -0800 NGie Cooper wrote: > On Wed, Jan 6, 2016 at 4:14 PM, Gleb Smirnoff > wrote: > > Author: glebius > > Date: Thu Jan 7 00:14:42 2016 > > New Revision: 293284 > > URL: https://svnweb.freebsd.org/changeset/base/293284 > > > > Log: > > Historically we have two fields in tcpcb to describe sender MSS: > > t_maxopd, and t_maxseg. This dualism emerged with T/TCP, but was > > not properly cleaned up after T/TCP removal. After all permutations > > over the years the result is that t_maxopd stores a minimum of peer > > offered MSS and MTU reduced by minimum protocol header. And > > t_maxseg stores (t_maxopd - TCPOLEN_TSTAMP_APPA) if timestamps are > > in action, or is equal to t_maxopd otherwise. That's a very rough > > estimate of MSS reduced by options length. Throughout the code it > > was used in places, where preciseness was not important, like cwnd > > or ssthresh calculations. > > > > With this change: > > > > - t_maxopd goes away. > > - t_maxseg now stores MSS not adjusted by options. > > - new function tcp_maxseg() is provided, that calculates MSS > > reduced by options length. The functions gives a better estimate, > > since it takes into account SACK state as well. > > > > Reviewed by: jtl > > Differential Revision: https://reviews.freebsd.org/D3593 > > This broke the build. > > From https://jenkins.freebsd.org/job/FreeBSD_HEAD_i386/2053/ : > > /usr/src/sys/modules/tcp/fastpath/../../../netinet/tcp_stacks/fastpath.c:481:6: > error: no member named 't_maxopd' in 'struct tcpcb' > --- all_subdir_sound --- > --- all_subdir_sb8 --- > ===> sound/driver/sb8 (all) > --- all_subdir_tcp/fastpath --- > if (DELAY_ACK(tp, tlen)) { > ^ ~~ > /usr/src/sys/modules/tcp/fastpath/../../../netinet/tcp_stacks/fastpath.c:167:19: > note: expanded from macro 'DELAY_ACK' > (tlen <= tp->t_maxopd) > && \ ^ > /usr/src/sys/modules/tcp/fastpath/../../../netinet/tcp_stacks/fastpath.c:606:8: > error: no member named 't_maxopd' in 'struct tcpcb' > if (DELAY_ACK(tp, tlen) && tlen != 0) > ^ ~~ > /usr/src/sys/modules/tcp/fastpath/../../../netinet/tcp_stacks/fastpath.c:167:19: > note: expanded from macro 'DELAY_ACK' > (tlen <= tp->t_maxopd) > && \ ^ > --- all_subdir_sound --- > --- all_subdir_sb16 --- > ctfconvert -L VERSION -g sb16.o > --- all_subdir_tcp/fastpath --- > /usr/src/sys/modules/tcp/fastpath/../../../netinet/tcp_stacks/fastpath.c:1545:8: > error: no member named 't_maxopd' in 'struct tcpcb' > if (DELAY_ACK(tp, tlen)) > ^ ~~ > /usr/src/sys/modules/tcp/fastpath/../../../netinet/tcp_stacks/fastpath.c:167:19: > note: expanded from macro 'DELAY_ACK' > (tlen <= tp->t_maxopd) > && \ ^ > 3 errors generated. > *** [fastpath.o] Error code 1 amd64 also ... --- dragon_saver.o --- /usr/local/libexec/ccache/world/cc -target x86_64-unknown-freebsd11.0 --sysroot=/media/da0s1/obj/usr/src/tmp -B/media/da0s1/obj/usr/src/tmp/usr/bin -O2 -pipe -march=native -fno-strict-aliasing -Werror -D_KERNEL -DKLD_MODULE -nostdinc -DHAVE_KERNEL_OPTION_HEADERS -include /media/da0s1/obj/usr/src/sys/mk11/opt_global.h -I. -I/usr/src/sys -fno-common -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -I/media/da0s1/obj/usr/src/sys/mk11 -mcmodel=kernel -mno-red-zone -mno-mmx -mno-sse -msoft-float -fno-asynchronous-unwind-tables -ffreestanding -fwrapv -fstack-protector -Wall -Wredundant-decls -Wnested-externs -Wstrict-prototypes -Wmissing-prototypes -Wpointer-arith -Winline -Wcast-qual -Wundef -Wno-pointer-sign -D__printf__=__freebsd_kprintf__ -Wmissing-include-dirs -fdiagnostics-show-option -Wno-unknown-pragmas -Wno-error-tautological-compare -Wno-error-empty-body -Wno-error-parentheses-equality -Wno-error-unused-function -Wno-error-pointer-sign -Wno-error-shift-negative-v alue -fvectorize -fslp-vectorize -fblocks -fcolor-diagnostics -mno-aes -mno-avx -std=iso9899:1999 -c /usr/src/sys/modules/syscons/dragon/../../../dev/syscons/dragon/dragon_saver.c -o dragon_saver.o --- all_subdir_tcp/fastpath --- /usr/src/sys/modules/tcp/fastpath/../../../netinet/tcp_stacks/fastpath.c:481:6: error: no member named 't_maxopd' in 'struct tcpcb' if (DELAY_ACK(tp, tlen)) { ^ ~~ /usr/src/sys/modules/tcp/fastpath/../../../netinet/tcp_stacks/fastpath.c:167:19: note: expanded from macro 'DELAY_ACK' (tlen <= tp->t_maxopd) && \ ^ /usr/src/sys/modules/tcp/fastpath/../../../netinet/tcp_stacks/fastpath.c:606:8: error: no member named 't_maxopd' in 'struct tcpcb' if (DELAY_ACK(tp, tlen) && tlen != 0) ^ ~~ /usr/src/sys/modules/tcp/fastpath/../../../netinet/tcp_stacks/fastpath.c:167:19: note: expanded from macro 'DELAY_ACK' (tlen <= tp->t_maxopd) && \ ^ /usr/src/sys/modules/tcp/fastpath/../../../netinet/tcp_stacks/fastpath.c:1545:8: error: no member named 't_maxopd' in 'struct tcpcb' if (DELAY_ACK(tp, tlen)) ^ ~~ /usr/src/sys/modules/tcp/fastpath/../../../netinet/tcp_stacks/fastpath.c:167:19: note: expanded from macro 'DELAY_ACK' (tlen <= tp->t_maxopd) && \ ^ 3 errors generated. *** [fastpath.o] Error code 1 make[4]: stopped in /usr/src/sys/modules/tcp/fastpath 1 error make[4]: stopped in /usr/src/sys/modules/tcp/fastpath *** [all_subdir_tcp/fastpath] Error code 2 make[3]: stopped in /usr/src/sys/modules --- all_subdir_sound --- A failure has been detected in another branch of the parallel make make[6]: stopped in /usr/src/sys/modules/sound/driver/ich *** [all_subdir_ich] Error code 2 make[5]: stopped in /usr/src/sys/modules/sound/driver 1 error make[5]: stopped in /usr/src/sys/modules/sound/driver *** [all_subdir_driver] Error code 2 make[4]: stopped in /usr/src/sys/modules/sound 1 error make[4]: stopped in /usr/src/sys/modules/sound *** [all_subdir_sound] Error code 2 make[3]: stopped in /usr/src/sys/modules --- all_subdir_syscons --- A failure has been detected in another branch of the parallel make make[5]: stopped in /usr/src/sys/modules/syscons/dragon *** [all_subdir_dragon] Error code 2 make[4]: stopped in /usr/src/sys/modules/syscons 1 error make[4]: stopped in /usr/src/sys/modules/syscons *** [all_subdir_syscons] Error code 2 make[3]: stopped in /usr/src/sys/modules --- all_subdir_sysvipc --- A failure has been detected in another branch of the parallel make make[5]: stopped in /usr/src/sys/modules/sysvipc/sysvshm *** [all_subdir_sysvshm] Error code 2 make[4]: stopped in /usr/src/sys/modules/sysvipc 1 error make[4]: stopped in /usr/src/sys/modules/sysvipc *** [all_subdir_sysvipc] Error code 2 make[3]: stopped in /usr/src/sys/modules 4 errors make[3]: stopped in /usr/src/sys/modules *** [modules-all] Error code 2 make[2]: stopped in /media/da0s1/obj/usr/src/sys/mk11 1 error make[2]: stopped in /media/da0s1/obj/usr/src/sys/mk11 *** [buildkernel] Error code 2 make[1]: stopped in /usr/src 1 error make[1]: stopped in /usr/src *** [buildkernel] Error code 2 make: stopped in /usr/src 1 error make: stopped in /usr/src