On 03/21/2017 09:43 PM, Dan Carpenter wrote: > The "need_cache_flush" variable is never set to false. When the > variable is true that means we print a warning message at the end of > the function. > > Fixes: 3418d036c81d ("raid5-ppl: Partial Parity Log write logging implementation") > Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> Good catch, thanks. I thought gcc would warn about such issues, I'll have to be more careful. Reviewed-by: Artur Paszkiewicz <artur.paszkiewicz@xxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html