Re: [PATCH 0/2] vgaarb: Fixes for partial VGA opt-out

From: Ville Syrjälä
Date: Fri Aug 30 2013 - 08:41:30 EST


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älä <ville.syrjala@xxxxxxxxxxxxxxx>

>
> Alex
>
> ---
>
> Alex Williamson (2):
> vgaarb: Don't disable resources that are not owned
> vgaarb: Fix VGA decodes changes
>
>
> drivers/gpu/vga/vgaarb.c | 50 ++++++++++++++++++++--------------------------
> 1 file changed, 22 insertions(+), 28 deletions(-)

--
Ville Syrjälä
Intel OTC
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/