Revert "gpu: nvgpu: support user fence updates"

This reverts commit 0c46f8a5e112c08c172ee2c692832e1753ffbcce.

We should not support tracking of MAX/threshold value for syncpoint allocated
by user space
Hence revert this patch

Bug 200326065
Jira NVGPU-179

Change-Id: I2df8f8c13fdac91c0814b11a2b7dee30153409d4
Signed-off-by: Deepak Nibade <dnibade@nvidia.com>
Reviewed-on: https://git-master.nvidia.com/r/1665827
Reviewed-by: svc-mobile-coverity <svc-mobile-coverity@nvidia.com>
GVS: Gerrit_Virtual_Submit
Reviewed-by: Terje Bergstrom <tbergstrom@nvidia.com>
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>

1 files changed, 1 insertions, 21 deletions

diff --git a/drivers/gpu/nvgpu/common/linux/channel.c b/drivers/gpu/nvgpu/common/linux/channel.c
index 35fb3023..a725cd6b 100644
--- a/drivers/gpu/nvgpu/common/linux/channel.c
+++ b/drivers/gpu/nvgpu/common/linux/channel.c
@@ -720,8 +720,7 @@ int gk20a_submit_channel_gpfifo(struct channel_gk20a *c,
                 return -EINVAL;
 
         if ((flags & (NVGPU_SUBMIT_GPFIFO_FLAGS_FENCE_WAIT |
-                      NVGPU_SUBMIT_GPFIFO_FLAGS_FENCE_GET |
-                      NVGPU_SUBMIT_GPFIFO_FLAGS_USER_FENCE_UPDATE)) &&
+                      NVGPU_SUBMIT_GPFIFO_FLAGS_FENCE_GET)) &&
             !fence)
                 return -EINVAL;
 
@@ -758,16 +757,6 @@ int gk20a_submit_channel_gpfifo(struct channel_gk20a *c,
                         (g->can_railgate && !c->deterministic) ||
                         !skip_buffer_refcounting;
 
-        /*
-         * If User is adding increments to the pushbuffer and doing all job
-         * tracking, then no need for kernel tracking here
-         * User should ensure that all pre-requisites for fast submit are met
-         * Fail the submit if that's not the case
-         */
-        if (need_job_tracking &&
-            (flags & NVGPU_SUBMIT_GPFIFO_FLAGS_USER_FENCE_UPDATE))
-                return -EINVAL;
-
         if (need_job_tracking) {
                 bool need_sync_framework = false;
 
@@ -879,15 +868,6 @@ int gk20a_submit_channel_gpfifo(struct channel_gk20a *c,
                 goto clean_up;
         }
 
-        if (flags & NVGPU_SUBMIT_GPFIFO_FLAGS_USER_FENCE_UPDATE) {
-                /*
-                 * User space adds increments in the pushbuffer, so just
-                 * handle the threshold book keeping in kernel by adding
-                 * number of syncpoint increments to threshold
-                 */
-                c->sync->add_user_incrs(c->sync, fence->value);
-        }
-
         if (need_job_tracking) {
                 err = channel_gk20a_alloc_job(c, &job);
                 if (err)