On Mon, Oct 22, 2012 at 5:39 AM, Miao Xie <mi...@cn.fujitsu.com> wrote:
> Step to reproduce:
>  # mkfs.btrfs <disk>
>  # mount <disk> <mnt>
>  # btrfs sub create <mnt>/subv0
>  # btrfs sub snap <mnt> <mnt>/subv0/snap0
>  # change <mnt>/subv0 from R/W to R/O
>  # btrfs sub del <mnt>/subv0/snap0
>
> We deleted the snapshot successfully. I think we should not be able to delete
> the snapshot since the parent subvolume is R/O.

snap0 isn't read-only in that case, right?  From a user interaction
standpoint, this seems like it just forces a user to rm -rf rather
btrfs sub del, which strikes me as a bit ham-handed when all we really
care about is leaving a (the?) directory entry where snap0 used to be.
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to