On Tue, Jan 03, 2023 at 09:08:20PM +0800, Jiang Jiacheng wrote:
> From: zhengchuan <zhengch...@huawei.com>
> 
> Firstly, we need to get migration pids, add
> virDomainMigrateGetMigrationPids() for migration pin.
> 
> Signed-off-by:zhengchuan<zhengch...@huawei.com>
> ---
>  include/libvirt/libvirt-domain.h |  3 +++
>  src/driver-hypervisor.h          |  5 ++++
>  src/libvirt-domain.c             | 39 ++++++++++++++++++++++++++++++++
>  src/libvirt_public.syms          |  5 ++++
>  src/qemu/qemu_domain.c           |  2 ++
>  src/qemu/qemu_domain.h           |  2 ++
>  src/qemu/qemu_driver.c           | 31 +++++++++++++++++++++++++
>  src/remote/remote_driver.c       |  1 +
>  src/remote/remote_protocol.x     | 17 +++++++++++++-
>  src/remote_protocol-structs      |  8 +++++++
>  10 files changed, 112 insertions(+), 1 deletion(-)
> 
> diff --git a/include/libvirt/libvirt-domain.h 
> b/include/libvirt/libvirt-domain.h
> index 295fd30c93..e11f2795f1 100644
> --- a/include/libvirt/libvirt-domain.h
> +++ b/include/libvirt/libvirt-domain.h
> @@ -6457,4 +6457,7 @@ int virDomainStartDirtyRateCalc(virDomainPtr domain,
>                                  int seconds,
>                                  unsigned int flags);
>  
> +char *virDomainMigrateGetMigrationPids(virDomainPtr domain,
> +                                       unsigned int flags);

Exposing QEMU thread PIDs in the public libvirt API is not something
we should be doing. We've explicitly aimed to avoid exposing the notion
of PIDs in our API in general.


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 :|

Reply via email to