From e5148791f65cef066fb20fb44f90fb53f7308f80 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Marek=20Ol=C5=A1=C3=A1k?= Date: Sun, 19 Nov 2017 00:24:40 +0100 Subject: [PATCH] st/dri: remove dead code and incorrect comment around make_current MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit Core Mesa already handles flushing based on ContextReleaseBehavior, so the comment is wrong. Also, old_st is always NULL, because unbind_context always precedes make_current. Reviewed-by: Nicolai Hähnle --- src/gallium/state_trackers/dri/dri_context.c | 13 ------------- 1 file changed, 13 deletions(-) diff --git a/src/gallium/state_trackers/dri/dri_context.c b/src/gallium/state_trackers/dri/dri_context.c index f5400ab21e6..d123f9c5534 100644 --- a/src/gallium/state_trackers/dri/dri_context.c +++ b/src/gallium/state_trackers/dri/dri_context.c @@ -236,11 +236,6 @@ dri_unbind_context(__DRIcontext * cPriv) if (st->thread_finish) st->thread_finish(st); - /* For conformance, unbind is supposed to flush the context. - * However, if we do it here we might end up flushing a partially - * destroyed context. Instead, we flush in dri_make_current and - * in dri_destroy_context which should cover all the cases. - */ stapi->make_current(stapi, NULL, NULL, NULL); } } @@ -257,14 +252,6 @@ dri_make_current(__DRIcontext * cPriv, struct dri_context *ctx = dri_context(cPriv); struct dri_drawable *draw = dri_drawable(driDrawPriv); struct dri_drawable *read = dri_drawable(driReadPriv); - struct st_context_iface *old_st = ctx->stapi->get_current(ctx->stapi); - - if (old_st && old_st->thread_finish) - old_st->thread_finish(old_st); - - /* Flush the old context here so we don't have to flush on unbind() */ - if (old_st && old_st != ctx->st) - old_st->flush(old_st, ST_FLUSH_FRONT, NULL); ++ctx->bind_count; -- 2.30.2