Vitor de Lima has posted comments on this change.

Change subject: core, engine, webadmin: Consider only online logical CPUs
......................................................................


Patch Set 3: Verified+1

Verified this patch by creating a fake Host with three cores, disabling the 
core with id 1 and trying to pin a vCPU of a VM into the disabled core and 
later into the core with id 2.

(This covers the two problematic cases in the original code, disabled cores 
with an id smaller than the total number of CPUs and CPUs with ids larger or 
equal to this total)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ib20f20d6003502fe2f2864abb5ad8f4f5a19ecdc
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Vitor de Lima <vdel...@redhat.com>
Gerrit-Reviewer: Eli Mesika <emes...@redhat.com>
Gerrit-Reviewer: Jiří Moskovčák <jmosk...@redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com>
Gerrit-Reviewer: Roy Golan <rgo...@redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjeli...@redhat.com>
Gerrit-Reviewer: Vitor de Lima <vdel...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to