Couple of hotplug cleanups

[libvirt] [PATCH v2 0/2] Couple of hotplug cleanups

Posted by John Ferlan 7 years, 2 months ago

Necromancing some old branches... 

v1: https://www.redhat.com/archives/libvir-list/2017-October/msg00930.html

Changes since v1 - beef up commit messages and drop patches 3 & 4 as
that's a bit more involved and Jan seemed to imply he eventually may
send patches dealing with the same thing - so I'll wait to see those.

John Ferlan (2):
  conf,qemu: Check for NULL addrs in virDomainUSBAddressRelease
  conf,qemu: Check for NULL addrs in virDomainUSBAddressEnsure

 src/conf/domain_addr.c         |  5 ++++-
 src/conf/domain_addr.h         |  4 ++--
 src/qemu/qemu_domain_address.c |  7 ++-----
 src/qemu/qemu_hotplug.c        | 31 ++++++++++---------------------
 4 files changed, 18 insertions(+), 29 deletions(-)

-- 
2.13.6

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] [PATCH v2 0/2] Couple of hotplug cleanups

Posted by Michal Privoznik 7 years, 2 months ago

On 02/14/2018 07:29 PM, John Ferlan wrote:
> Necromancing some old branches... 
> 
> v1: https://www.redhat.com/archives/libvir-list/2017-October/msg00930.html
> 
> Changes since v1 - beef up commit messages and drop patches 3 & 4 as
> that's a bit more involved and Jan seemed to imply he eventually may
> send patches dealing with the same thing - so I'll wait to see those.
> 
> John Ferlan (2):
>   conf,qemu: Check for NULL addrs in virDomainUSBAddressRelease
>   conf,qemu: Check for NULL addrs in virDomainUSBAddressEnsure
> 
>  src/conf/domain_addr.c         |  5 ++++-
>  src/conf/domain_addr.h         |  4 ++--
>  src/qemu/qemu_domain_address.c |  7 ++-----
>  src/qemu/qemu_hotplug.c        | 31 ++++++++++---------------------
>  4 files changed, 18 insertions(+), 29 deletions(-)
> 

ACK to both.

Michal

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] [PATCH v2 0/2] Couple of hotplug cleanups

Posted by John Ferlan 7 years, 2 months ago

ping?

Tks -

On 02/14/2018 01:29 PM, John Ferlan wrote:
> Necromancing some old branches... 
> 
> v1: https://www.redhat.com/archives/libvir-list/2017-October/msg00930.html
> 
> Changes since v1 - beef up commit messages and drop patches 3 & 4 as
> that's a bit more involved and Jan seemed to imply he eventually may
> send patches dealing with the same thing - so I'll wait to see those.
> 
> John Ferlan (2):
>   conf,qemu: Check for NULL addrs in virDomainUSBAddressRelease
>   conf,qemu: Check for NULL addrs in virDomainUSBAddressEnsure
> 
>  src/conf/domain_addr.c         |  5 ++++-
>  src/conf/domain_addr.h         |  4 ++--
>  src/qemu/qemu_domain_address.c |  7 ++-----
>  src/qemu/qemu_hotplug.c        | 31 ++++++++++---------------------
>  4 files changed, 18 insertions(+), 29 deletions(-)
> 

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list