diff options
author | Marek Olšák <[email protected]> | 2018-06-18 15:53:47 -0400 |
---|---|---|
committer | Marek Olšák <[email protected]> | 2018-06-28 22:27:25 -0400 |
commit | 41f80373b46604f585497086f971a43aeea7f0c1 (patch) | |
tree | 5b26412b041486e97291638452172a4f66d3623e | |
parent | fb28bf23db00d16b1b060c732e71e665a36c190b (diff) |
radeonsi: fix memory exhaustion issue with DCC statistics gathering with DRI2
Cc: 18.1 <[email protected]>
-rw-r--r-- | src/gallium/drivers/radeonsi/si_blit.c | 30 |
1 files changed, 27 insertions, 3 deletions
diff --git a/src/gallium/drivers/radeonsi/si_blit.c b/src/gallium/drivers/radeonsi/si_blit.c index 72adc21b3dc..0fd69f3a6c5 100644 --- a/src/gallium/drivers/radeonsi/si_blit.c +++ b/src/gallium/drivers/radeonsi/si_blit.c @@ -1332,9 +1332,33 @@ static void si_flush_resource(struct pipe_context *ctx, } /* Always do the analysis even if DCC is disabled at the moment. */ - if (tex->dcc_gather_statistics && tex->separate_dcc_dirty) { - tex->separate_dcc_dirty = false; - vi_separate_dcc_process_and_reset_stats(ctx, tex); + if (tex->dcc_gather_statistics) { + bool separate_dcc_dirty = tex->separate_dcc_dirty; + + /* If the color buffer hasn't been unbound and fast clear hasn't + * been used, separate_dcc_dirty is false, but there may have been + * new rendering. Check if the color buffer is bound and assume + * it's dirty. + * + * Note that DRI2 never unbinds window colorbuffers, which means + * the DCC pipeline statistics query would never be re-set and would + * keep adding new results until all free memory is exhausted if we + * didn't do this. + */ + if (!separate_dcc_dirty) { + for (unsigned i = 0; i < sctx->framebuffer.state.nr_cbufs; i++) { + if (sctx->framebuffer.state.cbufs[i] && + sctx->framebuffer.state.cbufs[i]->texture == res) { + separate_dcc_dirty = true; + break; + } + } + } + + if (separate_dcc_dirty) { + tex->separate_dcc_dirty = false; + vi_separate_dcc_process_and_reset_stats(ctx, tex); + } } } |