From owner-freebsd-arch@freebsd.org Tue Mar 16 17:13:49 2021 Return-Path: Delivered-To: freebsd-arch@mailman.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.nyi.freebsd.org (Postfix) with ESMTP id 7CA205B5F32; Tue, 16 Mar 2021 17:13:49 +0000 (UTC) (envelope-from noloader@gmail.com) Received: from mail-io1-xd32.google.com (mail-io1-xd32.google.com [IPv6:2607:f8b0:4864:20::d32]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (2048 bits) client-digest SHA256) (Client CN "smtp.gmail.com", Issuer "GTS CA 1O1" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4F0Kd92zJzz4qTr; Tue, 16 Mar 2021 17:13:49 +0000 (UTC) (envelope-from noloader@gmail.com) Received: by mail-io1-xd32.google.com with SMTP id k2so38028522ioh.5; Tue, 16 Mar 2021 10:13:49 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:reply-to:from:date:message-id :subject:to:cc; bh=LdTU8JNN19De/6YG0m9exoj67s4UkjHWivqTBVMyxSY=; b=lzupkKG+FvaRp0PjWDQE//WskdEXBwWGqjcfk0s3qNm0UhClL6rVvkKbYnbLuxwXbd gDhgWWnxI4uVg5n8zu6/s1wIen+WkfDRi7lOvwHuYFsVHtRw+KRv3cURfwdleRQ83Bhx 2TKY6MWBMA9WJg/OxsHWWwg4TRtSM19KAD/tYsvN159YiEtQex3UrpAAl8E0K19O+rVg XhGrHTNbF8dMfILdUwqqcHEDUD3/UoNG+fkzH1b9utonmDzBBehfXCzoUdD3MbVy1nxM ASvQLQfXvBd7Qs5LSU/5ZheMUcm6UPj8c6vmNxT+JgN1FNc2Vk7my1vW+/3/MnaWxAhN D8ag== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:reply-to :from:date:message-id:subject:to:cc; bh=LdTU8JNN19De/6YG0m9exoj67s4UkjHWivqTBVMyxSY=; b=AEzpK4atKHaU+zbederG490Wa0bO1/jn9g22xqopqyi4zRUXw7oOTBUbjgkt4QMV3Q VY81878N0tdV7Van0lvv7Ab/BC1+ESHpL1xV4eeT0EX3ExN+1J2/F2pk9184nnr2IUFa 7kMCiQi69Np/UzNHhAHXV7PHKpkzvlzFM+dcGu3jVqAEG8AKnGyh+IZaUW0wxrHRf6/r BadijlgDR7MA0+GCca8MUSQRBAR4c+E104iF63O6P6P8Khzj/IXJaOtZmE+ER6vhN81W WfuVpUaADmAmmCY0TEX22BcsRYMg3u0Rit+vASDDY8YrcZwz4JCfKyKYsd2i5IrNbumv wfgQ== X-Gm-Message-State: AOAM533WWmTyE3HRMF7fLOZOnvIcUAFCRfxo0VJrbnESnRxYi06suhA6 3lUEDzMQl388sirJRz53JdMU94bKnDIJWzPPuLlnHaIu3fxg6w== X-Google-Smtp-Source: ABdhPJxgZRgSO4zUBdqu3fymybt0U9uYN1cRmYhzSIgpDyYUk/THlqMVFC+AkJMlND6puMJWfIfum+GKTxRS9BPmCZY= X-Received: by 2002:a5d:8552:: with SMTP id b18mr4169939ios.124.1615914827889; Tue, 16 Mar 2021 10:13:47 -0700 (PDT) MIME-Version: 1.0 References: In-Reply-To: Reply-To: noloader@gmail.com From: Jeffrey Walton Date: Tue, 16 Mar 2021 13:13:36 -0400 Message-ID: Subject: Re: Removing WireGuard Support From FreeBSD Base To: Kyle Evans Cc: freebsd-arch@freebsd.org, FreeBSD Hackers , WireGuard mailing list Content-Type: text/plain; charset="UTF-8" X-Rspamd-Queue-Id: 4F0Kd92zJzz4qTr X-Spamd-Bar: ---- Authentication-Results: mx1.freebsd.org; none X-Spamd-Result: default: False [-4.00 / 15.00]; REPLY(-4.00)[] X-Mailman-Approved-At: Tue, 16 Mar 2021 20:34:02 +0000 X-BeenThere: freebsd-arch@freebsd.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Discussion related to FreeBSD architecture List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 16 Mar 2021 17:13:49 -0000 Hi Kyle, I'm going to top post because there's only two points to raise. Sorry about that. I don't have a dog in this fight, so take this with a grain of salt... > Netgate, pfSense, nor the original developer deserved the level of > scorn and criticism that they've received in the past days from both the > press and the community at large. I have not really seen any scorn or criticism by the press or community. But maybe my feeds are missing something... > In the next day or so, I will be committing a removal of all WireGuard > related bits from our 'main' branch, including the work that I recently > committed. It will be followed up by a removal of the implementation > from stable/13, and we will seek appropriate approval to remove it > from releng/13.0 as well. Please, do not be concerned by any of this; > this is being done with mutual support from all parties. The thing I find unusual is, the move appears to lack technical justification. The best I can tell, the reasons seem to be political. But like I said, maybe my feeds are missing something... As a naive outsider, if you are going to yank it, then the technical reasons for the action should be clearly enumerated. Everything else is just chatter or noise. The move just looks like a bunch of bruised egos and sour grapes. Maybe a good middle ground would be to take the existing code and put it in a Wireguard branch. Those who wish to keep Wireguard out of FreeBSD mainline have done so. FreeBSD users who wish to use Wireguard can build the Wireguard branch. And those who wish to improve Wireguard have a working branch for patches. Later, the branch can be re-merged back to master. Jeff On Tue, Mar 16, 2021 at 12:51 PM Kyle Evans wrote: > > You may have recently noticed some chatter around the internet about > FreeBSD's in-kernel WireGuard implementation, and the work we've done > on it in the last week. You may have also noticed additional chatter > afterwards with regards to the original implementation. I'd like to give > some context and information with regards to the current situation, as > well as provide some insight into the future as one of the developers > involved. > > With regard to the original implementation, this will be my only > commentary on the matter. I'm a developer, and I'm passionate > about the work that I do- often to a fault. I've said some things that > I regret; the accusations that Scott Long alluded to in an e-mail on FreeBSD > mailing lists were indeed made by me, and his phrasing of what I > said was much kinder than it could have been. These were mistakes, > and I'm going to own that. However, my personal belief is that neither > Netgate, pfSense, nor the original developer deserved the level of > scorn and criticism that they've received in the past days from both the > press and the community at large. > > In the next day or so, I will be committing a removal of all WireGuard > related bits from our 'main' branch, including the work that I recently > committed. It will be followed up by a removal of the implementation > from stable/13, and we will seek appropriate approval to remove it > from releng/13.0 as well. Please, do not be concerned by any of this; > this is being done with mutual support from all parties. > > Did the original implementation have issues? Yes, it did. Are we > certain that our new version -doesn't- have issues? I believe it > doesn't, but it hasn't been through thorough enough review. We hacked > on this for a week, and we all reviewed each others' work in the > process. The problem is that this work, in particular, is a driver with fairly > severe security implications. Review by "three developers working > and beating on it" is not the higher bar that we should be > holding this to. While I believed I was doing what's right for the > community, it's become clear that what's right for the community is > to take a step back and do this the right way. > > Note that we're not dropping this effort. We will continue iterating > on this out-of-tree, and we will go through the proper review > channels. Folks will be unhappy in the interim because we're removing > it right now, but in the end we will have a better FreeBSD because of > it. There will be a kernel module available in ports at some point, > but not before it's ready. > > Moving forward, myself, members of Netgate, and members of the larger > community *are* working together on strictly technical details. I urge > anyone with an interest in reviewing the driver to also get in touch with me. > Please, let's move forward as a community on this.