From: "Tobin C. Harding" <me@tobin.cc>
To: "Theodore Ts'o" <tytso@mit.edu>
Cc: "Tobin C. Harding" <me@tobin.cc>,
Linus Torvalds <torvalds@linux-foundation.org>,
Randy Dunlap <rdunlap@infradead.org>,
Steven Rostedt <rostedt@goodmis.org>,
Kees Cook <keescook@chromium.org>,
Anna-Maria Gleixner <anna-maria@linutronix.de>,
Andrew Morton <akpm@linux-foundation.org>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Arnd Bergmann <arnd@arndb.de>,
Andy Shevchenko <andy.shevchenko@gmail.com>,
linux-kernel@vger.kernel.org
Subject: [PATCH v8 0/4] enable early printing of hashed pointers
Date: Wed, 20 Jun 2018 14:20:42 +1000 [thread overview]
Message-ID: <20180620042046.30806-1-me@tobin.cc> (raw)
Ted,
Version 8 changes patch 2 which you previously acked. Change is to
remove integer cast as suggested on LKML. I've kept the ACK.
Changes made to patch with reviewed-by tag from Steve were acknowledged
on LKML, I've also kept Steve's tag.
Currently printing pointers early in the boot sequence can result in a
dummy string '(____ptrval____)' being printed. While resolving this
issue it was noticed that we can use the hw RNG if available for hashing
pointers.
Patch one and two do the ground work to be able to use hw RNG removing
from get_random_bytes_arch() the call to get_random_bytes() and
returning the number of bytes of random material successfully returned.
Patch three uses the hw RNG to get keying material if it is available.
Patch four further assists debugging early in the boot sequence for
machines that do not have a hw RNG by adding a command line option
'debug_boot_weak_hash'. If enabled, non-cryptographically secure hashing
is used instead of siphash so we can hash at any time.
thanks,
Tobin.
v8
- Remove pointless EXPORT_SYMBOL on static variable (thanks Steve).
- Remove unnecessary integer cast from min_t() argument (thanks Andy).
v7
- Remove unused variable, clearing compiler warning (found by Stephen
Rothwell's linux-next build infrastructure).
v6
- Rebase on top of Steve's patch (fixing race condition). Uses static
branch instead of memory barrier.
v5
- Use 'upside-down-xmas-tree' style to declare local variables (Steve)
- Added Reviewed-by tag from Steve (patch 2 and 3).
v4
- remove last patch of series (command line option patch)
v3
- Add __ro_after_init (suggested by Kees).
v2
- Use min_t() instead of min() (thanks checkpatch).
- Add __must_check to function declaration (thanks Steve).
- Use hw RNG by default if available (as originally suggested by Kees).
- Add command line option to use cryptographically insecure hashing.
If debug_early_boot is enabled use hash_long() instead of siphash
(as requested by Steve, and solves original problem for Anna-Maria).
- Added Acked-by tag from Ted (patch 1 and 2)
Tobin C. Harding (4):
random: Fix whitespace pre random-bytes work
random: Return nbytes filled from hw RNG
vsprintf: Use hw RNG for ptr_key
vsprintf: Add command line option debug_boot_weak_hash
.../admin-guide/kernel-parameters.txt | 9 +++++++
drivers/char/random.c | 19 ++++++-------
include/linux/random.h | 2 +-
lib/vsprintf.c | 27 ++++++++++++++++++-
4 files changed, 46 insertions(+), 11 deletions(-)
--
2.17.1
next reply other threads:[~2018-06-20 4:21 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-06-20 4:20 Tobin C. Harding [this message]
2018-06-20 4:20 ` [PATCH v8 1/4] random: Fix whitespace pre random-bytes work Tobin C. Harding
2018-06-20 4:20 ` [PATCH v8 2/4] random: Return nbytes filled from hw RNG Tobin C. Harding
2018-06-20 4:20 ` [PATCH v8 3/4] vsprintf: Use hw RNG for ptr_key Tobin C. Harding
2018-06-20 4:20 ` [PATCH v8 4/4] vsprintf: Add command line option debug_boot_weak_hash Tobin C. Harding
2018-06-20 16:09 ` Randy Dunlap
2018-06-20 22:30 ` Tobin C. Harding
2018-06-20 22:36 ` Randy Dunlap
2018-06-20 23:22 ` Tobin C. Harding
2018-06-20 23:38 ` Randy Dunlap
2018-06-21 3:15 ` Tobin C. Harding
2018-06-21 4:09 ` Randy Dunlap
2018-06-21 5:40 ` Tobin C. Harding
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20180620042046.30806-1-me@tobin.cc \
--to=me@tobin.cc \
--cc=akpm@linux-foundation.org \
--cc=andy.shevchenko@gmail.com \
--cc=anna-maria@linutronix.de \
--cc=arnd@arndb.de \
--cc=gregkh@linuxfoundation.org \
--cc=keescook@chromium.org \
--cc=linux-kernel@vger.kernel.org \
--cc=rdunlap@infradead.org \
--cc=rostedt@goodmis.org \
--cc=torvalds@linux-foundation.org \
--cc=tytso@mit.edu \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox