Martin Sivák has posted comments on this change.

Change subject: Add scheduling filter that checks NUMA compatibility
......................................................................


Patch Set 1:

Gilad, please check if the basic logic is ok. The NUMA fields in VM class are 
not documented so I used the logic from VdsBroker - empty getNumaNodeList means 
no NUMA support.

I also noticed that we have couple of numa related scheduling messages:

    VAR__DETAIL__NOT_MEMORY_PINNED_NUMA,
    VAR__DETAIL__NUMA_PINNING_FAILED,

But those are not used anywhere. Do you have the logic that would check the 
numa vs. vNuma nodes compatibility?

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I0a6a0883a2098c26971ca69641675da9eb724a3d
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Martin Sivák <[email protected]>
Gerrit-Reviewer: Gilad Chaplik <[email protected]>
Gerrit-Reviewer: Jiří Moskovčák <[email protected]>
Gerrit-Reviewer: Martin Sivák <[email protected]>
Gerrit-Reviewer: Roy Golan <[email protected]>
Gerrit-Reviewer: Tomer Saban <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to