On Wed, Feb 28, 2018 at 12:26 AM, Chris Wilson <ch...@chris-wilson.co.uk> wrote: > Quoting Andy Lutomirski (2018-02-24 00:07:23) >> On Tue, Feb 13, 2018 at 11:26 PM, Rodrigo Vivi <rodrigo.v...@intel.com> >> wrote: >> > From: Andy Lutomirski <l...@kernel.org> >> > >> > + >> > + dev_priv->psr.activate_timer.expires = jiffies - 1; >> >> That can't possibly be okay. > > As an initialisation value, set to the previous jiffie? You can set it > to the current jiffie, but then you have the issue of not noticing the > update to the current jiffie. > > So how is this any more incorrect?
I don't think you can just write to fields in struct timer_list like that. _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
