Re: [libav-devel] [PATCH 6/9] lavf: VAAPI scale filter

2016-03-19 Thread Mark Thompson
On 19/03/16 08:47, Anton Khirnov wrote: > Quoting Mark Thompson (2016-03-16 00:00:09) >> + >> +static int scale_vaapi_pipeline_uninit(ScaleVAAPIContext *ctx) >> +{ >> +if (ctx->va_context != VA_INVALID_ID) { > > Some tests reveal that these checks are not good, because this function > can be

Re: [libav-devel] [PATCH 6/9] lavf: VAAPI scale filter

2016-03-19 Thread Anton Khirnov
Quoting Mark Thompson (2016-03-16 00:00:09) > + > +static int scale_vaapi_pipeline_uninit(ScaleVAAPIContext *ctx) > +{ > +if (ctx->va_context != VA_INVALID_ID) { Some tests reveal that these checks are not good, because this function can be called without the context/config ids ever being

Re: [libav-devel] [PATCH 6/9] lavf: VAAPI scale filter

2016-03-19 Thread Diego Biurrun
On Tue, Mar 15, 2016 at 11:00:09PM +, Mark Thompson wrote: > --- a/libavfilter/allfilters.c > +++ b/libavfilter/allfilters.c > @@ -108,6 +108,7 @@ void avfilter_register_all(void) > REGISTER_FILTER(TRANSPOSE, transpose, vf); > REGISTER_FILTER(TRIM, trim,

[libav-devel] [PATCH 6/9] lavf: VAAPI scale filter

2016-03-15 Thread Mark Thompson
--- configure| 3 + libavfilter/Makefile | 1 + libavfilter/allfilters.c | 1 + libavfilter/vf_scale_vaapi.c | 451 +++ 4 files changed, 456 insertions(+) create mode 100644 libavfilter/vf_scale_vaapi.c diff --git