On 09.07.2012 18:10, Jerome Glisse wrote:
On Mon, Jul 9, 2012 at 11:59 AM, Michel Dänzer michel@daenzer.net wrote:
On Mon, 2012-07-09 at 12:41 +0200, Christian König wrote:
Hi,
The following patchset tries to save and restore the not yet processed commands from the rings in case of a lockup and with that should make a userspace problem with a single application far less problematic.
The first four patches are just stuff this patchset is based upon, followed by four patches which fix various bugs found while working on this feature.
Followed by patches which change the way how memory is saved/restored on suspend/resume, basically before we have unpinned most of the buffer objects so it could be move from vram into system memory. But that is mostly unnecessary cause the buffer object either are already in system memory or their content can be easily reinitialized.
The last three patches implement the actual tracking and restoring of commands in case of a lockup. Please take a look and review.
Patches 3, 5 and 14 are
Reviewed-by: Michel Dänzer michel.daenzer@amd.com
Patch 1-9 are Reviewed-by: Jerome Glisse jglisse@redhat.com
Other looks good but i want to test them too and spend a bit more time to double check few things. Will try to do that tomorrow.
Just send out v2 of the patchset. Mainly it integrates your idea of just saving rptr right before we call into the IB, but also contains all the other comments and fixes from Michel.
Cheers, Christian.