Re: [PATCH 2/2] fault-inject: add ratelimit option

From: Dmitry Monakhov
Date: Wed Aug 20 2014 - 10:36:27 EST


On Wed, 20 Aug 2014 23:20:18 +0900, Akinobu Mita <akinobu.mita@xxxxxxxxx> wrote:
> This looks a bit redundant. NULL pointer to "%pd" format produces
> "(null)" string, so this printk and if-else can be removed.
>
> Also, this message line is a bit longer than usual kernel message.
> Should we put '\n' after "forcing a failure."?
>
> This patch looks good to me. Please feel free to add my ACK.
Ok. Here is updated version.