From owner-cvs-all@FreeBSD.ORG Thu May 12 22:30:33 2005 Return-Path: Delivered-To: cvs-all@freebsd.org Received: from green.homeunix.org (freefall.freebsd.org [216.136.204.21]) by hub.freebsd.org (Postfix) with ESMTP id A03DC16A4CE; Thu, 12 May 2005 22:30:33 +0000 (GMT) Received: from green.homeunix.org (green@localhost [127.0.0.1]) by green.homeunix.org (8.13.3/8.13.1) with ESMTP id j4CMUXiq012108; Thu, 12 May 2005 18:30:33 -0400 (EDT) (envelope-from green@green.homeunix.org) Received: (from green@localhost) by green.homeunix.org (8.13.3/8.13.1/Submit) id j4CMUXGt012107; Thu, 12 May 2005 18:30:33 -0400 (EDT) (envelope-from green) Date: Thu, 12 May 2005 18:30:32 -0400 From: Brian Fundakowski Feldman To: Gleb Smirnoff Message-ID: <20050512223032.GE2058@green.homeunix.org> References: <200505121511.j4CFBUlT076772@repoman.freebsd.org> <20050512151749.GC2058@green.homeunix.org> <20050512215825.GD43447@cell.sick.ru> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20050512215825.GD43447@cell.sick.ru> User-Agent: Mutt/1.5.6i cc: cvs-src@FreeBSD.org cc: src-committers@FreeBSD.org cc: cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sbin/ipfw ipfw.8 ipfw2.c src/sys/netinet ip_divert.c ip_fw.h ip_fw2.c X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 12 May 2005 22:30:34 -0000 On Fri, May 13, 2005 at 01:58:25AM +0400, Gleb Smirnoff wrote: > Brian, > > On Thu, May 12, 2005 at 11:17:49AM -0400, Brian Fundakowski Feldman wrote: > B> On Thu, May 12, 2005 at 03:11:30PM +0000, Brian Feldman wrote: > B> > green 2005-05-12 15:11:30 UTC > B> > > B> > FreeBSD src repository > B> > > B> > Modified files: (Branch: RELENG_5) > B> > sbin/ipfw ipfw.8 ipfw2.c > B> > sys/netinet ip_divert.c ip_fw.h ip_fw2.c > B> > Log: > B> > MFC: IPFW ALTQ(4) classification support, diverted traffic match rules, > B> > and the TCP packet data length match rule. > B> > > B> > Revision Changes Path > B> > 1.150.2.7 +55 -2 src/sbin/ipfw/ipfw.8 > B> > 1.54.2.5 +259 -33 src/sbin/ipfw/ipfw2.c > B> > 1.98.2.4 +16 -12 src/sys/netinet/ip_divert.c > B> > 1.89.2.4 +17 -3 src/sys/netinet/ip_fw.h > B> > 1.70.2.11 +92 -1 src/sys/netinet/ip_fw2.c > B> > B> Gleb, would you like to merge the subsequent ip_divert.c changes? > > Definitely, I would like! > > You have broken ng_ksocket operation in RELENG_5 the same way you did it in > HEAD. However, HEAD is for untested changes, so I saw no problem here. After > some silence from you, I have fixed it in rev. 1.109. > > However, RELENG_5 is for _tested_ changes, not for stepping on the same > errors again. Since you asked me for merge, this means that you knew about > breakage you are committing. > > Why didn't you incorporated 1.109 into MFC? > > If this moment is not clear to you, why didn't you send me > email _before_ committing? I usually respond quickly. > > What was the reason to rush in and break things in RELENG_5? > > P.S. I can't quickly merge it now, because it needs testing and I have no time > and testbox now at home at 2 PM. So this will be left unfixed until tomorrow. If > you want, you can test and commit it yourself. The testcase is the same I've sent > you long time ago: > > http://lists.freebsd.org/pipermail/freebsd-net/2004-November/005562.html I'm trying not to step on anyone's toes by merging "their" code unless it's a dire emergency. I noticed the issue in question after doing the MFC and someone else saying something orthogonal to it, then happening upon it in the CVS history. It can be difficult to figure out why someone did piecemeal merges, so after realizing what happened, and considering that you were the only one affected, I concluded that you would be the best person to sort it out. So, in any case, sorry for missing a part of the MFC that is essential to you. I would prefer it if you would do what you propose and fix it at your leisure, because it would take me a whole day just to figure out how to work with netgraph again enough to know what it is that I am verifying. It looks like you've already MFCed the rest of the changes that you had made just after those to ip_divert.c? -- Brian Fundakowski Feldman \'[ FreeBSD ]''''''''''\ <> green@FreeBSD.org \ The Power to Serve! \ Opinions expressed are my own. \,,,,,,,,,,,,,,,,,,,,,,\