From: James Almer <jamrial@gmail.com> To: ffmpeg-devel@ffmpeg.org Subject: [FFmpeg-devel] [PATCH v2 1/6] avutil: add an API to handle Three Dimensional Reference Displays Information Date: Mon, 3 Feb 2025 19:35:41 -0300 Message-ID: <20250203223546.12224-1-jamrial@gmail.com> (raw) As defined in section G.14.3.2.3 of ITU-T H.265, it's required for proper signaling of MV-HEVC. Signed-off-by: James Almer <jamrial@gmail.com> --- Better namespace, and now only allocating the required display reference entries. libavutil/Makefile | 2 + libavutil/ref_displays_info.c | 53 +++++++++++ libavutil/ref_displays_info.h | 169 ++++++++++++++++++++++++++++++++++ 3 files changed, 224 insertions(+) create mode 100644 libavutil/ref_displays_info.c create mode 100644 libavutil/ref_displays_info.h diff --git a/libavutil/Makefile b/libavutil/Makefile index f8031815bd..67e942243e 100644 --- a/libavutil/Makefile +++ b/libavutil/Makefile @@ -76,6 +76,7 @@ HEADERS = adler32.h \ random_seed.h \ rc4.h \ rational.h \ + ref_displays_info.h \ refstruct.h \ replaygain.h \ ripemd.h \ @@ -166,6 +167,7 @@ OBJS = adler32.o \ pixelutils.o \ random_seed.o \ rational.o \ + ref_displays_info.o \ refstruct.o \ reverse.o \ rc4.o \ diff --git a/libavutil/ref_displays_info.c b/libavutil/ref_displays_info.c new file mode 100644 index 0000000000..a51aa42b54 --- /dev/null +++ b/libavutil/ref_displays_info.c @@ -0,0 +1,53 @@ +/* + * This file is part of FFmpeg. + * + * FFmpeg is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public + * License as published by the Free Software Foundation; either + * version 2.1 of the License, or (at your option) any later version. + * + * FFmpeg is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public + * License along with FFmpeg; if not, write to the Free Software + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA + */ + +#include <stddef.h> +#include <stdint.h> + +#include "avassert.h" +#include "mem.h" +#include "ref_displays_info.h" + +AV3DReferenceDisplaysInfo *av_ref_displays_info_alloc(unsigned int num_ref_displays, + size_t *out_size) +{ + struct CombinedStruct { + AV3DReferenceDisplaysInfo i; + AV3DReferenceDisplay r; + }; + const size_t ref_offset = offsetof(struct CombinedStruct, r); + size_t size = ref_offset; + AV3DReferenceDisplaysInfo *rdi; + + if (num_ref_displays > (SIZE_MAX - size) / sizeof(AV3DReferenceDisplay)) + return NULL; + size += sizeof(AV3DReferenceDisplay) * num_ref_displays; + + rdi = av_mallocz(size); + if (!rdi) + return NULL; + + rdi->num_ref_displays = num_ref_displays; + rdi->ref_size = sizeof(AV3DReferenceDisplay); + rdi->ref_offset = ref_offset; + + if (out_size) + *out_size = size; + + return rdi; +} diff --git a/libavutil/ref_displays_info.h b/libavutil/ref_displays_info.h new file mode 100644 index 0000000000..02ff4fcd98 --- /dev/null +++ b/libavutil/ref_displays_info.h @@ -0,0 +1,169 @@ +/* + * This file is part of FFmpeg. + * + * FFmpeg is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public + * License as published by the Free Software Foundation; either + * version 2.1 of the License, or (at your option) any later version. + * + * FFmpeg is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public + * License along with FFmpeg; if not, write to the Free Software + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA + */ + +/** + * @file + * @ingroup lavu_video_3d_reference_displays_info + * 3D Reference Displays Information + */ + +#ifndef AVUTIL_REF_DISPLAYS_INFO_H +#define AVUTIL_REF_DISPLAYS_INFO_H + +#include <stddef.h> +#include <stdint.h> + +#include "libavutil/avassert.h" + +/** + * @defgroup lavu_video_3d_reference_displays_info 3D Reference Displays Information + * @ingroup lavu_video + * + * The 3D Reference Displays Information describes information about the reference display + * width(s) and reference viewing distance(s) as well as information about the corresponding + * reference stereo pair(s). + * @{ + */ + +/** + * This structure describes information about the reference display width(s) and reference + * viewing distance(s) as well as information about the corresponding reference stereo pair(s). + * See section G.14.3.2.3 of ITU-T H.265 for more information. + * + * @note The struct must be allocated with av_ref_displays_info_alloc() and its size is not a + * part of the public ABI. + */ +typedef struct AV3DReferenceDisplaysInfo { + /** + * The number of reference displays that are signalled in this struct. + * Allowed range is 1 to 32, inclusive. + */ + unsigned int num_ref_displays; + + /** + * Offset in bytes from the beginning of this structure at which the array + * of reference displays starts. + */ + size_t ref_offset; + + /** + * Size of each reference display entry in bytes. May not match sizeof(AV3DReferenceDisplay). + */ + size_t ref_size; + + /** + * The exponent of the maximum allowable truncation error for + * {exponent,mantissa}_ref_display_width as given by 2<sup>(-prec_ref_display_width)</sup>. + */ + uint8_t prec_ref_display_width; + + /** + * A flag to indicate the presence of reference viewing distance. + * If false, the values of prec_ref_viewing_dist, exponent_ref_viewing_distance, + * and mantissa_ref_viewing_distance are undefined. + */ + uint8_t ref_viewing_distance_flag; + + /** + * The exponent of the maximum allowable truncation error for + * {exponent,mantissa}_ref_viewing_distance as given by 2<sup>^(-prec_ref_viewing_dist)</sup>. + * The value of prec_ref_viewing_dist shall be in the range of 0 to 31, inclusive. + */ + uint8_t prec_ref_viewing_dist; +} AV3DReferenceDisplaysInfo; + +/** + * Data structure for storing a reference display information. + * It is allocated as a part of AV3DReferenceDisplaysInfo and should be retrieved with + * av_ref_displays_info_display(). + * + * sizeof(AV3DReferenceDisplay) is not a part of the ABI and new fields may be + * added to it. + */ +typedef struct AV3DReferenceDisplay { + /** + * The ViewId of the left view of a stereo pair corresponding to the n-th reference display. + */ + uint16_t left_view_id; + + /** + * The ViewId of the left view of a stereo pair corresponding to the n-th reference display. + */ + uint16_t right_view_id; + + /** + * The exponent part of the reference display width of the n-th reference display. + */ + uint8_t exponent_ref_display_width; + + /** + * The mantissa part of the reference display width of the n-th reference display. + */ + uint8_t mantissa_ref_display_width; + + /** + * Tthe exponent part of the reference viewing distance of the n-th reference display. + */ + uint8_t exponent_ref_viewing_distance; + + /** + * The mantissa part of the reference viewing distance of the n-th reference display. + */ + uint8_t mantissa_ref_viewing_distance; + + /** + * An array of flags to indicates that the information about additional horizontal shift of + * the left and right views for the n-th reference display is present. + */ + uint8_t additional_shift_present_flag; + + /** + * The recommended additional horizontal shift for a stereo pair corresponding to the n-th + * reference baseline and the n-th reference display. + */ + int16_t num_sample_shift; +} AV3DReferenceDisplay; + +/** + * Get the reference display at the specified {@code idx}. + * Must be between 0 and num_ref_displays - 1. + */ +static av_always_inline AV3DReferenceDisplay* +av_ref_displays_info_display(AV3DReferenceDisplaysInfo *rdi, unsigned int idx) +{ + av_assert0(idx < rdi->num_ref_displays); + return (AV3DReferenceDisplay *)((uint8_t *)rdi + rdi->ref_offset + + idx * rdi->ref_size); +} + +/** + * Allocates memory for AV3DReferenceDisplaysInfo, plus an array of + * {@code num_ref_displays} AV3DReferenceDisplay and initializes the variables. + * Can be freed with a normal av_free() call. + * + * @param num_ref_displays the number of reference displays. + * @return the newly allocated struct or NULL on failure + */ +AV3DReferenceDisplaysInfo *av_ref_displays_info_alloc(unsigned int num_ref_displays, + size_t *out_size); + +/** + * @} + */ + +#endif /* AVUTIL_REF_DISPLAYS_INFO_H */ -- 2.48.1 _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
next reply other threads:[~2025-02-03 22:36 UTC|newest] Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top 2025-02-03 22:35 James Almer [this message] 2025-02-03 22:35 ` [FFmpeg-devel] [PATCH v2 2/6] avutil/frame: add a 3D Reference Displays Information side data type James Almer 2025-02-03 22:35 ` [FFmpeg-devel] [PATCH v2 3/6] avfilter/vf_showinfo: add support for 3D Reference Displays Information side data James Almer 2025-02-03 22:35 ` [FFmpeg-devel] [PATCH 4/6] avcodec/hevc/sei: ensure num_ref_displays is not set unless the SEI message is valid James Almer 2025-02-03 22:35 ` [FFmpeg-devel] [PATCH 5/6] avcodec/hevc/sei: dynamically allocate 3D Reference Displays Information SEI messages James Almer 2025-02-06 1:56 ` Michael Niedermayer 2025-02-06 2:50 ` [FFmpeg-devel] [PATCH v2 " James Almer 2025-02-03 22:35 ` [FFmpeg-devel] [PATCH v2 6/6] avcodec/hevc/hevcdec: export 3D Reference Displays side data James Almer 2025-02-05 21:07 ` Michael Niedermayer 2025-02-05 21:16 ` James Almer 2025-02-05 21:36 ` Michael Niedermayer
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20250203223546.12224-1-jamrial@gmail.com \ --to=jamrial@gmail.com \ --cc=ffmpeg-devel@ffmpeg.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel This inbox may be cloned and mirrored by anyone: git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git # If you have public-inbox 1.1+ installed, you may # initialize and index your mirror using the following commands: public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \ ffmpegdev@gitmailbox.com public-inbox-index ffmpegdev Example config snippet for mirrors. AGPL code for this site: git clone https://public-inbox.org/public-inbox.git