Roy Golan has posted comments on this change.

Change subject: core: add Opteron G5 model
......................................................................


Patch Set 1: (1 inline comment)

....................................................
File backend/manager/dbscripts/upgrade/pre_upgrade/0000_config.sql
Line 418: select fn_db_add_config_value('SendVmTicketUID','true','3.2');
Line 419: select fn_db_add_config_value('ServerCPUList','2:Intel Xeon w/o 
XD/NX:vmx,sse2:qemu64,-nx,+sse2; 3:Intel Xeon:vmx,sse2,nx:qemu64,+sse2; 4:Intel 
Conroe Family:vmx,sse2,nx,cx16,ssse3:qemu64,+sse2,+cx16,+ssse3; 5:Intel Penryn 
Family:vmx,sse2,nx,cx16,ssse3,sse4_1:qemu64,+sse2,+cx16,+ssse3,+sse4.1; 6:Intel 
Nehalem 
Family:vmx,sse2,nx,cx16,ssse3,sse4_1,sse4_2,popcnt:qemu64,+sse2,+cx16,+ssse3,+sse4.1,+sse4.2,+popcnt;
 2:AMD Opteron G1 w/o NX:svm,sse2:qemu64,-nx,+sse2; 3:AMD Opteron 
G1:svm,sse2,nx:qemu64,+sse2; 4:AMD Opteron 
G2:svm,sse2,nx,cx16:qemu64,+sse2,+cx16; 5:AMD Opteron 
G3:svm,sse2,nx,cx16,sse4a,misalignsse,popcnt,abm:qemu64,+sse2,+cx16,+sse4a,+misalignsse,+popcnt,+abm;','2.2');
Line 420: select fn_db_add_config_value('ServerCPUList','3:Intel Conroe 
Family:vmx,nx,model_Conroe:Conroe; 4:Intel Penryn 
Family:vmx,nx,model_Penryn:Penryn; 5:Intel Nehalem 
Family:vmx,nx,model_Nehalem:Nehalem; 6:Intel Westmere 
Family:aes,vmx,nx,model_Westmere:Westmere; 2:AMD Opteron 
G1:svm,nx,model_Opteron_G1:Opteron_G1; 3:AMD Opteron 
G2:svm,nx,model_Opteron_G2:Opteron_G2; 4:AMD Opteron 
G3:svm,nx,model_Opteron_G3:Opteron_G3;','3.0');
Line 421: select fn_db_add_config_value('ServerCPUList','3:Intel Conroe 
Family:vmx,nx,model_Conroe:Conroe; 4:Intel Penryn 
Family:vmx,nx,model_Penryn:Penryn; 5:Intel Nehalem 
Family:vmx,nx,model_Nehalem:Nehalem; 6:Intel Westmere 
Family:aes,vmx,nx,model_Westmere:Westmere; 2:AMD Opteron 
G1:svm,nx,model_Opteron_G1:Opteron_G1; 3:AMD Opteron 
G2:svm,nx,model_Opteron_G2:Opteron_G2; 4:AMD Opteron 
G3:svm,nx,model_Opteron_G3:Opteron_G3;','3.1');
Line 422: select fn_db_add_config_value('ServerCPUList','3:Intel Conroe 
Family:vmx,nx,model_Conroe:Conroe; 4:Intel Penryn 
Family:vmx,nx,model_Penryn:Penryn; 5:Intel Nehalem 
Family:vmx,nx,model_Nehalem:Nehalem; 6:Intel Westmere 
Family:aes,vmx,nx,model_Westmere:Westmere; 7:Intel SandyBridge 
Family:vmx,nx,model_SandyBridge:SandyBridge; 2:AMD Opteron 
G1:svm,nx,model_Opteron_G1:Opteron_G1; 3:AMD Opteron 
G2:svm,nx,model_Opteron_G2:Opteron_G2; 4:AMD Opteron 
G3:svm,nx,model_Opteron_G3:Opteron_G3; 5:AMD Opteron 
G4:svm,nx,model_Opteron_G4:Opteron_G4; 6:AMD Opteron 
G5:smx,model_Opteron_G5:Opteron_G5;','3.2');
the flag is useless (same for all other reported) - 
I have run a test with 2 of my hosts - set off the nx flag - vdsm keeps 
reporting my model. I think I should open a libvirt bug here for they are 
comparing the host cpu against a given xml.

vdsm is relying on the model reported by libvirt so all we need is the model 
string (don't need vmx|smx as well). the set of flags which are being checked 
upon is read from /usr/share/libvirt/cpu_map.xml - if the host is superset or 
identical in its flags then the model is returned.
Line 423: select fn_db_add_config_value('ServerRebootTimeout','300','general');
Line 424: select 
fn_db_add_config_value('SetupNetworksPollingTimeout','3','general');
Line 425: -- Add shareable disk property in vdc_options to support only 3.1 
version.
Line 426: select fn_db_add_config_value('ShareableDiskEnabled','false','2.2');


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I6eb396a88898924459e6ba5ff58356adddc56707
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Roy Golan <rgo...@redhat.com>
Gerrit-Reviewer: Itamar Heim <ih...@redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com>
Gerrit-Reviewer: Roy Golan <rgo...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to