Itamar Heim has posted comments on this change.

Change subject: core: fix cpu info for AMD Opterom Processor G5
......................................................................


Patch Set 1:

actually that's not correct (the comment, the patch seems fine).
the problem is a simple spelling mistake in the name of the virtualization flag 
(VMX for intel, SVM for AMD). for some reason, the G5 model was created with 
SMX as the virt flag, instead of the usual SVM.
this has nothing to do with model detection, only with virt flag detection (and 
nx flag detection) which we use to assure not only the model is correct, but 
also the virt and nx flags are there.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ica30c91f572229e9cbd0277dd84852c8f99bd50e
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Arik Hadas <[email protected]>
Gerrit-Reviewer: Arik Hadas <[email protected]>
Gerrit-Reviewer: Eli Mesika <[email protected]>
Gerrit-Reviewer: Itamar Heim <[email protected]>
Gerrit-Reviewer: Omer Frenkel <[email protected]>
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to