Received: by 2002:a05:6a10:a0d1:0:0:0:0 with SMTP id j17csp1083983pxa; Thu, 20 Aug 2020 01:54:55 -0700 (PDT) X-Google-Smtp-Source: ABdhPJz/hzyJT9XhH6EtbR4EK1zEwfxueG9fpzsThtxZYn1g7xw2ZNpr1lhnJKj2VuxaiaSBldbq X-Received: by 2002:a17:907:4303:: with SMTP id nh3mr2303041ejb.520.1597913695229; Thu, 20 Aug 2020 01:54:55 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1597913695; cv=none; d=google.com; s=arc-20160816; b=LZVZLeu5qvPC/KPnD7z5OIcdpl4Ge2cqVgqsmxX1syoD6pC5k1SvMqYxwV0kpTmZe0 prf3ULS4TW/wvpDt2jHHZMbdXKEFd3zqtg3v/9fe+B2pRjuurF0dT3uEVr77ncaFXyk9 GCaD8+lDrgnnvkz+NVWLAuFFXPZChmLFMtHZQshghBgUALx0Ux0QdRta/Pc782FSmHt9 CIzmhk3+Klw2GjrZxzJ5NK2lcNAQqiFLNckmev7BrXtqJ//0pAbeNIVqKFMVVHmbb50M Ajd64Y4AJwmDupeYhARtqZkA7t1jpyV/dnri5DEcRyLSkEDfSBDnEbKNLpbq2Ej6kGIN eTHw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=5dSGSAiOchgXa0bateO3AteEhDNKiuRbQ7iJbxrg0gc=; b=mHFfzsgMcS5pDCOXTO5293Sy82OvvpvFOVryrLiFNRKC8SYEgibYsGzisVitPkdev3 9f7U7Ab57rhxGdbvZ+TEL20SRXTV6VJ2aQZrUphSmlbO/CCg98WxPked8aIaz8I666hC MWx7t2sQm7/+7AkrEoB27RHIfERShy2nP3ql3YTP5ZW13MKGGuWcMdkDv0rN1yuO85B1 2OiNY5CtoXBNyfEChxr8sqr6LJ5TehqeBPd+lMUCeEb5W+mcUSGQIEXQ3e/R7pEQcemG bhC02vJn+WOINoC2cuUq+Y07clZlrjV8jPcCyEg4DwbS5H5WwzxATSbFwC/SeP6Jn/+o YMtA== ARC-Authentication-Results: i=1; mx.google.com; dkim=fail header.i=@chronox.de header.s=strato-dkim-0002 header.b=BmdFYWc0; spf=pass (google.com: domain of linux-crypto-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-crypto-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id yl26si704703ejb.625.2020.08.20.01.54.29; Thu, 20 Aug 2020 01:54:55 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-crypto-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=fail header.i=@chronox.de header.s=strato-dkim-0002 header.b=BmdFYWc0; spf=pass (google.com: domain of linux-crypto-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-crypto-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726875AbgHTIxq (ORCPT + 99 others); Thu, 20 Aug 2020 04:53:46 -0400 Received: from mo4-p04-ob.smtp.rzone.de ([81.169.146.176]:23399 "EHLO mo4-p04-ob.smtp.rzone.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726868AbgHTIxM (ORCPT ); Thu, 20 Aug 2020 04:53:12 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; t=1597913584; s=strato-dkim-0002; d=chronox.de; h=References:In-Reply-To:Message-ID:Date:Subject:Cc:To:From: X-RZG-CLASS-ID:X-RZG-AUTH:From:Subject:Sender; bh=5dSGSAiOchgXa0bateO3AteEhDNKiuRbQ7iJbxrg0gc=; b=BmdFYWc0k428IDiEJHej8BagV9U7YhYK6miMQ63LypOrFPW4DTY6TY+/FXvPr+/drI Qe+yFgbjtKjePmZspIxttkZFgfMrHuDcjOeW1Bqa0swZDtSkuWp57m30rJgMPwnG0Y0s GV2tDQnwPA1CG3hLl8qOxlJdgPpQ8QcyWFmSSgbj4B0JJcXYIuYvFcbIifPj7/n3M0CK VnQz86q1y/3H5SSR8xb7+OVZg0Bw69he9sHpvMJ4B2UXJ/FN3oKoyeZ5zMdp2C3O1RSU G5jonJK1tPJAGxUZFopsGPDJxPUASE1HmA5qNSgZfcfeae7+0kxmn6GesJ7cfRQaIX1/ qXlA== X-RZG-AUTH: ":P2ERcEykfu11Y98lp/T7+hdri+uKZK8TKWEqNyiHySGSa9k9xm0dNS3IdRAZAL+p6A==" X-RZG-CLASS-ID: mo00 Received: from positron.chronox.de by smtp.strato.de (RZmta 46.10.7 DYNA|AUTH) with ESMTPSA id 002e9aw7K8njBSw (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256 bits)) (Client did not present a certificate); Thu, 20 Aug 2020 10:49:45 +0200 (CEST) From: Stephan =?ISO-8859-1?Q?M=FCller?= To: Arnd Bergmann Cc: Greg Kroah-Hartman , linux-crypto@vger.kernel.org, LKML , linux-api@vger.kernel.org, "Eric W. Biederman" , "Alexander E. Patrakov" , "Ahmed S. Darwish" , "Theodore Y. Ts'o" , Willy Tarreau , Matthew Garrett , Vito Caputo , Andreas Dilger , Jan Kara , Ray Strode , William Jon McCann , zhangjs , Andy Lutomirski , Florian Weimer , Lennart Poettering , Nicolai Stange , "Peter, Matthias" , Marcelo Henrique Cerri , Roman Drahtmueller , Neil Horman , Randy Dunlap , Julia Lawall , Dan Carpenter Subject: [PATCH v32 03/12] LRNG - sysctls and /proc interface Date: Thu, 20 Aug 2020 10:40:43 +0200 Message-ID: <21893267.6Emhk5qWAg@positron.chronox.de> In-Reply-To: <2544450.mvXUDI8C0e@positron.chronox.de> References: <2544450.mvXUDI8C0e@positron.chronox.de> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="iso-8859-1" Sender: linux-crypto-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-crypto@vger.kernel.org The LRNG sysctl interface provides the same controls as the existing /dev/random implementation. These sysctls behave identically and are implemented identically. The goal is to allow a possible merge of the existing /dev/random implementation with this implementation which implies that this patch tries have a very close similarity. Yet, all sysctls are documented at [1]. In addition, it provides the file lrng_type which provides details about the LRNG: =2D the name of the DRNG that produces the random numbers for /dev/random, /dev/urandom, getrandom(2) =2D the hash used to produce random numbers from the entropy pool =2D the number of secondary DRNG instances =2D indicator whether the LRNG operates SP800-90B compliant =2D indicator whether a high-resolution timer is identified - only with a high-resolution timer the interrupt noise source will deliver sufficient entropy =2D indicator whether the LRNG has been minimally seeded (i.e. is the secondary DRNG seeded with at least 128 bits of of entropy) =2D indicator whether the LRNG has been fully seeded (i.e. is the secondary DRNG seeded with at least 256 bits of entropy) [1] https://www.chronox.de/lrng.html CC: "Eric W. Biederman" CC: "Alexander E. Patrakov" CC: "Ahmed S. Darwish" CC: "Theodore Y. Ts'o" CC: Willy Tarreau CC: Matthew Garrett CC: Vito Caputo CC: Andreas Dilger CC: Jan Kara CC: Ray Strode CC: William Jon McCann CC: zhangjs CC: Andy Lutomirski CC: Florian Weimer CC: Lennart Poettering CC: Nicolai Stange Reviewed-by: Marcelo Henrique Cerri Reviewed-by: Roman Drahtmueller Tested-by: Roman Drahtm=FCller Tested-by: Marcelo Henrique Cerri Tested-by: Neil Horman Signed-off-by: Stephan Mueller =2D-- drivers/char/lrng/Makefile | 1 + drivers/char/lrng/lrng_interfaces.c | 1 - drivers/char/lrng/lrng_internal.h | 4 + drivers/char/lrng/lrng_proc.c | 163 ++++++++++++++++++++++++++++ 4 files changed, 168 insertions(+), 1 deletion(-) create mode 100644 drivers/char/lrng/lrng_proc.c diff --git a/drivers/char/lrng/Makefile b/drivers/char/lrng/Makefile index 0a32f22c2c1a..e69c176f0161 100644 =2D-- a/drivers/char/lrng/Makefile +++ b/drivers/char/lrng/Makefile @@ -9,3 +9,4 @@ obj-y +=3D lrng_pool.o lrng_aux.o \ lrng_interfaces.o \ =20 obj-$(CONFIG_NUMA) +=3D lrng_numa.o +obj-$(CONFIG_SYSCTL) +=3D lrng_proc.o diff --git a/drivers/char/lrng/lrng_interfaces.c b/drivers/char/lrng/lrng_i= nterfaces.c index 7fc04e54d834..184108b6e1cb 100644 =2D-- a/drivers/char/lrng/lrng_interfaces.c +++ b/drivers/char/lrng/lrng_interfaces.c @@ -34,7 +34,6 @@ static DECLARE_WAIT_QUEUE_HEAD(lrng_write_wait); static DECLARE_WAIT_QUEUE_HEAD(lrng_init_wait); static struct fasync_struct *fasync; =20 =2Dstruct ctl_table random_table[]; /********************************** Helper *******************************= ****/ =20 /* Is the DRNG seed level too low? */ diff --git a/drivers/char/lrng/lrng_internal.h b/drivers/char/lrng/lrng_int= ernal.h index 495e00ae5ee1..a52c4cf367e2 100644 =2D-- a/drivers/char/lrng/lrng_internal.h +++ b/drivers/char/lrng/lrng_internal.h @@ -117,7 +117,11 @@ void lrng_cc20_init_state_boot(struct chacha20_state *= state); =20 /********************************** /proc ********************************= *****/ =20 +#ifdef CONFIG_SYSCTL +void lrng_pool_inc_numa_node(void); +#else static inline void lrng_pool_inc_numa_node(void) { } +#endif =20 /****************************** LRNG interfaces **************************= *****/ =20 diff --git a/drivers/char/lrng/lrng_proc.c b/drivers/char/lrng/lrng_proc.c new file mode 100644 index 000000000000..c569a269b07a =2D-- /dev/null +++ b/drivers/char/lrng/lrng_proc.c @@ -0,0 +1,163 @@ +// SPDX-License-Identifier: GPL-2.0 OR BSD-2-Clause +/* + * LRNG proc and sysctl interfaces + * + * Copyright (C) 2016 - 2020, Stephan Mueller + */ + +#include +#include +#include +#include +#include + +#include "lrng_internal.h" + +/* + * This function is used to return both the bootid UUID, and random + * UUID. The difference is in whether table->data is NULL; if it is, + * then a new UUID is generated and returned to the user. + * + * If the user accesses this via the proc interface, the UUID will be + * returned as an ASCII string in the standard UUID format; if via the + * sysctl system call, as 16 bytes of binary data. + */ +static int lrng_proc_do_uuid(struct ctl_table *table, int write, + void __user *buffer, size_t *lenp, loff_t *ppos) +{ + struct ctl_table fake_table; + unsigned char buf[64], tmp_uuid[16], *uuid; + + uuid =3D table->data; + if (!uuid) { + uuid =3D tmp_uuid; + generate_random_uuid(uuid); + } else { + static DEFINE_SPINLOCK(bootid_spinlock); + + spin_lock(&bootid_spinlock); + if (!uuid[8]) + generate_random_uuid(uuid); + spin_unlock(&bootid_spinlock); + } + + sprintf(buf, "%pU", uuid); + + fake_table.data =3D buf; + fake_table.maxlen =3D sizeof(buf); + + return proc_dostring(&fake_table, write, buffer, lenp, ppos); +} + +static int lrng_proc_do_entropy(struct ctl_table *table, int write, + void *buffer, size_t *lenp, loff_t *ppos) +{ + struct ctl_table fake_table; + int entropy_count; + + entropy_count =3D lrng_avail_entropy(); + + fake_table.data =3D &entropy_count; + fake_table.maxlen =3D sizeof(entropy_count); + + return proc_dointvec(&fake_table, write, buffer, lenp, ppos); +} + +static int lrng_sysctl_poolsize =3D LRNG_POOL_SIZE_BITS; +static int lrng_min_write_thresh; +static int lrng_max_write_thresh =3D LRNG_POOL_SIZE_BITS; +static char lrng_sysctl_bootid[16]; +static int lrng_drng_reseed_max_min; + +struct ctl_table random_table[] =3D { + { + .procname =3D "poolsize", + .data =3D &lrng_sysctl_poolsize, + .maxlen =3D sizeof(int), + .mode =3D 0444, + .proc_handler =3D proc_dointvec, + }, + { + .procname =3D "entropy_avail", + .maxlen =3D sizeof(int), + .mode =3D 0444, + .proc_handler =3D lrng_proc_do_entropy, + }, + { + .procname =3D "write_wakeup_threshold", + .data =3D &lrng_write_wakeup_bits, + .maxlen =3D sizeof(int), + .mode =3D 0644, + .proc_handler =3D proc_dointvec_minmax, + .extra1 =3D &lrng_min_write_thresh, + .extra2 =3D &lrng_max_write_thresh, + }, + { + .procname =3D "boot_id", + .data =3D &lrng_sysctl_bootid, + .maxlen =3D 16, + .mode =3D 0444, + .proc_handler =3D lrng_proc_do_uuid, + }, + { + .procname =3D "uuid", + .maxlen =3D 16, + .mode =3D 0444, + .proc_handler =3D lrng_proc_do_uuid, + }, + { + .procname =3D "urandom_min_reseed_secs", + .data =3D &lrng_drng_reseed_max_time, + .maxlen =3D sizeof(int), + .mode =3D 0644, + .proc_handler =3D proc_dointvec, + .extra1 =3D &lrng_drng_reseed_max_min, + }, + { } +}; + +/* Number of online DRNGs */ +static u32 numa_drngs =3D 1; + +void lrng_pool_inc_numa_node(void) +{ + numa_drngs++; +} + +static int lrng_proc_type_show(struct seq_file *m, void *v) +{ + struct lrng_drng *lrng_drng_init =3D lrng_drng_init_instance(); + unsigned long flags =3D 0; + unsigned char buf[300]; + + lrng_drng_lock(lrng_drng_init, &flags); + snprintf(buf, sizeof(buf), + "DRNG name: %s\n" + "Hash for reading entropy pool: %s\n" + "DRNG security strength: %d bits\n" + "number of DRNG instances: %u\n" + "SP800-90B compliance: %s\n" + "High-resolution timer: %s\n" + "LRNG minimally seeded: %s\n" + "LRNG fully seeded: %s\n", + lrng_drng_init->crypto_cb->lrng_drng_name(), + lrng_drng_init->crypto_cb->lrng_hash_name(), + LRNG_DRNG_SECURITY_STRENGTH_BITS, numa_drngs, + lrng_sp80090b_compliant() ? "true" : "false", + lrng_pool_highres_timer() ? "true" : "false", + lrng_state_min_seeded() ? "true" : "false", + lrng_state_fully_seeded() ? "true" : "false"); + lrng_drng_unlock(lrng_drng_init, &flags); + + seq_write(m, buf, strlen(buf)); + + return 0; +} + +static int __init lrng_proc_type_init(void) +{ + proc_create_single("lrng_type", 0444, NULL, &lrng_proc_type_show); + return 0; +} + +module_init(lrng_proc_type_init); =2D-=20 2.26.2