On Wed, Dec 20, 2017 at 02:09:18PM -0600, Petros Koutoupis wrote: > Minor code cleanup: Fixed alignment and a reduction of lines in header file. > > Signed-off-by: Petros Koutoupis <petros@xxxxxxxxxxxxxxxxxxx> The patch is whitespace damaged, unfortunately. Also, the cleanup actually isn't the best cleanup. We should really be changing ext4_chksum() to take a struct super instead of a struct ext4_sb_info, so we can call ext4_std_error() so we can properly log an error without immediately crashing the kernel. The chksum shouldn't really ever fail, but if it does, calling ext4_error() is better than bringing down the whole system with a BUG_ON. Cheers, - Ted
