On Thu, Sep 22, 2022 at 04:35:51PM +0200, Martin Kletzander wrote:
>On Tue, Sep 13, 2022 at 12:30:31PM -0700, Lin Yang wrote:
>>AMX was introduced in QEMU commit 1f16764f7d4515bfd5e4ae0aae814fa280a7d0c8
>>and following commits.
>
>Some of them not pushed yet I believe:
>
>https://lists.gnu.org/archive/html/qemu-devel/2022-08/msg01936.html
>
>Otherwise the patch looks good (I'm not sure what's the planned usage of
>the python script, but this patch just adds features that'd be missing
>in the future, so I think it's fine).
>
>Reviewed-by: Martin Kletzander <mkletzan(a)redhat.com>
>
I still can't see the other commits in QEMU mentioning the other
features, but I can't see how those missing could cause an issue, so in
the end I removed the "and following commits".
We also require a Signed-off-by for contributions as written in our
contribution guidelines:
https://libvirt.org/hacking.html
If you say I can add one for you with the current e-mail address there's
no need to resend this patch and I can add it before pushing.
Thanks for the patch.
Totally okay to me. Please sign with “Lin Yang <lin.a.yang(a)intel.com>".
Thank you so much for reviewing this patch.
Thanks,
Lin.