On Sat 03-09-16 17:33:39, Eric Ren wrote:

Hi Geliang,

On 08/31/2016 08:23 PM, Geliang Tang wrote:

quoted

There are some repetitive code in jbd2_journal_init_dev() and
jbd2_journal_init_inode(). So this patch extracts the common code into
jbd2_journal_init() helper to simplify the code. And fix the coding
style warnings reported by checkpatch.pl by the way.

journal_init_common() is already there for the same purpose, right?
Do we really need another helper?

Agreed, just move the common code into journal_init_common(). Also the
__getblk() in jbd2_journal_init_dev() should be getblk_unmovable() so that
would be a good preparatory fix to remove the need of the gfp argument.


								Honza
-- 
Jan Kara [off-list ref]
SUSE Labs, CR