Received: by 2002:ac0:a5a6:0:0:0:0:0 with SMTP id m35-v6csp1043473imm; Wed, 26 Sep 2018 10:39:30 -0700 (PDT) X-Google-Smtp-Source: ACcGV63y0n3YQVTBKoaVt77nwJIq3z0XxhKmAjeANrUysYP2gc6MUWz5orqD+DwFizqnWd4JQt+C X-Received: by 2002:a17:902:d201:: with SMTP id t1-v6mr7110797ply.63.1537983570910; Wed, 26 Sep 2018 10:39:30 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1537983570; cv=none; d=google.com; s=arc-20160816; b=ILTeddOQpIKOet7hFEdhvByYk8paGMhP/hMNx0qLYW53dA8WFmsiuDCQIcjwQ7b5BZ XOyQopjlRN3q0H8pxKi4LC2vfUsvuOKtuSeXt4thcdNMfkumVCv4PG4erUrg7/ZopEei fIfQjvWcYMnJr/kSVhKzVBiDTbe5Q8twp75Ah08g/cIcIYsl2S+4k5QEyvW9xyT1Jdv/ m2Zc9ACq6f4a0ahfwcZOKRXWiHy6OdjxBleEgWtXE3mZz+CpbIiXwJrOqJ8UFbRQzznd lPBuVg/kv8fHMNjrTCrPbvvaYJL6Y6RnrRZ2VNWhVS5JCfx/IX9CZy/rGZUjUrpc9BhT x63Q== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date:dkim-signature; bh=Vw9uJ6A47RDrP2qh7XTzZl5dWxAl/fQKH9RDEYfC4aI=; b=or9PHxy2aifqg9FLUnY/fSZGmYW/QIN4ry+OkeEONjer8LQm33Clb2bS/o5O0lxFvi zMQp7PQhiU0rYrxIlUNMFmRAwdIwPlsa6e1GTgrBGJgnLybt/E6AQIaCfiuXGo5nkRwZ faQ/1RU2VgBkrDyPqIk113NyjHXo8mzxJmni7NBzzODN3KjBDGrZA9JVb04jMNmEkLki SnLdju5Bh98JaCm+hUaXq2t2Ufki4fTkBzNY71Doo5XcAFDc3EjfGRrQtjCtHWuESq9q 9jiq8tFySzGiiyQChnsv43jDQPb6UJ+7qwOXb9RXB8V3uRu1htQyuUuA7h/dAw80vAKT /B9g== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=SVdga3UB; 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=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id t6-v6si5775386pga.635.2018.09.26.10.39.15; Wed, 26 Sep 2018 10:39:30 -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=@kernel.org header.s=default header.b=SVdga3UB; 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=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728614AbeIZXvX (ORCPT + 99 others); Wed, 26 Sep 2018 19:51:23 -0400 Received: from mail.kernel.org ([198.145.29.99]:46684 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727280AbeIZXvX (ORCPT ); Wed, 26 Sep 2018 19:51:23 -0400 Received: from gmail.com (unknown [104.132.51.88]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id A51112152F; Wed, 26 Sep 2018 17:37:22 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1537983442; bh=xuJZE6SN+NcWwElZWRzB3oyL2/JpmUCDXZQaipOqLls=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=SVdga3UBZCklH/UCt1IIxGHpZXa/NNhrkl5Fn4gsUOnMVFLrS5cWEOMBgXQVUIXiF w+aXfBWX/dbIOZZS6lMxbdH1D/SSu1pFlLaWvfPVxxuovB311Pn+0BCfgjVKthrnFR l4fnHCb8dy4GY0x2HEOzv6fiLbuvTiOMQyOkX7Nk= Date: Wed, 26 Sep 2018 10:37:21 -0700 From: Eric Biggers To: "Jason A. Donenfeld" Cc: Ard Biesheuvel , Herbert Xu , Thomas Gleixner , LKML , Netdev , Linux Crypto Mailing List , David Miller , Greg Kroah-Hartman , Samuel Neves , Andrew Lutomirski , Jean-Philippe Aumasson , Russell King - ARM Linux , linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH net-next v6 07/23] zinc: ChaCha20 ARM and ARM64 implementations Message-ID: <20180926173720.GA248906@gmail.com> References: <20180925145622.29959-1-Jason@zx2c4.com> <20180925145622.29959-8-Jason@zx2c4.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.10.1 (2018-07-13) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Sep 26, 2018 at 05:41:12PM +0200, Jason A. Donenfeld wrote: > On Wed, Sep 26, 2018 at 4:02 PM Ard Biesheuvel > wrote: > > I don't think it makes sense to keep > > it simple now and add the complexity later (and the same concern > > applies to async support btw). > > Ugh, no. I don't want to add needless complexity, period. Zinc is > synchronous, not asynchronous. It provides software implementations. > That's what it does. While many of your reviews have been useful, many > of your comments indicate some desire to change and mold the purpose > and focus of Zinc away from Zinc's intents. Stop that. It's not going > to become a bloated mess of "things Ard wanted and quipped about on > LKML." Things like these only serve to filibuster the patchset > indefinitely. But maybe that's what you'd like all along? Hard to > tell, honestly. So, no, sorry, Zinc isn't gaining an async interface > right now. Can you please stop accusing Ard of "filibustering" your patchset? Spending too long in non-preemptible code is a real problem even on non-RT systems. syzkaller has been reporting bugs where the kernel spins too long without any preemption points, both in crypto-related code and elsewhere in the kernel. So we've had to add explicit preemption points to address those, as otherwise users can lock up all CPUs for tens of seconds. The issue being discussed here is basically the same except here preemption is being explicitly disabled via kernel_neon_begin(), so it becomes a problem even on non-CONFIG_PREEMPT kernels. It's much better to address this problem (which is a regression from the current skcipher API which only maps a page at a time) up front rather than have to rush to patch it after the fact once the syzbot reports start coming in. - Eric