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

[Gagandeep Singh]

On Tue, Aug 14, 2018 at 12:46 PM Gagandeep Singh <deepgagan231197 at gmail.com>
wrote:

>
>
> 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
>

Updated patches are in new thread. Sorry for inconvenience.

Gagandeep Singh


> _______________________________________________
>> ffmpeg-devel mailing list
>> ffmpeg-devel at ffmpeg.org
>> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>>
>