gpu: nvgpu: Remove IOCTL FREE_OBJ_CTX

We have never used the IOCTL FREE_OBJ_CTX. Using it leads to context
being only partially available, and can lead to use-after-free.

Bug 1834225

Change-Id: I9d2b632ab79760f8186d02e0f35861b3a6aae649
Signed-off-by: Terje Bergstrom <tbergstrom@nvidia.com>
Reviewed-on: http://git-master/r/1250004
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>

1 files changed, 0 insertions, 22 deletions

diff --git a/drivers/gpu/nvgpu/vgpu/gr_vgpu.c b/drivers/gpu/nvgpu/vgpu/gr_vgpu.c
index 89223091..01f5e1a5 100644
--- a/drivers/gpu/nvgpu/vgpu/gr_vgpu.c
+++ b/drivers/gpu/nvgpu/vgpu/gr_vgpu.c
@@ -421,7 +421,6 @@ static void vgpu_gr_free_channel_ctx(struct channel_gk20a *c)
 
         memset(&c->ch_ctx, 0, sizeof(struct channel_ctx_gk20a));
 
-        c->num_objects = 0;
         c->first_init = false;
 }
 
@@ -586,8 +585,6 @@ static int vgpu_gr_alloc_obj_ctx(struct channel_gk20a  *c,
                 c->first_init = true;
         }
 
-        c->num_objects++;
-
         gk20a_dbg_fn("done");
         return 0;
 out:
@@ -599,24 +596,6 @@ out:
         return err;
 }
 
-static int vgpu_gr_free_obj_ctx(struct channel_gk20a  *c,
-                                struct nvgpu_free_obj_ctx_args *args)
-{
-        gk20a_dbg_fn("");
-
-        if (c->num_objects == 0)
-                return 0;
-
-        c->num_objects--;
-
-        if (c->num_objects == 0) {
-                c->first_init = false;
-                gk20a_disable_channel(c);
-        }
-
-        return 0;
-}
-
 static int vgpu_gr_init_gr_config(struct gk20a *g, struct gr_gk20a *gr)
 {
         struct vgpu_priv_data *priv = vgpu_get_priv_data(g);
@@ -1073,7 +1052,6 @@ void vgpu_init_gr_ops(struct gpu_ops *gops)
         gops->gr.detect_sm_arch = vgpu_gr_detect_sm_arch;
         gops->gr.free_channel_ctx = vgpu_gr_free_channel_ctx;
         gops->gr.alloc_obj_ctx = vgpu_gr_alloc_obj_ctx;
-        gops->gr.free_obj_ctx = vgpu_gr_free_obj_ctx;
         gops->gr.alloc_gr_ctx = vgpu_gr_alloc_gr_ctx;
         gops->gr.free_gr_ctx = vgpu_gr_free_gr_ctx;
         gops->gr.bind_ctxsw_zcull = vgpu_gr_bind_ctxsw_zcull;