From owner-freebsd-current@FreeBSD.ORG Fri Aug 17 21:29:52 2012 Return-Path: Delivered-To: current@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id EE001106564A; Fri, 17 Aug 2012 21:29:51 +0000 (UTC) (envelope-from kob6558@gmail.com) Received: from mail-wg0-f50.google.com (mail-wg0-f50.google.com [74.125.82.50]) by mx1.freebsd.org (Postfix) with ESMTP id 1206C8FC0C; Fri, 17 Aug 2012 21:29:50 +0000 (UTC) Received: by wgbds11 with SMTP id ds11so3703627wgb.31 for ; Fri, 17 Aug 2012 14:29:43 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type; bh=HzxPJAV6e2xVyti7Ia4qFjlE/8Y8KTyRsYzsC7fsj4w=; b=Cc99uxehz4K9mqQHy0XedcaF4WsTF90ize7TKJ+jAMPAqw2kwYoiYZEssD3jBTyh1m oPkRRX2KJrk9/u3ddzEIs8Iv8Q/kx78Cys7JRx+wbCyoFedc+rAXOqpvdeMFrwcGxOlE W9sM2xxGFZ7wTqHuR5I+ElhW0Hi4ApWbbsOtCb8s+xixphtkh3FC29pqajIVkt6Vf/3B 2J2naxiLISJI5pjm/Mdlv5/9t218Impr+47tLl+i9Fhr+N2EUou6Hp/nenhTVfLLJpZM J+9YJ/RnFxYviwLY8J1eTHF/LTH6qzQwoynHPKjuFzuQ3ZCxUI05NZTxVZZrfyGMmi26 BSjw== MIME-Version: 1.0 Received: by 10.180.107.103 with SMTP id hb7mr8179365wib.3.1345238983838; Fri, 17 Aug 2012 14:29:43 -0700 (PDT) Received: by 10.223.63.76 with HTTP; Fri, 17 Aug 2012 14:29:43 -0700 (PDT) In-Reply-To: <1345223506.27688.116.camel@revolution.hippie.lan> References: <157941699.20120815004542@serebryakov.spb.ru> <502AE8B5.9090106@FreeBSD.org> <502B775D.7000101@FreeBSD.org> <1849591745.20120815144006@serebryakov.spb.ru> <1345139226.27688.48.camel@revolution.hippie.lan> <174138639.20120817143840@serebryakov.spb.ru> <1345215393.27688.85.camel@revolution.hippie.lan> <1345223506.27688.116.camel@revolution.hippie.lan> Date: Fri, 17 Aug 2012 14:29:43 -0700 Message-ID: From: Kevin Oberman To: Ian Lepore Content-Type: text/plain; charset=UTF-8 Cc: Adrian Chadd , current@freebsd.org Subject: Re: CURRENT as gateway on not-so-fast hardware: where is a bottlneck? X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 17 Aug 2012 21:29:52 -0000 On Fri, Aug 17, 2012 at 10:11 AM, Ian Lepore wrote: > On Fri, 2012-08-17 at 09:58 -0700, Adrian Chadd wrote: >> On 17 August 2012 07:56, Ian Lepore wrote: >> >> > That result actually matches my expectation... it fixed only a part of >> > your problem. I suspected (without very good evidence) that you may >> > have two unrelated problems; hopefully now that we've eliminated one the >> > other will be easier to find. >> > >> > I've submitted a PR with that patch attached, since it has now been >> > shown to fix a problem on two different sets of (similar) hardware: >> > >> > http://www.freebsd.org/cgi/query-pr.cgi?pr=170705 >> >> Hm, who's a good person to review this stuff? Maybe bde? >> > > No! Not bde! He'll notice that I violated style(9) by accidentally > leaving an extra blank line between a comment block and the function > definition. :) (There are probably more violations than that -- I did > this when I was first trying to come to grips with the differences > between style(9) and the almost-style(9) standards we use at work.) > > When I first proposed the changes, jhb remarked that they sounded good, > but as far as I know, nobody reviewed the actual diff when I posted it. > It looks like bde and phk were the primary maintainers back when this > code was being more actively worked on. Why not bde? Everyone needs to learn what the term "bruceification" means. Believe me, there IS good reason for programming style and almost everyone with a commit bit gets close. bde will provide a reminder of any of those things you forgot were in style(9). This is something we should appreciate, even if it does sting a bit. -- R. Kevin Oberman, Network Engineer E-mail: kob6558@gmail.com