New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Hardcoded "VideoRam 65536" in xorg-qubes.conf.template is too low for multiple 4k displays #3174

Closed
tonsilware opened this Issue Oct 14, 2017 · 8 comments

Comments

@tonsilware

tonsilware commented Oct 14, 2017

Qubes OS version:

R3.2


General notes:

Yesterday I upgraded two of my displays to 4K (the third is 2560x1400). This resulted in major QubesOS AppVM issues - like AppVMs not registering clicks beyond a certain X/Y co-ordinate and "Run command in VM" frequently just not working at all.

I thought I'd try investigate: It seems that on each AppVM "qubes-set-monitor-layout" calls "xrandr". I discovered that xrandr was throwing an error when the requested resolution was above a certain level. So I increased the VideoRAM in /etc/X11/xorg-qubes.conf.template on all my templates and that seems to have solved the problem.


Expected behavior:

Perhaps, if possible, rather than having the VideoRAM hardcoded at "65536" it should be dynamically calculated and populated (like the other values in the template) to ensure that there is sufficient VideoRAM for the AppVM when it starts (based upon the resolution at the time of the AppVM starting). e.g. something like:

minVideoRAM = (total resolution width) * (total resolution height) * (bytes per pixel) + (some additional head room?)
VideoRAM = max(minVideoRAM, 65536)

@andrewdavidwong andrewdavidwong added this to the Release 3.2 updates milestone Oct 14, 2017

marmarek added a commit to marmarek/qubes-gui-agent-linux that referenced this issue Mar 22, 2018

Add extra video RAM for dynamic display attach
Since video ram is again dynamically calculated to the current needs,
add some extra margin to allow dynamic display attach.

For now make it large enough for one more FullHD display.

QubesOS/qubes-issues#3174
QubesOS/qubes-issues#3634

@marmarek marmarek referenced this issue in QubesOS/qubes-gui-agent-linux Mar 22, 2018

Merged

Dynamic video ram #36

@marmarek

This comment has been minimized.

Show comment
Hide comment
@marmarek

marmarek Mar 22, 2018

Member

Created pull request for this (QubesOS/qubes-gui-agent-linux#36). Can you verify if that solve the problem?

cc @TheBubblePopped

Member

marmarek commented Mar 22, 2018

Created pull request for this (QubesOS/qubes-gui-agent-linux#36). Can you verify if that solve the problem?

cc @TheBubblePopped

@tonsilware

This comment has been minimized.

Show comment
Hide comment
@tonsilware

tonsilware Mar 23, 2018

Thanks, I've tested the changes and that solves the problem on my system (4 monitors in total: 2x4k, 1x(2560x1440) and 1xFHD).

tonsilware commented Mar 23, 2018

Thanks, I've tested the changes and that solves the problem on my system (4 monitors in total: 2x4k, 1x(2560x1440) and 1xFHD).

@qubesos-bot

This comment has been minimized.

Show comment
Hide comment
@qubesos-bot

qubesos-bot May 2, 2018

Automated announcement from builder-github

The package gui-agent-linux has been pushed to the r4.0 testing repository for the CentOS centos7 template.
To test this update, please install it with the following command:

sudo yum update --enablerepo=qubes-vm-r4.0-current-testing

Changes included in this update

Automated announcement from builder-github

The package gui-agent-linux has been pushed to the r4.0 testing repository for the CentOS centos7 template.
To test this update, please install it with the following command:

sudo yum update --enablerepo=qubes-vm-r4.0-current-testing

Changes included in this update

@qubesos-bot qubesos-bot referenced this issue in QubesOS/updates-status May 2, 2018

Closed

gui-agent-linux v4.0.10 (r4.0) #501

@qubesos-bot

This comment has been minimized.

Show comment
Hide comment
@qubesos-bot

qubesos-bot May 2, 2018

Automated announcement from builder-github

The component gui-agent-linux (including package pulseaudio-qubes-4.0.10-1.fc26) has been pushed to the r4.0 testing repository for the Fedora template.
To test this update, please install it with the following command:

sudo yum update --enablerepo=qubes-vm-r4.0-current-testing

Changes included in this update

Automated announcement from builder-github

The component gui-agent-linux (including package pulseaudio-qubes-4.0.10-1.fc26) has been pushed to the r4.0 testing repository for the Fedora template.
To test this update, please install it with the following command:

sudo yum update --enablerepo=qubes-vm-r4.0-current-testing

Changes included in this update

@qubesos-bot

This comment has been minimized.

Show comment
Hide comment
@qubesos-bot

qubesos-bot May 2, 2018

Automated announcement from builder-github

The package qubes-gui-agent_4.0.10-1+deb9u1 has been pushed to the r4.0 testing repository for the Debian template.
To test this update, first enable the testing repository in /etc/apt/sources.list.d/qubes-*.list by uncommenting the line containing stretch-testing (or appropriate equivalent for your template version), then use the standard update command:

sudo apt-get update && sudo apt-get dist-upgrade

Changes included in this update

Automated announcement from builder-github

The package qubes-gui-agent_4.0.10-1+deb9u1 has been pushed to the r4.0 testing repository for the Debian template.
To test this update, first enable the testing repository in /etc/apt/sources.list.d/qubes-*.list by uncommenting the line containing stretch-testing (or appropriate equivalent for your template version), then use the standard update command:

sudo apt-get update && sudo apt-get dist-upgrade

Changes included in this update

@qubesos-bot

This comment has been minimized.

Show comment
Hide comment
@qubesos-bot

qubesos-bot May 14, 2018

Automated announcement from builder-github

The package gui-agent-linux has been pushed to the r4.0 stable repository for the Fedora centos7 template.
To install this update, please use the standard update command:

sudo yum update

Changes included in this update

Automated announcement from builder-github

The package gui-agent-linux has been pushed to the r4.0 stable repository for the Fedora centos7 template.
To install this update, please use the standard update command:

sudo yum update

Changes included in this update

@qubesos-bot

This comment has been minimized.

Show comment
Hide comment
@qubesos-bot

qubesos-bot May 14, 2018

Automated announcement from builder-github

The package qubes-gui-agent_4.0.10-1+deb9u1 has been pushed to the r4.0 stable repository for the Debian template.
To install this update, please use the standard update command:

sudo apt-get update && sudo apt-get dist-upgrade

Changes included in this update

Automated announcement from builder-github

The package qubes-gui-agent_4.0.10-1+deb9u1 has been pushed to the r4.0 stable repository for the Debian template.
To install this update, please use the standard update command:

sudo apt-get update && sudo apt-get dist-upgrade

Changes included in this update

@qubesos-bot

This comment has been minimized.

Show comment
Hide comment
@qubesos-bot

qubesos-bot May 14, 2018

Automated announcement from builder-github

The component gui-agent-linux (including package pulseaudio-qubes-4.0.10-1.fc26) has been pushed to the r4.0 stable repository for the Fedora template.
To install this update, please use the standard update command:

sudo yum update

Changes included in this update

Automated announcement from builder-github

The component gui-agent-linux (including package pulseaudio-qubes-4.0.10-1.fc26) has been pushed to the r4.0 stable repository for the Fedora template.
To install this update, please use the standard update command:

sudo yum update

Changes included in this update

marmarek added a commit to marmarek/qubes-gui-agent-linux that referenced this issue Jun 4, 2018

Revert "Use fixed amount of memory"
This reverts commit 998bdb0.

The value used here is too small for multiple 4k monitors, restore
dynamic calculation.

Fixes QubesOS/qubes-issues#3174
Fixes QubesOS/qubes-issues#3634

(cherry picked from commit ac21967)

marmarek added a commit to marmarek/qubes-gui-agent-linux that referenced this issue Jun 4, 2018

Add extra video RAM for dynamic display attach
Since video ram is again dynamically calculated to the current needs,
add some extra margin to allow dynamic display attach.

For now make it large enough for one more FullHD display.

QubesOS/qubes-issues#3174
QubesOS/qubes-issues#3634

(cherry picked from commit b5acac3)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment