* Re: [PATCH] vfs: set proper level for printk in fs/super.c [not found] ` <20100320133848.GD5331@bicker> @ 2010-03-22 12:14 ` "潘卫平(Peter Pan)" 0 siblings, 0 replies; 2+ messages in thread From: @ 2010-03-22 12:14 UTC (permalink / raw) To: Dan Carpenter; +Cc: kernel-janitors, linux-fsdevel On 03/20/2010 09:38 PM, Dan Carpenter wrote: > On Sat, Mar 20, 2010 at 09:10:07PM +0800, "潘卫平(Peter Pan)" wrote: >> Signed-off-by: Peter Pan(潘卫平)<wppan@redflag-linux.com> >> --- >> fs/super.c | 4 ++-- >> 1 files changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/fs/super.c b/fs/super.c >> index f35ac60..60d720f 100644 >> --- a/fs/super.c >> +++ b/fs/super.c >> @@ -312,7 +312,7 @@ void generic_shutdown_super(struct super_block *sb) >> >> /* Forget any remaining inodes */ >> if (invalidate_inodes(sb)) { >> - printk("VFS: Busy inodes after unmount of %s. " >> + printk(KERN_INFO "VFS: Busy inodes after unmount of %s. " >> "Self-destruct in 5 seconds. Have a nice day...\n", > > I'm pretty sure that both of these should be a higher priority than > KERN_INFO. > > regards, > dan carpenter > >> sb->s_id); >> } >> @@ -642,7 +642,7 @@ static void do_emergency_remount(struct work_struct >> *work) >> } >> spin_unlock(&sb_lock); >> kfree(work); >> - printk("Emergency Remount complete\n"); >> + printk(KERN_INFO "Emergency Remount complete\n"); >> } >> >> void emergency_remount(void) >> -- >> 1.6.6 >> >> >> -- >> To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in >> the body of a message to majordomo@vger.kernel.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html > Signed-off-by: 潘卫平(Peter Pan) <wppan@redflag-linux.com> --- fs/super.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/fs/super.c b/fs/super.c index f35ac60..baffd23 100644 --- a/fs/super.c +++ b/fs/super.c @@ -312,7 +312,7 @@ void generic_shutdown_super(struct super_block *sb) /* Forget any remaining inodes */ if (invalidate_inodes(sb)) { - printk("VFS: Busy inodes after unmount of %s. " + printk(KERN_WARNING "VFS: Busy inodes after unmount of %s. " "Self-destruct in 5 seconds. Have a nice day...\n", sb->s_id); } @@ -642,7 +642,7 @@ static void do_emergency_remount(struct work_struct *work) } spin_unlock(&sb_lock); kfree(work); - printk("Emergency Remount complete\n"); + printk(KERN_NOTICE "Emergency Remount complete\n"); } void emergency_remount(void) -- 1.6.6 -- Peter Pan(潘卫平) Red Flag Software Co.,Ltd ^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] vfs: set proper level for printk in fs/super.c @ 2010-03-22 12:14 ` "潘卫平(Peter Pan)" 0 siblings, 0 replies; 2+ messages in thread From: "潘卫平(Peter Pan)" @ 2010-03-22 12:14 UTC (permalink / raw) To: Dan Carpenter; +Cc: kernel-janitors, linux-fsdevel On 03/20/2010 09:38 PM, Dan Carpenter wrote: > On Sat, Mar 20, 2010 at 09:10:07PM +0800, "潘卫平(Peter Pan)" wrote: >> Signed-off-by: Peter Pan(潘卫平)<wppan@redflag-linux.com> >> --- >> fs/super.c | 4 ++-- >> 1 files changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/fs/super.c b/fs/super.c >> index f35ac60..60d720f 100644 >> --- a/fs/super.c >> +++ b/fs/super.c >> @@ -312,7 +312,7 @@ void generic_shutdown_super(struct super_block *sb) >> >> /* Forget any remaining inodes */ >> if (invalidate_inodes(sb)) { >> - printk("VFS: Busy inodes after unmount of %s. " >> + printk(KERN_INFO "VFS: Busy inodes after unmount of %s. " >> "Self-destruct in 5 seconds. Have a nice day...\n", > > I'm pretty sure that both of these should be a higher priority than > KERN_INFO. > > regards, > dan carpenter > >> sb->s_id); >> } >> @@ -642,7 +642,7 @@ static void do_emergency_remount(struct work_struct >> *work) >> } >> spin_unlock(&sb_lock); >> kfree(work); >> - printk("Emergency Remount complete\n"); >> + printk(KERN_INFO "Emergency Remount complete\n"); >> } >> >> void emergency_remount(void) >> -- >> 1.6.6 >> >> >> -- >> To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in >> the body of a message to majordomo@vger.kernel.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html > Signed-off-by: 潘卫平(Peter Pan) <wppan@redflag-linux.com> --- fs/super.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/fs/super.c b/fs/super.c index f35ac60..baffd23 100644 --- a/fs/super.c +++ b/fs/super.c @@ -312,7 +312,7 @@ void generic_shutdown_super(struct super_block *sb) /* Forget any remaining inodes */ if (invalidate_inodes(sb)) { - printk("VFS: Busy inodes after unmount of %s. " + printk(KERN_WARNING "VFS: Busy inodes after unmount of %s. " "Self-destruct in 5 seconds. Have a nice day...\n", sb->s_id); } @@ -642,7 +642,7 @@ static void do_emergency_remount(struct work_struct *work) } spin_unlock(&sb_lock); kfree(work); - printk("Emergency Remount complete\n"); + printk(KERN_NOTICE "Emergency Remount complete\n"); } void emergency_remount(void) -- 1.6.6 -- Peter Pan(潘卫平) Red Flag Software Co.,Ltd -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply related [flat|nested] 2+ messages in thread
end of thread, other threads:[~2010-03-22 12:44 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <4BA4C92F.5070007@redflag-linux.com>
[not found] ` <20100320133848.GD5331@bicker>
2010-03-22 12:14 ` [PATCH] vfs: set proper level for printk in fs/super.c
2010-03-22 12:14 ` "潘卫平(Peter Pan)"
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.