Quoting Tvrtko Ursulin (2019-07-30 08:33:28) > > On 30/07/2019 00:47, Daniele Ceraolo Spurio wrote: > > For symmetry with intel_gt_resume and to hide more stuff from the top > > level under intel_gt. Note that the switch_to_kernel_context_sync has > > not been moved dure to the locking and ordering requirements that exist > > Typo in due. > > Should we add intel_gt_suspend_early/late, with the former dealing with > switch_to_kernel_context_sync? Although that raises an interesting > conundrum since it operates on GEM and GT. Best then to leave it > unwrapped for now I think. > > Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx> Ah, I wouldn't apply this patch yet for exactly that reason: we shouldn't be driving GT suspend from inside GEM, so would wait until we have a few more pieces of the puzzle reviewed^W unravelled. -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
