qemu_agent: Rework domain object handling in open

[PATCH 0/4] qemu_agent: Rework domain object handling in open

Michal Privoznik posted 4 patches 2 years, 5 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.1635493104.git.mprivozn@redhat.com

src/qemu/qemu_agent.c   | 17 ++++++++---------
src/qemu/qemu_agent.h   |  2 --
src/qemu/qemu_monitor.c |  5 -----
src/qemu/qemu_process.c | 20 --------------------
4 files changed, 8 insertions(+), 36 deletions(-)

Expand all Fold all

[PATCH 0/4] qemu_agent: Rework domain object handling in open

Posted by Michal Privoznik 2 years, 5 months ago

The most important patch is the first one because it fixes a race
condition. The rest is just a cleanup I've noticed while looking at the
code.

Michal Prívozník (4):
  qemu_agent: Rework domain object locking when opening agent
  qemuAgentOpen: Rework domain object refcounting
  qemu_agent: Drop destroy callback
  qemuMonitorOpen: Rework domain object refcounting

 src/qemu/qemu_agent.c   | 17 ++++++++---------
 src/qemu/qemu_agent.h   |  2 --
 src/qemu/qemu_monitor.c |  5 -----
 src/qemu/qemu_process.c | 20 --------------------
 4 files changed, 8 insertions(+), 36 deletions(-)

-- 
2.32.0

Re: [PATCH 0/4] qemu_agent: Rework domain object handling in open

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

On a Friday in 2021, Michal Privoznik wrote:
>The most important patch is the first one because it fixes a race
>condition. The rest is just a cleanup I've noticed while looking at the
>code.
>
>Michal Prívozník (4):
>  qemu_agent: Rework domain object locking when opening agent
>  qemuAgentOpen: Rework domain object refcounting
>  qemu_agent: Drop destroy callback
>  qemuMonitorOpen: Rework domain object refcounting
>
> src/qemu/qemu_agent.c   | 17 ++++++++---------
> src/qemu/qemu_agent.h   |  2 --
> src/qemu/qemu_monitor.c |  5 -----
> src/qemu/qemu_process.c | 20 --------------------
> 4 files changed, 8 insertions(+), 36 deletions(-)
>

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

Jano