qemu: Open chardev logfile on behalf of QEMU

[PATCH 0/4] qemu: Open chardev logfile on behalf of QEMU

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

src/conf/domain_conf.c    |  2 +-
src/conf/domain_conf.h    |  2 +-
src/qemu/qemu_command.c   | 68 +++++++++++++++++++++++++++++----------
src/qemu/qemu_domain.c    | 13 ++++----
src/qemu/qemu_domain.h    |  4 +--
src/qemu/qemu_driver.c    |  4 +--
src/qemu/qemu_saveimage.c |  5 +--
7 files changed, 66 insertions(+), 32 deletions(-)

Expand all Fold all

[PATCH 0/4] qemu: Open chardev logfile on behalf of QEMU

Posted by Michal Privoznik 2 years, 8 months ago

See 4/4 for explanation.

Michal Prívozník (4):
  virDomainDefGetSecurityLabelDef: Fix const correctness
  qemuDomainOpenFile: Take virDomainDef instead of virDomainObj
  qemuDomainOpenFile: Take @cfg instead of driver
  qemu: Open chardev logfile on behalf of QEMU

 src/conf/domain_conf.c    |  2 +-
 src/conf/domain_conf.h    |  2 +-
 src/qemu/qemu_command.c   | 68 +++++++++++++++++++++++++++++----------
 src/qemu/qemu_domain.c    | 13 ++++----
 src/qemu/qemu_domain.h    |  4 +--
 src/qemu/qemu_driver.c    |  4 +--
 src/qemu/qemu_saveimage.c |  5 +--
 7 files changed, 66 insertions(+), 32 deletions(-)

-- 
2.31.1

Re: [PATCH 0/4] qemu: Open chardev logfile on behalf of QEMU

Posted by Pavel Hrdina 2 years, 8 months ago

On Fri, Aug 06, 2021 at 05:22:34PM +0200, Michal Privoznik wrote:
> See 4/4 for explanation.
> 
> Michal Prívozník (4):
>   virDomainDefGetSecurityLabelDef: Fix const correctness
>   qemuDomainOpenFile: Take virDomainDef instead of virDomainObj
>   qemuDomainOpenFile: Take @cfg instead of driver
>   qemu: Open chardev logfile on behalf of QEMU

Reviewed-by: Pavel Hrdina <phrdina@redhat.com>

Re: [PATCH 0/4] qemu: Open chardev logfile on behalf of QEMU

Posted by Jano Tomko 2 years, 8 months ago

On a %A in %Y, Michal Privoznik wrote:
> See 4/4 for explanation.
> 
> Michal Prívozník (4):
>   virDomainDefGetSecurityLabelDef: Fix const correctness
>   qemuDomainOpenFile: Take virDomainDef instead of virDomainObj
>   qemuDomainOpenFile: Take @cfg instead of driver
>   qemu: Open chardev logfile on behalf of QEMU
> 
>  src/conf/domain_conf.c    |  2 +-
>  src/conf/domain_conf.h    |  2 +-
>  src/qemu/qemu_command.c   | 68 +++++++++++++++++++++++++++++----------
>  src/qemu/qemu_domain.c    | 13 ++++----
>  src/qemu/qemu_domain.h    |  4 +--
>  src/qemu/qemu_driver.c    |  4 +--
>  src/qemu/qemu_saveimage.c |  5 +--
>  7 files changed, 66 insertions(+), 32 deletions(-)
> 

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