diff options
author | Boris Brezillon <[email protected]> | 2019-10-01 16:24:27 +0200 |
---|---|---|
committer | Boris Brezillon <[email protected]> | 2019-10-07 11:38:26 +0200 |
commit | 19546108d3dd5541a189e36df4ea83b3f519e48f (patch) | |
tree | 5d396690047c0178d91cc959b6d81122cf2a22c4 /src/gallium | |
parent | 555c0de8c6f5b8be8c28ee92a816c7bb711afde5 (diff) |
Revert "st/dri2: Implement DRI2bufferDamageExtension"
This reverts commit 492ffbed63a2a62759224b1c7d45aa7923d8f542.
BACK_LEFT attachment can be outdated when the user calls
KHR_partial_update(), leading to a damage region update on the
wrong pipe_resource object.
Let's not expose the ->set_damage_region() method until the core is
fixed to handle that properly.
Cc: [email protected]
Signed-off-by: Boris Brezillon <[email protected]>
Acked-by: Daniel Stone <[email protected]>
Diffstat (limited to 'src/gallium')
-rw-r--r-- | src/gallium/drivers/panfrost/pan_screen.c | 1 | ||||
-rw-r--r-- | src/gallium/include/pipe/p_screen.h | 17 | ||||
-rw-r--r-- | src/gallium/state_trackers/dri/dri2.c | 35 |
3 files changed, 0 insertions, 53 deletions
diff --git a/src/gallium/drivers/panfrost/pan_screen.c b/src/gallium/drivers/panfrost/pan_screen.c index 55c66e0c9a7..acc5b476316 100644 --- a/src/gallium/drivers/panfrost/pan_screen.c +++ b/src/gallium/drivers/panfrost/pan_screen.c @@ -753,7 +753,6 @@ panfrost_create_screen(int fd, struct renderonly *ro) screen->base.get_compiler_options = panfrost_screen_get_compiler_options; screen->base.fence_reference = panfrost_fence_reference; screen->base.fence_finish = panfrost_fence_finish; - screen->base.set_damage_region = panfrost_resource_set_damage_region; panfrost_resource_screen_init(screen); diff --git a/src/gallium/include/pipe/p_screen.h b/src/gallium/include/pipe/p_screen.h index 9a1fc37280e..a275a7c7900 100644 --- a/src/gallium/include/pipe/p_screen.h +++ b/src/gallium/include/pipe/p_screen.h @@ -483,23 +483,6 @@ struct pipe_screen { bool (*is_parallel_shader_compilation_finished)(struct pipe_screen *screen, void *shader, unsigned shader_type); - - /** - * Set the damage region (called when KHR_partial_update() is invoked). - * This function is passed an array of rectangles encoding the damage area. - * rects are using the bottom-left origin convention. - * nrects = 0 means 'reset the damage region'. What 'reset' implies is HW - * specific. For tile-based renderers, the damage extent is typically set - * to cover the whole resource with no damage rect (or a 0-size damage - * rect). This way, the existing resource content is reloaded into the - * local tile buffer for every tile thus making partial tile update - * possible. For HW operating in immediate mode, this reset operation is - * likely to be a NOOP. - */ - void (*set_damage_region)(struct pipe_screen *screen, - struct pipe_resource *resource, - unsigned int nrects, - const struct pipe_box *rects); }; diff --git a/src/gallium/state_trackers/dri/dri2.c b/src/gallium/state_trackers/dri/dri2.c index 574ddaea5c7..d42727c6cd6 100644 --- a/src/gallium/state_trackers/dri/dri2.c +++ b/src/gallium/state_trackers/dri/dri2.c @@ -1873,36 +1873,6 @@ static const __DRI2interopExtension dri2InteropExtension = { }; /** - * \brief the DRI2bufferDamageExtension set_damage_region method - */ -static void -dri2_set_damage_region(__DRIdrawable *dPriv, unsigned int nrects, int *rects) -{ - struct dri_drawable *drawable = dri_drawable(dPriv); - struct pipe_resource *resource = drawable->textures[ST_ATTACHMENT_BACK_LEFT]; - struct pipe_screen *screen = resource->screen; - struct pipe_box *boxes = NULL; - - if (nrects) { - boxes = CALLOC(nrects, sizeof(*boxes)); - assert(boxes); - - for (unsigned int i = 0; i < nrects; i++) { - int *rect = &rects[i * 4]; - - u_box_2d(rect[0], rect[1], rect[2], rect[3], &boxes[i]); - } - } - - screen->set_damage_region(screen, resource, nrects, boxes); - FREE(boxes); -} - -static __DRI2bufferDamageExtension dri2BufferDamageExtension = { - .base = { __DRI2_BUFFER_DAMAGE, 1 }, -}; - -/** * \brief the DRI2ConfigQueryExtension configQueryb method */ static int @@ -2003,7 +1973,6 @@ static const __DRIextension *dri_screen_extensions[] = { &dri2GalliumConfigQueryExtension.base, &dri2ThrottleExtension.base, &dri2FenceExtension.base, - &dri2BufferDamageExtension.base, &dri2InteropExtension.base, &dri2NoErrorExtension.base, &driBlobExtension.base, @@ -2019,7 +1988,6 @@ static const __DRIextension *dri_robust_screen_extensions[] = { &dri2ThrottleExtension.base, &dri2FenceExtension.base, &dri2InteropExtension.base, - &dri2BufferDamageExtension.base, &dri2Robustness.base, &dri2NoErrorExtension.base, &driBlobExtension.base, @@ -2082,9 +2050,6 @@ dri2_init_screen(__DRIscreen * sPriv) } } - if (pscreen->set_damage_region) - dri2BufferDamageExtension.set_damage_region = dri2_set_damage_region; - if (pscreen->get_param(pscreen, PIPE_CAP_DEVICE_RESET_STATUS_QUERY)) { sPriv->extensions = dri_robust_screen_extensions; screen->has_reset_status_query = true; |