* [Qemu-devel] [PATCH v3] Sort the help info shown in monitor at runtime
@ 2011-10-12 3:32 Wayne Xia
2011-10-12 6:39 ` Wenyi Gao
2011-11-01 22:21 ` Anthony Liguori
0 siblings, 2 replies; 3+ messages in thread
From: Wayne Xia @ 2011-10-12 3:32 UTC (permalink / raw)
To: qemu-devel; +Cc: Wayne Xia
This patch would try sort the command list in monitor at runtime. As a result,
command help and help info would show a more friendly sorted command list.
For eg:
(qemu)help
acl_add
acl_policy
acl_remove
acl_reset
acl_show
balloon
block_passwd
...
the command list is sorted.
v3: using qsort function to sort the command list.
Signed-off-by: Wayne Xia <xiawenc@linux.vnet.ibm.com>
---
monitor.c | 30 ++++++++++++++++++++++++++----
1 files changed, 26 insertions(+), 4 deletions(-)
diff --git a/monitor.c b/monitor.c
index 31b212a..a172167 100644
--- a/monitor.c
+++ b/monitor.c
@@ -195,8 +195,8 @@ static inline int mon_print_count_get(const Monitor *mon) { return 0; }
static QLIST_HEAD(mon_list, Monitor) mon_list;
-static const mon_cmd_t mon_cmds[];
-static const mon_cmd_t info_cmds[];
+static mon_cmd_t mon_cmds[];
+static mon_cmd_t info_cmds[];
static const mon_cmd_t qmp_cmds[];
static const mon_cmd_t qmp_query_cmds[];
@@ -2726,13 +2726,14 @@ int monitor_get_fd(Monitor *mon, const char *fdname)
return -1;
}
-static const mon_cmd_t mon_cmds[] = {
+/* mon_cmds and info_cmds would be sorted at runtime */
+static mon_cmd_t mon_cmds[] = {
#include "hmp-commands.h"
{ NULL, NULL, },
};
/* Please update hmp-commands.hx when adding or changing commands */
-static const mon_cmd_t info_cmds[] = {
+static mon_cmd_t info_cmds[] = {
{
.name = "version",
.args_type = "",
@@ -5068,6 +5069,25 @@ static void monitor_event(void *opaque, int event)
}
}
+static int
+compare_mon_cmd(const void *a, const void *b)
+{
+ return strcmp(((const mon_cmd_t *)a)->name,
+ ((const mon_cmd_t *)b)->name);
+}
+
+static void sortcmdlist(void)
+{
+ int array_num;
+ int elem_size = sizeof(mon_cmd_t);
+
+ array_num = sizeof(mon_cmds)/elem_size-1;
+ qsort((void *)mon_cmds, array_num, elem_size, compare_mon_cmd);
+
+ array_num = sizeof(info_cmds)/elem_size-1;
+ qsort((void *)info_cmds, array_num, elem_size, compare_mon_cmd);
+}
+
/*
* Local variables:
@@ -5110,6 +5130,8 @@ void monitor_init(CharDriverState *chr, int flags)
QLIST_INSERT_HEAD(&mon_list, mon, entry);
if (!default_mon || (flags & MONITOR_IS_DEFAULT))
default_mon = mon;
+
+ sortcmdlist();
}
static void bdrv_password_cb(Monitor *mon, const char *password, void *opaque)
--
1.7.6
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [Qemu-devel] [PATCH v3] Sort the help info shown in monitor at runtime
2011-10-12 3:32 [Qemu-devel] [PATCH v3] Sort the help info shown in monitor at runtime Wayne Xia
@ 2011-10-12 6:39 ` Wenyi Gao
2011-11-01 22:21 ` Anthony Liguori
1 sibling, 0 replies; 3+ messages in thread
From: Wenyi Gao @ 2011-10-12 6:39 UTC (permalink / raw)
To: Wayne Xia; +Cc: qemu-devel
On Wed, 2011-10-12 at 11:32 +0800, Wayne Xia wrote:
> This patch would try sort the command list in monitor at runtime. As a result,
> command help and help info would show a more friendly sorted command list.
> For eg:
> (qemu)help
> acl_add
> acl_policy
> acl_remove
> acl_reset
> acl_show
> balloon
> block_passwd
> ...
> the command list is sorted.
>
> v3: using qsort function to sort the command list.
>
> Signed-off-by: Wayne Xia <xiawenc@linux.vnet.ibm.com>
> ---
> monitor.c | 30 ++++++++++++++++++++++++++----
> 1 files changed, 26 insertions(+), 4 deletions(-)
>
> diff --git a/monitor.c b/monitor.c
> index 31b212a..a172167 100644
> --- a/monitor.c
> +++ b/monitor.c
> @@ -195,8 +195,8 @@ static inline int mon_print_count_get(const Monitor *mon) { return 0; }
>
> static QLIST_HEAD(mon_list, Monitor) mon_list;
>
> -static const mon_cmd_t mon_cmds[];
> -static const mon_cmd_t info_cmds[];
> +static mon_cmd_t mon_cmds[];
> +static mon_cmd_t info_cmds[];
>
> static const mon_cmd_t qmp_cmds[];
> static const mon_cmd_t qmp_query_cmds[];
> @@ -2726,13 +2726,14 @@ int monitor_get_fd(Monitor *mon, const char *fdname)
> return -1;
> }
>
> -static const mon_cmd_t mon_cmds[] = {
> +/* mon_cmds and info_cmds would be sorted at runtime */
> +static mon_cmd_t mon_cmds[] = {
> #include "hmp-commands.h"
> { NULL, NULL, },
> };
>
> /* Please update hmp-commands.hx when adding or changing commands */
> -static const mon_cmd_t info_cmds[] = {
> +static mon_cmd_t info_cmds[] = {
> {
> .name = "version",
> .args_type = "",
> @@ -5068,6 +5069,25 @@ static void monitor_event(void *opaque, int event)
> }
> }
>
> +static int
> +compare_mon_cmd(const void *a, const void *b)
> +{
> + return strcmp(((const mon_cmd_t *)a)->name,
> + ((const mon_cmd_t *)b)->name);
> +}
> +
> +static void sortcmdlist(void)
> +{
> + int array_num;
> + int elem_size = sizeof(mon_cmd_t);
> +
> + array_num = sizeof(mon_cmds)/elem_size-1;
> + qsort((void *)mon_cmds, array_num, elem_size, compare_mon_cmd);
> +
> + array_num = sizeof(info_cmds)/elem_size-1;
> + qsort((void *)info_cmds, array_num, elem_size, compare_mon_cmd);
> +}
> +
>
> /*
> * Local variables:
> @@ -5110,6 +5130,8 @@ void monitor_init(CharDriverState *chr, int flags)
> QLIST_INSERT_HEAD(&mon_list, mon, entry);
> if (!default_mon || (flags & MONITOR_IS_DEFAULT))
> default_mon = mon;
> +
> + sortcmdlist();
> }
>
> static void bdrv_password_cb(Monitor *mon, const char *password, void *opaque)
Tested-by: Wenyi Gao <wenyi@linux.vnet.ibm.com>
Work nice.
Wenyi Gao
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [Qemu-devel] [PATCH v3] Sort the help info shown in monitor at runtime
2011-10-12 3:32 [Qemu-devel] [PATCH v3] Sort the help info shown in monitor at runtime Wayne Xia
2011-10-12 6:39 ` Wenyi Gao
@ 2011-11-01 22:21 ` Anthony Liguori
1 sibling, 0 replies; 3+ messages in thread
From: Anthony Liguori @ 2011-11-01 22:21 UTC (permalink / raw)
To: Wayne Xia; +Cc: qemu-devel
On 10/11/2011 10:32 PM, Wayne Xia wrote:
> This patch would try sort the command list in monitor at runtime. As a result,
> command help and help info would show a more friendly sorted command list.
> For eg:
> (qemu)help
> acl_add
> acl_policy
> acl_remove
> acl_reset
> acl_show
> balloon
> block_passwd
> ...
> the command list is sorted.
>
> v3: using qsort function to sort the command list.
>
> Signed-off-by: Wayne Xia<xiawenc@linux.vnet.ibm.com>
Applied. Thanks.
Regards,
Anthony Liguori
> ---
> monitor.c | 30 ++++++++++++++++++++++++++----
> 1 files changed, 26 insertions(+), 4 deletions(-)
>
> diff --git a/monitor.c b/monitor.c
> index 31b212a..a172167 100644
> --- a/monitor.c
> +++ b/monitor.c
> @@ -195,8 +195,8 @@ static inline int mon_print_count_get(const Monitor *mon) { return 0; }
>
> static QLIST_HEAD(mon_list, Monitor) mon_list;
>
> -static const mon_cmd_t mon_cmds[];
> -static const mon_cmd_t info_cmds[];
> +static mon_cmd_t mon_cmds[];
> +static mon_cmd_t info_cmds[];
>
> static const mon_cmd_t qmp_cmds[];
> static const mon_cmd_t qmp_query_cmds[];
> @@ -2726,13 +2726,14 @@ int monitor_get_fd(Monitor *mon, const char *fdname)
> return -1;
> }
>
> -static const mon_cmd_t mon_cmds[] = {
> +/* mon_cmds and info_cmds would be sorted at runtime */
> +static mon_cmd_t mon_cmds[] = {
> #include "hmp-commands.h"
> { NULL, NULL, },
> };
>
> /* Please update hmp-commands.hx when adding or changing commands */
> -static const mon_cmd_t info_cmds[] = {
> +static mon_cmd_t info_cmds[] = {
> {
> .name = "version",
> .args_type = "",
> @@ -5068,6 +5069,25 @@ static void monitor_event(void *opaque, int event)
> }
> }
>
> +static int
> +compare_mon_cmd(const void *a, const void *b)
> +{
> + return strcmp(((const mon_cmd_t *)a)->name,
> + ((const mon_cmd_t *)b)->name);
> +}
> +
> +static void sortcmdlist(void)
> +{
> + int array_num;
> + int elem_size = sizeof(mon_cmd_t);
> +
> + array_num = sizeof(mon_cmds)/elem_size-1;
> + qsort((void *)mon_cmds, array_num, elem_size, compare_mon_cmd);
> +
> + array_num = sizeof(info_cmds)/elem_size-1;
> + qsort((void *)info_cmds, array_num, elem_size, compare_mon_cmd);
> +}
> +
>
> /*
> * Local variables:
> @@ -5110,6 +5130,8 @@ void monitor_init(CharDriverState *chr, int flags)
> QLIST_INSERT_HEAD(&mon_list, mon, entry);
> if (!default_mon || (flags& MONITOR_IS_DEFAULT))
> default_mon = mon;
> +
> + sortcmdlist();
> }
>
> static void bdrv_password_cb(Monitor *mon, const char *password, void *opaque)
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2011-11-01 22:21 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-10-12 3:32 [Qemu-devel] [PATCH v3] Sort the help info shown in monitor at runtime Wayne Xia
2011-10-12 6:39 ` Wenyi Gao
2011-11-01 22:21 ` Anthony Liguori
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).