On 23.02.2012 18:32, Jerome Glisse wrote:
2012/2/23 Mathias Fröhlich <Mathias.Froehlich@xxxxxxx>
Christian,
On Thursday, February 23, 2012 15:18:42 Christian König
wrote:
> The function radeon_bo_list_validate can cause a
> bo to move, resulting in a different sync_obj
> and a dependency to wait for this move to finish.
>
> Signed-off-by: Christian König < deathsimple@xxxxxxxxxxx>
> Reviewed-by: Alex Deucher < alexander.deucher@xxxxxxx>
lockups that I experience since some time every now and then.
The usual symptom is that I get the
radeon 0000:01:00.0: GPU lockup CP stall for more than
10000msec
GPU lockup (waiting for 0x00682AC3 last fence id 0x00682AC2)
[...]
kernel message. Each time with the fence being off by one like
in the example
above.
If this change has the potential to fix this issue I think
this particular
patch should be considered for the current upstream kernel
release.
Mathias
No this patch doesn't. This patch is all about getting proper sync
btw different rings.
That's unfortunately true, since we wasn't able to release any code
that makes direct use of the different rings (yet) it shouldn't
really matter in practice. I Just wanted to have that fix upstream
since it is an obvious bug.
Christian.
Sorry for the deception.
Cheers,
Jerome
|
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel
