From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: [PATCH 0/2] vgaarb: Fixes for partial VGA opt-out Date: Fri, 30 Aug 2013 15:41:19 +0300 Message-ID: <20130830124119.GH11428@intel.com> References: <20130815222835.27642.15330.stgit@bling.home> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Content-Disposition: inline In-Reply-To: <20130815222835.27642.15330.stgit@bling.home> Sender: linux-kernel-owner@vger.kernel.org To: Alex Williamson Cc: linux-kernel@vger.kernel.org, airlied@redhat.com, intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org On Thu, Aug 15, 2013 at 04:37:47PM -0600, Alex Williamson wrote: > I'm trying to add support for VGA arbitration on newer Intel graphics > devices. The existing code attempts to do this, but appear to have > not been updated since GMCH devices roamed the Earth. On newer > devices like Haswell, we can disable VGA memory through an MSR on the > device, but we rely on the VGA arbiter to manage VGA IO using the PCI > COMMAND register. In trying to unregister legacy VGA memory, I found > that the VGA arbiter still wanted to disable both memory and IO on > the device and that it forgot to actually program the device to > disable IO when the decoding is updated. This series attempts to fix > both of those. Thanks, The series looks good to me. Reviewed-by: Ville Syrj=E4l=E4 >=20 > Alex >=20 > --- >=20 > Alex Williamson (2): > vgaarb: Don't disable resources that are not owned > vgaarb: Fix VGA decodes changes >=20 >=20 > drivers/gpu/vga/vgaarb.c | 50 ++++++++++++++++++++----------------= ---------- > 1 file changed, 22 insertions(+), 28 deletions(-) --=20 Ville Syrj=E4l=E4 Intel OTC