mirror of
https://github.com/hardkernel/linux.git
synced 2026-06-08 11:50:43 +09:00
sync: reorder sync_fence_release
Previously fence's pts were freed before the were the fence was removed from the global fence list. This led to a race with the debugfs support where it would iterate over sync_pts that had been freed. Change-Id: Ia3ddbf77de42ca593fc2dc353b5d04e42ddf3946 Signed-off-by: Erik Gilling <konkers@android.com>
This commit is contained in:
@@ -520,12 +520,12 @@ static int sync_fence_release(struct inode *inode, struct file *file)
|
||||
struct sync_fence *fence = file->private_data;
|
||||
unsigned long flags;
|
||||
|
||||
sync_fence_free_pts(fence);
|
||||
|
||||
spin_lock_irqsave(&sync_fence_list_lock, flags);
|
||||
list_del(&fence->sync_fence_list);
|
||||
spin_unlock_irqrestore(&sync_fence_list_lock, flags);
|
||||
|
||||
sync_fence_free_pts(fence);
|
||||
|
||||
kfree(fence);
|
||||
|
||||
return 0;
|
||||
|
||||
Reference in New Issue
Block a user