[FFmpeg-devel] [PATCH] Optimization: support for libx264's mb_info

[Carotti, Elias]

On Sun, 2023-06-18 at 12:18 +0200, Stefano Sabatini wrote:
> CAUTION: This email originated from outside of the organization. Do
> not click links or open attachments unless you can confirm the sender
> and know the content is safe.
> 
> 
> 
> On date Monday 2023-06-12 17:28:10 +0000, Carotti, Elias wrote:
> > Hi Stefano,
> > Here is the revised patch according to your suggestions. This
> > should
> > allow for efficient inlining of the methods computing the map of
> > skipped macrobloks.
> > Best,
> > Elias
> [...]
> > 
> > From 0e7979250231edbe0b845cee96c473bc6e07d46b Mon Sep 17 00:00:00
> > 2001
> > From: Elias Carotti <eliascrt _at_ amazon _dot_ it>
> > Date: Wed, 19 Apr 2023 11:49:39 +0200
> > Subject: [PATCH] Add support for libx264's MB_INFO
> > 
> > libx264's x264_image_properties_t, which is passed to the encoding
> > function,
> > contains a field to pass down information on the portions of the
> > frame which
> > changed with respect to the previous one (used for prediction) to
> > mark
> > unchanged macroblocks P_SKIP.
> > ---
> >  libavcodec/libx264.c   | 91
> > ++++++++++++++++++++++++++++++++++++++++++
> >  libavutil/Makefile     |  4 ++
> >  libavutil/frame.h      | 10 +++++
> >  libavutil/video_hint.c | 89
> > +++++++++++++++++++++++++++++++++++++++++
> >  libavutil/video_hint.h | 87
> > ++++++++++++++++++++++++++++++++++++++++
> >  5 files changed, 281 insertions(+)
> >  create mode 100644 libavutil/video_hint.c
> >  create mode 100644 libavutil/video_hint.h
> 
> Note: maybe this should be split in two parts, one about libavutil
> and
> one about libx264. The libavutil part should also contain a minor
> bump
> for the new feature, same for libavcodec (with corresponding changes
> in doc/APIchanges).
> 
> I don't know if we track internal dependencies (to define the minor
> libavutil version needed for the new libx264 feature in
> libavcodec). What's the current practice here?
> 
> Looks good to me otherwise (still it would be good if someone else
> can
> have a look at it).
....

Hi all,
please find the updated patch split into two parts. The first part with
the changes to libavutil is appended at the end of this email, while
the second patch with the changes to libavcodec will be in a separate
mail.
Best, 
Elias 






NICE SRL, viale Monte Grappa 3/5, 20124 Milano, Italia, Registro delle Imprese di Milano Monza Brianza Lodi REA n. 2096882, Capitale Sociale: 10.329,14 EUR i.v., Cod. Fisc. e P.IVA 01133050052, Societa con Socio Unico


-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-lavu-add-AVVideoHint-API.patch
Type: text/x-patch
Size: 10310 bytes
Desc: 0001-lavu-add-AVVideoHint-API.patch
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20230621/cdf02fd6/attachment.bin>