draw: corrections to allow for different cliptest cases
[mesa.git] / src / gallium / winsys / r600 / drm / radeon_ctx.c
index bd050c4cf90d96e9326584c4d46bcd9862e491c3..7ccb52459055729b564f3916619e3a5a3cd3bb3d 100644 (file)
 #include "radeon_drm.h"
 #include "bof.h"
 
-int radeon_ctx_set_bo_new(struct radeon_ctx *ctx, struct radeon_bo *bo)
+static int radeon_ctx_set_bo_new(struct radeon_ctx *ctx, struct radeon_ws_bo *bo)
 {
-       void *ptr;
-
-       ptr = realloc(ctx->bo, sizeof(struct radeon_bo) * (ctx->nbo + 1));
-       if (ptr == NULL) {
-               return -ENOMEM;
-       }
-       ctx->bo = ptr;
-       ctx->bo[ctx->nbo] = bo;
+       if (ctx->nbo >= RADEON_CTX_MAX_PM4)
+               return -EBUSY;
+       /* take a reference to the kernel bo */
+       radeon_bo_reference(ctx->radeon, &ctx->bo[ctx->nbo], radeon_bo_pb_get_bo(bo->pb));
        ctx->nbo++;
        return 0;
 }
 
-struct radeon_bo *radeon_ctx_get_bo(struct radeon_ctx *ctx, unsigned reloc)
-{
-       struct radeon_cs_reloc *greloc;
-       unsigned i;
-
-       greloc = (void *)(((u8 *)ctx->reloc) + reloc * 4);
-       for (i = 0; i < ctx->nbo; i++) {
-               if (ctx->bo[i]->handle == greloc->handle) {
-                       return radeon_bo_incref(ctx->radeon, ctx->bo[i]);
-               }
-       }
-       fprintf(stderr, "%s no bo for reloc[%d 0x%08X] %d\n", __func__, reloc, greloc->handle, ctx->nbo);
-       return NULL;
-}
-
-void radeon_ctx_get_placement(struct radeon_ctx *ctx, unsigned reloc, u32 *placement)
+static void radeon_ctx_get_placement(struct radeon_ctx *ctx, unsigned reloc, u32 *placement)
 {
        struct radeon_cs_reloc *greloc;
        unsigned i;
@@ -76,66 +57,74 @@ void radeon_ctx_get_placement(struct radeon_ctx *ctx, unsigned reloc, u32 *place
        }
 }
 
-struct radeon_ctx *radeon_ctx(struct radeon *radeon)
+void radeon_ctx_clear(struct radeon_ctx *ctx)
 {
-       struct radeon_ctx *ctx;
+       for (int i = 0; i < ctx->nbo; i++) {
+               radeon_bo_reference(ctx->radeon, &ctx->bo[i], NULL);
+       }
+       ctx->ndwords = RADEON_CTX_MAX_PM4;
+       ctx->cdwords = 0;
+       ctx->nreloc = 0;
+       ctx->nbo = 0;
+}
 
+struct radeon_ctx *radeon_ctx_init(struct radeon *radeon)
+{
+       struct radeon_ctx *ctx;
        if (radeon == NULL)
                return NULL;
-       ctx = calloc(1, sizeof(*ctx));
-       if (ctx == NULL)
-               return NULL;
+       ctx = calloc(1, sizeof(struct radeon_ctx));
        ctx->radeon = radeon_incref(radeon);
+       radeon_ctx_clear(ctx);
+       ctx->pm4 = malloc(RADEON_CTX_MAX_PM4 * 4);
+       if (ctx->pm4 == NULL) {
+               radeon_ctx_fini(ctx);
+               return NULL;
+       }
+       ctx->reloc = malloc(sizeof(struct radeon_cs_reloc) * RADEON_CTX_MAX_PM4);
+       if (ctx->reloc == NULL) {
+               radeon_ctx_fini(ctx);
+               return NULL;
+       }
+       ctx->bo = calloc(sizeof(void *), RADEON_CTX_MAX_PM4);
+       if (ctx->bo == NULL) {
+               radeon_ctx_fini(ctx);
+               return NULL;
+       }
        return ctx;
 }
 
-struct radeon_ctx *radeon_ctx_incref(struct radeon_ctx *ctx)
-{
-       ctx->refcount++;
-       return ctx;
-}
-
-struct radeon_ctx *radeon_ctx_decref(struct radeon_ctx *ctx)
+void radeon_ctx_fini(struct radeon_ctx *ctx)
 {
        unsigned i;
 
        if (ctx == NULL)
-               return NULL;
-       if (--ctx->refcount > 0) {
-               return NULL;
-       }
+               return;
 
-       for (i = 0; i < ctx->ndraw; i++) {
-               ctx->draw[i] = radeon_draw_decref(ctx->draw[i]);
-       }
        for (i = 0; i < ctx->nbo; i++) {
-               ctx->bo[i] = radeon_bo_decref(ctx->radeon, ctx->bo[i]);
+               radeon_bo_reference(ctx->radeon, &ctx->bo[i], NULL);
        }
        ctx->radeon = radeon_decref(ctx->radeon);
-       free(ctx->state);
-       free(ctx->draw);
        free(ctx->bo);
        free(ctx->pm4);
        free(ctx->reloc);
-       memset(ctx, 0, sizeof(*ctx));
        free(ctx);
-       return NULL;
 }
 
 static int radeon_ctx_state_bo(struct radeon_ctx *ctx, struct radeon_state *state)
 {
        unsigned i, j;
        int r;
-
+       struct radeon_bo *state_bo;
        if (state == NULL)
                return 0;
        for (i = 0; i < state->nbo; i++) {
                for (j = 0; j < ctx->nbo; j++) {
-                       if (state->bo[i] == ctx->bo[j])
+                       state_bo = radeon_bo_pb_get_bo(state->bo[i]->pb);
+                       if (state_bo == ctx->bo[j])
                                break;
                }
                if (j == ctx->nbo) {
-                       radeon_bo_incref(ctx->radeon, state->bo[i]);
                        r = radeon_ctx_set_bo_new(ctx, state->bo[i]);
                        if (r)
                                return r;
@@ -152,17 +141,19 @@ int radeon_ctx_submit(struct radeon_ctx *ctx)
        uint64_t chunk_array[2];
        int r = 0;
 
-       if (!ctx->cpm4)
+       if (!ctx->cdwords)
                return 0;
+
+       radeon_bo_pbmgr_flush_maps(ctx->radeon->kman);
 #if 0
-       for (r = 0; r < ctx->cpm4; r++) {
+       for (r = 0; r < ctx->cdwords; r++) {
                fprintf(stderr, "0x%08X\n", ctx->pm4[r]);
        }
 #endif
        drmib.num_chunks = 2;
        drmib.chunks = (uint64_t)(uintptr_t)chunk_array;
        chunks[0].chunk_id = RADEON_CHUNK_ID_IB;
-       chunks[0].length_dw = ctx->cpm4;
+       chunks[0].length_dw = ctx->cdwords;
        chunks[0].chunk_data = (uint64_t)(uintptr_t)ctx->pm4;
        chunks[1].chunk_id = RADEON_CHUNK_ID_RELOCS;
        chunks[1].length_dw = ctx->nreloc * sizeof(struct radeon_cs_reloc) / 4;
@@ -176,26 +167,25 @@ int radeon_ctx_submit(struct radeon_ctx *ctx)
        return r;
 }
 
-static int radeon_ctx_reloc(struct radeon_ctx *ctx, struct radeon_bo *bo,
+static int radeon_ctx_reloc(struct radeon_ctx *ctx, struct radeon_ws_bo *bo,
                        unsigned id, unsigned *placement)
 {
        unsigned i;
-       struct radeon_cs_reloc *ptr;
+       unsigned bo_handle = radeon_ws_bo_get_handle(bo);
 
        for (i = 0; i < ctx->nreloc; i++) {
-               if (ctx->reloc[i].handle == bo->handle) {
+               if (ctx->reloc[i].handle == bo_handle) {
                        ctx->pm4[id] = i * sizeof(struct radeon_cs_reloc) / 4;
                        return 0;
                }
        }
-       ptr = realloc(ctx->reloc, sizeof(struct radeon_cs_reloc) * (ctx->nreloc + 1));
-       if (ptr == NULL)
-               return -ENOMEM;
-       ctx->reloc = ptr;
-       ptr[ctx->nreloc].handle = bo->handle;
-       ptr[ctx->nreloc].read_domain = placement[0] | placement [1];
-       ptr[ctx->nreloc].write_domain = placement[0] | placement [1];
-       ptr[ctx->nreloc].flags = 0;
+       if (ctx->nreloc >= RADEON_CTX_MAX_PM4) {
+               return -EBUSY;
+       }
+       ctx->reloc[ctx->nreloc].handle = bo_handle;
+       ctx->reloc[ctx->nreloc].read_domain = placement[0] | placement [1];
+       ctx->reloc[ctx->nreloc].write_domain = placement[0] | placement [1];
+       ctx->reloc[ctx->nreloc].flags = 0;
        ctx->pm4[id] = ctx->nreloc * sizeof(struct radeon_cs_reloc) / 4;
        ctx->nreloc++;
        return 0;
@@ -208,132 +198,79 @@ static int radeon_ctx_state_schedule(struct radeon_ctx *ctx, struct radeon_state
 
        if (state == NULL)
                return 0;
-       memcpy(&ctx->pm4[ctx->id], state->pm4, state->cpm4 * 4);
+       if (state->cpm4 > ctx->ndwords) {
+               return -EBUSY;
+       }
+       memcpy(&ctx->pm4[ctx->cdwords], state->pm4, state->cpm4 * 4);
        for (i = 0; i < state->nreloc; i++) {
                rid = state->reloc_pm4_id[i];
                bid = state->reloc_bo_id[i];
-               cid = ctx->id + rid;
+               cid = ctx->cdwords + rid;
                r = radeon_ctx_reloc(ctx, state->bo[bid], cid,
                                        &state->placement[bid * 2]);
                if (r) {
-                       fprintf(stderr, "%s state %d failed to reloc\n", __func__, state->type);
+                       fprintf(stderr, "%s state %d failed to reloc\n", __func__, state->stype->stype);
                        return r;
                }
        }
-       ctx->id += state->cpm4;
+       ctx->cdwords += state->cpm4;
+       ctx->ndwords -= state->cpm4;
        return 0;
 }
 
 int radeon_ctx_set_query_state(struct radeon_ctx *ctx, struct radeon_state *state)
 {
-       void *tmp;
        int r = 0;
 
        /* !!! ONLY ACCEPT QUERY STATE HERE !!! */
-       if (state->type != R600_QUERY_BEGIN_TYPE && state->type != R600_QUERY_END_TYPE) {
-               return -EINVAL;
-       }
        r = radeon_state_pm4(state);
        if (r)
                return r;
-       if ((ctx->draw_cpm4 + state->cpm4) > RADEON_CTX_MAX_PM4) {
-               /* need to flush */
-               return -EBUSY;
-       }
-       if (state->cpm4 >= RADEON_CTX_MAX_PM4) {
-               fprintf(stderr, "%s single state too big %d, max %d\n",
-                       __func__, state->cpm4, RADEON_CTX_MAX_PM4);
-               return -EINVAL;
-       }
-       tmp = realloc(ctx->state, (ctx->nstate + 1) * sizeof(void*));
-       if (tmp == NULL)
-               return -ENOMEM;
-       ctx->state = tmp;
-       ctx->state[ctx->nstate++] = radeon_state_incref(state);
        /* BEGIN/END query are balanced in the same cs so account for END
         * END query when scheduling BEGIN query
         */
-       if (state->type == R600_QUERY_BEGIN_TYPE) {
-               ctx->draw_cpm4 += state->cpm4 * 2;
+       switch (state->stype->stype) {
+       case R600_STATE_QUERY_BEGIN:
+               /* is there enough place for begin & end */
+               if ((state->cpm4 * 2) > ctx->ndwords)
+                       return -EBUSY;
+               ctx->ndwords -= state->cpm4;
+               break;
+       case R600_STATE_QUERY_END:
+               ctx->ndwords += state->cpm4;
+               break;
+       default:
+               return -EINVAL;
        }
-       return 0;
+       return radeon_ctx_state_schedule(ctx, state);
 }
 
-int radeon_ctx_set_draw_new(struct radeon_ctx *ctx, struct radeon_draw *draw)
+int radeon_ctx_set_draw(struct radeon_ctx *ctx, struct radeon_draw *draw)
 {
-       struct radeon_draw *pdraw = NULL;
-       struct radeon_draw **ndraw;
-       struct radeon_state *nstate, *ostate;
-       unsigned cpm4, i, cstate;
-       void *tmp;
+       unsigned previous_cdwords;
        int r = 0;
+       int i;
 
-       ndraw = realloc(ctx->draw, sizeof(void*) * (ctx->ndraw + 1));
-       if (ndraw == NULL)
-               return -ENOMEM;
-       ctx->draw = ndraw;
-       for (i = 0; i < draw->nstate; i++) {
+       for (i = 0; i < ctx->radeon->max_states; i++) {
                r = radeon_ctx_state_bo(ctx, draw->state[i]);
                if (r)
                        return r;
        }
-       r = radeon_draw_check(draw);
-       if (r)
-               return r;
-       if (draw->cpm4 >= RADEON_CTX_MAX_PM4) {
-               fprintf(stderr, "%s single draw too big %d, max %d\n",
-                       __func__, draw->cpm4, RADEON_CTX_MAX_PM4);
-               return -EINVAL;
-       }
-       tmp = realloc(ctx->state, (ctx->nstate + draw->nstate) * sizeof(void*));
-       if (tmp == NULL)
-               return -ENOMEM;
-       ctx->state = tmp;
-       pdraw = ctx->cdraw;
-       for (i = 0, cpm4 = 0, cstate = ctx->nstate; i < draw->nstate - 1; i++) {
-               nstate = draw->state[i];
-               if (nstate) {
-                       if (pdraw && pdraw->state[i]) {
-                               ostate = pdraw->state[i];
-                               if (ostate->pm4_crc != nstate->pm4_crc) {
-                                       ctx->state[cstate++] = nstate;
-                                       cpm4 += nstate->cpm4;
-                               }
-                       } else {
-                               ctx->state[cstate++] = nstate;
-                               cpm4 += nstate->cpm4;
+       previous_cdwords = ctx->cdwords;
+       for (i = 0; i < ctx->radeon->max_states; i++) {
+               if (draw->state[i]) {
+                       r = radeon_ctx_state_schedule(ctx, draw->state[i]);
+                       if (r) {
+                               ctx->cdwords = previous_cdwords;
+                               return r;
                        }
                }
        }
-       /* The last state is the draw state always add it */
-       if (draw->state[i] == NULL) {
-               fprintf(stderr, "%s no draw command\n", __func__);
-               return -EINVAL;
-       }
-       ctx->state[cstate++] = draw->state[i];
-       cpm4 += draw->state[i]->cpm4;
-       if ((ctx->draw_cpm4 + cpm4) > RADEON_CTX_MAX_PM4) {
-               /* need to flush */
-               return -EBUSY;
-       }
-       ctx->draw_cpm4 += cpm4;
-       ctx->nstate = cstate;
-       ctx->draw[ctx->ndraw++] = draw;
-       ctx->cdraw = draw;
-       return 0;
-}
-
-int radeon_ctx_set_draw(struct radeon_ctx *ctx, struct radeon_draw *draw)
-{
-       int r;
 
-       radeon_draw_incref(draw);
-       r = radeon_ctx_set_draw_new(ctx, draw);
-       if (r)
-               radeon_draw_decref(draw);
-       return r;
+       return 0;
 }
 
+#if 0
 int radeon_ctx_pm4(struct radeon_ctx *ctx)
 {
        unsigned i;
@@ -345,9 +282,6 @@ int radeon_ctx_pm4(struct radeon_ctx *ctx)
        if (ctx->pm4 == NULL)
                return -EINVAL;
        for (i = 0, ctx->id = 0; i < ctx->nstate; i++) {
-               r = radeon_ctx_state_schedule(ctx, ctx->state[i]);
-               if (r)
-                       return r;
        }
        if (ctx->id != ctx->draw_cpm4) {
                fprintf(stderr, "%s miss predicted pm4 size %d for %d\n",
@@ -357,12 +291,13 @@ int radeon_ctx_pm4(struct radeon_ctx *ctx)
        ctx->cpm4 = ctx->draw_cpm4;
        return 0;
 }
+#endif
 
 void radeon_ctx_dump_bof(struct radeon_ctx *ctx, const char *file)
 {
        bof_t *bcs, *blob, *array, *bo, *size, *handle, *device_id, *root;
        unsigned i;
-
+       unsigned bo_size;
        root = device_id = bcs = blob = array = bo = size = handle = NULL;
        root = bof_object();
        if (root == NULL)
@@ -375,7 +310,6 @@ void radeon_ctx_dump_bof(struct radeon_ctx *ctx, const char *file)
        bof_decref(device_id);
        device_id = NULL;
        /* dump relocs */
-printf("%d relocs\n", ctx->nreloc);
        blob = bof_blob(ctx->nreloc * 16, ctx->reloc);
        if (blob == NULL)
                goto out_err;
@@ -384,8 +318,7 @@ printf("%d relocs\n", ctx->nreloc);
        bof_decref(blob);
        blob = NULL;
        /* dump cs */
-printf("%d pm4\n", ctx->cpm4);
-       blob = bof_blob(ctx->cpm4 * 4, ctx->pm4);
+       blob = bof_blob(ctx->cdwords * 4, ctx->pm4);
        if (blob == NULL)
                goto out_err;
        if (bof_object_set(root, "pm4", blob))
@@ -400,7 +333,8 @@ printf("%d pm4\n", ctx->cpm4);
                bo = bof_object();
                if (bo == NULL)
                        goto out_err;
-               size = bof_int32(ctx->bo[i]->size);
+               bo_size = ctx->bo[i]->size;
+               size = bof_int32(bo_size);
                if (size == NULL)
                        goto out_err;
                if (bof_object_set(bo, "size", size))
@@ -415,7 +349,7 @@ printf("%d pm4\n", ctx->cpm4);
                bof_decref(handle);
                handle = NULL;
                radeon_bo_map(ctx->radeon, ctx->bo[i]);
-               blob = bof_blob(ctx->bo[i]->size, ctx->bo[i]->data);
+               blob = bof_blob(bo_size, ctx->bo[i]->data);
                radeon_bo_unmap(ctx->radeon, ctx->bo[i]);
                if (blob == NULL)
                        goto out_err;
@@ -431,7 +365,6 @@ printf("%d pm4\n", ctx->cpm4);
        if (bof_object_set(root, "bo", array))
                goto out_err;
        bof_dump_file(root, file);
-printf("done dump\n");
 out_err:
        bof_decref(blob);
        bof_decref(array);