Am 10.08.2016 um 08:02 schrieb zhoucm1:


On 2016年08月08日 19:04, Christian König wrote:
I need to work on the S3 issue again now, going to come back to this patch set when I have more time.
I've addressed your previous comments, are you free?

Well if you figure out why the heck the UVD engine sometimes crashes since I changed how usage is detected I'm all yours :)

how about the other patches?

They looked rather good to me, maybe Alex or somebody else could take a look as well.

What you should keep in mind is a) please try to avoid using the scheduler for recovery, I really don't like the idea that we start to mess with userspace submissions before anything else is stable again.

And b) when we use the same reservation object for two BOs we need to set the parent, otherwise we can run into a race which gets freed first. That applies even when the BOs are reference counted like your shadow BOs are.


Thanks,
David Zhou

Regards,
Christian.

_______________________________________________
amd-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/amd-gfx


_______________________________________________
amd-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to