i965: Drop unused alignment parameter from brw_bo_alloc().
authorKenneth Graunke <kenneth@whitecape.org>
Mon, 26 Mar 2018 22:58:05 +0000 (15:58 -0700)
committerKenneth Graunke <kenneth@whitecape.org>
Wed, 28 Mar 2018 01:41:44 +0000 (18:41 -0700)
brw_bo_alloc no longer uses this parameter, so there's no point.

Reviewed-by: Jason Ekstrand <jason@jlekstrand.net>
14 files changed:
src/mesa/drivers/dri/i965/brw_blorp.c
src/mesa/drivers/dri/i965/brw_bufmgr.c
src/mesa/drivers/dri/i965/brw_bufmgr.h
src/mesa/drivers/dri/i965/brw_performance_query.c
src/mesa/drivers/dri/i965/brw_pipe_control.c
src/mesa/drivers/dri/i965/brw_program.c
src/mesa/drivers/dri/i965/brw_program_cache.c
src/mesa/drivers/dri/i965/brw_queryobj.c
src/mesa/drivers/dri/i965/gen6_queryobj.c
src/mesa/drivers/dri/i965/gen6_sol.c
src/mesa/drivers/dri/i965/intel_batchbuffer.c
src/mesa/drivers/dri/i965/intel_buffer_objects.c
src/mesa/drivers/dri/i965/intel_screen.c
src/mesa/drivers/dri/i965/intel_upload.c

index a9c6dc4d50141d69d583662362915b230142174b..44064fc0cf30a2e23869be67dcb6da73db6d3c16 100644 (file)
@@ -827,7 +827,7 @@ blorp_get_client_bo(struct brw_context *brw,
        * data which we need to copy into a BO.
        */
       struct brw_bo *bo =
-         brw_bo_alloc(brw->bufmgr, "tmp_tex_subimage_src", size, 64);
+         brw_bo_alloc(brw->bufmgr, "tmp_tex_subimage_src", size);
       if (bo == NULL) {
          perf_debug("intel_texsubimage: temp bo creation failed: size = %u\n",
                     size);
index 295211e89f06bdaa7e02869e5566176ebfba875b..83c0286e0da9a9e6102ca40d75468e513cfd8d4b 100644 (file)
@@ -428,7 +428,7 @@ err:
 
 struct brw_bo *
 brw_bo_alloc(struct brw_bufmgr *bufmgr,
-             const char *name, uint64_t size, uint64_t alignment)
+             const char *name, uint64_t size)
 {
    return bo_alloc_internal(bufmgr, name, size, 0, I915_TILING_NONE, 0);
 }
index f33b6b5a3be410d1d0ae6d7d5b1c756da540c5c9..65e76b27d1bbec62f0aca0e9310cee90e51e22cc 100644 (file)
@@ -175,7 +175,7 @@ struct brw_bo {
  * using brw_bo_map() to be used by the CPU.
  */
 struct brw_bo *brw_bo_alloc(struct brw_bufmgr *bufmgr, const char *name,
-                            uint64_t size, uint64_t alignment);
+                            uint64_t size);
 
 /**
  * Allocate a tiled buffer object.
index 4144aa3c9b81eabc501cc45b16147f8df3cd9fef..ef5401a21b48037febb25d1b6abcd202101f2bd0 100644 (file)
@@ -1236,8 +1236,7 @@ brw_begin_perf_query(struct gl_context *ctx,
       }
 
       obj->oa.bo =
-         brw_bo_alloc(brw->bufmgr, "perf. query OA MI_RPC bo",
-                      MI_RPC_BO_SIZE, 64);
+         brw_bo_alloc(brw->bufmgr, "perf. query OA MI_RPC bo", MI_RPC_BO_SIZE);
 #ifdef DEBUG
       /* Pre-filling the BO helps debug whether writes landed. */
       void *map = brw_bo_map(brw, obj->oa.bo, MAP_WRITE);
@@ -1293,7 +1292,7 @@ brw_begin_perf_query(struct gl_context *ctx,
 
       obj->pipeline_stats.bo =
          brw_bo_alloc(brw->bufmgr, "perf. query pipeline stats bo",
-                            STATS_BO_SIZE, 64);
+                      STATS_BO_SIZE);
 
       /* Take starting snapshots. */
       snapshot_statistics_registers(brw, obj, 0);
index 71f79b18474695c30bd5090dec0b8afc83d7025f..02278be6d6290e1b85e2d916d32c1e21e84b13dc 100644 (file)
@@ -572,7 +572,7 @@ brw_init_pipe_control(struct brw_context *brw,
     * the gen6 workaround because it involves actually writing to
     * the buffer, and the kernel doesn't let us write to the batch.
     */
-   brw->workaround_bo = brw_bo_alloc(brw->bufmgr, "workaround", 4096, 4096);
+   brw->workaround_bo = brw_bo_alloc(brw->bufmgr, "workaround", 4096);
    if (brw->workaround_bo == NULL)
       return -ENOMEM;
 
index 23743639dc8cd4ee75a86cdbc6871e7a7198e4f4..4579a58b820987255853f0cda19f618e71836abb 100644 (file)
@@ -342,7 +342,7 @@ brw_get_scratch_bo(struct brw_context *brw,
    }
 
    if (!old_bo) {
-      *scratch_bo = brw_bo_alloc(brw->bufmgr, "scratch bo", size, 4096);
+      *scratch_bo = brw_bo_alloc(brw->bufmgr, "scratch bo", size);
    }
 }
 
@@ -437,7 +437,7 @@ brw_alloc_stage_scratch(struct brw_context *brw,
 
    stage_state->scratch_bo =
       brw_bo_alloc(brw->bufmgr, "shader scratch space",
-                   per_thread_size * thread_count, 4096);
+                   per_thread_size * thread_count);
 }
 
 void brwInitFragProgFuncs( struct dd_function_table *functions )
@@ -466,7 +466,7 @@ brw_init_shader_time(struct brw_context *brw)
    const int max_entries = 2048;
    brw->shader_time.bo =
       brw_bo_alloc(brw->bufmgr, "shader time",
-                   max_entries * BRW_SHADER_TIME_STRIDE * 3, 4096);
+                   max_entries * BRW_SHADER_TIME_STRIDE * 3);
    brw->shader_time.names = rzalloc_array(brw, const char *, max_entries);
    brw->shader_time.ids = rzalloc_array(brw, int, max_entries);
    brw->shader_time.types = rzalloc_array(brw, enum shader_time_shader_type,
index 14c356db321a4595e36fdfe1101cb14ca2aee805..ce11f1d070308f1cb4700b6186a145facbc8b05e 100644 (file)
@@ -221,7 +221,7 @@ brw_cache_new_bo(struct brw_cache *cache, uint32_t new_size)
    perf_debug("Copying to larger program cache: %u kB -> %u kB\n",
               (unsigned) cache->bo->size / 1024, new_size / 1024);
 
-   new_bo = brw_bo_alloc(brw->bufmgr, "program cache", new_size, 64);
+   new_bo = brw_bo_alloc(brw->bufmgr, "program cache", new_size);
    if (can_do_exec_capture(brw->screen))
       new_bo->kflags = EXEC_OBJECT_CAPTURE;
 
@@ -388,7 +388,7 @@ brw_init_caches(struct brw_context *brw)
    cache->items =
       calloc(cache->size, sizeof(struct brw_cache_item *));
 
-   cache->bo = brw_bo_alloc(brw->bufmgr, "program cache", 16384, 64);
+   cache->bo = brw_bo_alloc(brw->bufmgr, "program cache", 16384);
    if (can_do_exec_capture(brw->screen))
       cache->bo->kflags = EXEC_OBJECT_CAPTURE;
 
index f667f55b6bfe33a1305888be90b1a5617450c32d..da3df6b4bd5c9cc76c55a509b6fce9071a34dbf3 100644 (file)
@@ -287,7 +287,7 @@ brw_begin_query(struct gl_context *ctx, struct gl_query_object *q)
        * the system was doing other work, such as running other applications.
        */
       brw_bo_unreference(query->bo);
-      query->bo = brw_bo_alloc(brw->bufmgr, "timer query", 4096, 4096);
+      query->bo = brw_bo_alloc(brw->bufmgr, "timer query", 4096);
       brw_write_timestamp(brw, query->bo, 0);
       break;
 
@@ -449,7 +449,7 @@ ensure_bo_has_space(struct gl_context *ctx, struct brw_query_object *query)
          brw_queryobj_get_results(ctx, query);
       }
 
-      query->bo = brw_bo_alloc(brw->bufmgr, "query", 4096, 1);
+      query->bo = brw_bo_alloc(brw->bufmgr, "query", 4096);
       query->last_index = 0;
    }
 }
@@ -529,7 +529,7 @@ brw_query_counter(struct gl_context *ctx, struct gl_query_object *q)
    assert(q->Target == GL_TIMESTAMP);
 
    brw_bo_unreference(query->bo);
-   query->bo = brw_bo_alloc(brw->bufmgr, "timestamp query", 4096, 4096);
+   query->bo = brw_bo_alloc(brw->bufmgr, "timestamp query", 4096);
    brw_write_timestamp(brw, query->bo, 0);
 
    query->flushed = false;
index b43ca0ee478e7cfcbf37e669edab984913912b7a..75060d47210a630b9d962b788ab0930185540fa7 100644 (file)
@@ -329,7 +329,7 @@ gen6_begin_query(struct gl_context *ctx, struct gl_query_object *q)
 
    /* Since we're starting a new query, we need to throw away old results. */
    brw_bo_unreference(query->bo);
-   query->bo = brw_bo_alloc(brw->bufmgr, "query results", 4096, 4096);
+   query->bo = brw_bo_alloc(brw->bufmgr, "query results", 4096);
 
    /* For ARB_query_buffer_object: The result is not available */
    set_query_availability(brw, query, false);
index 355acd421893264dcd50a38b47ad48d3feb0b24f..0c830c7342c31e41ab14db9b16fee6e1b603a642 100644 (file)
@@ -195,9 +195,9 @@ brw_new_transform_feedback(struct gl_context *ctx, GLuint name)
    _mesa_init_transform_feedback_object(&brw_obj->base, name);
 
    brw_obj->offset_bo =
-      brw_bo_alloc(brw->bufmgr, "transform feedback offsets", 16, 64);
+      brw_bo_alloc(brw->bufmgr, "transform feedback offsets", 16);
    brw_obj->prim_count_bo =
-      brw_bo_alloc(brw->bufmgr, "xfb primitive counts", 16384, 64);
+      brw_bo_alloc(brw->bufmgr, "xfb primitive counts", 16384);
 
    return &brw_obj->base;
 }
index d824ff272ece9170f0364e742feb01fcefc4259c..9147ff829399b2732364b31794e664e77c5f5a7b 100644 (file)
@@ -191,7 +191,7 @@ recreate_growing_buffer(struct brw_context *brw,
    struct intel_batchbuffer *batch = &brw->batch;
    struct brw_bufmgr *bufmgr = screen->bufmgr;
 
-   grow->bo = brw_bo_alloc(bufmgr, name, size, 4096);
+   grow->bo = brw_bo_alloc(bufmgr, name, size);
    grow->bo->kflags = can_do_exec_capture(screen) ? EXEC_OBJECT_CAPTURE : 0;
    grow->partial_bo = NULL;
    grow->partial_bo_map = NULL;
@@ -352,7 +352,7 @@ grow_buffer(struct brw_context *brw,
       finish_growing_bos(grow);
    }
 
-   struct brw_bo *new_bo = brw_bo_alloc(bufmgr, bo->name, new_size, bo->align);
+   struct brw_bo *new_bo = brw_bo_alloc(bufmgr, bo->name, new_size);
 
    /* Copy existing data to the new larger buffer */
    grow->partial_bo_map = grow->map;
index 8268040911aa701551890baedc38c964db0444a7..9deb5e7434a3929258093e6817cceec6d11bab26 100644 (file)
@@ -96,7 +96,7 @@ alloc_buffer_object(struct brw_context *brw,
        */
       size += 64 * 32; /* max read length of 64 256-bit units */
    }
-   intel_obj->buffer = brw_bo_alloc(brw->bufmgr, "bufferobj", size, 64);
+   intel_obj->buffer = brw_bo_alloc(brw->bufmgr, "bufferobj", size);
 
    /* the buffer might be bound as a uniform buffer, need to update it
     */
@@ -290,7 +290,7 @@ brw_buffer_subdata(struct gl_context *ctx,
                     intel_obj->valid_data_start,
                     intel_obj->valid_data_end);
          struct brw_bo *temp_bo =
-            brw_bo_alloc(brw->bufmgr, "subdata temp", size, 64);
+            brw_bo_alloc(brw->bufmgr, "subdata temp", size);
 
          brw_bo_subdata(temp_bo, 0, size, data);
 
@@ -462,7 +462,7 @@ brw_map_buffer_range(struct gl_context *ctx,
       intel_obj->map_extra[index] = (uintptr_t) offset % alignment;
       intel_obj->range_map_bo[index] =
          brw_bo_alloc(brw->bufmgr, "BO blit temp",
-                      length + intel_obj->map_extra[index], alignment);
+                      length + intel_obj->map_extra[index]);
       void *map = brw_bo_map(brw, intel_obj->range_map_bo[index], access);
       obj->Mappings[index].Pointer = map + intel_obj->map_extra[index];
       return obj->Mappings[index].Pointer;
index dfb889221d84e23de17c1f96b413c2924a0ad796..dcb98dab03a1a0c624470d3689d7f20fac3859c8 100644 (file)
@@ -1906,11 +1906,11 @@ intel_detect_pipelined_register(struct intel_screen *screen,
    bool success = false;
 
    /* Create a zero'ed temporary buffer for reading our results */
-   results = brw_bo_alloc(screen->bufmgr, "registers", 4096, 0);
+   results = brw_bo_alloc(screen->bufmgr, "registers", 4096);
    if (results == NULL)
       goto err;
 
-   bo = brw_bo_alloc(screen->bufmgr, "batchbuffer", 4096, 0);
+   bo = brw_bo_alloc(screen->bufmgr, "batchbuffer", 4096);
    if (bo == NULL)
       goto err_results;
 
index e4297bf22b41c4615d69fe6307f3deb984d43ad7..f165a7b4322469872a0bb7219d0b69a8e0da888a 100644 (file)
@@ -86,7 +86,7 @@ brw_upload_space(struct brw_uploader *upload,
    assert((upload->bo == NULL) == (upload->map == NULL));
    if (!upload->bo) {
       upload->bo = brw_bo_alloc(upload->bufmgr, "streamed data",
-                                MAX2(upload->default_size, size), 4096);
+                                MAX2(upload->default_size, size));
       upload->map = brw_bo_map(NULL, upload->bo,
                                MAP_READ | MAP_WRITE |
                                MAP_PERSISTENT | MAP_ASYNC);