From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [Intel-gfx] [PATCH v2] Partially revert "drm/i915: unconditionally use mt forcewake on hsw/ivb" Date: Wed, 10 Jul 2013 08:04:14 +0200 Message-ID: <20130710060414.GF18285@phenom.ffwll.local> References: <1373410831-1347-1-git-send-email-linux@roeck-us.net> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <1373410831-1347-1-git-send-email-linux@roeck-us.net> Sender: linux-kernel-owner@vger.kernel.org To: Guenter Roeck Cc: intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org, David Airlie , Daniel Vetter , linux-kernel@vger.kernel.org, Mika Kuoppala List-Id: intel-gfx@lists.freedesktop.org On Tue, Jul 09, 2013 at 04:00:31PM -0700, Guenter Roeck wrote: > This patch partially reverts commit 36ec8f877481449bdfa072e6adf2060869e2b970 for > IvyBridge CPUs. > > The original commit results in repeated 'Timed out waiting for forcewake old > ack to clear' messages on a Supermicro C7H61 board (BIOS version 2.00 and 2.00b) > with i7-3770K CPU. It ultimately results in a hangup if the system is highly > loaded. Reverting the commit for IvyBridge CPUs fixes the issue. > > Issue a warning if the CPU is IvyBridge and mt forcewake is disabled, since > this condition can result in secondary issues. > > Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=60541 > Cc: Jesse Barnes > Cc: Daniel Vetter > Cc: Mika Kuoppala > Signed-off-by: Guenter Roeck Picked up for -fixes, thanks for the patch. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch