On Mon, Apr 10, 2017 at 02:58:34PM +0000, Saarinen, Jani wrote: > HI, > > -----Original Message----- > > From: Intel-gfx [mailto:intel-gfx-bounces@xxxxxxxxxxxxxxxxxxxxx] On Behalf > > Of Chris Wilson > > Sent: Monday, April 10, 2017 5:26 PM > > To: Wajdeczko, Michal <Michal.Wajdeczko@xxxxxxxxx> > > Cc: intel-gfx@xxxxxxxxxxxxxxxxxxxxx > > Subject: Re: [PATCH v2] drm/i915: Don't allow overuse of > > __intel_wait_for_register_fw() > > > > On Mon, Apr 10, 2017 at 12:17:47PM +0000, Michal Wajdeczko wrote: > > > This function should not be called with long timeouts in atomic context. > > > Annotate it as might_sleep if timeout is longer than 10us. > > > > > > v2: fix comment (Michal) > > > > > > Signed-off-by: Michal Wajdeczko <michal.wajdeczko@xxxxxxxxx> > > > Suggested-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > > > Cc: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > > > > Reviewed-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > > > > And pushed, thanks for the patch and for including the limits in the doc. > > -Chris > Was there reason why pushed as clearly seen issues on pw run: https://patchwork.freedesktop.org/series/22774/ > Or is there known issues we are planning to fix now? Now same faults on on tip: https://intel-gfx-ci.01.org/CI/ Because I was blind. The bug was latent and fix is sent. -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
