From: Marek Olšák Date: Sat, 18 Nov 2017 23:24:40 +0000 (+0100) Subject: st/dri: remove dead code and incorrect comment around make_current X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=e5148791f65cef066fb20fb44f90fb53f7308f80;p=mesa.git st/dri: remove dead code and incorrect comment around make_current 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 --- 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;