Re: [Mesa-dev] [PATCH 1/2] vl: add motion adaptive deinterlacer

2014-02-15 Thread Andy Furniss
Grigori Goronzy wrote: On 15.02.2014 13:14, Andy Furniss wrote: Thanks Grigori for doing this - looks really good on HD stuff I've tested and of course is easily fast enough, unlike anything on the CPU at high res. Any plans for the future? Well, adding edge-guided spatial interpolation for

Re: [Mesa-dev] [PATCH 1/2] vl: add motion adaptive deinterlacer

2014-02-15 Thread Grigori Goronzy
On 15.02.2014 13:14, Andy Furniss wrote: Thanks Grigori for doing this - looks really good on HD stuff I've tested and of course is easily fast enough, unlike anything on the CPU at high res. Any plans for the future? Well, adding edge-guided spatial interpolation for the temporal-spatial mo

Re: [Mesa-dev] [PATCH 1/2] vl: add motion adaptive deinterlacer

2014-02-15 Thread Ilia Mirkin
On Sat, Feb 15, 2014 at 7:14 AM, Andy Furniss wrote: > At the start or when I press f on mplayer to toggle between full/window > I see a frame of old/uninitialised video mem - does this happen on nvidia? I definitely see something funny. It seemed like the deinterlace was essentially "turned off"

Re: [Mesa-dev] [PATCH 1/2] vl: add motion adaptive deinterlacer

2014-02-15 Thread Andy Furniss
Andy Furniss wrote: Ilia Mirkin wrote: On Sat, Feb 15, 2014 at 5:10 AM, Christian König wrote: That's most likely an issue with st/vdpau. I implemented a small hack to clear the black bars of a surface only once. No idea how exactly it happens, but Grigoris patch seems to somehow break the log

Re: [Mesa-dev] [PATCH 1/2] vl: add motion adaptive deinterlacer

2014-02-15 Thread Andy Furniss
Ilia Mirkin wrote: On Sat, Feb 15, 2014 at 5:10 AM, Christian König wrote: That's most likely an issue with st/vdpau. I implemented a small hack to clear the black bars of a surface only once. No idea how exactly it happens, but Grigoris patch seems to somehow break the logic there. Anyway we

Re: [Mesa-dev] [PATCH 1/2] vl: add motion adaptive deinterlacer

2014-02-15 Thread Ilia Mirkin
On Sat, Feb 15, 2014 at 5:10 AM, Christian König wrote: > That's most likely an issue with st/vdpau. I implemented a small hack to > clear the black bars of a surface only once. No idea how exactly it happens, > but Grigoris patch seems to somehow break the logic there. > > Anyway we need to rewor

Re: [Mesa-dev] [PATCH 1/2] vl: add motion adaptive deinterlacer

2014-02-15 Thread Christian König
That's most likely an issue with st/vdpau. I implemented a small hack to clear the black bars of a surface only once. No idea how exactly it happens, but Grigoris patch seems to somehow break the logic there. Anyway we need to rework this part when getting DRI3 anyway (much more efficiently an

Re: [Mesa-dev] [PATCH 1/2] vl: add motion adaptive deinterlacer

2014-02-14 Thread Ilia Mirkin
Grigori, I just tried this out on a few interlaced videos I have, and it works really quite well! One thing I did notice is that when playing back content with mplayer (and -vo vdpau:deint=3) in full screen mode, the black bars are now stale/uninitialized textures. This does not happen if I don't

Re: [Mesa-dev] [PATCH 1/2] vl: add motion adaptive deinterlacer

2014-02-14 Thread Christian König
A really nice piece of work, thx allot. Both patches reviewed and pushed upstream. Cheers, Christian. Am 13.02.2014 21:32, schrieb Grigori Goronzy: --- src/gallium/auxiliary/Makefile.sources | 3 +- src/gallium/auxiliary/vl/vl_deint_filter.c | 491 + src/g

[Mesa-dev] [PATCH 1/2] vl: add motion adaptive deinterlacer

2014-02-13 Thread Grigori Goronzy
--- src/gallium/auxiliary/Makefile.sources | 3 +- src/gallium/auxiliary/vl/vl_deint_filter.c | 491 + src/gallium/auxiliary/vl/vl_deint_filter.h | 78 + 3 files changed, 571 insertions(+), 1 deletion(-) create mode 100644 src/gallium/auxiliary/vl/vl_deint_f