1
=== modified file 'src/server/compositor/default_display_buffer_compositor.cpp'
2
--- src/server/compositor/default_display_buffer_compositor.cpp	2016-08-01 14:43:19 +0000
3
+++ src/server/compositor/default_display_buffer_compositor.cpp	2016-08-17 03:03:20 +0000
4
@@ -85,9 +85,17 @@
5
85
        report->renderables_in_frame(this, renderable_list);
85
        report->renderables_in_frame(this, renderable_list);
6
86
        report->rendered_frame(this);
86
        report->rendered_frame(this);
7
87
87
11
88
        // Release the buffers we did use back to the clients, before starting
88
        /*
12
89
        // on the potentially slow flip().
89
         * This is used for the 'early release' optimization to release buffers
13
90
        // FIXME: This clear() call is blocking a little because we drive IPC here (LP: #1395421)
90
         * we did use back to clients before starting on the potentially slow
14
91
         * post() call.
15
92
         * FIXME: This clear() call is blocking a little because we drive IPC
16
93
         *        here (LP: #1395421). However if the early release
17
94
         *        optimization is disabled or absent (LP: #1561418) then this
18
95
         *        clear() doesn't contribute anything. In that case the
19
96
         *        problematic IPC (LP: #1395421) will instead occur in buffer
20
97
         *        acquisition calls when we composite the next frame.
21
98
         */
22
91
        renderable_list.clear();
99
        renderable_list.clear();
23
92
    }
100
    }
24
93
101
Reviewer	Review Type	Date Requested	Status
Cemil Azizoglu (community)		2016-08-16	Approve on 2016-08-18
Mir CI Bot	continuous-integration		Approve on 2016-08-17
Review via email: mp+302994@code.launchpad.net