drm/amdgpu: Fix amdgpu_sync_add_later to preserve explicit flag.
authorAndrey Grodzovsky <andrey.grodzovsky@amd.com>
Tue, 5 Dec 2017 13:03:13 +0000 (08:03 -0500)
committerAlex Deucher <alexander.deucher@amd.com>
Thu, 7 Dec 2017 16:48:56 +0000 (11:48 -0500)
Signed-off-by: Andrey Grodzovsky <andrey.grodzovsky@amd.com>
Reviewed-by: Christian König <christian.koenig@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/amdgpu/amdgpu_sync.c

index ebe1ffb..bb79fd3 100644 (file)
@@ -120,7 +120,7 @@ static void amdgpu_sync_keep_later(struct dma_fence **keep,
  * Tries to add the fence to an existing hash entry. Returns true when an entry
  * was found, false otherwise.
  */
-static bool amdgpu_sync_add_later(struct amdgpu_sync *sync, struct dma_fence *f)
+static bool amdgpu_sync_add_later(struct amdgpu_sync *sync, struct dma_fence *f, bool explicit)
 {
        struct amdgpu_sync_entry *e;
 
@@ -129,6 +129,10 @@ static bool amdgpu_sync_add_later(struct amdgpu_sync *sync, struct dma_fence *f)
                        continue;
 
                amdgpu_sync_keep_later(&e->fence, f);
+
+               /* Preserve eplicit flag to not loose pipe line sync */
+               e->explicit |= explicit;
+
                return true;
        }
        return false;
@@ -148,12 +152,11 @@ int amdgpu_sync_fence(struct amdgpu_device *adev, struct amdgpu_sync *sync,
 
        if (!f)
                return 0;
-
        if (amdgpu_sync_same_dev(adev, f) &&
            amdgpu_sync_get_owner(f) == AMDGPU_FENCE_OWNER_VM)
                amdgpu_sync_keep_later(&sync->last_vm_update, f);
 
-       if (amdgpu_sync_add_later(sync, f))
+       if (amdgpu_sync_add_later(sync, f, explicit))
                return 0;
 
        e = kmem_cache_alloc(amdgpu_sync_slab, GFP_KERNEL);