Hi Chao,
On 2018/11/20 11:24, Chao Yu wrote:
>> +/*
>> + * it's safe to decrease since the workgroup isn't visible
>> + * and refcount >= 2 (cannot be freezed).
>> + */
>> +atomic_dec(>refcount);
> How about:
>
> #define
On 2018/11/14 23:25, Gao Xiang wrote:
> When the managed cache is enabled, the last reference count
> of a workgroup must be used for its workstation.
>
> Otherwise, it could lead to incorrect (un)freezes in
> the reclaim path, and it would be harmful.
>
> A typical race as follows:
>
> Thread
When the managed cache is enabled, the last reference count
of a workgroup must be used for its workstation.
Otherwise, it could lead to incorrect (un)freezes in
the reclaim path, and it would be harmful.
A typical race as follows:
Thread 1 (In the reclaim path) Thread 2
workgroup_freeze(grp,
