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

[Saul Wold <sgw@linux.intel.com>]

On Mon, 2016-03-28 at 23:33 +0100, Richard Purdie wrote:
> 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.
> 
Yeah, I had a thought that this might cause a MACHINE specific issue,
but wanted to get eyes on the issue.

Your right this could be another example of where the Runtime Machine
Config could help, I will add it to Jianxun's list.

> 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...
> 
Unfortunately X will start correctly with the newer AccelMethod which
this particular hardware does not handle well.

Let's hold off on this patch for now, we understand that basics, better
than we did yesterday and can get it logged in the Bug.

Sau!


> Cheers,
> 
> Richard