[Intel-gfx] [PATCH] Xv: free tearing on textured video

[Kristian Høgsberg]

On Wed, Feb 25, 2009 at 11:30 AM, Keith Packard <keithp at keithp.com> wrote:
> On Wed, 2009-02-25 at 15:56 +0800, Zou, Nanhai wrote:
>
>> I suggest turn this option default to be true or not use an option at all.
>> We already have too many options to handle.
>
> Ideally, this code would do the right thing automatically, and be
> configured when it 'automatic' wasn't working right. Nanhai's suggestion
> of turning it on by default is a good one, as is Dave's suggestion of
> using an Xv attribute instead of a config file option.
>
> I'd suggest one further change -- only sync when the video is 'large',
> so that videos which are only a small part of the screen don't impact
> other applications running at the same time.

That's a good heuristic, something like 50% of available screen space
could work.  Default to 'auto' sounds good to me.

Kristian