gfs2: Allow lock_nolock mount to specify jid=X
authorBob Peterson <rpeterso@redhat.com>
Wed, 29 Apr 2020 13:45:54 +0000 (08:45 -0500)
committerAndreas Gruenbacher <agruenba@redhat.com>
Tue, 2 Jun 2020 17:45:05 +0000 (19:45 +0200)
Before this patch, a simple typo accidentally added \n to the jid=
string for lock_nolock mounts. This made it impossible to mount a
gfs2 file system with a journal other than journal0. Thus:

mount -tgfs2 -o hostdata="jid=1" <device> <mount pt>

Resulted in:
mount: wrong fs type, bad option, bad superblock on <device>

In most cases this is not a problem. However, for debugging and
testing purposes we sometimes want to test the integrity of other
journals. This patch removes the unnecessary \n and thus allows
lock_nolock users to specify an alternate journal.

Signed-off-by: Bob Peterson <rpeterso@redhat.com>
Signed-off-by: Andreas Gruenbacher <agruenba@redhat.com>
fs/gfs2/ops_fstype.c

index e2b69ffcc6a8414ed9c48dbe42f7be8fd0a2c57a..094f5fe7c00906aee304e273a5169157967aab7a 100644 (file)
@@ -880,7 +880,7 @@ fail:
 }
 
 static const match_table_t nolock_tokens = {
-       { Opt_jid, "jid=%d\n", },
+       { Opt_jid, "jid=%d", },
        { Opt_err, NULL },
 };