GRBM lock in vce_v3_0_check_soft_reset(void *handle)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Yes, it sounds like a bug.  It should take the mutex locally.

From: amd-gfx [mailto:amd-gfx-bounces@xxxxxxxxxxxxxxxxxxxxx] On Behalf Of StDenis, Tom
Sent: Thursday, August 11, 2016 9:42 AM
To: amd-gfx list
Subject: GRBM lock in vce_v3_0_check_soft_reset(void *handle)


I'm working on tidying up vce v3 and came across the function


static int vce_v3_0_check_soft_reset(void *handle)


Which writes to the GRBM_GFX_INDEX but doesn't take the mutex around it.  Nor does the parent function in amdgpu_device.c which calls all of the check_soft_reset callbacks.



Is this a bug?



Tom
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/amd-gfx/attachments/20160811/fc90155c/attachment.html>


[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux