[PATCH 07/14] qemuxml2argvmock: Drop virNuma* mocks

Kristina Hanicova khanicov at redhat.com
Mon Mar 13 15:31:32 UTC 2023


On Wed, Mar 8, 2023 at 12:17 PM Michal Privoznik <mprivozn at redhat.com>
wrote:

> Since qemuxml2argvtest is  now using virnumamock, there's no need
> for qemuxml2argvmock to offer reimplementation of virNuma*()
> functions. Also, the comment about CLang and FreeBSD (introduced
> in v4.3.0-40-g77ac204d14) is no longer true.  Looks like noinline
> attribute was the missing culprit.
>
> Signed-off-by: Michal Privoznik <mprivozn at redhat.com>
> ---
>  src/util/virnuma.h                            |  2 +-
>  ...-unavailable-restrictive.x86_64-latest.err |  2 +-
>  ...mnode-unavailable-strict.x86_64-latest.err |  2 +-
>  ...umatune-static-nodeset-exceed-hostnode.err |  2 +-
>  tests/qemuxml2argvmock.c                      | 41 -------------------
>  5 files changed, 4 insertions(+), 45 deletions(-)
>

I think you can also drop include of virnuma.h in tests/qemuxml2argvmock.c

Reviewed-by: Kristina Hanicova <khanicov at redhat.com>

Kristina
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20230313/209d82f7/attachment.htm>


More information about the libvir-list mailing list