Re: [PATCH 6/7] random: make /dev/urandom scalable for silly userspace programs

On Sun, Aug 21, 2016 at 12:53:15PM +0300, Jan Varho wrote:

On Mon, Jun 13, 2016 at 6:48 PM, Theodore Ts'o [off-list ref] wrote:

quoted

+static inline void maybe_reseed_primary_crng(void)
+{
+       if (crng_init > 2 &&
+           time_after(jiffies, primary_crng.init_time + CRNG_RESEED_INTERVAL))
+               crng_reseed(&primary_crng, &input_pool);
+}

Is the above function (which is now in 4.8-rc2) supposed to do
something? It seems to have no callers and the maximum value of
crng_init is 2.

It's dead code.  Its function got moved into _extra_crng(), and you're
right, these days crng_init never gets above 2.  Thanks for pointing
that out.  I'll take it out as a cleanup patch.

     	   	     	       	 	 - Ted