Skip to content
This repository has been archived by the owner on Nov 1, 2021. It is now read-only.

backend/drm: only import buffers once #2276

Closed
emersion opened this issue Jun 11, 2020 · 1 comment · Fixed by #2568
Closed

backend/drm: only import buffers once #2276

emersion opened this issue Jun 11, 2020 · 1 comment · Fixed by #2568

Comments

@emersion
Copy link
Member

Right now each time the DRM backend wants to scan-out a client buffer it imports it via GBM. On some drivers I believe this degrades performance. For instance with #2240 and AMD, I end up with the compositor using near 100% CPU, spending a lot of time in Addr::V2::CoordEq::solve in radeonsi.so.

Instead we should import each buffer once, and then re-use the gbm_bo.

@emersion
Copy link
Member Author

The AMD-specific issue may be fixed by https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/5398

Would still be a good idea to import buffers once.

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

Successfully merging a pull request may close this issue.

1 participant