On Fri, Jul 26, 2019 at 10:19:07PM +0200, Thomas Gleixner wrote: > While looking at CONFIG_PREEMPT dependencies treewide the #ifdef in > crypto_yield() matched. > > CONFIG_PREEMPT and CONFIG_PREEMPT_VOLUNTARY are mutually exclusive so the > extra !CONFIG_PREEMPT conditional is redundant. > > cond_resched() has only an effect when CONFIG_PREEMPT_VOLUNTARY is set, > otherwise it's a stub which the compiler optimizes out. > > Remove the whole conditional. > > No functional change. > > Signed-off-by: Thomas Gleixner <tglx@xxxxxxxxxxxxx> > Cc: linux-crypto@xxxxxxxxxxxxxxx > Cc: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx> > Cc: "David S. Miller" <davem@xxxxxxxxxxxxx> > --- > include/crypto/algapi.h | 2 -- > 1 file changed, 2 deletions(-) Patch applied. Thanks. -- Email: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx> Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt