From owner-freebsd-net@FreeBSD.ORG Thu May 28 17:21:03 2015 Return-Path: Delivered-To: freebsd-net@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id D2EA81A2; Thu, 28 May 2015 17:21:03 +0000 (UTC) (envelope-from pkelsey@gmail.com) Received: from mail-yh0-x236.google.com (mail-yh0-x236.google.com [IPv6:2607:f8b0:4002:c01::236]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (Client CN "smtp.gmail.com", Issuer "Google Internet Authority G2" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 8DD8269B; Thu, 28 May 2015 17:21:03 +0000 (UTC) (envelope-from pkelsey@gmail.com) Received: by yhpn97 with SMTP id n97so12364834yhp.0; Thu, 28 May 2015 10:21:02 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:sender:in-reply-to:references:date:message-id:subject :from:to:cc:content-type; bh=rQNgLX2cmCuEujaHbIRlQUETz8PVQsnzy5ANVGc+CMM=; b=hsGYe34T/sz3PFZXSxYa4+O3a/tiyRdQO/UVinr87e21HM/Pt5d7Kbc5VzXFZ9GObE 8/3JUODC+dwBKXB9Hkkunfu79uMBss1RKhRsHnpHvrLfCQDQ/9OoB5C8fkR5QuRRW4ii 5HXw+a+xjEOeBCuz2Giw++pJIyhhKMISzLUyuKlzICBcOaGKQGihzUdTge6W7aVgMLMr LRKB6Ndc+BIBfIuy22MV6jVc2uPz1oSp9VpYLTZsjuy3dcikG3uc6nS3ohc9TrWJD3jn ZiQ4JKFMYDHZyZNte5LXmi0FT40Li7dB6X/+8JdNFPuVBtRRO4Pp56PfnCkzJBZ3QlwB jeQg== MIME-Version: 1.0 X-Received: by 10.236.198.138 with SMTP id v10mr3662219yhn.113.1432833662707; Thu, 28 May 2015 10:21:02 -0700 (PDT) Sender: pkelsey@gmail.com Received: by 10.13.201.71 with HTTP; Thu, 28 May 2015 10:21:02 -0700 (PDT) In-Reply-To: <20150528171853.GU95600@strugglingcoder.info> References: <20150528045551.GS95600@strugglingcoder.info> <55674C6B.9050700@delphij.net> <20150528171853.GU95600@strugglingcoder.info> Date: Thu, 28 May 2015 13:21:02 -0400 X-Google-Sender-Auth: Qt4FTilrX-bwvI23X_Y8abqVKtE Message-ID: Subject: Re: Looking for input on "locally patch tcpdump or merge in latest release from upstream?" From: Patrick Kelsey To: hiren panchasara Cc: d@delphij.net, "freebsd-net@freebsd.org" , delphij@freebsd.org Content-Type: text/plain; charset=UTF-8 X-Content-Filtered-By: Mailman/MimeDel 2.1.20 X-BeenThere: freebsd-net@freebsd.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: Networking and TCP/IP with FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 28 May 2015 17:21:03 -0000 Yes, I'll give it a go. I haven't yet done a dry-run to see how hairy it might be, so no predictions there. Thanks, Patrick On Thu, May 28, 2015 at 1:18 PM, hiren panchasara < hiren@strugglingcoder.info> wrote: > On 05/28/15 at 10:12P, Xin Li wrote: > > -----BEGIN PGP SIGNED MESSAGE----- > > Hash: SHA512 > > > > Hi, Hiren, > > > > On 05/27/15 21:55, hiren panchasara wrote: > > > On 05/28/15 at 12:40P, Patrick Kelsey wrote: > > >> Hi, > > >> > > >> I've had a patch for a capsicum-related issue in tcpdump sitting > > >> around since last September ( > > >> https://lists.freebsd.org/pipermail/freebsd-current/2014-September/05 > > 2049.html) > > >> > > >> > > that is still needed and that I want finally address in the tree (the pa > > tch > > >> was reviewed by rwatson@ and pjd@ back then). > > >> > > >> This issue was patched separately in the upstream tcpdump sources > > >> in February ( > > >> https://github.com/the-tcpdump-group/tcpdump/commit/887bf88fd058f8c0e > > f9a5af1a95b43753e3ad2eb), > > >> > > >> > > along with a refactor of the associated capsicum code, and that work has > > >> been present in tcpdump releases since 4.7.3 ( > > >> http://www.tcpdump.org/tcpdump-changes.txt). > > >> > > >> The last tcpdump release imported into the FreeBSD tree was 4.6.2 > > >> ( http://svnweb.freebsd.org/base/vendor/tcpdump/). > > >> > > >> tcpdump release import/merges have recently resulted in some > > >> confusion/lost local patches due to the extent of the diffs > > >> (e.g., the thread at > > >> https://lists.freebsd.org/pipermail/svn-src-head/2015-February/067853 > > .html). > > >> > > >> > > >> > > I see three possible ways to proceed: > > >> > > >> 1. Apply the minimal-local-diff patch from last September to our > > >> local tcpdump sources. This seems like it might contribute to a > > >> future difficult/lossy tcpdump vendor import/merge. > > >> > > >> 2. Import tcpdump 4.7.3 or later to address this issue. Are > > >> there any reasons why this might not be desired? I don't have a > > >> feel for when/why past tcpdump vendor imports have been performed > > >> or avoided. > > >> > > >> 3. Cherry-pick the upstream patch and apply it to our local > > >> sources, directly addressing only this issue and avoiding future > > >> tcpdump vendor import/merge problems related to this issue. > > >> > > >> I'm looking for input on the above. If left to my own devices, > > >> I'd go with (3). > > > > > > Latest upstream release is 4.7.4 and the one before that was 4.6.2 > > > which we already have in the tree. I think we should get latest > > > instead of picking bits and pieces when possible. > > > > > > CCing Xin for his input as he has been doing last few imports. > > > > Yes, I think we should do the new import. (Are you willing to do > > that? I'm doing it only because nobody else were doing it...) > > I think Patrick volunteered to do the import. If he can't, I'll do it. > > Cheers, > Hiren >