zink: lift some renderpass mechanics up a level in the api

(should be) no functional changes

Reviewed-by: Dave Airlie <airlied@redhat.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/16476>

src/gallium/drivers/zink/zink_context.c

@@ -1999,14 +1999,26 @@ zink_update_vk_sample_locations(struct zink_context *ctx)
 void
 zink_batch_rp(struct zink_context *ctx)
 {
+   if (ctx->batch.in_rp)
+      return;
+   unsigned clear_buffers;
+   clear_buffers = zink_begin_render_pass(ctx);
    if (!ctx->batch.in_rp)
-      zink_begin_render_pass(ctx);
+      return; //dead swapchain
+   if (ctx->render_condition.query)
+      zink_start_conditional_render(ctx);
+   zink_clear_framebuffer(ctx, clear_buffers);
 }
 
 void
 zink_batch_no_rp(struct zink_context *ctx)
 {
-   zink_end_render_pass(ctx);
+   if (!ctx->batch.in_rp)
+      return;
+   if (ctx->render_condition.query)
+      zink_stop_conditional_render(ctx);
+   if (ctx->gfx_pipeline_state.render_pass)
+      zink_end_render_pass(ctx);
    assert(!ctx->batch.in_rp);
 }
 

src/gallium/drivers/zink/zink_render_pass.c

@@ -591,7 +591,7 @@ begin_render_pass(struct zink_context *ctx)
    return clear_buffers;
 }
 
-void
+unsigned
 zink_begin_render_pass(struct zink_context *ctx)
 {
    setup_framebuffer(ctx);
@@ -630,19 +630,13 @@ zink_begin_render_pass(struct zink_context *ctx)
       ctx->gfx_pipeline_state.render_pass = rp;
    }
    assert(ctx->gfx_pipeline_state.render_pass);
-   unsigned clear_buffers = begin_render_pass(ctx);
-
-   if (ctx->render_condition.query)
-      zink_start_conditional_render(ctx);
-   zink_clear_framebuffer(ctx, clear_buffers);
+   return begin_render_pass(ctx);
 }
 
 void
 zink_end_render_pass(struct zink_context *ctx)
 {
    if (ctx->batch.in_rp) {
-      if (ctx->render_condition.query)
-         zink_stop_conditional_render(ctx);
       VKCTX(CmdEndRenderPass)(ctx->batch.state->cmdbuf);
       for (unsigned i = 0; i < ctx->fb_state.nr_cbufs; i++) {
          struct zink_ctx_surface *csurf = (struct zink_ctx_surface*)ctx->fb_state.cbufs[i];

src/gallium/drivers/zink/zink_render_pass.h

@@ -89,7 +89,7 @@ zink_destroy_render_pass(struct zink_screen *screen,
                          struct zink_render_pass *rp);
 
 
-void
+unsigned
 zink_begin_render_pass(struct zink_context *ctx);
 void
 zink_end_render_pass(struct zink_context *ctx);