Re: [PATCH v3] qemu,qmp: convert do_inject_nmi() to QObject, QError

Luiz Capitulino
2010-12-16T11:12:33Z

On Thu, 16 Dec 2010 12:51:14 +0200
Avi Kivity <avi@redhat.com> wrote:

> On 12/16/2010 12:48 PM, Luiz Capitulino wrote:
> > Ok, I didn't know that, but I had another idea: the command could accept
> > either a single cpu index or a list:
> >
> >
> > { "execute": "inject-nmi", "arguments": { "cpus": 2 } }
> >
> > { "execute": "inject-nmi", "arguments": { "cpus": [1, 2, 3, 4] } }
> >
> > This has the feature of injecting the nmi in just some cpus, although I'm
> > not sure this is going to be desired/useful.
> >
> > If we agree on this we'll have to wait because the monitor doesn't currently
> > support "hybrid" arguments.
>
> I hope it never does. They're hard to support in old-school statically
> typed languages.

We could accept only a list then.