From owner-svn-src-user@freebsd.org Sat Mar 24 04:35:17 2018 Return-Path: Delivered-To: svn-src-user@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 5DBF3F6A98B for ; Sat, 24 Mar 2018 04:35:17 +0000 (UTC) (envelope-from freebsd@pdx.rh.CN85.dnsmgr.net) Received: from pdx.rh.CN85.dnsmgr.net (br1.CN84in.dnsmgr.net [69.59.192.140]) (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 A176977BD6; Sat, 24 Mar 2018 04:35:16 +0000 (UTC) (envelope-from freebsd@pdx.rh.CN85.dnsmgr.net) Received: from pdx.rh.CN85.dnsmgr.net (localhost [127.0.0.1]) by pdx.rh.CN85.dnsmgr.net (8.13.3/8.13.3) with ESMTP id w2O4ZBZf034491; Fri, 23 Mar 2018 21:35:11 -0700 (PDT) (envelope-from freebsd@pdx.rh.CN85.dnsmgr.net) Received: (from freebsd@localhost) by pdx.rh.CN85.dnsmgr.net (8.13.3/8.13.3/Submit) id w2O4ZBIn034490; Fri, 23 Mar 2018 21:35:11 -0700 (PDT) (envelope-from freebsd) From: "Rodney W. Grimes" Message-Id: <201803240435.w2O4ZBIn034490@pdx.rh.CN85.dnsmgr.net> Subject: Re: svn commit: r331461 - in user/markj/netdump/sys: kern netinet/netdump sys vm In-Reply-To: To: cem@freebsd.org Date: Fri, 23 Mar 2018 21:35:11 -0700 (PDT) CC: "Rodney W. Grimes" , src-committers , svn-src-user@freebsd.org Reply-To: rgrimes@freebsd.org X-Mailer: ELM [version 2.4ME+ PL121h (25)] MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset=US-ASCII X-BeenThere: svn-src-user@freebsd.org X-Mailman-Version: 2.1.25 Precedence: list List-Id: "SVN commit messages for the experimental " user" src tree" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 24 Mar 2018 04:35:17 -0000 > The code review link is https://reviewboard.west.isilon.com/r/67553/ . > It won't do you much good outside our corporate network, though. Then it should probably not be mentioned in a FreeBSD svn commit? > Here's a summary of the my feedback, which are pretty straightforward > style-y cleanups and could easily be read out of the diff itself: > > 1. Move external function declarations from two .c files to a header > (sys/mbuf.h). > 2. Annotate unused parameters with __unused. > 3. Replace unused M_FOO flags with 0 in mb_ctor_clust() invocation. > Since valid M_FOO values are non-zero, this should result in an > assertion if the APIs suddenly start using the passed flag value. > 4. Add an assertion that mutually exclusive flags to zone_ctor > (NOBUCKET | MAXBUCKET) are indeed mutually exclusive (NOBUCKET is new > to this branch). Thank you > The only change that isn't related to my feedback was: Drop trashing > netdump mbufs during zone import subroutine. > > There are some further review comments which may eventually end up in > this branch. The intent is to merge this branch to the main line at some point, so it is of benifit to have the commit messages here hold the same quality as if it was a commit to -current, correct? > Conrad > > On Fri, Mar 23, 2018 at 7:46 PM, Rodney W. Grimes > wrote: > >> Author: markj > >> Date: Fri Mar 23 20:29:34 2018 > >> New Revision: 331461 > >> URL: https://svnweb.freebsd.org/changeset/base/331461 > >> > >> Log: > >> Address some but not all review feedback from cem. > > > > Please be detailed in what is changing, as the above > > only tells us where it came from in a vague way as > > no review number is even cited. > > > > I know this is on a private branch, but when/if it > > is merged this becomes part of the main line. > -- Rod Grimes rgrimes@freebsd.org