[PATCH] btrfs-progs: Remove the get_fslist error message.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



When calling get_fslist on a kernel doesn't support
BTRFS_IOC_GET_FSLIST, an error message will output but the codes will
continue using the old way to scan mounted fs.

Since the error message will fail xfstest/btrfs/006 testcase and there
is a fallback way, the message should not be printed.

Also other error message in get_fslist is not needed since the functions
in utils.c should not print any thing and caller will handle the error.
This patch just remove the unneeded perror messages.

Signed-off-by: Qu Wenruo <quwenruo@xxxxxxxxxxxxxx>
---
 utils.c | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/utils.c b/utils.c
index 33b3ddc..364c191 100644
--- a/utils.c
+++ b/utils.c
@@ -2209,10 +2209,8 @@ int get_fslist(struct btrfs_ioctl_fslist **out_fslist, u64 *out_count)
 
 	fd = open("/dev/btrfs-control", O_RDWR);
 	e = errno;
-	if (fd < 0) {
-		perror("failed to open /dev/btrfs-control");
+	if (fd < 0)
 		return -e;
-	}
 
 	/* space to hold 512 fsids, doesn't matter if small
 	 * it would fail and return count so then we try again
@@ -2238,8 +2236,6 @@ again:
 		free(fsargs);
 		goto again;
 	} else if (ret < 0) {
-		printf("ERROR: scan_fsid ioctl failed - %s\n",
-			strerror(e));
 		ret = -e;
 		goto out;
 	}
-- 
1.9.0

--
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




[Index of Archives]     [Linux Filesystem Development]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux