On 12/05/2017 08:41 AM, Andrey Grodzovsky wrote: > Signed-off-by: Andrey Grodzovsky <[email protected]> > --- > 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); _______________________________________________ amd-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/amd-gfx
