Mesa (master): mesa/st: take ownership rather than adding reference for new renderbuffers
Timothy Arceri
tarceri at kemper.freedesktop.org
Mon Apr 10 01:06:09 UTC 2017
Module: Mesa
Branch: master
Commit: d0791ac2eddd3d9aaae91034ae897c9628f66174
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=d0791ac2eddd3d9aaae91034ae897c9628f66174
Author: Timothy Arceri <tarceri at itsqueeze.com>
Date: Sat Apr 8 10:54:56 2017 +1000
mesa/st: take ownership rather than adding reference for new renderbuffers
This avoids locking in the reference calls and fixes a leak after the
RefCount initialisation was change from 0 to 1.
Fixes: 32141e53d1520 (mesa: tidy up renderbuffer RefCount initialisation)
Reviewed-by: Emil Velikov <emil.velikov at collabora.com>
Tested-by: Bartosz Tomczyk <bartosz.tomczyk86 at gmail.com>
---
src/mesa/state_tracker/st_manager.c | 18 +++++++++++++-----
1 file changed, 13 insertions(+), 5 deletions(-)
diff --git a/src/mesa/state_tracker/st_manager.c b/src/mesa/state_tracker/st_manager.c
index f469ebd3a0..a91dc76d22 100644
--- a/src/mesa/state_tracker/st_manager.c
+++ b/src/mesa/state_tracker/st_manager.c
@@ -312,13 +312,21 @@ st_framebuffer_add_renderbuffer(struct st_framebuffer *stfb,
return FALSE;
if (idx != BUFFER_DEPTH) {
- _mesa_add_renderbuffer(&stfb->Base, idx, rb);
+ _mesa_add_renderbuffer_without_ref(&stfb->Base, idx, rb);
+ return TRUE;
}
- else {
- if (util_format_get_component_bits(format, UTIL_FORMAT_COLORSPACE_ZS, 0))
- _mesa_add_renderbuffer(&stfb->Base, BUFFER_DEPTH, rb);
- if (util_format_get_component_bits(format, UTIL_FORMAT_COLORSPACE_ZS, 1))
+
+ bool rb_ownership_taken = false;
+ if (util_format_get_component_bits(format, UTIL_FORMAT_COLORSPACE_ZS, 0)) {
+ _mesa_add_renderbuffer_without_ref(&stfb->Base, BUFFER_DEPTH, rb);
+ rb_ownership_taken = true;
+ }
+
+ if (util_format_get_component_bits(format, UTIL_FORMAT_COLORSPACE_ZS, 1)) {
+ if (rb_ownership_taken)
_mesa_add_renderbuffer(&stfb->Base, BUFFER_STENCIL, rb);
+ else
+ _mesa_add_renderbuffer_without_ref(&stfb->Base, BUFFER_STENCIL, rb);
}
return TRUE;
More information about the mesa-commit
mailing list