于 2011年10月26日 14:43, Osier Yang 写道: > With the patch, output of "kvm list" will be: > > % ./kvm list > PID NAME STATE > ------------------------------------ > 5270 test running > test shut off > default shut off > > "not running" is changed into "shut off", as it might > be other states which are "not running" in future? such > as "paused". > > Signed-off-by: Osier Yang <jyang@xxxxxxxxxx> > --- > tools/kvm/builtin-list.c | 10 +++++++--- > 1 files changed, 7 insertions(+), 3 deletions(-) > > diff --git a/tools/kvm/builtin-list.c b/tools/kvm/builtin-list.c > index 2ff3472..4cabd59 100644 > --- a/tools/kvm/builtin-list.c > +++ b/tools/kvm/builtin-list.c > @@ -33,6 +33,9 @@ static const struct option list_options[] = { > OPT_END() > }; > > +#define KVM_INSTANCE_RUNNING "running" > +#define KVM_INSTANCE_SHUTOFF "shut off" > + > void kvm_list_help(void) > { > usage_with_options(list_usage, list_options); > @@ -71,7 +74,7 @@ static int print_guest(const char *name, int sock) > if (strncmp(comm, PROCESS_NAME, strlen(PROCESS_NAME))) > goto cleanup; > > - printf("%5d %s\n", pid, name); > + printf("%5d %-20s %s\n", pid, name, KVM_INSTANCE_RUNNING); I intended to use "6s" here > > free(comm); > > @@ -109,7 +112,7 @@ static int kvm_list_rootfs(void) > if (dirent->d_type == DT_DIR && > strcmp(dirent->d_name, ".") && > strcmp(dirent->d_name, "..")) > - printf(" %s (not running)\n", dirent->d_name); > + printf("%5s %-20s %s\n", "", dirent->d_name, KVM_INSTANCE_SHUTOFF); likewise, s/5/6/ > } > > return 0; > @@ -134,7 +137,8 @@ int kvm_cmd_list(int argc, const char **argv, const char *prefix) > if (!run && !rootfs) > run = rootfs = true; > > - printf(" PID GUEST\n"); > + printf("%6s %-20s %s\n", "PID", "NAME", "STATE"); > + printf("------------------------------------\n"); > > if (run) { > r = kvm_list_running_instances(); -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html