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

Verify if XShmPutImage() works fine when VT-d is enabled #23

Closed
marmarek opened this Issue Mar 8, 2015 · 2 comments

Comments

Projects
None yet
1 participant
@marmarek
Member

marmarek commented Mar 8, 2015

Reported by joanna on 20 Apr 2010 22:06 UTC
According to this message:
http://groups.google.com/group/qubes-devel/msg/9d23cd9c26127ed7

The XShmPutImage() function, that is used by guid in Dom0 to draw the actual app's pixmap, might attempt to schedule a DMA from the SHM region to video memory. The SHM memory in our case belongs originaly to DomU and is being brought to Dom0 via xc_map_foreign_pages() (see http://qubes-os.org/gitweb/?p=rafal/gui.git;a=blob;f=shmoverride/shmoverride.c;h=77be2127948436013dc8a57ea069901a5d33ec03;hb=HEAD#l48).

If xc_map_foreign_pages() doesn't update IOMMU/VT-d tables so that the mapped pages belong also to Dom0, than the graphics card, that is VT-d assigned to Dom0, would not be able to do DMA read from those pages, and so the XShmPutImage() might not work correctly (not at all in fact).

TODO: check if xc_map_foreign_pages() updates IOMMU/VT-d tables, and if not discuss a way to add such behaviour on xen-devel and implement a patch.

Migrated-From: https://wiki.qubes-os.org/ticket/23

@marmarek

This comment has been minimized.

Show comment
Hide comment
@marmarek

marmarek Mar 8, 2015

Member

Comment by joanna on 13 May 2010 12:39 UTC
No need to worry! All should work fine -- see this thread:

http://lists.xensource.com/archives/html/xen-devel/2010-05/msg00667.html

Member

marmarek commented Mar 8, 2015

Comment by joanna on 13 May 2010 12:39 UTC
No need to worry! All should work fine -- see this thread:

http://lists.xensource.com/archives/html/xen-devel/2010-05/msg00667.html

@marmarek

This comment has been minimized.

Show comment
Hide comment
@marmarek

marmarek Mar 8, 2015

Member

Modified by joanna on 27 Jul 2010 14:58 UTC

Member

marmarek commented Mar 8, 2015

Modified by joanna on 27 Jul 2010 14:58 UTC

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment