Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754269AbdGNP4E (ORCPT ); Fri, 14 Jul 2017 11:56:04 -0400 Received: from merlin.infradead.org ([205.233.59.134]:37722 "EHLO merlin.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753382AbdGNP4D (ORCPT ); Fri, 14 Jul 2017 11:56:03 -0400 Date: Fri, 14 Jul 2017 17:55:41 +0200 From: Peter Zijlstra To: Ilia Mirkin Cc: Tobias Klausmann , Karol Herbst , Mike Galbraith , David Airlie , "nouveau@lists.freedesktop.org" , LKML , "dri-devel@lists.freedesktop.org" , Ben Skeggs Subject: Re: [Nouveau] [regression drm/noveau] suspend to ram -> BOOM: exception RIP: drm_calc_vbltimestamp_from_scanoutpos+335 Message-ID: <20170714155541.onocqn5o4f7v2mlh@hirez.programming.kicks-ass.net> References: <1499853345.23742.8.camel@gmx.de> <1499858703.23742.25.camel@gmx.de> <1500039368.5763.12.camel@gmx.de> <1500039693.5763.15.camel@gmx.de> <6341f91f-1c56-2188-f55b-2f47d23262fc@mni.thm.de> <31d327dd-bc3d-cb80-45a9-6a8eb9eb2a9d@mni.thm.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: NeoMutt/20170609 (1.8.3) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 597 Lines: 16 On Fri, Jul 14, 2017 at 11:20:01AM -0400, Ilia Mirkin wrote: > On Fri, Jul 14, 2017 at 11:19 AM, Tobias Klausmann > wrote: > > The conversion is a nice catch, but i'd like to have a bit more context, see > > below! > > > > With a better description: > > > > Tobias Klausmann > > I don't think it was meant as a serious patch. WARN_ON_ONCE should > work. The fix isn't to remove all instances of WARN_ON_ONCE. The fix > is to fix WARN_ON_ONCE. Quite so. Clearly I buggered it for modules; that really wasn't the plan.