Re: [PATCH] xf86-video-intel: Add UXA PACKAGECONFIG and 20-intel.conf

[Richard Purdie <richard.purdie@linuxfoundation.org>]

On Mon, 2016-03-28 at 11:07 -0700, Saul Wold wrote:
> This patch adds the UXA Packageconfig option to handle older Intel
> Graphics
> the uxa code when enabled needs a patch from upstream due to a change
> in the
> API of the Xserver.
> 
> Also added a generic 20-intel.conf with a commented line to enable
> the uxa
> AccelMethod option.
> 
> The Packageconfig only needs to be enabled in genericx86-64 since
> other
> MACHINE configs (such as meta-intel) include the VAAPI acceleration
> that
> handle the graphics hardware corretly.

This doesn't really look right to me. It makes the driver MACHINE
specific which is generally bad and will fail various tests. You likely
need to detect this at runtime on the relevant hardward and enable
if/as/when needed.

Is there some way that X can correctly autodetect what its running on
and do the right thing without requiring manual config? The fact we're
having to do that seems to fly in the face of the "configurationless"
goals X has had more recently...

Cheers,

Richard