Arik Hadas has submitted this change and it was merged. Change subject: core: Video RAM size settings reworked ...................................................................... core: Video RAM size settings reworked Video RAM sizes are currently set in a wrong way. Especially one of the most important parameters, `vgamem', is not set at all. That may prevent setups with high resolution monitors from working. This patch tries to set more reasonable video RAM sizes. It replaces the old video RAM size computation (including removal of some unused related code) with a new one, based on guest OS. The guest OS specifics are set in osinfo. We try to keep the computation as simple as possible, for the following reasons: 1. Nobody can say what the perfect values are. 2. Things are likely to change in QEMU in future. 3. We try to be reasonably conservative not to break current setups. Note that recent VDSM version is needed to set `vgamem' value (it's ignored in older versions). As we can't be sure about all older versions of VDSM, libvirt and QEMU, we send the same values as before to clusters of older versions. These may be not the best values but it's the safest approach. We may provide means to override the memory values via osinfo in a followup patch. Change-Id: I96ab9d40621ad12ec2505a2de75e3ff5af2f59d0 Bug-Url: https://bugzilla.redhat.com/1275539 Signed-off-by: Milan Zamazal <mzamazal@redhat.com> --- A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/utils/VideoDeviceSettings.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/utils/VmDeviceUtils.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/osinfo/OsRepository.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/utils/VmDeviceCommonUtils.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/OsRepositoryImpl.java M backend/manager/modules/utils/src/main/javacc/org/ovirt/engine/core/utils/osinfo/osinfo.jj M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsProperties.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VmInfoBuilder.java M packaging/conf/osinfo-defaults.properties 9 files changed, 155 insertions(+), 48 deletions(-) Approvals: Jenkins CI: Passed CI tests Shmuel Leib Melamud: Looks good to me, but someone else must approve Arik Hadas: Looks good to me, approved Milan Zamazal: Verified -- To view, visit https://gerrit.ovirt.org/50092 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I96ab9d40621ad12ec2505a2de75e3ff5af2f59d0 Gerrit-PatchSet: 10 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Milan Zamazal <mzamazal@redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek@redhat.com> Gerrit-Reviewer: Milan Zamazal <mzamazal@redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi@redhat.com> Gerrit-Reviewer: Shmuel Leib Melamud <smelamud@redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>