Re: [FFmpeg-devel] [PATCH] avcodec/vp9: mark frame as finished on decode_tiles() failure

2017-12-14 Thread Michael Niedermayer 
On Wed, Dec 13, 2017 at 11:00:37PM -0800, James Zern wrote:
> On Wed, Dec 13, 2017 at 5:02 PM, Michael Niedermayer
>  wrote:
> > Fixes deadlock with framethreads
> > Fixes: 
> > Netflix_Aerial_1080p_60fps_8bit_420.y4m.vp9.noaltref.webm.ivf.s69372_r01-05_b6-.ivf
> > Fixes: 
> > Netflix_Aerial_1080p_60fps_10bit_420.y4m.vp9.noaltref.webm.ivf.s149104_r01-05_b6-.ivf
> > Fixes: ducks_take_off_444_720p50.y4m.vp9.webm.ivf.s107375_r01-05_b6-.ivf
> >
> > Reported-by: James Zern 
> > Signed-off-by: Michael Niedermayer 
> > ---
> >  libavcodec/vp9.c | 4 +++-
> >  1 file changed, 3 insertions(+), 1 deletion(-)
> >
> 
> lgtm. tested locally, thanks for having a look.

will apply

thanks
[...]

-- 
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Everything should be made as simple as possible, but not simpler.
-- Albert Einstein


signature.asc
Description: PGP signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] avcodec/vp9: mark frame as finished on decode_tiles() failure

2017-12-13 Thread James Zern 
On Wed, Dec 13, 2017 at 5:02 PM, Michael Niedermayer
 wrote:
> Fixes deadlock with framethreads
> Fixes: 
> Netflix_Aerial_1080p_60fps_8bit_420.y4m.vp9.noaltref.webm.ivf.s69372_r01-05_b6-.ivf
> Fixes: 
> Netflix_Aerial_1080p_60fps_10bit_420.y4m.vp9.noaltref.webm.ivf.s149104_r01-05_b6-.ivf
> Fixes: ducks_take_off_444_720p50.y4m.vp9.webm.ivf.s107375_r01-05_b6-.ivf
>
> Reported-by: James Zern 
> Signed-off-by: Michael Niedermayer 
> ---
>  libavcodec/vp9.c | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
>

lgtm. tested locally, thanks for having a look.
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel