Messages in this thread | | | From | "Brown, Aaron F" <> | Subject | RE: [Intel-wired-lan] [PATCH v2 1/1] e1000e: Undo e1000e_pm_freeze if __e1000_shutdown fails | Date | Wed, 7 Jun 2017 01:07:33 +0000 |
| |
> From: Intel-wired-lan [mailto:intel-wired-lan-bounces@osuosl.org] On Behalf > Of Jeff Kirsher > Sent: Tuesday, June 6, 2017 1:46 PM > To: David Miller <davem@davemloft.net>; Nikula, Jani > <jani.nikula@intel.com> > Cc: Ursulin, Tvrtko <tvrtko.ursulin@intel.com>; daniel.vetter@ffwll.ch; intel- > gfx@lists.freedesktop.org; linux-kernel@vger.kernel.org; > jani.nikula@linux.intel.com; chris@chris-wilson.co.uk; Ertman, David M > <david.m.ertman@intel.com>; intel-wired-lan@lists.osuosl.org; dri- > devel@lists.freedesktop.org; netdev@vger.kernel.org; airlied@gmail.com > Subject: Re: [Intel-wired-lan] [PATCH v2 1/1] e1000e: Undo > e1000e_pm_freeze if __e1000_shutdown fails > > On Fri, 2017-06-02 at 14:14 -0400, David Miller wrote: > > From: Jani Nikula <jani.nikula@intel.com> > > Date: Wed, 31 May 2017 18:50:43 +0300 > > > > > From: Chris Wilson <chris@chris-wilson.co.uk> > > > > > > An error during suspend (e100e_pm_suspend), > > > > ... > > > lead to complete failure: > > > > ... > > > The unwind failures stems from commit 2800209994f8 ("e1000e: > > > Refactor PM > > > flows"), but it may be a later patch that introduced the non- > > > recoverable > > > behaviour. > > > > > > Fixes: 2800209994f8 ("e1000e: Refactor PM flows") > > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=99847 > > > Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com> > > > Cc: Jeff Kirsher <jeffrey.t.kirsher@intel.com> > > > Cc: Dave Ertman <davidx.m.ertman@intel.com> > > > Cc: Bruce Allan <bruce.w.allan@intel.com> > > > Cc: intel-wired-lan@lists.osuosl.org > > > Cc: netdev@vger.kernel.org > > > Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> > > > [Jani: bikeshed repainted] > > > Signed-off-by: Jani Nikula <jani.nikula@intel.com> > > > > Jeff, please make sure this gets submitted to me soon. > > Expect it later tonight, just finishing up testing.
Tested-by: Aaron Brown <aaron.f.brown@intel.com>
| |