Re: [libvirt] [PATCH 0/4] qemu: fix vcpupin/emulatorpin with --config on VMs with automatic NUMA pinning
On Wed, Sep 21, 2016 at 04:50:52PM +0200, Michal Privoznik wrote: On 21.09.2016 15:54, Martin Kletzander wrote: On Wed, Sep 14, 2016 at 08:39:05AM +0200, Peter Krempa wrote: The automatic nodeset would creep into the persistent definition for live VMs with automatic pinning which is incorrect. https://bugzilla.redhat.com/show_bug.cgi?id=1365779 Peter Krempa (4): conf: Introduce virDomainObjGetOneDefState qemu: domain: Add macro to simplify access to vm private data I must say I don't like this one ^^, it's way more readable to just use the variable which can be just optimized out, but since we use that already in multiple places, meh ACK series, I guess. Agreed. It makes debugging way worse and also does not work well with autocomplete in my vim. Revert it please. I would suggest removing all at the same time. The conversation about how some people don't like it, even though it doesn't matter, can happen in that series then. signature.asc Description: Digital signature -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH 0/4] qemu: fix vcpupin/emulatorpin with --config on VMs with automatic NUMA pinning
On 21.09.2016 15:54, Martin Kletzander wrote: > On Wed, Sep 14, 2016 at 08:39:05AM +0200, Peter Krempa wrote: >> The automatic nodeset would creep into the persistent definition for >> live VMs >> with automatic pinning which is incorrect. >> >> https://bugzilla.redhat.com/show_bug.cgi?id=1365779 >> >> Peter Krempa (4): >> conf: Introduce virDomainObjGetOneDefState >> qemu: domain: Add macro to simplify access to vm private data > > I must say I don't like this one ^^, it's way more readable to just use > the variable which can be just optimized out, but since we use that > already in multiple places, meh ACK series, I guess. Agreed. It makes debugging way worse and also does not work well with autocomplete in my vim. Revert it please. Michal -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH 0/4] qemu: fix vcpupin/emulatorpin with --config on VMs with automatic NUMA pinning
On Wed, Sep 21, 2016 at 15:54:34 +0200, Martin Kletzander wrote: > On Wed, Sep 14, 2016 at 08:39:05AM +0200, Peter Krempa wrote: > >The automatic nodeset would creep into the persistent definition for live VMs > >with automatic pinning which is incorrect. > > > >https://bugzilla.redhat.com/show_bug.cgi?id=1365779 > > > >Peter Krempa (4): > > conf: Introduce virDomainObjGetOneDefState > > qemu: domain: Add macro to simplify access to vm private data > > I must say I don't like this one ^^, it's way more readable to just use > the variable which can be just optimized out, but since we use that > already in multiple places, meh ACK series, I guess. Thanks, pushed. Peter -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH 0/4] qemu: fix vcpupin/emulatorpin with --config on VMs with automatic NUMA pinning
On Wed, Sep 14, 2016 at 08:39:05AM +0200, Peter Krempa wrote: The automatic nodeset would creep into the persistent definition for live VMs with automatic pinning which is incorrect. https://bugzilla.redhat.com/show_bug.cgi?id=1365779 Peter Krempa (4): conf: Introduce virDomainObjGetOneDefState qemu: domain: Add macro to simplify access to vm private data I must say I don't like this one ^^, it's way more readable to just use the variable which can be just optimized out, but since we use that already in multiple places, meh ACK series, I guess. qemu: driver: Don't return automatic NUMA vCPU pinning data for persistentDef qemu: driver: Don't return automatic NUMA emulator pinning data for persistentDef src/conf/domain_conf.c | 40 src/conf/domain_conf.h | 3 +++ src/libvirt_private.syms | 1 + src/qemu/qemu_domain.h | 3 +++ src/qemu/qemu_driver.c | 23 +-- 5 files changed, 56 insertions(+), 14 deletions(-) -- 2.10.0 -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list signature.asc Description: Digital signature -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH 0/4] qemu: fix vcpupin/emulatorpin with --config on VMs with automatic NUMA pinning
On Wed, Sep 14, 2016 at 08:39:05 +0200, Peter Krempa wrote: > The automatic nodeset would creep into the persistent definition for live VMs > with automatic pinning which is incorrect. > > https://bugzilla.redhat.com/show_bug.cgi?id=1365779 > > Peter Krempa (4): > conf: Introduce virDomainObjGetOneDefState > qemu: domain: Add macro to simplify access to vm private data > qemu: driver: Don't return automatic NUMA vCPU pinning data for > persistentDef > qemu: driver: Don't return automatic NUMA emulator pinning data for > persistentDef Ping. -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list