From owner-svn-src-all@freebsd.org Mon Apr 2 08:41:33 2018 Return-Path: Delivered-To: svn-src-all@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id ABF3AF7CDF1; Mon, 2 Apr 2018 08:41:33 +0000 (UTC) (envelope-from freebsd@omnilan.de) Received: from mx0.gentlemail.de (mx0.gentlemail.de [IPv6:2a00:e10:2800::a130]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 32B0874A02; Mon, 2 Apr 2018 08:41:32 +0000 (UTC) (envelope-from freebsd@omnilan.de) Received: from mh0.gentlemail.de (mh0.gentlemail.de [IPv6:2a00:e10:2800::a135]) by mx0.gentlemail.de (8.14.5/8.14.5) with ESMTP id w328fUo6043979; Mon, 2 Apr 2018 10:41:30 +0200 (CEST) (envelope-from freebsd@omnilan.de) Received: from titan.inop.mo1.omnilan.net (s1.omnilan.de [217.91.127.234]) (using TLSv1 with cipher ECDHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by mh0.gentlemail.de (Postfix) with ESMTPSA id 3B718372; Mon, 2 Apr 2018 10:41:29 +0200 (CEST) Message-ID: <5AC1ECB8.6050709@omnilan.de> Date: Mon, 02 Apr 2018 10:41:28 +0200 From: Harry Schmalzbauer Organization: OmniLAN User-Agent: Mozilla/5.0 (X11; U; FreeBSD i386; de-DE; rv:1.9.2.8) Gecko/20100906 Lightning/1.0b2 Thunderbird/3.1.2 MIME-Version: 1.0 To: Mark Johnston CC: svn-src-head@freebsd.org, Sean Bruno , src-committers@freebsd.org, glebius@freebsd.org, svn-src-all@freebsd.org Subject: Re: svn commit: r324508 - head/sys/kern References: <201710102221.v9AML554015782@repo.freebsd.org> <5ABF55FC.6030102@omnilan.de> <20180401164726.GA953@raichu> In-Reply-To: <20180401164726.GA953@raichu> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 8bit X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.2.7 (mx0.gentlemail.de [IPv6:2a00:e10:2800::a130]); Mon, 02 Apr 2018 10:41:30 +0200 (CEST) X-Milter: Spamilter (Reciever: mx0.gentlemail.de; Sender-ip: ; Sender-helo: mh0.gentlemail.de; ) X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.25 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: Mon, 02 Apr 2018 08:41:33 -0000 Bezüglich Mark Johnston's Nachricht vom 01.04.2018 18:47 (localtime): > On Sat, Mar 31, 2018 at 11:33:48AM +0200, Harry Schmalzbauer wrote: >> Bezüglich Sean Bruno's Nachricht vom 11.10.2017 00:21 (localtime): >>> Author: sbruno >>> Date: Tue Oct 10 22:21:05 2017 >>> New Revision: 324508 >>> URL: https://svnweb.freebsd.org/changeset/base/324508 >>> >>> Log: >>> match sendfile() error handling to send(). >>> >>> Sendfile() should match the error checking order of send() which >>> is currently: >>> >>> SBS_CANTSENDMORE >>> so_error >>> SS_ISCONNECTED >>> >>> Submitted by: Jason Eggleston >>> Reviewed by: glebius >>> MFC after: 2 weeks >>> Sponsored by: Limelight Networks >>> Differential Revision: https://reviews.freebsd.org/D12633 >>> >>> Modified: >>> head/sys/kern/kern_sendfile.c >>> >> >> I'm still applying this one locally to stable/11. >> Is it going to be MFCd before 11.2? >> >> There are a view more candidates: >> https://svnweb.freebsd.org/base?view=revision&revision=324601 along with >> r324448 >> https://svnweb.freebsd.org/base?view=revision&revision=327596 >> https://svnweb.freebsd.org/base?view=revision&revision=328977 >> These were flagged for MFC with expired defer time. >> And tomorrow this one was supposed to be MFCd also: >> https://svnweb.freebsd.org/base?view=revision&revision=331130 > > I think r324446 and r324448 need to be MFCed before most of these can go > in. I MFCed the other commits (r317567, r324508) that you asked about in > other threads. Thanks for MFCing the unrelated other two (r317567, r324508) from cem@! Hope someone finds time to sort out the dependiencies of r324446, r324448 and this r324508 along with r324601, r327596, r328977. Currently I don't have a sendfile() test case, but for some reason I stumbled across this MFCflagged commit some time ago and I guess 11.2 shouldn't ship without. Thanks, -harry