From mboxrd@z Thu Jan 1 00:00:00 1970 From: Matthew Garrett Subject: Re: [PATCH 2/2] drm/i915: By default, enable RC6 on IVB and SNB when reasonable Date: Tue, 13 Dec 2011 03:45:58 +0000 Message-ID: <20111213034558.GA26750@srcf.ucam.org> References: <1321684889-18691-1-git-send-email-keithp@keithp.com> <1323474829-2501-1-git-send-email-keithp@keithp.com> <1323474829-2501-3-git-send-email-keithp@keithp.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <1323474829-2501-3-git-send-email-keithp@keithp.com> Sender: linux-kernel-owner@vger.kernel.org To: Keith Packard Cc: intel-gfx@lists.freedesktop.org, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, Ted Phelps , Peter , Lukas Hejtmanek , Andrew Lutomirski List-Id: dri-devel@lists.freedesktop.org On Fri, Dec 09, 2011 at 03:53:49PM -0800, Keith Packard wrote: > RC6 should always work on IVB, and should work on SNB whenever IO > remapping is disabled. RC6 never works on Ironlake. Make the default > value for the parameter follow these guidelines. Setting the value > to either 0 or 1 will force the specified behavior. I still don't like this. We're in a situation where we clearly have to disable one feature or the other on SNB - but we're disabling the one that's going to be useful to a large number of people, and leaving the niche feature enabled. If we're going to merge this then let's turn off iommu on SNB by default. -- Matthew Garrett | mjg59@srcf.ucam.org