Re: [FFmpeg-devel] [PATCH 3/4] lavf/concat: switch to new BSF API.

2016-06-03 Thread Nicolas George
L'octidi 8 prairial, an CCXXIV, Nicolas George a écrit :
> It seems the memory management is not as negligible as I expected. The
> result is a ~2.3% slow-down for ~90k frames from AVI, which is approximately
> equal to the standard deviation on 10 runs. I suspect this is acceptable
> compared to the overhead of more complex containers.

No comments? I think this overhead is acceptable and this, if nobody objects
I will push soon the series.

Regards,

-- 
  Nicolas George


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


Re: [FFmpeg-devel] [PATCH 3/4] lavf/concat: switch to new BSF API.

2016-05-26 Thread Nicolas George
Le primidi 1er prairial, an CCXXIV, Michael Niedermayer a écrit :
> can you do a quick time ffmpeg ... with -codec copy just to double
> check ? theres no major effect

It seems the memory management is not as negligible as I expected. The
result is a ~2.3% slow-down for ~90k frames from AVI, which is approximately
equal to the standard deviation on 10 runs. I suspect this is acceptable
compared to the overhead of more complex containers.

Regards,

-- 
  Nicolas George


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


Re: [FFmpeg-devel] [PATCH 3/4] lavf/concat: switch to new BSF API.

2016-05-19 Thread Michael Niedermayer
On Thu, May 19, 2016 at 10:54:39PM +0200, Nicolas George wrote:
> Le primidi 1er prairial, an CCXXIV, Michael Niedermayer a écrit :
> > is the overhead from the null filter insiginicant ?
> 
> I have not actually made a benchmark, but I believe it is. Look in
> libavcodec/bsf.c: the overhead is limited to a few checks and a
> av_packet_move_ref().

can you do a quick time ffmpeg ... with -codec copy just to double
check ? theres no major effect


[...]


-- 
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Dictatorship: All citizens are under surveillance, all their steps and
actions recorded, for the politicians to enforce control.
Democracy: All politicians are under surveillance, all their steps and
actions recorded, for the citizens to enforce control.


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


Re: [FFmpeg-devel] [PATCH 3/4] lavf/concat: switch to new BSF API.

2016-05-19 Thread Nicolas George
Le primidi 1er prairial, an CCXXIV, Michael Niedermayer a écrit :
> is the overhead from the null filter insiginicant ?

I have not actually made a benchmark, but I believe it is. Look in
libavcodec/bsf.c: the overhead is limited to a few checks and a
av_packet_move_ref().

Regards,

-- 
  Nicolas George


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