qemu: Move originalMemlock

[PATCH 0/4] qemu: Move originalMemlock

Michal Privoznik posted 4 patches 1 year, 11 months ago

Download series mbox

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

src/conf/domain_conf.h |  3 --
src/qemu/qemu_domain.c | 76 +++++++++++++++++++++++-------------------
src/qemu/qemu_domain.h |  3 ++
3 files changed, 45 insertions(+), 37 deletions(-)

Expand all Fold all

[PATCH 0/4] qemu: Move originalMemlock

Posted by Michal Privoznik 1 year, 11 months ago

As I tried to explain the purpose of virDomainObj to somebody I've
came across this very weirdly placed member: originalMemlock. Let's move
it into qemuDomainObjPrivate. But after I've done so I've realized the
error label is needless. So that's dropped too.

Michal Prívozník (4):
  conf: Move virDomainObj::originalMemlock into qemuDomainObjPrivate
  qemu_domain: Format qemuDomainObjPrivate::originalMemlock
  qemu_domain: Drop needless free from qemuDomainObjPrivateXMLParse()
  qemu_domain: Drop needless 'error' label in
    qemuDomainObjPrivateXMLParse()

 src/conf/domain_conf.h |  3 --
 src/qemu/qemu_domain.c | 76 +++++++++++++++++++++++-------------------
 src/qemu/qemu_domain.h |  3 ++
 3 files changed, 45 insertions(+), 37 deletions(-)

-- 
2.35.1

Re: [PATCH 0/4] qemu: Move originalMemlock

Posted by Peter Krempa 1 year, 11 months ago

On Wed, May 11, 2022 at 17:03:15 +0200, Michal Privoznik wrote:
> As I tried to explain the purpose of virDomainObj to somebody I've
> came across this very weirdly placed member: originalMemlock. Let's move
> it into qemuDomainObjPrivate. But after I've done so I've realized the
> error label is needless. So that's dropped too.
> 
> Michal Prívozník (4):
>   conf: Move virDomainObj::originalMemlock into qemuDomainObjPrivate
>   qemu_domain: Format qemuDomainObjPrivate::originalMemlock
>   qemu_domain: Drop needless free from qemuDomainObjPrivateXMLParse()
>   qemu_domain: Drop needless 'error' label in
>     qemuDomainObjPrivateXMLParse()

Reviewed-by: Peter Krempa <pkrempa@redhat.com>