Intel - XVideo

[Eric Anholt]

On Thu, 2007-06-14 at 10:30 +0200, Lukas Hejtmanek wrote:
> On Wed, Jun 13, 2007 at 04:41:21PM -0700, Eric Anholt wrote:
> > I'd forgotten about that code.  It was quite broken, so I had gone with
> > planar to packed conversion to just get something working.  I've fixed
> > up the planar path now, which should help with XV performance issues on
> > this hardware.
> 
> Well, I saw the commit. However, for i965 there is still planar to packed
> copy. Is this limitation of i965 hw or the code is not just ready for i965?

Not a limitation of the hardware at all.  Just nobody's written the
planar pixel shader for 965 yet.

> > Since we're doing the entire YUV -> RGB conversion in our pixel shader,
> > we certainly could.  Nobody has bothered yet.
> 
> OK, I see. However, I have no idea how to change brightness when packed format
> is used - it seems that internal matrix is used. Am I right?

On the 915, we use the built in colorspace conversion, but we could turn
that off and get YUV data out of the sampling, which we could then
adjust to our needs.  The 965 code does the YUV -> RGB in the pixel
shader already.

> > 1) requires the DRI to avoid spinning, so we'll just have tearing if you
> > can't do DRI.
> > 2) requires i830, i855, or 915+ to do it from the command stream.  To
> > fix that, we might be able to use the scanline window wait, or I guess
> > use the DRM to wait for a vblank synchronously before emitting our draw.
> > 
> > My work in progress patch for this is at
> > git://people.freedesktop.org/~anholt/xf86-video-intel in the t-v-vblank
> > branch.  Right now it crashes the 945GM I've been testing on.  If
> > someone wanted to track this down, it would be great.
> 
> what exactly means i830, i855 or 915+ ? Does it mean that it does not work
> only on i845 and i865?

Yeah.  Granted, we don't even have textured video on pre-915 yet, but it
would be nice if we did so we could get proper compositing results.

-- 
Eric Anholt                             anholt at FreeBSD.org
eric at anholt.net                         eric.anholt at intel.com

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 187 bytes
Desc: This is a digitally signed message part
URL: <http://lists.x.org/archives/xorg/attachments/20070614/f27a3627/attachment.pgp>