On Thu, Nov 21, 2013 at 11:32:36AM +0800, Anand Jain wrote:
> On 11/20/2013 10:18 PM, David Sterba wrote:
> >On Fri, Nov 15, 2013 at 07:25:34PM +0800, Anand Jain wrote:
> >> static int btrfs_scan_kernel(void *search)
> >> {
> >>- int ret = 0, fd;
> >>- FILE *f;
> >>- struct mntent *mnt;
> >>- struct btrfs_ioctl_fs_info_args fs_info_arg;
> >>- struct btrfs_ioctl_dev_info_args *dev_info_arg = NULL;
> >>- struct btrfs_ioctl_space_args *space_info_arg;
> >>+ int ret = 0;
> >> char label[BTRFS_LABEL_SIZE];
> >>-
> >>- f = setmntent("/proc/self/mounts", "r");
> >>- if (f == NULL)
> >>- return 1;
> >>-
> >>- memset(label, 0, sizeof(label));
> >>- while ((mnt = getmntent(f)) != NULL) {
> >>- if (strcmp(mnt->mnt_type, "btrfs"))
> >>+ char mnt[BTRFS_PATH_NAME_MAX + 1];
> >>+ struct btrfs_ioctl_fslist *fslist;
> >>+ struct btrfs_ioctl_fslist *fslist_saved;
> >
> > [CC] cmds-send.o
> >cmds-filesystem.c: In function ‘btrfs_scan_kernel’:
> >cmds-filesystem.c:461:29: warning: variable ‘fslist_saved’ set but not used
> >[-Wunused-but-set-variable]
>
> Thanks David. That was something for later todo which I missed.
> sorry my mistake. Sent out v5.
Thanks, patch replaced.
--
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
