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 > amd-gfx at lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/amd-gfx
