On 12/07/2015 11:38 PM, Chandan Rajendra wrote:
On Monday 07 Dec 2015 14:06:42 Qu Wenruo wrote:
Introduce a new mount option "nologreplay" to co-operate with "ro" mount
option to get real readonly mount, like "norecovery" in ext* and xfs.
Since the new parse_options() need to check new flags at remount time,
so add a new parameter for parse_options().
Hello Qu,
The patch should add an entry into btrfs_show_options() for 'mount' command to
be able to display an entry corresponding to 'nologreplay' mount option.
Thanks for the advice.
I just forgot show_mount_options().
And for option to disable "nologreplay", I follow the behavior of xfs
and ext4 "norecovery" behavior.
Which means no option to disable "norecovery", as it is only used for
debugging case.
So the only method is to umount it and mount it again.
Even after adding such an entry to btrfs_show_options(),
the following occurs,
# mount -o nologreplay,ro /dev/loop6 /mnt/
# mount | grep loop6
/dev/loop6 on /mnt type btrfs (ro,relatime,seclabel,nologreplay,space_cache,subvolid=5,subvol=/)
# mount -o remount,rw /dev/loop6 /mnt/
# mount | grep loop6
/dev/loop6 on /mnt type btrfs (rw,relatime,seclabel,nologreplay,space_cache,subvolid=5,subvol=/)
As can be seen from the last line, both 'rw' and 'nologreplay' options are set.
I'll double check it, maybe it bypassed btrfs_parse_options(), and
bypassed the check in it.
Thanks,
Qu
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
