Dan Kenigsberg has submitted this change and it was merged.

Change subject: Change source of CPU sockets/cores to libvirt capabilities
......................................................................


Change source of CPU sockets/cores to libvirt capabilities

Bug-Url: http://bugzilla.redhat.com/833425

/proc/cpuinfo is not enough to provide full CPU
sockets/cores/threads information in some systems,
e.g. multi NUMA AMD Magny-Cours processors. To bypass
different/unusual architectures and avoid duplication
of code between vdsm and libvirt, this patch changes
the source of CPU topology from /proc/cpuinfo to
libvirt API.

Since libvirt ignores funky topologies [1] with
assimetric number of sockets per numa node and
cores per socket, this patch assumes that all
numa nodes has the same number of sockets and
all sockets has the same number of cores.

Merged with Laszlo's changes in http://gerrit.ovirt.org/8302

[1] - 
http://libvirt.org/git/?p=libvirt.git;a=commit;h=ac9dd4a676f21b5e3ca6dbe0526f2a6709072beb

Change-Id: I1619e3d9e042bc801c988f099d3b84922f4e03d3
Signed-off-by: Amador Pahim <[email protected]>
---
M tests/Makefile.am
M tests/capsTests.py
A tests/caps_libvirt.out
M vdsm.spec.in
M vdsm/caps.py
M vdsm/clientIF.py
6 files changed, 194 insertions(+), 17 deletions(-)

Approvals:
  Amador Pahim: Verified
  Dan Kenigsberg: Looks good to me, approved


--
To view, visit http://gerrit.ovirt.org/5481
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I1619e3d9e042bc801c988f099d3b84922f4e03d3
Gerrit-PatchSet: 20
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Amador Pahim <[email protected]>
Gerrit-Reviewer: Amador Pahim <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Doron Fediuck <[email protected]>
Gerrit-Reviewer: Douglas Schilling Landgraf <[email protected]>
Gerrit-Reviewer: Gal Hammer <[email protected]>
Gerrit-Reviewer: Laszlo Hornyak <[email protected]>
Gerrit-Reviewer: Michal Skrivanek <[email protected]>
Gerrit-Reviewer: Shu Ming <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to