Hi Ryusuke Konishi, The patch is resent. Thanks very much for your review and suggestions. Regards, Li Hong ------------------ cut here ----------------------- >From d5ecb6ca6bafa2c9aa7058aee8918b94eea9b993 Mon Sep 17 00:00:00 2001 From: Li Hong <lihong.hi@xxxxxxxxx> Date: Tue, 6 Apr 2010 01:23:04 +0800 Subject: [PATCH] nilfs2: add a print message after loading nilfs2 Printing a message after loading a file system is a practice. Add this to provide a better user-friendly experience. Signed-off-by: Li Hong <lihong.hi@xxxxxxxxx> --- fs/nilfs2/super.c | 7 ++++++- 1 files changed, 6 insertions(+), 1 deletions(-) diff --git a/fs/nilfs2/super.c b/fs/nilfs2/super.c index 2e767bc..8777172 100644 --- a/fs/nilfs2/super.c +++ b/fs/nilfs2/super.c @@ -1186,7 +1186,12 @@ static int __init init_nilfs_fs(void) return err; err = register_filesystem(&nilfs_fs_type); - return err; + if (err) + return err; + + printk(KERN_INFO "NILFS version 2 loaded\n"); + return 0; + } static void __exit exit_nilfs_fs(void) -- 1.6.3.3 On Thu, Apr 08, 2010 at 04:21:41PM +0900, Ryusuke Konishi wrote: > On Thu, 8 Apr 2010 13:45:05 +0800, Li Hong <lihong.hi@xxxxxxxxx> wrote: > > 2010/4/8 Ryusuke Konishi <ryusuke@xxxxxxxx>: > > > On Wed, 7 Apr 2010 22:20:58 +0800, Li Hong <lihong.hi@xxxxxxxxx> wrote: > > >> Hi KONISHI Ryusuke, > > >> > > >> Any feedbacks on these patches? > > >> > > >> Thanks, > > >> Li Hong > > > > > > Well, adding a load message is ok, but it should be more moderate like > > > "nilfs2 file system loaded\n". > > Yes. I just thought that printing a message after loading a file > > system is a practice. > > You may choose any string that you think is suitable :) > > How about "NILFS version 2 loaded\n" ? > > Seems that most messages have prefix "NILFS: ..." or "NILFS (device > %s):". > > > > I think this sort of change should be discussed in the sub topic > > > mailing list (i.e. linux-nilfs). You don't have to cc every patch to > > > linux-kernel. > > Ok. No problem. > > > > > And, please include some explanation in the change log to clarify the > > > aim or motivation. > > I took it as a very simple patch and thought a header was enough. I can add more > > explanations if here has a demand. > > You thought "printing a message after loading a file system is a > practice.", so it is the reason :) > > Thanks, > Ryusuke Konishi -- To unsubscribe from this list: send the line "unsubscribe linux-nilfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html