hwrng: Make explicit that max >= 32 always
authorPrasannaKumar Muralidharan <prasannatsmkumar@gmail.com>
Fri, 18 Nov 2016 17:30:10 +0000 (23:00 +0530)
committerHerbert Xu <herbert@gondor.apana.org.au>
Mon, 21 Nov 2016 14:50:45 +0000 (22:50 +0800)
As hw_random core calls ->read with max > 32 or more, make it explicit.
Also remove checks involving 'max' being less than 8.

Signed-off-by: PrasannaKumar Muralidharan <prasannatsmkumar@gmail.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
drivers/char/hw_random/msm-rng.c
drivers/char/hw_random/pic32-rng.c
drivers/char/hw_random/pseries-rng.c
include/linux/hw_random.h

index 96fb986402eb4e1d6682ac05377979c92efde2cd..841fee845ec9e18610769912e1cc054955fe5d6d 100644 (file)
@@ -90,10 +90,6 @@ static int msm_rng_read(struct hwrng *hwrng, void *data, size_t max, bool wait)
        /* calculate max size bytes to transfer back to caller */
        maxsize = min_t(size_t, MAX_HW_FIFO_SIZE, max);
 
-       /* no room for word data */
-       if (maxsize < WORD_SZ)
-               return 0;
-
        ret = clk_prepare_enable(rng->clk);
        if (ret)
                return ret;
index 11dc9b7c09cef7dbf168f9cc156b0d7accedfc66..9b5e68a71d016016fac96ab5f2a27f46b970fb49 100644 (file)
@@ -62,9 +62,6 @@ static int pic32_rng_read(struct hwrng *rng, void *buf, size_t max,
        u32 t;
        unsigned int timeout = RNG_TIMEOUT;
 
-       if (max < 8)
-               return 0;
-
        do {
                t = readl(priv->base + RNGRCNT) & RCNT_MASK;
                if (t == 64) {
index 63ce51d09af1be79dab9b65c8320001d3ee568d2..d9f46b437cc20d93556957168cc6440c0cb96785 100644 (file)
@@ -28,7 +28,6 @@
 static int pseries_rng_read(struct hwrng *rng, void *data, size_t max, bool wait)
 {
        u64 buffer[PLPAR_HCALL_BUFSIZE];
-       size_t size = max < 8 ? max : 8;
        int rc;
 
        rc = plpar_hcall(H_RANDOM, (unsigned long *)buffer);
@@ -36,10 +35,10 @@ static int pseries_rng_read(struct hwrng *rng, void *data, size_t max, bool wait
                pr_err_ratelimited("H_RANDOM call failed %d\n", rc);
                return -EIO;
        }
-       memcpy(data, buffer, size);
+       memcpy(data, buffer, 8);
 
        /* The hypervisor interface returns 64 bits */
-       return size;
+       return 8;
 }
 
 /**
index 34a0dc18f3271ef01b14642c5117274c12fe18f1..bee0827766a397d6d5ff3a2f8cdf42b97925f3c8 100644 (file)
@@ -30,8 +30,7 @@
  *                     Must not be NULL.    *OBSOLETE*
  * @read:              New API. drivers can fill up to max bytes of data
  *                     into the buffer. The buffer is aligned for any type
- *                     and max is guaranteed to be >= to that alignment
- *                     (either 4 or 8 depending on architecture).
+ *                     and max is a multiple of 4 and >= 32 bytes.
  * @priv:              Private data, for use by the RNG driver.
  * @quality:           Estimation of true entropy in RNG's bitstream
  *                     (per mill).