On 10/22/2010 08:17 AM, Luiz Capitulino wrote:
You and Gerd asked about the purpose of this command, turns out that it's
only useful for developing new USB devices for QEMU, so I thought it would
be better to restrict it, so that people don't start using this the
wrong way or worse, we can't drop/break it because some tool is now using it.

Need HMP commands have no support associated with them. Given QMP, there's no reason to script HMP commands.

But a long standing policy in QEMU has been to avoid conditional code because it results in dead code. I think this policy has actually worked well for us historically.

Regards,

Anthony Liguori

Anyway, maybe a good doc will do, this was just a small suggestion.

Objections, Anthony?

Not with better docs.

Regards,

Anthony Liguori

This is just adding a HMP command.  Is that the right approach or
was that an unintentional consequence of rebasing post-HMP/QMP
split?

I don't think this should be available under QMP, it's more a debugging
command for USB developers.


Regards,

Anthony Liguori


cheers,
   Gerd








Reply via email to