Bug 1434550
Summary: | virsh host capabilities shows incorrect number of sockets | ||||||
---|---|---|---|---|---|---|---|
Product: | [Community] Virtualization Tools | Reporter: | Richard W.M. Jones <rjones> | ||||
Component: | libvirt | Assignee: | Libvirt Maintainers <libvirt-maint> | ||||
Status: | CLOSED CANTFIX | QA Contact: | |||||
Severity: | unspecified | Docs Contact: | |||||
Priority: | unspecified | ||||||
Version: | unspecified | CC: | jdenemar, libvirt-maint, pkrempa, rbalakri, srwx4096 | ||||
Target Milestone: | --- | ||||||
Target Release: | --- | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Whiteboard: | |||||||
Fixed In Version: | Doc Type: | If docs needed, set a value | |||||
Doc Text: | Story Points: | --- | |||||
Clone Of: | Environment: | ||||||
Last Closed: | 2017-03-28 13:18:54 UTC | Type: | Bug | ||||
Regression: | --- | Mount Type: | --- | ||||
Documentation: | --- | CRM: | |||||
Verified Versions: | Category: | --- | |||||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||
Cloudforms Team: | --- | Target Upstream Version: | |||||
Embargoed: | |||||||
Bug Depends On: | |||||||
Bug Blocks: | 910269 | ||||||
Attachments: |
|
Description
Richard W.M. Jones
2017-03-21 17:27:00 UTC
Is this a NUMA machine? The number of sockets is counted per NUMA node. It is similar to what virNodeGetInfo returns, just the number of NUMA cells are completely ignored here. Use other parts of the capabilities XML if you want to see the real CPU topology. Created attachment 1265346 [details]
capabilities.xml
It is a NUMA machine (in as much as every machine with > 1 sockets
these days is NUMA). This is a bit unexpected as I'd expect
"sockets" to be the number of sockets. It does show
<cells num='2'> which I guess refers to NUMA cells.
Full capabilities XML attached.
Hi Richard, I am new here. It seems to me it's a design decision when it was implemented, it maybe a design flaw as we see it today. I am simply practicing to contribute some code. Do we need to actually fix it or leave as it is? Dan As Jirka said, this was a wrong decision originally and it can't be changed now. If you are unsure how to aggregate data from the topology XML, you can check out libvirt-python.git/examples/topology.py |