Re: [PATCH 02/10] drm/i915/guc: Initialize GuC before restarting engines

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Oct 05, 2017 at 09:36:31AM +0000, Chris Wilson wrote:
> Quoting Michał Winiarski (2017-10-05 10:13:41)
> > Now that we're handling request resubmission the same way as regular
> > submission (from the tasklet), we can move GuC initialization earlier,
> > before restarting the engines. This way, we're no longer being in the
> > state of flux during engine restart - we're already in user requested
> > submission mode.
> 
> Do you remember why it was the other way around?

IIRC we were resubmitting directly from "intel_uc_init_hw".
I guess doing that without calling ->init_hw() first was harmful :)

-Michał

> 
> > Cc: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
> > Cc: Michal Wajdeczko <michal.wajdeczko@xxxxxxxxx>
> > Cc: Michel Thierry <michel.thierry@xxxxxxxxx>
> > Cc: Mika Kuoppala <mika.kuoppala@xxxxxxxxx>
> > Cc: Oscar Mateo <oscar.mateo@xxxxxxxxx>
> > Signed-off-by: Michał Winiarski <michal.winiarski@xxxxxxxxx>
> Reviewed-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
> -Chris
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux