random: ignore GRND_RANDOM in getentropy(2)

The separate blocking pool is going away.  Start by ignoring
GRND_RANDOM in getentropy(2).

This should not materially break any API.  Any code that worked
without this change should work at least as well with this change.

Signed-off-by: Andy Lutomirski <luto@kernel.org>
Link: https://lore.kernel.org/r/705c5a091b63cc5da70c99304bb97e0109be0a26.1577088521.git.luto@kernel.org
Signed-off-by: Theodore Ts'o <tytso@mit.edu>

diff --git a/drivers/char/random.c b/drivers/char/random.c
index b7e2ad7eafcaf23f4f28ef08f0ce656884401354..a92dfea038ceb993a7ade90f6c23b69ba62d3b63 100644 (file)
--- a/drivers/char/random.c
+++ b/drivers/char/random.c
@@ -2207,9 +2207,6 @@ SYSCALL_DEFINE3(getrandom, char __user *, buf, size_t, count,
        if (count > INT_MAX)
                count = INT_MAX;
 
-       if (flags & GRND_RANDOM)
-               return _random_read(flags & GRND_NONBLOCK, buf, count);
-
        if (!(flags & GRND_INSECURE) && !crng_ready()) {
                if (flags & GRND_NONBLOCK)
                        return -EAGAIN;

diff --git a/include/uapi/linux/random.h b/include/uapi/linux/random.h
index c092d20088d3b4e156fe6be74bd492b3e3055ec3..dcc1b3e6106fe20de80d06d0c842fdb12fb561e2 100644 (file)
--- a/include/uapi/linux/random.h
+++ b/include/uapi/linux/random.h
@@ -48,7 +48,7 @@ struct rand_pool_info {
  * Flags for getrandom(2)
  *
  * GRND_NONBLOCK       Don't block and return EAGAIN instead
- * GRND_RANDOM         Use the /dev/random pool instead of /dev/urandom
+ * GRND_RANDOM         No effect
  * GRND_INSECURE       Return non-cryptographic random bytes
  */
 #define GRND_NONBLOCK  0x0001