diff options
| author | Kevin Wolf <kwolf@redhat.com> | 2014-11-21 17:43:57 +0100 |
|---|---|---|
| committer | Kevin Wolf <kwolf@redhat.com> | 2014-11-24 13:52:10 +0100 |
| commit | 24bf10dac389942b58f0d24e96590c927251b3d6 (patch) | |
| tree | c6c477ab383aef474d21ce9d7279fc1673d4322e /block | |
| parent | 0e88f478508b566152c6681f4889ed9830a2c0a5 (diff) | |
Revert "qemu-img info: show nocow info"
This reverts commit 000c4dfff4d7686e2fba3066a477a1290ed60622.
The main reason for reverting this commit before the 2.2 release is that
it adds a QAPI interface that we don't want to keep: The 'nocow' flag
doesn't generally make sense for block nodes, but only for the raw-posix
driver. It should therefore be part of ImageInfoSpecific rather than
ImageInfo.
The commit contains more problems, but unlike the API stability issue
they wouldn't justify reverting it.
Conflicts:
block/qapi.c
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
Diffstat (limited to 'block')
| -rw-r--r-- | block/qapi.c | 26 |
1 files changed, 0 insertions, 26 deletions
diff --git a/block/qapi.c b/block/qapi.c index 1301144bbc..a87a34a8b4 100644 --- a/block/qapi.c +++ b/block/qapi.c @@ -29,13 +29,6 @@ #include "qapi/qmp-output-visitor.h" #include "qapi/qmp/types.h" #include "sysemu/block-backend.h" -#ifdef __linux__ -#include <linux/fs.h> -#include <sys/ioctl.h> -#ifndef FS_NOCOW_FL -#define FS_NOCOW_FL 0x00800000 /* Do not cow file */ -#endif -#endif BlockDeviceInfo *bdrv_block_device_info(BlockDriverState *bs) { @@ -180,9 +173,6 @@ void bdrv_query_image_info(BlockDriverState *bs, int ret; Error *err = NULL; ImageInfo *info; -#ifdef __linux__ - int fd, attr; -#endif size = bdrv_getlength(bs); if (size < 0) { @@ -212,18 +202,6 @@ void bdrv_query_image_info(BlockDriverState *bs, info->format_specific = bdrv_get_specific_info(bs); info->has_format_specific = info->format_specific != NULL; -#ifdef __linux__ - /* get NOCOW info */ - fd = qemu_open(bs->filename, O_RDONLY | O_NONBLOCK); - if (fd >= 0) { - if (ioctl(fd, FS_IOC_GETFLAGS, &attr) == 0 && (attr & FS_NOCOW_FL)) { - info->has_nocow = true; - info->nocow = true; - } - qemu_close(fd); - } -#endif - backing_filename = bs->backing_file; if (backing_filename[0] != '\0') { info->backing_filename = g_strdup(backing_filename); @@ -655,8 +633,4 @@ void bdrv_image_info_dump(fprintf_function func_fprintf, void *f, func_fprintf(f, "Format specific information:\n"); bdrv_image_info_specific_dump(func_fprintf, f, info->format_specific); } - - if (info->has_nocow && info->nocow) { - func_fprintf(f, "NOCOW flag: set\n"); - } } |