Message ID | 1345042466-31743-4-git-send-email-benoit@irqsave.net |
---|---|
State | New |
Headers | show |
On 08/15/2012 08:54 AM, Benoît Canet wrote: > This additionnal --machine=json option make qemu-img info output on > stdout a JSON formated representation of the image informations. > > --machine=json was choosen instead of --format=json because the > info command already have a -f parameter. Do we want a counterpart '--machine=text' or '--machine=human' to explicitly specify current output format? > @@ -1113,10 +1174,20 @@ static int img_info(int argc, char **argv) > char backing_filename[1024]; > char backing_filename2[1024]; > BlockDriverInfo bdi; > + ImageInfo *image_info; > > fmt = NULL; > + machine = NULL; > for(;;) { > - c = getopt(argc, argv, "f:h"); > + int option_index = 0; > + static struct option long_options[] = { > + {"help", no_argument, 0, 'h'}, > + {"format", required_argument, 0, 'f'}, > + {"machine", required_argument, 0, 'm'}, > + {0, 0, 0, 0} > + }; > + c = getopt_long(argc, argv, "f:h", > + long_options, &option_index); getopt_long is not in POSIX. Are you sure this will compile on all target platforms?
Le Wednesday 15 Aug 2012 à 10:23:32 (-0600), Eric Blake a écrit : > On 08/15/2012 08:54 AM, Benoît Canet wrote: > > This additionnal --machine=json option make qemu-img info output on > > stdout a JSON formated representation of the image informations. > > > > --machine=json was choosen instead of --format=json because the > > info command already have a -f parameter. > > Do we want a counterpart '--machine=text' or '--machine=human' to > explicitly specify current output format? > > > @@ -1113,10 +1174,20 @@ static int img_info(int argc, char **argv) > > char backing_filename[1024]; > > char backing_filename2[1024]; > > BlockDriverInfo bdi; > > + ImageInfo *image_info; > > > > fmt = NULL; > > + machine = NULL; > > for(;;) { > > - c = getopt(argc, argv, "f:h"); > > + int option_index = 0; > > + static struct option long_options[] = { > > + {"help", no_argument, 0, 'h'}, > > + {"format", required_argument, 0, 'f'}, > > + {"machine", required_argument, 0, 'm'}, > > + {0, 0, 0, 0} > > + }; > > + c = getopt_long(argc, argv, "f:h", > > + long_options, &option_index); > > getopt_long is not in POSIX. Are you sure this will compile on all > target platforms? benoit@Laure:~/qemu$ git grep getopt_long fsdev/virtfs-proxy-helper.c: c = getopt_long(argc, argv, "p:nh?f:s:u:g:", helper_opts, qemu-ga.c: while ((ch = getopt_long(argc, argv, sopt, lopt, &opt_ind)) != -1) { qemu-io.c: while ((c = getopt_long(argc, argv, sopt, lopt, &opt_index)) != -1) { qemu-nbd.c: while ((ch = getopt_long(argc, argv, sopt, lopt, &opt_ind)) != -1) { savevm.c:#define getopt_long_only getopt_long A few occurence of getopg_long are already in qemu. Is it a sufficient reason to keep getopt_long ? If not what would be the alternatives ? Benoît > > -- > Eric Blake eblake@redhat.com +1-919-301-3266 > Libvirt virtualization library http://libvirt.org >
On Wed, 15 Aug 2012 16:54:26 +0200 "Benoît Canet" <benoit.canet@gmail.com> wrote: > This additionnal --machine=json option make qemu-img info output on > stdout a JSON formated representation of the image informations. > > --machine=json was choosen instead of --format=json because the > info command already have a -f parameter. Looks good to me, a few minor comments below. > > example: > { > "snapshots": [ > { > "name": "truc", > "date-sec": 1345034924, > "date-nsec": 870405000, > "vm-clock-sec": 4647590161, > "id": "1", > "vm-state-size": 80805256 > }, > { > "name": "onx", > "date-sec": 1345034927, > "date-nsec": 914633000, > "vm-clock-sec": 7065282574, > "id": "2", > "vm-state-size": 75927370 > }, > { > "name": "list", > "date-sec": 1345034939, > "date-nsec": 39119000, > "vm-clock-sec": 10108695046, > "id": "3", > "vm-state-size": 75890567 > } > ], > "virtual-size": 1073741824, > "filename": "snapshot.img", > "cluster-size": 65536, > "format": "qcow2", > "actual-size": 242614272, > "encrypted": false, > "backing-filename": "truc5.raw", > "dirty-flag": false > } > > Signed-off-by: Benoit Canet <benoit@irqsave.net> > --- > Makefile | 3 +- > qemu-img.c | 126 +++++++++++++++++++++++++++++++++++++++++++++++++++++++----- > 2 files changed, 118 insertions(+), 11 deletions(-) > > diff --git a/Makefile b/Makefile > index ab82ef3..9ba064b 100644 > --- a/Makefile > +++ b/Makefile > @@ -160,7 +160,8 @@ tools-obj-y = $(oslib-obj-y) $(trace-obj-y) qemu-tool.o qemu-timer.o \ > iohandler.o cutils.o iov.o async.o > tools-obj-$(CONFIG_POSIX) += compatfd.o > > -qemu-img$(EXESUF): qemu-img.o $(tools-obj-y) $(block-obj-y) > +qemu-img$(EXESUF): qemu-img.o $(tools-obj-y) $(block-obj-y) $(qapi-obj-y) \ > + qapi-visit.o qapi-types.o > qemu-nbd$(EXESUF): qemu-nbd.o $(tools-obj-y) $(block-obj-y) > qemu-io$(EXESUF): qemu-io.o cmd.o $(tools-obj-y) $(block-obj-y) > > diff --git a/qemu-img.c b/qemu-img.c > index 80cfb9b..1a80414 100644 > --- a/qemu-img.c > +++ b/qemu-img.c > @@ -21,12 +21,16 @@ > * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN > * THE SOFTWARE. > */ > +#include "qapi-visit.h" > +#include "qapi/qmp-output-visitor.h" > +#include "qjson.h" > #include "qemu-common.h" > #include "qemu-option.h" > #include "qemu-error.h" > #include "osdep.h" > #include "sysemu.h" > #include "block_int.h" > +#include <getopt.h> > #include <stdio.h> > > #ifdef _WIN32 > @@ -84,6 +88,7 @@ static void help(void) > " '-p' show progress of command (only certain commands)\n" > " '-S' indicates the consecutive number of bytes that must contain only zeros\n" > " for qemu-img to create a sparse image during conversion\n" > + " '-m' or '--machine' takes the format in which the output must be done (json)\n" > "\n" > "Parameters to check subcommand:\n" > " '-r' tries to repair any inconsistencies that are found during the check.\n" > @@ -1102,10 +1107,66 @@ static void dump_snapshots(BlockDriverState *bs) > g_free(sn_tab); > } > > +static void collect_snapshots(BlockDriverState *bs , ImageInfo *image_info) > +{ > + int i, sn_count; > + QEMUSnapshotInfo *sn_tab; > + SnapshotInfoList *sn_info_list, *cur_item = NULL, *head = NULL; > + sn_count = bdrv_snapshot_list(bs, &sn_tab); > + > + for (i = 0; i < sn_count; i++) { > + sn_info_list = g_new0(SnapshotInfoList, 1); > + > + sn_info_list->value = g_new0(SnapshotInfo, 1); > + sn_info_list->value->id = g_strdup(sn_tab[i].id_str); > + sn_info_list->value->name = g_strdup(sn_tab[i].name); > + sn_info_list->value->vm_state_size = sn_tab[i].vm_state_size; > + sn_info_list->value->date_sec = sn_tab[i].date_sec; > + sn_info_list->value->date_nsec = sn_tab[i].date_nsec; > + sn_info_list->value->vm_clock_sec = sn_tab[i].vm_clock_nsec; > + > + /* XXX: waiting for the qapi to support GSList */ > + if (!cur_item) { You can drop head and use imagine_info->snapshots instead. > + head = cur_item = sn_info_list; > + } else { > + cur_item->next = sn_info_list; > + cur_item = sn_info_list; > + } > + > + } > + > + g_free(sn_tab); > + image_info->snapshots = head; > +} > + > +static void dump_json_image_info(ImageInfo *image_info) > +{ > + Error *errp = NULL; > + QString *str; > + QmpOutputVisitor *ov = qmp_output_visitor_new(); > + QObject *obj; > + visit_type_ImageInfo(qmp_output_get_visitor(ov), > + &image_info, NULL, &errp); > + obj = qmp_output_get_qobject(ov); > + str = qobject_to_json_pretty(obj); > + assert(str != NULL); > + qstring_append_chr(str, '\n'); This call is not needed, just add \n to the printf call below. > + printf("%s", qstring_get_str(str)); > + qobject_decref(obj); > + qmp_output_visitor_cleanup(ov); > + QDECREF(str); > +} > + > +#define PRINTH(human, args...) do { \ > + if (human) { \ > + printf(args); \ > + } } while (0); > + > static int img_info(int argc, char **argv) > { > int c; > - const char *filename, *fmt; > + bool human = true; > + const char *filename, *fmt, *machine; > BlockDriverState *bs; > char size_buf[128], dsize_buf[128]; > uint64_t total_sectors; > @@ -1113,10 +1174,20 @@ static int img_info(int argc, char **argv) > char backing_filename[1024]; > char backing_filename2[1024]; > BlockDriverInfo bdi; > + ImageInfo *image_info; > > fmt = NULL; > + machine = NULL; > for(;;) { > - c = getopt(argc, argv, "f:h"); > + int option_index = 0; > + static struct option long_options[] = { > + {"help", no_argument, 0, 'h'}, > + {"format", required_argument, 0, 'f'}, > + {"machine", required_argument, 0, 'm'}, > + {0, 0, 0, 0} > + }; > + c = getopt_long(argc, argv, "f:h", > + long_options, &option_index); > if (c == -1) { > break; > } > @@ -1128,6 +1199,9 @@ static int img_info(int argc, char **argv) > case 'f': > fmt = optarg; > break; > + case 'm': > + machine = optarg; > + break; > } > } > if (optind >= argc) { > @@ -1135,6 +1209,11 @@ static int img_info(int argc, char **argv) > } > filename = argv[optind++]; > > + image_info = g_new0(ImageInfo, 1); > + if (machine && !strncmp(machine, "json", strlen("json"))) { > + human = false; > + } > + > bs = bdrv_new_open(filename, fmt, BDRV_O_FLAGS | BDRV_O_NO_BACKING); > if (!bs) { > return 1; > @@ -1148,39 +1227,66 @@ static int img_info(int argc, char **argv) > get_human_readable_size(dsize_buf, sizeof(dsize_buf), > allocated_size); > } > - printf("image: %s\n" > + PRINTH(human, "image: %s\n" > "file format: %s\n" > "virtual size: %s (%" PRId64 " bytes)\n" > "disk size: %s\n", > filename, bdrv_get_format_name(bs), size_buf, > (total_sectors * 512), > dsize_buf); > + image_info->filename = g_strdup(filename); > + image_info->format = g_strdup(bdrv_get_format_name(bs)); > + image_info->virtual_size = total_sectors * 512; > + image_info->actual_size = allocated_size >= 0 ? allocated_size : 0; > if (bdrv_is_encrypted(bs)) { > - printf("encrypted: yes\n"); > + PRINTH(human, "encrypted: yes\n"); > + image_info->encrypted = true; > + } else { > + image_info->encrypted = false; > } > if (bdrv_get_info(bs, &bdi) >= 0) { > if (bdi.cluster_size != 0) { > - printf("cluster_size: %d\n", bdi.cluster_size); > + PRINTH(human, "cluster_size: %d\n", bdi.cluster_size); > + image_info->cluster_size = bdi.cluster_size; > } > if (bdi.is_dirty) { > - printf("cleanly shut down: no\n"); > + PRINTH(human, "cleanly shut down: no\n"); > + image_info->dirty_flag = true; > + } else { > + image_info->dirty_flag = false; > } > } > bdrv_get_backing_filename(bs, backing_filename, sizeof(backing_filename)); > if (backing_filename[0] != '\0') { > + image_info->backing_filename = g_strdup(backing_filename); > bdrv_get_full_backing_filename(bs, backing_filename2, > sizeof(backing_filename2)); > - printf("backing file: %s", backing_filename); > + PRINTH(human, "backing file: %s", backing_filename); > if (strcmp(backing_filename, backing_filename2) != 0) { > - printf(" (actual path: %s)", backing_filename2); > + PRINTH(human, " (actual path: %s)", backing_filename2); > + } > + if (human) { > + putchar('\n'); > } > - putchar('\n'); > } > - dump_snapshots(bs); > + > + if (human) { > + dump_snapshots(bs); > + } else { > + collect_snapshots(bs, image_info); > + } > + > + if (!human) { > + dump_json_image_info(image_info); > + } I'd drop that test and move dump_json_image_info() to the else clause above. > + > + qapi_free_ImageInfo(image_info); > bdrv_delete(bs); > return 0; > } > > +#undef PRINTH > + > #define SNAPSHOT_LIST 1 > #define SNAPSHOT_CREATE 2 > #define SNAPSHOT_APPLY 3
diff --git a/Makefile b/Makefile index ab82ef3..9ba064b 100644 --- a/Makefile +++ b/Makefile @@ -160,7 +160,8 @@ tools-obj-y = $(oslib-obj-y) $(trace-obj-y) qemu-tool.o qemu-timer.o \ iohandler.o cutils.o iov.o async.o tools-obj-$(CONFIG_POSIX) += compatfd.o -qemu-img$(EXESUF): qemu-img.o $(tools-obj-y) $(block-obj-y) +qemu-img$(EXESUF): qemu-img.o $(tools-obj-y) $(block-obj-y) $(qapi-obj-y) \ + qapi-visit.o qapi-types.o qemu-nbd$(EXESUF): qemu-nbd.o $(tools-obj-y) $(block-obj-y) qemu-io$(EXESUF): qemu-io.o cmd.o $(tools-obj-y) $(block-obj-y) diff --git a/qemu-img.c b/qemu-img.c index 80cfb9b..1a80414 100644 --- a/qemu-img.c +++ b/qemu-img.c @@ -21,12 +21,16 @@ * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN * THE SOFTWARE. */ +#include "qapi-visit.h" +#include "qapi/qmp-output-visitor.h" +#include "qjson.h" #include "qemu-common.h" #include "qemu-option.h" #include "qemu-error.h" #include "osdep.h" #include "sysemu.h" #include "block_int.h" +#include <getopt.h> #include <stdio.h> #ifdef _WIN32 @@ -84,6 +88,7 @@ static void help(void) " '-p' show progress of command (only certain commands)\n" " '-S' indicates the consecutive number of bytes that must contain only zeros\n" " for qemu-img to create a sparse image during conversion\n" + " '-m' or '--machine' takes the format in which the output must be done (json)\n" "\n" "Parameters to check subcommand:\n" " '-r' tries to repair any inconsistencies that are found during the check.\n" @@ -1102,10 +1107,66 @@ static void dump_snapshots(BlockDriverState *bs) g_free(sn_tab); } +static void collect_snapshots(BlockDriverState *bs , ImageInfo *image_info) +{ + int i, sn_count; + QEMUSnapshotInfo *sn_tab; + SnapshotInfoList *sn_info_list, *cur_item = NULL, *head = NULL; + sn_count = bdrv_snapshot_list(bs, &sn_tab); + + for (i = 0; i < sn_count; i++) { + sn_info_list = g_new0(SnapshotInfoList, 1); + + sn_info_list->value = g_new0(SnapshotInfo, 1); + sn_info_list->value->id = g_strdup(sn_tab[i].id_str); + sn_info_list->value->name = g_strdup(sn_tab[i].name); + sn_info_list->value->vm_state_size = sn_tab[i].vm_state_size; + sn_info_list->value->date_sec = sn_tab[i].date_sec; + sn_info_list->value->date_nsec = sn_tab[i].date_nsec; + sn_info_list->value->vm_clock_sec = sn_tab[i].vm_clock_nsec; + + /* XXX: waiting for the qapi to support GSList */ + if (!cur_item) { + head = cur_item = sn_info_list; + } else { + cur_item->next = sn_info_list; + cur_item = sn_info_list; + } + + } + + g_free(sn_tab); + image_info->snapshots = head; +} + +static void dump_json_image_info(ImageInfo *image_info) +{ + Error *errp = NULL; + QString *str; + QmpOutputVisitor *ov = qmp_output_visitor_new(); + QObject *obj; + visit_type_ImageInfo(qmp_output_get_visitor(ov), + &image_info, NULL, &errp); + obj = qmp_output_get_qobject(ov); + str = qobject_to_json_pretty(obj); + assert(str != NULL); + qstring_append_chr(str, '\n'); + printf("%s", qstring_get_str(str)); + qobject_decref(obj); + qmp_output_visitor_cleanup(ov); + QDECREF(str); +} + +#define PRINTH(human, args...) do { \ + if (human) { \ + printf(args); \ + } } while (0); + static int img_info(int argc, char **argv) { int c; - const char *filename, *fmt; + bool human = true; + const char *filename, *fmt, *machine; BlockDriverState *bs; char size_buf[128], dsize_buf[128]; uint64_t total_sectors; @@ -1113,10 +1174,20 @@ static int img_info(int argc, char **argv) char backing_filename[1024]; char backing_filename2[1024]; BlockDriverInfo bdi; + ImageInfo *image_info; fmt = NULL; + machine = NULL; for(;;) { - c = getopt(argc, argv, "f:h"); + int option_index = 0; + static struct option long_options[] = { + {"help", no_argument, 0, 'h'}, + {"format", required_argument, 0, 'f'}, + {"machine", required_argument, 0, 'm'}, + {0, 0, 0, 0} + }; + c = getopt_long(argc, argv, "f:h", + long_options, &option_index); if (c == -1) { break; } @@ -1128,6 +1199,9 @@ static int img_info(int argc, char **argv) case 'f': fmt = optarg; break; + case 'm': + machine = optarg; + break; } } if (optind >= argc) { @@ -1135,6 +1209,11 @@ static int img_info(int argc, char **argv) } filename = argv[optind++]; + image_info = g_new0(ImageInfo, 1); + if (machine && !strncmp(machine, "json", strlen("json"))) { + human = false; + } + bs = bdrv_new_open(filename, fmt, BDRV_O_FLAGS | BDRV_O_NO_BACKING); if (!bs) { return 1; @@ -1148,39 +1227,66 @@ static int img_info(int argc, char **argv) get_human_readable_size(dsize_buf, sizeof(dsize_buf), allocated_size); } - printf("image: %s\n" + PRINTH(human, "image: %s\n" "file format: %s\n" "virtual size: %s (%" PRId64 " bytes)\n" "disk size: %s\n", filename, bdrv_get_format_name(bs), size_buf, (total_sectors * 512), dsize_buf); + image_info->filename = g_strdup(filename); + image_info->format = g_strdup(bdrv_get_format_name(bs)); + image_info->virtual_size = total_sectors * 512; + image_info->actual_size = allocated_size >= 0 ? allocated_size : 0; if (bdrv_is_encrypted(bs)) { - printf("encrypted: yes\n"); + PRINTH(human, "encrypted: yes\n"); + image_info->encrypted = true; + } else { + image_info->encrypted = false; } if (bdrv_get_info(bs, &bdi) >= 0) { if (bdi.cluster_size != 0) { - printf("cluster_size: %d\n", bdi.cluster_size); + PRINTH(human, "cluster_size: %d\n", bdi.cluster_size); + image_info->cluster_size = bdi.cluster_size; } if (bdi.is_dirty) { - printf("cleanly shut down: no\n"); + PRINTH(human, "cleanly shut down: no\n"); + image_info->dirty_flag = true; + } else { + image_info->dirty_flag = false; } } bdrv_get_backing_filename(bs, backing_filename, sizeof(backing_filename)); if (backing_filename[0] != '\0') { + image_info->backing_filename = g_strdup(backing_filename); bdrv_get_full_backing_filename(bs, backing_filename2, sizeof(backing_filename2)); - printf("backing file: %s", backing_filename); + PRINTH(human, "backing file: %s", backing_filename); if (strcmp(backing_filename, backing_filename2) != 0) { - printf(" (actual path: %s)", backing_filename2); + PRINTH(human, " (actual path: %s)", backing_filename2); + } + if (human) { + putchar('\n'); } - putchar('\n'); } - dump_snapshots(bs); + + if (human) { + dump_snapshots(bs); + } else { + collect_snapshots(bs, image_info); + } + + if (!human) { + dump_json_image_info(image_info); + } + + qapi_free_ImageInfo(image_info); bdrv_delete(bs); return 0; } +#undef PRINTH + #define SNAPSHOT_LIST 1 #define SNAPSHOT_CREATE 2 #define SNAPSHOT_APPLY 3
This additionnal --machine=json option make qemu-img info output on stdout a JSON formated representation of the image informations. --machine=json was choosen instead of --format=json because the info command already have a -f parameter. example: { "snapshots": [ { "name": "truc", "date-sec": 1345034924, "date-nsec": 870405000, "vm-clock-sec": 4647590161, "id": "1", "vm-state-size": 80805256 }, { "name": "onx", "date-sec": 1345034927, "date-nsec": 914633000, "vm-clock-sec": 7065282574, "id": "2", "vm-state-size": 75927370 }, { "name": "list", "date-sec": 1345034939, "date-nsec": 39119000, "vm-clock-sec": 10108695046, "id": "3", "vm-state-size": 75890567 } ], "virtual-size": 1073741824, "filename": "snapshot.img", "cluster-size": 65536, "format": "qcow2", "actual-size": 242614272, "encrypted": false, "backing-filename": "truc5.raw", "dirty-flag": false } Signed-off-by: Benoit Canet <benoit@irqsave.net> --- Makefile | 3 +- qemu-img.c | 126 +++++++++++++++++++++++++++++++++++++++++++++++++++++++----- 2 files changed, 118 insertions(+), 11 deletions(-)