[FFmpeg-devel] [PATCH] split mpegvideo.c
Michael Niedermayer
michaelni
Mon Jul 2 17:23:52 CEST 2007
Hi
On Mon, Jul 02, 2007 at 12:18:21PM +0200, Benoit Fouet wrote:
> Hi,
>
> Michael Niedermayer wrote:
> > Hi
> >
> > On Thu, Jun 28, 2007 at 01:38:23PM +0200, Benoit Fouet wrote:
> >
> >> Benoit Fouet wrote:
> >>
> >>> M?ns Rullg?rd wrote:
> >>>
> >>>
> >>>> Benoit Fouet <benoit.fouet at purplelabs.com> writes:
> >>>>
> >>>>
> >>>>
> >>>>> Hi,
> >>>>>
> >>>>> here is a first attempt to $subj
> >>>>>
> >>>>> what is done:
> >>>>> - remove everything flagged under CONFIG_ENCODERS from mpegvideo.c
> >>>>> (apart from the DCT_common_init() part)
> >>>>> - put those parts in mpegvideo_enc.c
> >>>>>
> >>>>>
> >>>>>
> >>>> Does this in any way affect performance? I can imagine calls from
> >>>> encoding functions to common functions no longer being inlined due to
> >>>> the split. Then again, there might not be any relevant such calls.
> >>>>
> >>>>
> >>>>
> >>>>
> >>> i didn't check performances yet. but this is a problem we might have, i
> >>> guess...
> >>>
> >>>
> >>>
> >> anyway, what would be the proper way to do such a benchmark.
> >> As it is something i never did in ffmpeg, any pointer on how to do it
> >> would be very welcome !
> >>
> >
> > nm anychangedobject.o
> >
> > and checking if new function names appear (simply cleaning the output of nm
> > with sed and diffing them works well), if so they are no longer inlined
> > (this will tell us what needs benchmarking)
> >
> >
>
> ok, i did that.
> it seems the only differences are the following ones:
> - copy_picture is now defined
> - MPV_common_defaults too
>
> the two of them were implicitely inlined by the compiler actually.
> i think copy_picture could be moved to the common header file to make it
> inlined again, but this should not be necessary for
> MPV_commmon_defaults, which is only called at initialization of codec.
neither of the 2 should be inlined, so theres no need to move them into
header
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
When you are offended at any man's fault, turn to yourself and study your
own failings. Then you will forget your anger. -- Epictetus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070702/f6147542/attachment.pgp>
More information about the ffmpeg-devel
mailing list