[PATCH 0/6] libxl: Misc cleanups and improvements

Jim Fehlig posted 6 patches 2 years, 9 months ago
Test syntax-check failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20210706211255.8891-1-jfehlig@suse.com
src/libxl/libxl_domain.c    | 384 ++++++++++++++++++------------------
src/libxl/libxl_domain.h    |  11 +-
src/libxl/libxl_driver.c    |  30 +--
src/libxl/libxl_migration.c |  15 +-
4 files changed, 218 insertions(+), 222 deletions(-)
[PATCH 0/6] libxl: Misc cleanups and improvements
Posted by Jim Fehlig 2 years, 9 months ago
My main objective with this series was to decompose libxlDomainStart a bit,
as it had become quite unwieldy. Along the way I made a few other small
improvements to the code.

Jim Fehlig (6):
  libxl: Drop unused 'cfg' parameter from libxlDomainSaveImageOpen
  libxl: Move managed save logic to libxlDomainStartNew
  libxl: Add a helper function to unprepare network devices
  libxl: Introduce libxlDomainStartPrepare
  libxl: Introduce libxlDomainStartPerform
  libxl: Add helper function for running the hook script

 src/libxl/libxl_domain.c    | 384 ++++++++++++++++++------------------
 src/libxl/libxl_domain.h    |  11 +-
 src/libxl/libxl_driver.c    |  30 +--
 src/libxl/libxl_migration.c |  15 +-
 4 files changed, 218 insertions(+), 222 deletions(-)

-- 
2.31.1


Re: [PATCH 0/6] libxl: Misc cleanups and improvements
Posted by Jano Tomko 2 years, 9 months ago
On a %A in %Y, Jim Fehlig wrote:
> My main objective with this series was to decompose libxlDomainStart a bit,
> as it had become quite unwieldy. Along the way I made a few other small
> improvements to the code.
> 
> Jim Fehlig (6):
>   libxl: Drop unused 'cfg' parameter from libxlDomainSaveImageOpen
>   libxl: Move managed save logic to libxlDomainStartNew
>   libxl: Add a helper function to unprepare network devices
>   libxl: Introduce libxlDomainStartPrepare
>   libxl: Introduce libxlDomainStartPerform
>   libxl: Add helper function for running the hook script
> 
>  src/libxl/libxl_domain.c    | 384 ++++++++++++++++++------------------
>  src/libxl/libxl_domain.h    |  11 +-
>  src/libxl/libxl_driver.c    |  30 +--
>  src/libxl/libxl_migration.c |  15 +-
>  4 files changed, 218 insertions(+), 222 deletions(-)
> 

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

Jano