On 07/24/2013 02:18 AM, Eric Sandeen wrote:
Have you tested it? What's the difference in output?
- ext4_msg(sb, KERN_INFO, "mounted filesystem with%s. "
- "Opts: %s%s%s", descr, sbi->s_es->s_mount_opts, + ext4_msg(sb, KERN_INFO, "mounted filesystem with%s " + "%s%s%s mount option(s)", descr, sbi->s_es->s_mount_opts, *sbi->s_es->s_mount_opts ? "; " : "", orig_data); From mounted filesystem with Opts: (null) re-mounted. Opts:(null) To mounted filesystem with (null) mount option(s) re-mounted (null) mount option(s) That (null) could change to no or none I suppose but it still feels a bit off thou maybe this output sample would be better? if = (null) mounted filesystem re-mounted filesystem elif mounted filesystem with x,y.x mount option(s) re-mounted filesystem with x,y.x mount option(s) JBG -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html