| Message ID | 148419040712.32674.16288936788225568096.stgit@birch.djwong.org (mailing list archive) |
|---|---|
| State | Accepted |
| Headers | show |
On Wed, Jan 11, 2017 at 07:06:47PM -0800, Darrick J. Wong wrote: > Prefix the perror output of the dedupe command consistently. All the > other perror calls reference the ioctl name directly, so we might as > well do that for all the dedupe cases. Looks good, Reviewed-by: Christoph Hellwig <hch@lst.de> -- To unsubscribe from this list: send the line "unsubscribe linux-xfs" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/io/reflink.c b/io/reflink.c index 41cb884..4c822b3 100644 --- a/io/reflink.c +++ b/io/reflink.c @@ -78,12 +78,12 @@ dedupe_ioctl( goto done; } if (info->status < 0) { - fprintf(stderr, "dedupe: %s\n", + fprintf(stderr, "XFS_IOC_FILE_EXTENT_SAME: %s\n", _(strerror(-info->status))); goto done; } if (info->status == XFS_EXTENT_DATA_DIFFERS) { - fprintf(stderr, "dedupe: %s\n", + fprintf(stderr, "XFS_IOC_FILE_EXTENT_SAME: %s\n", _("Extents did not match.")); goto done; }
Prefix the perror output of the dedupe command consistently. All the other perror calls reference the ioctl name directly, so we might as well do that for all the dedupe cases. Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com> --- io/reflink.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) -- To unsubscribe from this list: send the line "unsubscribe linux-xfs" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html