AlexIndustrial/mesa
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

turnip: improve binning pipe layout config

Browse Source 
The old code looks the same as GL driver, but we get things like
pipe_count = {32, 1}, which seems bad.

This uses similar logic as for tiles which produces a balanced pipe_count
width/height.

Signed-off-by: Jonathan Marek <jonathan@marek.ca>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/3142>
This commit is contained in:
Jonathan Marek
2019-12-17 17:22:46 -05:00
parent 9f9432d56c
commit 036230341f
1 changed files with 10 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/freedreno/vulkan/tu_cmd_buffer.c
+10 -15
View File
@@ -254,21 +254,16 @@ tu_tiling_config_update_pipe_layout(struct tu_tiling_config *tiling,
};
tiling->pipe_count = tiling->tile_count;
/* do not exceed max pipe count vertically */
while (tiling->pipe_count.height > max_pipe_count) {
tiling->pipe0.height += 2;
tiling->pipe_count.height =
(tiling->tile_count.height + tiling->pipe0.height - 1) /
tiling->pipe0.height;
}
/* do not exceed max pipe count */
while (tiling->pipe_count.width * tiling->pipe_count.height >
max_pipe_count) {
tiling->pipe0.width += 1;
tiling->pipe_count.width =
(tiling->tile_count.width + tiling->pipe0.width - 1) /
tiling->pipe0.width;
while (tiling->pipe_count.width * tiling->pipe_count.height > max_pipe_count) {
if (tiling->pipe0.width < tiling->pipe0.height) {
tiling->pipe0.width += 1;
tiling->pipe_count.width =
DIV_ROUND_UP(tiling->tile_count.width, tiling->pipe0.width);
} else {
tiling->pipe0.height += 1;
tiling->pipe_count.height =
DIV_ROUND_UP(tiling->tile_count.height, tiling->pipe0.height);
}
}
}