HID: debug: Use the __set_current_state()
When detecting an invalid list->hdev, the process needs to manually set its state to TASK_RUNNING and exit. In the original code, set_current_state() (which includes a memory barrier) is used here, but it is immediately followed by a mutex_unlock() call. Since mutex_unlock() internally includes a memory barrier, this ensures that all modifications within the critical section (including the process state) are visible to other CPUs. Therefore, replacing it with __set_current_state() (without an implicit barrier) avoids redundant memory barriers. Signed-off-by:Zihuan Zhang <zhangzihuan@kylinos.cn> Signed-off-by:
Jiri Kosina <jkosina@suse.com>
Loading
Please sign in to comment