[PATCH 0/5] Cleanup of callers of virDomainDefGetVcpu

Peter Krempa via Devel posted 5 patches 1 month, 1 week ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/cover.1753972379.git.pkrempa@redhat.com
src/ch/ch_domain.c          | 17 ++++++++++-------
src/ch/ch_domain.h          |  2 +-
src/ch/ch_process.c         |  2 +-
src/ch/ch_process.h         |  3 ---
src/conf/domain_postparse.c |  4 ----
src/qemu/qemu_domain.c      | 15 ---------------
src/qemu/qemu_domain.h      |  1 -
7 files changed, 12 insertions(+), 32 deletions(-)
[PATCH 0/5] Cleanup of callers of virDomainDefGetVcpu
Posted by Peter Krempa via Devel 1 month, 1 week ago
Fix few issues with how virDomainDefGetVcpu is called and in related
code.

Peter Krempa (5):
  Unexport virCHProcessSetupVcpu
  virDomainVcpuDefPostParse: Remove impossible check
  qemu: domain: Remove unused qemuDomainGetVcpuHalted
  virCHDomainRefreshThreadInfo: Don't trust vcpu ID returned by
    hypervisor
  virCHDomainRefreshThreadInfo: Remove illusion that caller cares about
    return value

 src/ch/ch_domain.c          | 17 ++++++++++-------
 src/ch/ch_domain.h          |  2 +-
 src/ch/ch_process.c         |  2 +-
 src/ch/ch_process.h         |  3 ---
 src/conf/domain_postparse.c |  4 ----
 src/qemu/qemu_domain.c      | 15 ---------------
 src/qemu/qemu_domain.h      |  1 -
 7 files changed, 12 insertions(+), 32 deletions(-)

-- 
2.50.1
Re: [PATCH 0/5] Cleanup of callers of virDomainDefGetVcpu
Posted by Ján Tomko via Devel 1 month ago
On a Thursday in 2025, Peter Krempa via Devel wrote:
>Fix few issues with how virDomainDefGetVcpu is called and in related
>code.
>
>Peter Krempa (5):
>  Unexport virCHProcessSetupVcpu
>  virDomainVcpuDefPostParse: Remove impossible check
>  qemu: domain: Remove unused qemuDomainGetVcpuHalted
>  virCHDomainRefreshThreadInfo: Don't trust vcpu ID returned by
>    hypervisor
>  virCHDomainRefreshThreadInfo: Remove illusion that caller cares about
>    return value
>
> src/ch/ch_domain.c          | 17 ++++++++++-------
> src/ch/ch_domain.h          |  2 +-
> src/ch/ch_process.c         |  2 +-
> src/ch/ch_process.h         |  3 ---
> src/conf/domain_postparse.c |  4 ----
> src/qemu/qemu_domain.c      | 15 ---------------
> src/qemu/qemu_domain.h      |  1 -
> 7 files changed, 12 insertions(+), 32 deletions(-)
>

Reviewed-by: Ján Tomko <jtomko@redhat.com>

Jano