[libvirt] [PATCH 0/3] Fixes for issues caused by IOThreads

John Ferlan jferlan at redhat.com
Tue Sep 16 00:13:51 UTC 2014


Well I guess it happens to everyone - hopefully only once though...

The IOThreads code really messed up a few things - git bisection for
the build is broken as of 5f6ad32c733a3bd158938aecabb0508a434ece95,
but that's resolved by 938fb12fad6d15c9fdb73f998c4e0ec1e278721f
which adds the 'niothreadspin && iothreadspin' definitions.

Secondarily it seems things got worse, because guests weren't able to
be started because of a very bad logic error (<= 0 vs. < 0 and specific
check that 0 is "OK" (meaning no IOThreads).

How that got by own self testing I'm still not quite sure, but it did
and it's a mea culpa for that.

Anyway, patch 1 fixes the running issue... Patch 2 fixes some spacing
issues that Eric pointed out privately (prefer "i + 1" vs. "i+1").
Patch 3 just makes sure to use the right cgroup setup/init parameters
since there is no "iothread0"

John Ferlan (3):
  qemu: Fix iothreads issue
  qemu_cgroup: Adjust spacing around incrementor
  qemu: Fix call in qemuDomainSetNumaParamsLive for virCgroupNewIOThread

 src/qemu/qemu_cgroup.c  | 7 ++++---
 src/qemu/qemu_driver.c  | 3 ++-
 src/qemu/qemu_process.c | 6 +++++-
 3 files changed, 11 insertions(+), 5 deletions(-)

-- 
1.9.3




More information about the libvir-list mailing list