Re: [PATCH] crypto: Remove redundant #ifdef in crypto_yield()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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



[Index of Archives]     [Kernel]     [Gnu Classpath]     [Gnu Crypto]     [DM Crypt]     [Netfilter]     [Bugtraq]

  Powered by Linux