From: Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer@amd.com>
Date: Tue, 15 Jun 2021 13:52:14 +0000 (+0200)
Subject: dlist: remove InstSize
X-Git-Tag: upstream/21.2.3~682
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=0843a7ebabb2305fc229e2b7cd8b214810f6d398;p=platform%2Fupstream%2Fmesa.git

dlist: remove InstSize

Instead store the instruction size alongside the opcode.

Reviewed-by: Marek Olšák <marek.olsak@amd.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/11493>
---

diff --git a/src/mesa/main/dlist.c b/src/mesa/main/dlist.c
index 94c7c76..d86fcbb 100644
--- a/src/mesa/main/dlist.c
+++ b/src/mesa/main/dlist.c
@@ -150,9 +150,6 @@
 
 /**
  * Display list opcodes.
- *
- * The fact that these identifiers are assigned consecutive
- * integer values starting at 0 is very important, see InstSize array usage)
  */
 typedef enum
 {
@@ -657,7 +654,15 @@ typedef enum
  */
 union gl_dlist_node
 {
-   OpCode opcode;
+   struct {
+#if !DETECT_OS_WINDOWS
+      OpCode opcode:16;
+#else
+      /* sizeof(Node) is 8 with MSVC/mingw, so use an explicit 16 bits type. */
+      uint16_t opcode;
+#endif
+      uint16_t InstSize;
+   };
    GLboolean b;
    GLbitfield bf;
    GLubyte ub;
@@ -778,14 +783,6 @@ union int64_pair
 #define BLOCK_SIZE 256
 
 
-
-/**
- * Number of nodes of storage needed for each instruction.
- * Sizes for dynamically allocated opcodes are stored in the context struct.
- */
-static GLuint InstSize[OPCODE_END_OF_LIST + 1];
-
-
 void mesa_print_display_list(GLuint list);
 
 
@@ -845,7 +842,7 @@ is_bitmap_list(const struct gl_display_list *dlist)
 {
    const Node *n = dlist->Head;
    if (n[0].opcode == OPCODE_BITMAP) {
-      n += InstSize[OPCODE_BITMAP];
+      n += n[0].InstSize;
       if (n[0].opcode == OPCODE_END_OF_LIST)
          return true;
    }
@@ -1113,8 +1110,6 @@ make_list(GLuint name, GLuint count)
    dlist->Name = name;
    dlist->Head = malloc(sizeof(Node) * count);
    dlist->Head[0].opcode = OPCODE_END_OF_LIST;
-   /* All InstSize[] entries must be non-zero */
-   InstSize[OPCODE_END_OF_LIST] = 1;
    return dlist;
 }
 
@@ -1374,8 +1369,8 @@ _mesa_delete_list(struct gl_context *ctx, struct gl_display_list *dlist)
             ;
       }
 
-      assert(InstSize[opcode] > 0);
-      n += InstSize[opcode];
+      assert(n[0].InstSize > 0);
+      n += n[0].InstSize;
    }
 }
 
@@ -1534,15 +1529,6 @@ dlist_alloc(struct gl_context *ctx, OpCode opcode, GLuint bytes, bool align8)
 
    assert(bytes <= BLOCK_SIZE * sizeof(Node));
 
-   if (InstSize[opcode] == 0) {
-      /* save instruction size now */
-      InstSize[opcode] = numNodes;
-   }
-   else {
-      /* make sure instruction size agrees */
-      assert(numNodes == InstSize[opcode]);
-   }
-
    if (sizeof(void *) > sizeof(Node) && align8
        && ctx->ListState.CurrentPos % 2 == 0) {
       /* The opcode would get placed at node[0] and the payload would start
@@ -1588,6 +1574,7 @@ dlist_alloc(struct gl_context *ctx, OpCode opcode, GLuint bytes, bool align8)
    if (nopNode) {
       assert(ctx->ListState.CurrentPos % 2 == 0); /* even value */
       n[0].opcode = OPCODE_NOP;
+      n[0].InstSize = 1;
       n++;
       /* The "real" opcode will now be at an odd location and the payload
        * will be at an even location.
@@ -1596,6 +1583,7 @@ dlist_alloc(struct gl_context *ctx, OpCode opcode, GLuint bytes, bool align8)
    ctx->ListState.CurrentPos += nopNode + numNodes;
 
    n[0].opcode = opcode;
+   n[0].InstSize = numNodes;
 
    return n;
 }
@@ -13436,8 +13424,8 @@ execute_list(struct gl_context *ctx, GLuint list)
       }
 
       /* increment n to point to next compiled command */
-      assert(InstSize[opcode] > 0);
-      n += InstSize[opcode];
+      assert(n[0].InstSize > 0);
+      n += n[0].InstSize;
    }
 }
 
@@ -14850,7 +14838,7 @@ print_list(struct gl_context *ctx, GLuint list, const char *fname)
                    opcode, (void *) n);
             } else {
                fprintf(f, "command %d, %u operands\n", opcode,
-                            InstSize[opcode]);
+                            n[0].InstSize);
                break;
             }
             FALLTHROUGH;
@@ -14863,8 +14851,8 @@ print_list(struct gl_context *ctx, GLuint list, const char *fname)
       }
 
       /* increment n to point to next compiled command */
-      assert(InstSize[opcode] > 0);
-      n += InstSize[opcode];
+      assert(n[0].InstSize > 0);
+      n += n[0].InstSize;
    }
 }
 
@@ -14941,8 +14929,8 @@ _mesa_glthread_execute_list(struct gl_context *ctx, GLuint list)
       }
 
       /* increment n to point to next compiled command */
-      assert(InstSize[opcode] > 0);
-      n += InstSize[opcode];
+      assert(n[0].InstSize > 0);
+      n += n[0].InstSize;
    }
 }
 
@@ -14979,15 +14967,8 @@ _mesa_install_dlist_vtxfmt(struct _glapi_table *disp,
 void
 _mesa_init_display_list(struct gl_context *ctx)
 {
-   static GLboolean tableInitialized = GL_FALSE;
    GLvertexformat *vfmt = &ctx->ListState.ListVtxfmt;
 
-   /* zero-out the instruction size table, just once */
-   if (!tableInitialized) {
-      memset(InstSize, 0, sizeof(InstSize));
-      tableInitialized = GL_TRUE;
-   }
-
    /* Display list */
    ctx->ListState.CallDepth = 0;
    ctx->ExecuteFlag = GL_TRUE;
@@ -14998,9 +14979,6 @@ _mesa_init_display_list(struct gl_context *ctx)
    /* Display List group */
    ctx->List.ListBase = 0;
 
-   InstSize[OPCODE_NOP] = 1;
-   InstSize[OPCODE_VERTEX_LIST] = 1 + align(sizeof(struct vbo_save_vertex_list), sizeof(Node)) / sizeof(Node);
-
 #define NAME_AE(x) _ae_##x
 #define NAME_CALLLIST(x) save_##x
 #define NAME(x) save_##x