amdgpu: remove amdgpu_ib

Not useful if we're gonna use BO handles directly.

Reviewed-by: Jammy Zhou <Jammy.Zhou@amd.com>
Reviewed-by: Christian König <christian.koenig@amd.com>

3 files changed, 22 insertions, 70 deletions

diff --git a/amdgpu/amdgpu.h b/amdgpu/amdgpu.h
index bef7bf58..cb30c0ae 100644
--- a/amdgpu/amdgpu.h
+++ b/amdgpu/amdgpu.h
@@ -125,13 +125,6 @@ typedef struct amdgpu_bo *amdgpu_bo_handle;
  */
 typedef struct amdgpu_bo_list *amdgpu_bo_list_handle;
 
-/**
- * Define handle to be used when dealing with command
- * buffers (a.k.a. ibs)
- *
- */
-typedef struct amdgpu_ib *amdgpu_ib_handle;
-
 
 /*--------------------------------------------------------------------------*/
 /* -------------------------- Structures ---------------------------------- */
@@ -305,7 +298,7 @@ struct amdgpu_gds_alloc_info {
 */
 struct amdgpu_cs_ib_alloc_result {
         /** IB allocation handle */
-        amdgpu_ib_handle handle;
+        amdgpu_bo_handle handle;
 
         /** Assigned GPU VM MC Address of command buffer */
         uint64_t        mc_address;
@@ -325,7 +318,7 @@ struct amdgpu_cs_ib_info {
         uint64_t      flags;
 
         /** Handle of command buffer */
-        amdgpu_ib_handle ib_handle;
+        amdgpu_bo_handle bo_handle;
 
         /**
          * Size of Command Buffer to be submitted.
@@ -964,7 +957,7 @@ int amdgpu_cs_alloc_ib(amdgpu_context_handle context,
  * \sa amdgpu_cs_alloc_ib()
  *
 */
-int amdgpu_cs_free_ib(amdgpu_ib_handle handle);
+int amdgpu_cs_free_ib(amdgpu_bo_handle handle);
 
 /**
  * Send request to submit command buffers to hardware.
diff --git a/amdgpu/amdgpu_cs.c b/amdgpu/amdgpu_cs.c
index 326e3d32..1429f26b 100644
--- a/amdgpu/amdgpu_cs.c
+++ b/amdgpu/amdgpu_cs.c
@@ -42,17 +42,21 @@
  *
  * \return  0 on success otherwise POSIX Error code
 */
-static int amdgpu_cs_create_ib(amdgpu_context_handle context,
-                               enum amdgpu_cs_ib_size ib_size,
-                               amdgpu_ib_handle *ib)
+int amdgpu_cs_alloc_ib(amdgpu_context_handle context,
+                       enum amdgpu_cs_ib_size ib_size,
+                       struct amdgpu_cs_ib_alloc_result *output)
 {
-        struct amdgpu_bo_alloc_request alloc_buffer;
+        struct amdgpu_bo_alloc_request alloc_buffer = {};
         struct amdgpu_bo_alloc_result info;
         int r;
         void *cpu;
-        struct amdgpu_ib *new_ib;
 
-        memset(&alloc_buffer, 0, sizeof(alloc_buffer));
+        if (NULL == context)
+                return -EINVAL;
+        if (NULL == output)
+                return -EINVAL;
+        if (ib_size >= AMDGPU_CS_IB_SIZE_NUM)
+                return -EINVAL;
 
         switch (ib_size) {
         case amdgpu_cs_ib_size_4K:
@@ -89,18 +93,9 @@ static int amdgpu_cs_create_ib(amdgpu_context_handle context,
                 return r;
         }
 
-        new_ib = malloc(sizeof(struct amdgpu_ib));
-        if (NULL == new_ib) {
-                amdgpu_bo_cpu_unmap(info.buf_handle);
-                amdgpu_bo_free(info.buf_handle);
-                return -ENOMEM;
-        }
-
-        new_ib->context = context;
-        new_ib->buf_handle = info.buf_handle;
-        new_ib->cpu = cpu;
-        new_ib->virtual_mc_base_address = info.virtual_mc_base_address;
-        *ib = new_ib;
+        output->handle = info.buf_handle;
+        output->cpu = cpu;
+        output->mc_address = info.virtual_mc_base_address;
         return 0;
 }
 
@@ -111,47 +106,18 @@ static int amdgpu_cs_create_ib(amdgpu_context_handle context,
  *
  * \return  0 on success otherwise POSIX Error code
 */
-int amdgpu_cs_free_ib(amdgpu_ib_handle ib)
+int amdgpu_cs_free_ib(amdgpu_bo_handle bo)
 {
         int r;
 
-        if (!ib)
+        if (!bo)
                 return -EINVAL;
 
-        r = amdgpu_bo_cpu_unmap(ib->buf_handle);
+        r = amdgpu_bo_cpu_unmap(bo);
         if (r)
                 return r;
 
-        r = amdgpu_bo_free(ib->buf_handle);
-        if (r)
-                return r;
-
-        free(ib);
-        return 0;
-}
-
-int amdgpu_cs_alloc_ib(amdgpu_context_handle context,
-                       enum amdgpu_cs_ib_size ib_size,
-                       struct amdgpu_cs_ib_alloc_result *output)
-{
-        int r;
-        amdgpu_ib_handle ib;
-
-        if (NULL == context)
-                return -EINVAL;
-        if (NULL == output)
-                return -EINVAL;
-        if (ib_size >= AMDGPU_CS_IB_SIZE_NUM)
-                return -EINVAL;
-
-        r = amdgpu_cs_create_ib(context, ib_size, &ib);
-        if (!r) {
-                output->handle = ib;
-                output->cpu = ib->cpu;
-                output->mc_address = ib->virtual_mc_base_address;
-        }
-
-        return r;
+        return amdgpu_bo_free(bo);
 }
 
 /**
@@ -346,8 +312,8 @@ static int amdgpu_cs_submit_one(amdgpu_context_handle context,
 
                 ib = &ibs_request->ibs[i];
 
-                chunk_data[i].ib_data.handle = ib->ib_handle->buf_handle->handle;
-                chunk_data[i].ib_data.va_start = ib->ib_handle->virtual_mc_base_address
+                chunk_data[i].ib_data.handle = ib->bo_handle->handle;
+                chunk_data[i].ib_data.va_start = ib->bo_handle->virtual_mc_base_address
                                                 + ib->offset_dw * 4;
                 chunk_data[i].ib_data.ib_bytes = ib->size * 4;
                 chunk_data[i].ib_data.ip_type = ibs_request->ip_type;
diff --git a/amdgpu/amdgpu_internal.h b/amdgpu/amdgpu_internal.h
index a4c29894..ee1cb61c 100644
--- a/amdgpu/amdgpu_internal.h
+++ b/amdgpu/amdgpu_internal.h
@@ -111,13 +111,6 @@ struct amdgpu_context {
         uint32_t id;
 };
 
-struct amdgpu_ib {
-        amdgpu_context_handle context;
-        amdgpu_bo_handle buf_handle;
-        void *cpu;
-        uint64_t virtual_mc_base_address;
-};
-
 /**
  * Functions.
  */