Received: by 2002:a05:6a10:1a4d:0:0:0:0 with SMTP id nk13csp5363387pxb; Sun, 13 Feb 2022 18:33:34 -0800 (PST) X-Google-Smtp-Source: ABdhPJynUB8QiYCpzYwF27sg6Kqp9TcMkN38xd3olsvbT4o89yllrxIrTO1isoDilED98t3Anq25 X-Received: by 2002:a63:904c:: with SMTP id a73mr7125633pge.539.1644806014368; Sun, 13 Feb 2022 18:33:34 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1644806014; cv=none; d=google.com; s=arc-20160816; b=fYrWESw+kSxwoJxCfc+7Y/KKzDnF34r8w16VRGhOPnlF9h9dznxk3V/RRPY5poJWv7 fzX+SxHuYvTBXTRcD/msvWUst+2dM9G7GpGGpAe5MpJD5zK1VRI3Ex59eKMDdr13Y+jf Q3Iu1yQUwLvu/J92un00fPQVadygoTJmEw5ydGtQI/RZIz6NEyQBefuf+28xP9sj4bIh oDGakVKMyKypPzLh+bS+5NmQxaCJpP5aSuw0J0tlZ7gxXGYWxWJZPEn3y+o7sBh8LOKw g9uVI2JkXMhSbVQQ0j8FzzbIR/u7HFsSeEr7a/DThm6lx9fdIIV5pbiAJbwdmaAYhvF9 585g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-disposition:mime-version :references:message-id:subject:cc:to:from:date; bh=nND2hGWmncbWHrqTxu4O4CX0+xbM3Ra2nRPeiO9xWtw=; b=a/8e/ku1KLmUiCl+rBtaJ/UvAIkZoZ2hfnLOH/1IKkkKj8TPHtWfpKa+W1BFjBOc+3 vCVmKBLZ9ZYJepykwDSRde/DJA1VPrSvqgxGUXK5dSDGRIkpU4fZMKT40PMVmILKP7IJ 2n9QLuZ3x+BNnJtRTmQKMFNhAm83Ces/NQz4DKS85CO4JhMAJFg9Wh3r9r+gMPOoHCWw vRRppyQcn/vRiT9HQ1mLd367FhQa98uWhzvzASwaaoUitaQDKrD0+UKWllewkZJaso34 fS5I0UO9IjyTcpy/5moVc7vpA3vvlrcE9pM4Vr0YVauiZkjLw2JDrS3lLH5x6W2plYeO B2Yw== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id k184si1257792pgd.817.2022.02.13.18.33.18; Sun, 13 Feb 2022 18:33:34 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234018AbiBMG5I (ORCPT + 99 others); Sun, 13 Feb 2022 01:57:08 -0500 Received: from mxb-00190b01.gslb.pphosted.com ([23.128.96.19]:40658 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233996AbiBMG5H (ORCPT ); Sun, 13 Feb 2022 01:57:07 -0500 Received: from isilmar-4.linta.de (isilmar-4.linta.de [136.243.71.142]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 30AA55DE58 for ; Sat, 12 Feb 2022 22:57:02 -0800 (PST) X-isilmar-external: YES X-isilmar-external: YES Received: from owl.dominikbrodowski.net (owl.brodo.linta [10.2.0.111]) by isilmar-4.linta.de (Postfix) with ESMTPSA id DC82F2013FF; Sun, 13 Feb 2022 06:56:59 +0000 (UTC) Received: by owl.dominikbrodowski.net (Postfix, from userid 1000) id 8CB19809A1; Sun, 13 Feb 2022 07:55:05 +0100 (CET) Date: Sun, 13 Feb 2022 07:55:05 +0100 From: Dominik Brodowski To: "Jason A. Donenfeld" Cc: linux-kernel@vger.kernel.org Subject: Re: [PATCH v2 00/10] random: re-group and re-document functions Message-ID: References: <20220212122318.623435-1-Jason@zx2c4.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20220212122318.623435-1-Jason@zx2c4.com> X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,SPF_HELO_NONE, SPF_NONE,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Am Sat, Feb 12, 2022 at 01:23:08PM +0100 schrieb Jason A. Donenfeld: > We previously had massive documentation comments and functions strewn > all about. This attempts to restore sanity by dividing the code into a > few sections: > > - Initialization and readiness waiting. > - Fast key erasure RNG. > - Entropy accumulation and extraction. > - Entropy collection. > - Userspace read/write. > - Sysctl. > > The result of this ordering and this grouping is only 2 forward > declarations, indicating that this is probably a sensible grouping. > Also, some documentation that was just hopelessly out of date been > removed. We'll later look into re-adding parts of this to the Linux man > pages project. > > No functional changes. Hi Jason, thanks for your massive patchset. While some of it was a bit tedious and repetetive to review, feel free to add my Reviewed-by: Dominik Brodowski to patchs 1-3, 6 and 8-10. For patches 4, 5 and 7, I have some comments; once addressed, feel free to add my reviewed-by tag to those as well. Thanks, Dominik