From owner-svn-src-head@freebsd.org Thu Aug 20 14:19:51 2015 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 6871E9BF235; Thu, 20 Aug 2015 14:19:51 +0000 (UTC) (envelope-from melifaro@ipfw.ru) Received: from forward2h.mail.yandex.net (forward2h.mail.yandex.net [IPv6:2a02:6b8:0:f05::2]) by mx1.freebsd.org (Postfix) with ESMTP id 256C2775; Thu, 20 Aug 2015 14:19:51 +0000 (UTC) (envelope-from melifaro@ipfw.ru) Received: from web13h.yandex.ru (web13h.yandex.ru [84.201.186.42]) by forward2h.mail.yandex.net (Yandex) with ESMTP id 6E52A701196; Thu, 20 Aug 2015 17:19:48 +0300 (MSK) Received: from 127.0.0.1 (localhost [127.0.0.1]) by web13h.yandex.ru (Yandex) with ESMTP id 434AE23C1A19; Thu, 20 Aug 2015 17:19:47 +0300 (MSK) Received: by web13h.yandex.ru with HTTP; Thu, 20 Aug 2015 17:19:46 +0300 From: Alexander V. Chernikov Envelope-From: melifaro@ipfw.ru To: Bjoern A. Zeeb , George Neville-Neil Cc: "src-committers@freebsd.org" , "svn-src-all@freebsd.org" , "svn-src-head@freebsd.org" In-Reply-To: References: <201508201205.t7KC5IoT012658@repo.freebsd.org> <46D09C16-85BD-4E99-8600-16280AC9397A@neville-neil.com> Subject: Re: svn commit: r286955 - in head/sys: net netinet netinet6 MIME-Version: 1.0 Message-Id: <129231440080386@web13h.yandex.ru> X-Mailer: Yamail [ http://yandex.ru ] 5.0 Date: Thu, 20 Aug 2015 17:19:46 +0300 Content-Transfer-Encoding: 8bit Content-Type: text/plain; charset=utf-8 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, 20 Aug 2015 14:19:51 -0000 20.08.2015, 16:29, "Bjoern A. Zeeb" : >>  On 20 Aug 2015, at 13:13 , George Neville-Neil wrote: >> >>  Why was this work not in Phabricator? This is a large change that has not been reviewed, >>  as far as I can tell, by anyone else on the project. I am tempted to ask that this be backed out >>  and reviewed BEFORE it goes into the tree. > > I probably wouldn’t go as far as backing out. > > Given this seems part of a larger change that seem to be streamed into HEAD and I have probably missed a posting or heads up or something on net@ [if someone could point me at that it would be very helpful], I appreciate things being broken into smaller reviewable bits. I have no clear idea of what the end goal is or will be or look like, which makes it hard to comment on design decisions. This is really the part of large change. The posting itself is here: https://wiki.freebsd.org/ProjectsRoutingProposal It was actually implemented (in slightly different way) in projects/routing branch. Unfortunately, I had to postpone that until recently. HEAD changed significantly since then, so merging is a bit complex. I've got routing part merged (and I'll update projects/routing branch within a few days). However, there are bunch of "general" lle changes, related to code "cleanup". So, my plan was to merge these, update projects/routing branch so anyone can clearly see the difference on fresh HEAD and create a bunch of reviews with proposed locking / timer changes for lle (and do the same for routing changes). But currently we're here.. > > I however, as I have stated in the past, would really really appreciate each change in PB, with the proposed commit message, and allow a few days of review or for people to comment on. I have spend two years on fixing bugs on the previous rewrite of that code. I am not exactly looking forward to see history repeat itself. I see no problem in sending changes there for design changes and I actually do prefer PB for that. I actually not against using PB for everything, but here we come very close to exact definition of "a few days". My intentions on the last commits were actually to ease the review of upcoming stuff, not to push-everything-without-review. > > Thanks, > Bjoern