Change in vdsm[ovirt-3.5]: Actually use the NUMA stats thread

msivak at redhat.com msivak at redhat.com
Mon May 18 13:35:39 UTC 2015


Martin Sivák has posted comments on this change.

Change subject: Actually use the NUMA stats thread
......................................................................


Patch Set 2:

(1 comment)

https://gerrit.ovirt.org/#/c/41059/2/vdsm/virt/vm.py
File vdsm/virt/vm.py:

Line 629: 
Line 630:             stats[dName].update(dLatency)
Line 631: 
Line 632:     def _getNumaStats(self, stats):
Line 633:         sInfo, vmNumaNodeRuntimeMap, sampleInterval = self.sampleNuma.getStats()
> If you go this way (which I don't really like, but I'll not make a case aga
I thought start will be the same as end with window size 1, but anyway. I will use the start value.
Line 634:         if vmNumaNodeRuntimeMap:
Line 635:             stats['vNodeRuntimeInfo'] = vmNumaNodeRuntimeMap
Line 636: 
Line 637:     def _getVmJobs(self, stats):


-- 
To view, visit https://gerrit.ovirt.org/41059
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ib9696adfed13c101ad52fd67796f2d141c810404
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.5
Gerrit-Owner: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list