On Wed, 20 Aug 2014 23:20:18 +0900, Akinobu Mita 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.