[PATCH] drm/amdgpu: Fix amdgpu_sync_add_later to preserve explicit flag.

[William Lewis]

On 12/05/2017 08:41 AM, Andrey Grodzovsky wrote:
> Signed-off-by: Andrey Grodzovsky <andrey.grodzovsky at amd.com>
> ---
>   drivers/gpu/drm/amd/amdgpu/amdgpu_sync.c | 9 ++++++---
>   1 file changed, 6 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_sync.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_sync.c
> index f3d1a25..84fd367 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_sync.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_sync.c
> @@ -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 */

Minor nitpick.  s/eplicit/explicit/

> +		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);