Re: [PATCH 1/5] Renaming weak prng invocations - prandom_bytes_state, prandom_u32_state

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Subject: Re: [PATCH 1/5] Renaming weak prng invocations - prandom_bytes_state, prandom_u32_state
From: Stanislaw Gruszka <stanislaw.gruszka@xxxxxxxxxxxxxxx>
Date: Wed, 14 Dec 2022 13:33:58 +0100
Cc: Vignesh Raghavendra <vigneshr@xxxxxx>, Peter Zijlstra <peterz@xxxxxxxxxxxxx>, Joonsoo Kim <iamjoonsoo.kim@xxxxxxx>, Roman Gushchin <roman.gushchin@xxxxxxxxx>, Rasmus Villemoes <linux@xxxxxxxxxxxxxxxxxx>, Alexei Starovoitov <ast@xxxxxxxxxx>, dri-devel@xxxxxxxxxxxxxxxxxxxxx, Song Liu <song@xxxxxxxxxx>, Eric Dumazet <edumazet@xxxxxxxxxx>, linux-mtd@xxxxxxxxxxxxxxxxxxx, Stanislav Fomichev <sdf@xxxxxxxxxx>, "H. Peter Anvin" <hpa@xxxxxxxxx>, Hyeonggon Yoo <42.hyeyoo@xxxxxxxxx>, Christoph Lameter <cl@xxxxxxxxx>, Daniel Borkmann <daniel@xxxxxxxxxxxxx>, Richard Weinberger <richard@xxxxxx>, x86@xxxxxxxxxx, John Fastabend <john.fastabend@xxxxxxxxx>, Andrii Nakryiko <andrii@xxxxxxxxxx>, ilay.bahat1@xxxxxxxxx, Ingo Molnar <mingo@xxxxxxxxxx>, David Rientjes <rientjes@xxxxxxxxxx>, Yonghong Song <yhs@xxxxxx>, Paolo Abeni <pabeni@xxxxxxxxxx>, "James E.J. Bottomley" <jejb@xxxxxxxxxxxxx>, Petr Mladek <pmladek@xxxxxxxx>, david.keisarschm@xxxxxxxxxxxxxxx, Dave Hansen <dave.hansen@xxxxxxxxxxxxxxx>, Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxxxxxxxx>, Miquel Raynal <miquel.raynal@xxxxxxxxxxx>, intel-gfx@xxxxxxxxxxxxxxxxxxxxx, Steven Rostedt <rostedt@xxxxxxxxxxx>, KP Singh <kpsingh@xxxxxxxxxx>, Jakub Kicinski <kuba@xxxxxxxxxx>, Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx>, Borislav Petkov <bp@xxxxxxxxx>, Hannes Reinecke <hare@xxxxxxx>, Andy Lutomirski <luto@xxxxxxxxxx>, Jiri Pirko <jiri@xxxxxxxxxx>, Thomas Gleixner <tglx@xxxxxxxxxxxxx>, Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>, bpf@xxxxxxxxxxxxxxx, Vlastimil Babka <vbabka@xxxxxxx>, Hao Luo <haoluo@xxxxxxxxxx>, linux-scsi@xxxxxxxxxxxxxxx, "Martin K. Petersen" <martin.petersen@xxxxxxxxxx>, linux-mm@xxxxxxxxx, netdev@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, Pekka Enberg <penberg@xxxxxxxxxx>, Sergey Senozhatsky <senozhatsky@xxxxxxxxxxxx>, aksecurity@xxxxxxxxx, Jiri Olsa <jolsa@xxxxxxxxxx>, Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>, Martin KaFai Lau <martin.lau@xxxxxxxxx>, "David S. Miller" <davem@xxxxxxxxxxxxx>
In-reply-to: <Y5c8KLzJFz/XZMiM@zx2c4.com>
References: <cover.1670778651.git.david.keisarschm@mail.huji.ac.il> <b3caaa5ac5fca4b729bf1ecd0d01968c09e6d083.1670778652.git.david.keisarschm@mail.huji.ac.il> <Y5c8KLzJFz/XZMiM@zx2c4.com>

On Mon, Dec 12, 2022 at 03:35:20PM +0100, Jason A. Donenfeld wrote:
> Please CC me on future revisions.
> 
> As of 6.2, the prandom namespace is *only* for predictable randomness.
> There's no need to rename anything. So nack on this patch 1/5.

It is not obvious (for casual developers like me) that p in prandom
stands for predictable. Some renaming would be useful IMHO.

Regards
Stanislaw

Follow-Ups:
- Re: [PATCH 1/5] Renaming weak prng invocations - prandom_bytes_state, prandom_u32_state
  - From: Eric Dumazet

References:
- [PATCH 1/5] Renaming weak prng invocations - prandom_bytes_state, prandom_u32_state
  - From: david . keisarschm
- Re: [PATCH 1/5] Renaming weak prng invocations - prandom_bytes_state, prandom_u32_state
  - From: Jason A. Donenfeld

Prev by Date: [PATCH v2 1/2] dt-bindings: display: imx: Describe drm binding for fsl, imx-lcdc
Next by Date: Re: renesas/master bisection: igt-kms-rockchip.kms_vblank.pipe-A-wait-forked on rk3399-gru-kevin
Previous by thread: Re: [PATCH 1/5] Renaming weak prng invocations - prandom_bytes_state, prandom_u32_state
Next by thread: Re: [PATCH 1/5] Renaming weak prng invocations - prandom_bytes_state, prandom_u32_state
Index(es):
- Date
- Thread

[Index of Archives] [Linux DRI Users] [Linux Intel Graphics] [Linux USB Devel] [Video for Linux] [Linux Audio Users] [Yosemite News] [Linux Kernel] [Linux SCSI] [XFree86] [Linux USB Devel] [Video for Linux] [Linux Audio Users] [Linux Kernel] [Linux SCSI] [XFree86]