David Sterba wrote on 2016/01/14 17:00 +0100:
On Tue, Jan 12, 2016 at 10:05:11AM +0800, Qu Wenruo wrote:
'Norecovery' mount option is almost a standard for all other
filesystems, including f2fs, ext4, gfs2, nilfs2, and xfs.
And since current 'recovery' mount option for btrfs is too generic, it's
better to change btrfs 'recovery' to 'usebackuproot' and then we can
safely add 'norecovery' and 'nologreplay' mount option for btrfs.
The patch itself is not huge and already tested by others since v2.
Only mount option name is changing during all these version changes.
So we have some kind of consensus about the options, but I think you
need to split the patches further.
It's never a bad idea to split patches :)
* the new option 'nologreplay'
* keep 'usebackuproot' as a new alias for 'recovery' (with the
deprecation warning)
* new 'norecovery' that switches on 'nologreplay' and 'nousebackuproot'
(ie. the desired "no writes")
There is a case in remount, when we start with 'ro,nologreplay' then
remount with just 'ro', the log should be replayed. The other
combinations should be easy. I haven't looked at the patch if it's
implemented that way though.
For that case, even remount with -o remount,ro the nologreplay mount
option won't be erased.
That's why we added a lot of "noXXXX" mount option, or we can't disable
some feature by remount.
And since I didn't add such "logreplay" mount option intentionally, so
there is no way to replay log unless umount the fs and mount it without
"nologreplay" again.
I'll update the patchset soon and hopes it can catch the rc1 merge window.
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
