[libvirt] [PATCH] qemu: Report error if qemu monitor command not found for BlockJob
Adam Litke
agl at us.ibm.com
Tue Aug 23 14:31:00 UTC 2011
On 08/22/2011 10:04 PM, Osier Yang wrote:
> Hi, Adam
>
> I likes the idea to wrap the checking as a function seperately, but the
> function
> won't work well if the command is "help info", though we don't have a use
> of "help info" yet.
>
> My point is since the function is intending to work for all the command,
> as a
> common function, it needs to consider some boudary too.
>
> How about below?
Good points. The function below looks good.
> qemuMonitorTextCommandNotFound(const char *cmd, const char *reply)
> {
> if (STRPREFIX(cmd, "info ")) {
> if (strstr(reply, "info version"))
> return 1;
> } else {
> if (strstr(reply, "unknown command:"))
> return 1;
> }
>
> return 0;
> }
>
> And there might be other different info qemu will output for a unknown
> command
> we don't known yet. Using "cmd" as an argument will allow us to extend
> the checking
> methods.
>
> Thanks
> Osier
--
Adam Litke
IBM Linux Technology Center
More information about the libvir-list
mailing list