[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH V10 17/17] hmp: add parameter device and -b for
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [PATCH V10 17/17] hmp: add parameter device and -b for info block |
Date: |
Tue, 2 Apr 2013 09:46:04 +0200 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
Am 29.03.2013 um 03:48 hat Wenchao Xia geschrieben:
> δΊ 2013-3-28 19:09, Kevin Wolf ει:
> >Am 22.03.2013 um 15:19 hat Wenchao Xia geschrieben:
> >> With these parameters, user can choose the information to be showed,
> >>to avoid message flood in the montior.
> >>
> >>Signed-off-by: Wenchao Xia <address@hidden>
> >
> >Aha, so here you actually introduce the 'device' parameter. If you can
> >have this patch first, and only then patch 16, then limiting the new
> >output to the form with a device specified should be trivial.
> >
> It is a bit indirect that "info block" and "info block <device>"
> show different on single device, so I introduced additional parameter
> '-b' to filter out info. With more thinking, I think it should be
> {
> .name = "block",
> .help = "show the block devices",
> .args_type = "verbose:-v,device:B?",
> .params = "[-v] [device]",
> .help = "show info of one block device or all block devices "
> "and detail of images with -v option",
> }
> Then by default "info block" so brief summary as old time. Since the
> "-v" parameter is filtering something out not present on patch 15,
> so I can't move this patch forward, hope you are OK with it.
'info block [-v] [<device>]' is okay with me. I would prefer if you
could arrange the patches so that without -v you never get verbose
output in any intermediate patch, but otherwise I don't mind about
the order of patches.
Kevin
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-devel] [PATCH V10 17/17] hmp: add parameter device and -b for info block,
Kevin Wolf <=