Are you referring to insert_work->smp_mb(); ?
Andrey
On 06/20/2018 10:50 AM, Christian König wrote:
>> +Â Â Â Â * Try to submit work so SQ_EDC_INFO can be accessed from
>> +Â Â Â Â * BH. If previous work submission hasn't finished yet
>> +Â Â Â Â * just print whatever info is possible directly from the ISR.
>> +Â Â Â Â */
>> +Â Â Â if (work_pending(&adev->gfx.sq_work.work)) {
>> +Â Â Â Â Â Â Â gfx_v8_0_parse_sq_irq(adev, ih_data);
>> +Â Â Â } else {
>> +Â Â Â Â Â Â Â adev->gfx.sq_work.ih_data = ih_data;
>> +Â Â Â Â Â Â Â /* Verify the new value visible in BH handler */
>> +Â Â Â Â Â Â Â smp_wmb();
>
> You can drop the barrier here and in gfx_v8_0_sq_irq_work_func(), the
> schedule_work() function is a barrier itself for both reads and writes
> anyway.
>
> Apart from that the both patches are Acked-by: Christian König
> <christian.koenig at amd.com>.
>
> Regards,
> Christian.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/amd-gfx/attachments/20180620/f5df7822/attachment.html>
