Skip to content

Commit

Permalink
hmp: Disable chardev-add and chardev-remove
Browse files Browse the repository at this point in the history
As a general rule, HMP commands must be built on top of the QMP API.
Luiz and others have worked long & hard to make HMP conform to this
rule.

Commit f108890 added chardev-add, in violation of this rule.  QMP
command chardev-add was added right before, with minimal features, and
the idea to complete it step by step, then switch over the HMP command
to use it.

Unfortunately, we're not there, yet, and we don't want to release with
chardev-add in a "HMP is more powerful than QMP" state.

Disable the HMP command for now, along with its chardev-remove buddy.

Signed-off-by: Markus Armbruster <armbru@redhat.com>
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
  • Loading branch information
Markus Armbruster authored and Anthony Liguori committed Feb 6, 2013
1 parent 543f341 commit 8a14952
Showing 1 changed file with 32 additions and 31 deletions.
63 changes: 32 additions & 31 deletions hmp-commands.hx
Original file line number Diff line number Diff line change
Expand Up @@ -1522,37 +1522,38 @@ passed since 1970, i.e. unix epoch.
@end table
ETEXI

{
.name = "chardev-add",
.args_type = "args:s",
.params = "args",
.help = "add chardev",
.mhandler.cmd = hmp_chardev_add,
},

STEXI
@item chardev_add args
@findex chardev_add

chardev_add accepts the same parameters as the -chardev command line switch.

ETEXI

{
.name = "chardev-remove",
.args_type = "id:s",
.params = "id",
.help = "remove chardev",
.mhandler.cmd = hmp_chardev_remove,
},

STEXI
@item chardev_remove id
@findex chardev_remove

Removes the chardev @var{id}.

ETEXI
HXCOMM Disabled for now, because it isn't built on top of QMP's chardev-add
HXCOMM {
HXCOMM .name = "chardev-add",
HXCOMM .args_type = "args:s",
HXCOMM .params = "args",
HXCOMM .help = "add chardev",
HXCOMM .mhandler.cmd = hmp_chardev_add,
HXCOMM },
HXCOMM
HXCOMM STEXI
HXCOMM @item chardev_add args
HXCOMM @findex chardev_add
HXCOMM
HXCOMM chardev_add accepts the same parameters as the -chardev command line switch.
HXCOMM
HXCOMM ETEXI
HXCOMM
HXCOMM {
HXCOMM .name = "chardev-remove",
HXCOMM .args_type = "id:s",
HXCOMM .params = "id",
HXCOMM .help = "remove chardev",
HXCOMM .mhandler.cmd = hmp_chardev_remove,
HXCOMM },
HXCOMM
HXCOMM STEXI
HXCOMM @item chardev_remove id
HXCOMM @findex chardev_remove
HXCOMM
HXCOMM Removes the chardev @var{id}.
HXCOMM
HXCOMM ETEXI

{
.name = "info",
Expand Down

0 comments on commit 8a14952

Please sign in to comment.