On Tue, Nov 07, 2023 at 02:15:34PM +0000, Daniel P. Berrangé wrote:
> Date: Tue, 7 Nov 2023 14:15:34 +0000
> From: "Daniel P. Berrangé" <berra...@redhat.com>
> Subject: Re: [PATCH] scripts/cpu-x86-uarch-abi.py: Fix parameter error of
> cmd
>
> On Wed, Oct 18, 2023 at 06:00:11PM +0800, Zhao Liu wrote:
> > From: Zhao Liu <zhao1....@intel.com>
> >
> > When run this script, there's the error:
> >
> > python3 scripts/cpu-x86-uarch-abi.py /tmp/qmp
> > Traceback (most recent call last):
> > File "/path-to-qemu/qemu/scripts/cpu-x86-uarch-abi.py", line 96, in
> > <module>
> > cpu = shell.cmd("query-cpu-model-expansion",
> > TypeError: QEMUMonitorProtocol.cmd() takes 2 positional arguments but 3
> > were given
> >
> > Commit 7f521b023bc28 ("scripts/cpu-x86-uarch-abi.py: use .command()
> > instead of .cmd()") converts the the original .cmd() to .command()
> > (which was later renamed to "cmd" to replace the original one).
> >
> > But the new .cmd() only accepts typing.Mapping as the parameter instead
> > of typing.Dict (see _qmp.execute()).
> >
> > Change the paremeters of "query-cpu-model-expansion" to typing.Mapping
> > format to fix this error.
> >
> > Fixes: 7f521b023bc28 ("scripts/cpu-x86-uarch-abi.py: use .command() instead
> > of .cmd()")
> > Signed-off-by: Zhao Liu <zhao1....@intel.com>
> > ---
> > scripts/cpu-x86-uarch-abi.py | 4 ++--
> > 1 file changed, 2 insertions(+), 2 deletions(-)
>
> Reviewed-by: Daniel P. Berrangé <berra...@redhat.com>
>
> I'll queue this one and sent a PULL before the final release.
>
Thanks!
Regards,
Zhao
>
> With regards,
> Daniel
> --
> |: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
> |: https://libvirt.org -o- https://fstop138.berrange.com :|
> |: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
>
