[FFmpeg-devel] [GSOC][PATCH] lavc/cfhd:frame threading support added for temporal transform decoding

[Gagandeep Singh]

On Tue, Aug 14, 2018 at 1:17 AM James Almer <jamrial at gmail.com> wrote:

> On 8/13/2018 4:29 PM, Gagandeep Singh wrote:
> > ---
> >  libavcodec/cfhd.c | 370 ++++++++++++++++++++++++++++------------------
> >  libavcodec/cfhd.h |   8 +-
> >  2 files changed, 235 insertions(+), 143 deletions(-)
> >
> > diff --git a/libavcodec/cfhd.c b/libavcodec/cfhd.c
> > index 2c538f0bbd..042f63c6d2 100644
> > --- a/libavcodec/cfhd.c
> > +++ b/libavcodec/cfhd.c
> > @@ -65,9 +65,13 @@ static av_cold int cfhd_init(AVCodecContext *avctx)
> >  {
> >      CFHDContext *s = avctx->priv_data;
> >
> > +    avctx->internal->allocate_progress = 1;
>
> Shouldn't this be under a !avctx->internal->is_copy check? Probably also
> the ff_cfhd_init_vlcs call below.
>
> Alternatively, add a new function meant to be used only with
> AVCodec->init_thread_copy().
>
> >      avctx->bits_per_raw_sample = 10;
> >      s->avctx                   = avctx;
> >      s->progressive             = 0;
> > +    s->i_frame.f = av_frame_alloc();
> > +    s->p_frame.f = av_frame_alloc();
> > +
> >
> >      return ff_cfhd_init_vlcs(s);
> >  }
>
>
Thanks for the feedback. Updated patch attatched.

Gagandeep Singh

> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: frame_threading_support_for_3d_transform_samples.patch
Type: text/x-patch
Size: 27366 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20180814/19b0b74a/attachment.bin>