From mboxrd@z Thu Jan 1 00:00:00 1970 From: Takashi Iwai Subject: Re: [PATCH] drm/i915: Suppress hotplug work during PM suspend/resume Date: Thu, 19 Apr 2012 18:32:22 +0200 Message-ID: References: <1334852586_26987@CP5-2952> Mime-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mx2.suse.de (cantor2.suse.de [195.135.220.15]) by gabe.freedesktop.org (Postfix) with ESMTP id AC9F49ED16 for ; Thu, 19 Apr 2012 09:32:22 -0700 (PDT) In-Reply-To: <1334852586_26987@CP5-2952> 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: Chris Wilson Cc: intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org At Thu, 19 Apr 2012 17:22:47 +0100, Chris Wilson wrote: > > On Thu, 19 Apr 2012 18:10:18 +0200, Takashi Iwai wrote: > > The hotplug work can be still kicked off via irq during PM, and this > > may conflict with the resume procedure. For example, eDP on SNB > > machine shows WARNING like below during the resume: > > Daniel and I were just discussing the number of work queues we have > active across suspend that like to poke hardware as we either try to > sleep or resume. Since we already have to quiesce the workers to disable > the device, I suggest we reuse the same code for suspend. Hm, sorry it's not clear to me which code you are referring to... Is it only in Daniel's next tree, or am I missing in Linus tree? thanks, Takashi