Change in vdsm[master]: vdsm: implement NUMA aware KSM policy

dudi at redhat.com dudi at redhat.com
Sun May 10 09:08:28 UTC 2015


Dudi Maroshi has posted comments on this change.

Change subject: vdsm: implement NUMA aware KSM policy
......................................................................


Patch Set 6: Verified+1

(2 comments)

https://gerrit.ovirt.org/#/c/40129/6/vdsm/supervdsmServer
File vdsm/supervdsmServer:

Line 309:                                         iommu_group)
Line 310:         with open(ruleFile, "w") as rf:
Line 311:             self.log.debug("Creating rule %s: %r", ruleFile, rule)
Line 312:             rf.write(rule)
Line 313: 
Change resulting from rebase. Not my commit.
Line 314:     @logDecorator
Line 315:     def rmAppropriateIommuGroup(self, iommu_group):
Line 316:         """
Line 317:         Remove udev rule in /etc/udev/rules.d/ created by


Line 319:         """
Line 320:         rule = os.path.join(_UDEV_RULE_FILE_DIR, _UDEV_RULE_FILE_PREFIX +
Line 321:                             "iommu_group_" + iommu_group + _UDEV_RULE_FILE_EXT)
Line 322:         self.log.debug("Removing rule %s", rule)
Line 323:         os.remove(rule)
Change resulting from rebase. Not my commit.
Line 324: 
Line 325:     @logDecorator
Line 326:     def ksmTune(self, tuningParams):
Line 327:         '''


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Iec607f9ef3284c1448bfc2831d125fc4d81b28d2
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dudi Maroshi <dudi at redhat.com>
Gerrit-Reviewer: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Dudi Maroshi <dudi at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Max Kovgan <mvk at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list