Received: by 2002:ac0:a582:0:0:0:0:0 with SMTP id m2-v6csp815186imm; Mon, 1 Oct 2018 20:19:44 -0700 (PDT) X-Google-Smtp-Source: ACcGV62DyAhPgt8e8f+XAzeExm7PbGGJ5adI1a575GJhvTrXjHOiATFZBH4OXScLjawaEA/YZTcF X-Received: by 2002:a17:902:a9c9:: with SMTP id b9-v6mr13979042plr.225.1538450384728; Mon, 01 Oct 2018 20:19:44 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1538450384; cv=none; d=google.com; s=arc-20160816; b=zDOG63fMsGWEFfxERogrMoiRizWmhHsgqJ3UnYkDjDQcjnSgqt42AUSiYCX6i7h0fQ yu+YAPTjm3Hs4vp7x/29JkSMgqMBHpdB/y2CVlNW2qLxbs08CuwoK2lfwA6y17sUX0fH ae3hp1X/M4I7wSJty7SGZqIq/+YgfrZ0lPN0huM52BnhFtAhQyyFYl3r4hqRBoz1yXsm MNKSukR6eF6MwgA+I9nreMmoHXDuyZrtAbfGaxsgBLUEKFeLzoWidx1L1ec0HEOMiiWp Fik6i6iR2EPoeZ9jxMxzLv8XT/LjIAo6HzIN1Ejrh7Yl8pMYBSxQXjwhq1z8vlJuTjO5 +0ng== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:dkim-signature; bh=UwSW5mcTJ83QseRvE+pgPvWQmoSu034M8m82xbibTLc=; b=m6u8M1eaLN7WenCOu8bWLppQkkdlOm17tOqVrDQHeXwL3zJtJbiYQdEz8g/fRu0oCd QmRGWwb2alJdyC9LskYquWFR7o4Tt4E9OCt2bFM/+HGow4vDx57ZWE0ohFMbc9m4CG1Z WSRbW03rSUV7160we57hDzPJam+pzfeUeIr7TtD/jMjClGeQsMhjT+GH1cbN+QvV78UE jwG44/YGr0ZPfZ2uHHHjhZOuydT5zn+KeA+4ZuvzT0fxaI7aSJWj+5ihUivUY7FQosyy 0cmLy5uBT3vCavk3m5fGvy0GkBkrEsbm9BCsOkOyZle+JWNaAJ9E1suAxtoeXwMKHnAc kigw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@zx2c4.com header.s=mail header.b=Wf5x1Y3G; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=zx2c4.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id r3-v6si14192487plb.221.2018.10.01.20.19.30; Mon, 01 Oct 2018 20:19:44 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@zx2c4.com header.s=mail header.b=Wf5x1Y3G; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=zx2c4.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726798AbeJBKAG (ORCPT + 99 others); Tue, 2 Oct 2018 06:00:06 -0400 Received: from frisell.zx2c4.com ([192.95.5.64]:46423 "EHLO frisell.zx2c4.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726239AbeJBKAG (ORCPT ); Tue, 2 Oct 2018 06:00:06 -0400 Received: by frisell.zx2c4.com (ZX2C4 Mail Server) with ESMTP id 2df64f0d; Tue, 2 Oct 2018 03:19:05 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha1; c=relaxed; d=zx2c4.com; h=mime-version :references:in-reply-to:from:date:message-id:subject:to:cc :content-type; s=mail; bh=Dbb/vV91vK4zck0wMbIfg2nFqn4=; b=Wf5x1Y 3GWW6UQf9hGyiJMheZTbsHTb8MlVX2f+jZRpkqCke8VPnmpy1Ywu7uF/exgcE9q0 bX3qacIT+JWfaWXKXLC5b5hJ+FnJxD1ZTBT5bVTfHa+eGlRE4j1M2h04ibRetcNA j4laLqchoLxFH6w+lHMKxecijWYKMmxYAgxW4zbW1q8QZvTzSFeZXuB+izP6MhQt TDeJ3Dnxfeo7tO9GiYHfV/aMhV6q5a/GteeeWfPUcZtfKKABT4Jmc9aXzRRS+vfj cWsyE/UOzm40KDWEPbf1SsYBPr3LOnwJSFQNJbGe1d/LUH2aqHoOqNNyQCKQxs8c CPgTfz4v6FFHEiQw== Received: by frisell.zx2c4.com (ZX2C4 Mail Server) with ESMTPSA id bc1d6ed6 (TLSv1.2:ECDHE-RSA-AES128-GCM-SHA256:128:NO); Tue, 2 Oct 2018 03:19:04 +0000 (UTC) Received: by mail-oi1-f177.google.com with SMTP id a203-v6so407820oib.0; Mon, 01 Oct 2018 20:19:04 -0700 (PDT) X-Gm-Message-State: ABuFfog5aW4jtiPfYOwx34ViRYMLeeAUMPtD5kJ6wOets8+t1hzqTJ6X VZVFmSbw+2FNYJpCOw9A3UPlD/npDaznmLQbQwE= X-Received: by 2002:aca:ce4c:: with SMTP id e73-v6mr6065045oig.225.1538450330758; Mon, 01 Oct 2018 20:18:50 -0700 (PDT) MIME-Version: 1.0 References: <20180925145622.29959-4-Jason@zx2c4.com> <20181002031542.dbxu44gvqrughlgw@gondor.apana.org.au> In-Reply-To: <20181002031542.dbxu44gvqrughlgw@gondor.apana.org.au> From: "Jason A. Donenfeld" Date: Tue, 2 Oct 2018 05:18:38 +0200 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH net-next v6 03/23] zinc: ChaCha20 generic C implementation and selftest To: Herbert Xu Cc: LKML , Netdev , Linux Crypto Mailing List , David Miller , Greg Kroah-Hartman , Samuel Neves , Andrew Lutomirski , Jean-Philippe Aumasson Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Herbert, On Tue, Oct 2, 2018 at 5:15 AM Herbert Xu wrote: > I don't have any problems with this interface. > > However, I'm very unhappy about the fact that this adds another > copy of the chacha20 implementation to the kernel. Either this > is better than the one that we already have or it isn't. > > So if it's better please replace the existing chacha20 C > implementation with this or otherwise just adapt the existing > implementation to this interface. > > If we're replacing the existing one then you can either replace > it directly within this patch or in a subsequent patch but please > do it in the same series because otherwise we could wind up with > two implementations for years to come. Indeed this series takes care of removing the other implementation later on. In case it helps to see things visualized, here's the layout of the whole series: https://git.kernel.org/pub/scm/linux/kernel/git/zx2c4/linux.git/log/?h=jd/wireguard Note that comments on this are probably a bit more useful than on the version posted to the mailing list, since I've already started incorporating quite a bit of feedback from v6, and so v7 will be posted sometime later containing these. Jason