Couple of almost trivial patches

[PATCH 0/3] Couple of almost trivial patches

Michal Privoznik posted 3 patches 4 years, 2 months ago

Download series mbox

Test syntax-check failed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/cover.1582196469.git.mprivozn@redhat.com

src/qemu/qemu_migration.c      | 66 +++++++++++++++++-----------------
src/qemu/qemu_process.c        |  9 +----
src/qemu/qemu_vhost_user_gpu.c | 10 +-----
src/util/virpidfile.c          |  2 +-
4 files changed, 36 insertions(+), 51 deletions(-)

Expand all Fold all

[PATCH 0/3] Couple of almost trivial patches

Posted by Michal Privoznik 4 years, 2 months ago

These stem out from my review of Marc-André's patches:

https://www.redhat.com/archives/libvir-list/2020-January/msg00648.html

Michal Prívozník (3):
  virpidfile: Set correct retval in virPidFileReadPath()
  qemu: Don't explicitly remove pidfile after
    virPidFileForceCleanupPath()
  qemu_migration: Rearrange some checks in qemuMigrationSrcIsAllowed()

 src/qemu/qemu_migration.c      | 66 +++++++++++++++++-----------------
 src/qemu/qemu_process.c        |  9 +----
 src/qemu/qemu_vhost_user_gpu.c | 10 +-----
 src/util/virpidfile.c          |  2 +-
 4 files changed, 36 insertions(+), 51 deletions(-)

-- 
2.24.1

Re: [PATCH 0/3] Couple of almost trivial patches

Posted by Ján Tomko 4 years, 2 months ago

On Thu, Feb 20, 2020 at 12:02:34PM +0100, Michal Privoznik wrote:
>These stem out from my review of Marc-André's patches:
>
>https://www.redhat.com/archives/libvir-list/2020-January/msg00648.html
>
>Michal Prívozník (3):
>  virpidfile: Set correct retval in virPidFileReadPath()
>  qemu: Don't explicitly remove pidfile after
>    virPidFileForceCleanupPath()
>  qemu_migration: Rearrange some checks in qemuMigrationSrcIsAllowed()
>
> src/qemu/qemu_migration.c      | 66 +++++++++++++++++-----------------
> src/qemu/qemu_process.c        |  9 +----
> src/qemu/qemu_vhost_user_gpu.c | 10 +-----
> src/util/virpidfile.c          |  2 +-
> 4 files changed, 36 insertions(+), 51 deletions(-)
>

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

Jano