libxl: finish use of {Begin, End}API pattern in APIs

[libvirt] [PATCH 0/3] libxl: finish use of {Begin, End}API pattern in APIs

Jim Fehlig posted 3 patches 6 years, 1 month ago

Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20180319231453.19954-1-jfehlig@suse.com

Test syntax-check passed

src/libxl/libxl_driver.c    | 29 ++++++++++++++++-------------
src/libxl/libxl_migration.c |  7 +++++--
2 files changed, 21 insertions(+), 15 deletions(-)

Expand all Fold all

[libvirt] [PATCH 0/3] libxl: finish use of {Begin, End}API pattern in APIs

Posted by Jim Fehlig 6 years, 1 month ago

This is a follow-up to John and I's recent work to improve locking and
ref counting of virDomainObj in the libxl driver. These patches change
the remaining APIs to use the begin and end API pattern used throughout
libxl and other drivers.

Jim Fehlig (3):
  libxl: lock virDomainObj after ListRemove
  libxl: call EndJob in error case
  libxl: convert DefineXMLFlags to use begin/end API pattern

 src/libxl/libxl_driver.c    | 29 ++++++++++++++++-------------
 src/libxl/libxl_migration.c |  7 +++++--
 2 files changed, 21 insertions(+), 15 deletions(-)

-- 
2.16.2

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

Re: [libvirt] [PATCH 0/3] libxl: finish use of {Begin, End}API pattern in APIs

Posted by John Ferlan 6 years, 1 month ago


On 03/19/2018 07:14 PM, Jim Fehlig wrote:
> This is a follow-up to John and I's recent work to improve locking and
> ref counting of virDomainObj in the libxl driver. These patches change
> the remaining APIs to use the begin and end API pattern used throughout
> libxl and other drivers.
> 
> Jim Fehlig (3):
>   libxl: lock virDomainObj after ListRemove
>   libxl: call EndJob in error case
>   libxl: convert DefineXMLFlags to use begin/end API pattern
> 
>  src/libxl/libxl_driver.c    | 29 ++++++++++++++++-------------
>  src/libxl/libxl_migration.c |  7 +++++--
>  2 files changed, 21 insertions(+), 15 deletions(-)
> 

Reviewed-by: John Ferlan <jferlan@redhat.com>
(series)

John

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