Commit 555c17dd authored by Committed by Kristian H. Kristensen
compositor-drm: refactor to avoid unnecessary allocation of KMS FBs
Currently, the drm backend will create and destroy a KMS FB for each frame. However, the bos for a gbm surface are reused (at least with mesa) so we can store the fb_id on it and destroy it only on the bo's destroy callback. To use the same path for scanning out client buffers, some refactor was needed. Previously, the bo for the client buffer was destroyed early so that gbm_surface_release_buffer() would not be called with it, since at the page flip handler output->scanout_buffer can be NULL even if the current frame is a client buffer. This was solved by adding a drm_fb structure that holds a gbm_bo, an fb_id, and information about the fb coming from a client buffer or not. A drm_fb is created in such a way that it is destroyed whenever the bo it references is destroyed. The fields current_* and next_* in drm_output are changed into only two pointers to drm_fb's.
Showing with 139 additions and 122 deletions