From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pf0-f197.google.com (mail-pf0-f197.google.com [209.85.192.197]) by kanga.kvack.org (Postfix) with ESMTP id E71F56B025E for ; Wed, 3 Aug 2016 19:39:47 -0400 (EDT) Received: by mail-pf0-f197.google.com with SMTP id o124so427701806pfg.1 for ; Wed, 03 Aug 2016 16:39:47 -0700 (PDT) Received: from pmta2.delivery5.ore.mailhop.org (pmta2.delivery5.ore.mailhop.org. [54.186.218.12]) by mx.google.com with ESMTPS id g9si11189924pfk.211.2016.08.03.16.39.45 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Wed, 03 Aug 2016 16:39:45 -0700 (PDT) From: Jason Cooper Subject: [PATCH v3 0/7] char/random: Simplify random address requests Date: Wed, 3 Aug 2016 23:39:06 +0000 Message-Id: <20160803233913.32511-1-jason@lakedaemon.net> In-Reply-To: <20160728204730.27453-1-jason@lakedaemon.net> References: <20160728204730.27453-1-jason@lakedaemon.net> Sender: owner-linux-mm@kvack.org List-ID: To: Kees Cook , Michael Ellerman , "Roberts, William C" , Yann Droneaud Cc: Linux-MM , LKML , kernel-hardening , Russell King - ARM Linux , Andrew Morton , Theodore Ts'o , Arnd Bergmann , gregkh@linuxfoundation.org, Catalin Marinas , Will Deacon , Ralf Baechle , benh@kernel.crashing.org, paulus@samba.org, "David S. Miller" , Thomas Gleixner , Ingo Molnar , "H . Peter Anvin" , x86@kernel.org, viro@zeniv.linux.org.uk, Nick Kralevich , Jeffrey Vander Stoep , Daniel Cashman , Jason Cooper Two previous attempts have been made to rework this API. The first can be found at: https://lkml.kernel.org/r/cover.1390770607.git.ydroneaud@opteya.com The second at: https://lkml.kernel.org/r/1469471141-25669-1-git-send-email-william.c.roberts@intel.com Previous versions of this series can been seen at: RFC: https://lkml.kernel.org/r/20160726030201.6775-1-jason@lakedaemon.net v1: https://lkml.kernel.org/r/20160728204730.27453-1-jason@lakedaemon.net v2: https://lkml.kernel.org/r/20160730154244.403-1-jason@lakedaemon.net In addition to incorporating ideas from these two previous efforts, this series adds several desirable features. First, we take the range as an argument directly, which removes math both before the call and inside the function. Second, we return the start address on error. All callers fell back to the start address on error, so we remove the need to check for errors. Third, we cap range to prevent overflow. Last, we use kerneldoc to describe the new function. If possible, I'd like to request Acks from the various subsystems so that we can merge this as one bisectable branch. Changes from v2: - s/randomize_addr/randomize_page/ (Kees Cook) - PAGE_ALIGN(start) if it wasn't (Kees Cook, Michael Ellerman) Changes from v1: - Explicitly mention page_aligned start assumption (Yann Droneaud) - pick random pages vice random addresses (Yann Droneaud) - catch range=0 last - Add Ack for arm64 (Will Deacon) Jason Cooper (7): random: Simplify API for random address requests x86: Use simpler API for random address requests ARM: Use simpler API for random address requests arm64: Use simpler API for random address requests tile: Use simpler API for random address requests unicore32: Use simpler API for random address requests random: Remove unused randomize_range() arch/arm/kernel/process.c | 3 +-- arch/arm64/kernel/process.c | 8 ++------ arch/tile/mm/mmap.c | 3 +-- arch/unicore32/kernel/process.c | 3 +-- arch/x86/kernel/process.c | 3 +-- arch/x86/kernel/sys_x86_64.c | 5 +---- drivers/char/random.c | 36 +++++++++++++++++++++++++----------- include/linux/random.h | 2 +- 8 files changed, 33 insertions(+), 30 deletions(-) -- 2.9.2 -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org