From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chris Wilson Subject: Re: [PATCH] drm/i915: Suppress hotplug work during PM suspend/resume Date: Thu, 10 May 2012 10:25:44 +0100 Message-ID: <1336641962_223397@CP5-2952> References: <1334858104.32618.28.camel@atropine> <20120510082919.GA4867@phenom.ffwll.local> <20120510090646.GD4867@phenom.ffwll.local> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from fireflyinternet.com (smtp.fireflyinternet.com [109.228.6.236]) by gabe.freedesktop.org (Postfix) with ESMTP id 94F1E9E9EE for ; Thu, 10 May 2012 02:26:07 -0700 (PDT) In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org Errors-To: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org To: Takashi Iwai , Daniel Vetter Cc: intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org On Thu, 10 May 2012 11:19:11 +0200, Takashi Iwai wrote: > At Thu, 10 May 2012 11:06:46 +0200, > Daniel Vetter wrote: > > This patch is not just for hotplug, but for all the delayed work and timer > > stuff the driver does. And we _do_ have a bug report that leaking the rps > > work (for snb+ turbo mode) across either a s/r cycle or a gpu reset kills > > the driver. > > Yes, but my point is that Chris's patch (at least the commit above > alone) won't fix the problem we faced. The patch I thought you two were talking about was: http://cgit.freedesktop.org/~ickle/linux-2.6/commit/?h=fastboot&id=a5f91dcc3cb7a9dc1214d5014ea6f0338824ad8e which is what I thought we had planned to fix the residual issue. -Chris -- Chris Wilson, Intel Open Source Technology Centre