On 07/31/2012 08:53 AM, Alexander Karbstein wrote:

> The command btrfs subvolume set-default 0 /path/to/fs changed the
> default subvolume to whatever subvolume was currently mounted on
> /path/to/fs. This patch changes this behaviour to set the default
> subvolume to BTRFS_FS_TREE_OBJECTID in case the user asks for
> subvolid=0
> 

> Signed-off-by: Alexander Karbstein <alexander.karbst...@gmail.com>
> ---
>  btrfs_cmds.c |    5 +++++
>  1 files changed, 5 insertions(+), 0 deletions(-)
> 
> diff --git a/btrfs_cmds.c b/btrfs_cmds.c
> index f2b6355..699d9b0 100644
> --- a/btrfs_cmds.c
> +++ b/btrfs_cmds.c
> @@ -964,6 +964,11 @@ int do_set_default_subvol(int nargs, char **argv)
>               fprintf(stderr, "ERROR: invalid tree id (%s)\n",subvolid);
>               return 30;
>       }
> +     
> +     /* Using the original root fs tree */
> +     if (objectid == 0ULL) {
> +             objectid = BTRFS_FS_TREE_OBJECTID;
> +     }
>       ret = ioctl(fd, BTRFS_IOC_DEFAULT_SUBVOL, &objectid);
>       e = errno;
>       close(fd);


Seems that you're using an old btrfs-prog source code, now the file is 
cmds-subvolume.c

And also checkpatch.pl complains about style errors while checking this.

thanks,
liubo
--
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