Royce Lv has posted comments on this change.

Change subject: add emulator pin support to create vm
......................................................................


Patch Set 11: (1 inline comment)

....................................................
Commit Message
Line 5: CommitDate: 2012-12-20 15:49:56 +0800
Line 6: 
Line 7: add emulator pin support to create vm
Line 8: 
Line 9: move cpupin to a seperate function 'appendTunable'
As we support 'vcpu-pin' to take advantage of  localization. we pin emulator 
for the same purpose. In my opinion, for SLA, if we pin all low privilege 
guests(their vcpu and emulator) to specific core, it reserves CPU resources for 
high privileged guests. Correct me if I'm wrong.
Line 10: to add cputune(e.g.cpu_share),memtune limit def when create vm.
Line 11: add emulator pin support to create vm,
Line 12: then the cpuPinning dict changs to be:
Line 13: {0:'1-4','emulator':'2'}


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ic616b825eca94682e1563ab51773d8a23351998d
Gerrit-PatchSet: 11
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Royce Lv <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Doron Fediuck <[email protected]>
Gerrit-Reviewer: Laszlo Hornyak <[email protected]>
Gerrit-Reviewer: Mark Wu <[email protected]>
Gerrit-Reviewer: Royce Lv <[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