[FFmpeg-devel] [PATCH 06/18] postproc/postprocess: Don't generally include arch-specific headers

2024-03-28 Thread Andreas Rheinhardt
Signed-off-by: Andreas Rheinhardt --- libpostproc/postprocess.c | 3 ++- libpostproc/postprocess_template.c | 5 - 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/libpostproc/postprocess.c b/libpostproc/postprocess.c index 0586e458b4..8f4de8b1e1 100644 --- a/libpostpro

[FFmpeg-devel] [PATCH 07/18] swscale/swscale_internal: Only include altivec header iff HAVE_ALTIVEC

2024-03-28 Thread Andreas Rheinhardt
Signed-off-by: Andreas Rheinhardt --- libswscale/swscale_internal.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libswscale/swscale_internal.h b/libswscale/swscale_internal.h index abeebbb002..2f6cc70946 100644 --- a/libswscale/swscale_internal.h +++ b/libswscale/swscale_internal.h @@ -3

[FFmpeg-devel] [PATCH 08/18] avcodec/msmpeg4: Don't include x86-specific header unconditionally

2024-03-28 Thread Andreas Rheinhardt
Signed-off-by: Andreas Rheinhardt --- libavcodec/msmpeg4.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/libavcodec/msmpeg4.c b/libavcodec/msmpeg4.c index a2c4c57728..e327bf36a7 100644 --- a/libavcodec/msmpeg4.c +++ b/libavcodec/msmpeg4.c @@ -27,13 +27,17 @@ * MSMPEG4

[FFmpeg-devel] [PATCH 09/18] swscale/swscale_internal: Hoist branch out of loop

2024-03-28 Thread Andreas Rheinhardt
Signed-off-by: Andreas Rheinhardt --- I wonder whether one can use aligned writes here? libswscale/swscale_internal.h | 34 ++ 1 file changed, 10 insertions(+), 24 deletions(-) diff --git a/libswscale/swscale_internal.h b/libswscale/swscale_internal.h index 2f6cc

[FFmpeg-devel] [PATCH 10/18] swscale/swscale_internal: Don't export internal function

2024-03-28 Thread Andreas Rheinhardt
sws_alloc_set_opts() can actually be made internal to utils.c. This commit does so. Signed-off-by: Andreas Rheinhardt --- libswscale/swscale_internal.h | 11 - libswscale/utils.c| 83 ++- 2 files changed, 43 insertions(+), 51 deletions(-) diff --g

[FFmpeg-devel] [PATCH 11/18] lib*/version: Use static_assert for static asserts

2024-03-28 Thread Andreas Rheinhardt
Also update the checks that guard against inserting a new enum entry in the middle of a range. Signed-off-by: Andreas Rheinhardt --- libavcodec/version.c| 16 +++- libavdevice/version.c | 5 +++-- libavfilter/version.c | 5 +++-- libavformat/version.c | 5 +++-- libavuti

[FFmpeg-devel] [PATCH 13/18] avcodec/arm/mpegvideo_arm: Use static_assert to check offsets

2024-03-28 Thread Andreas Rheinhardt
Also move AV_CHECK_OFFSET to its only user, namely lavc/arm/mpegvideo_arm.c and rename it to CHECK_OFFSET. Signed-off-by: Andreas Rheinhardt --- libavcodec/arm/mpegvideo_arm.c | 20 libavutil/internal.h | 8 2 files changed, 12 insertions(+), 16 deletions

[FFmpeg-devel] [PATCH 14/18] avutil/internal: Move libm inclusion to the beginning

2024-03-28 Thread Andreas Rheinhardt
Signed-off-by: Andreas Rheinhardt --- libavutil/internal.h | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/libavutil/internal.h b/libavutil/internal.h index e0b2c74b21..5772b8a612 100644 --- a/libavutil/internal.h +++ b/libavutil/internal.h @@ -40,6 +40,7 @@ #include #inc

[FFmpeg-devel] [PATCH 15/18] avutil/internal: Move FF_MEMORY_POISON to its only user

2024-03-28 Thread Andreas Rheinhardt
Namely mem.c. Signed-off-by: Andreas Rheinhardt --- libavutil/internal.h | 2 -- libavutil/mem.c | 2 ++ 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/libavutil/internal.h b/libavutil/internal.h index 5772b8a612..fe0b9d7a24 100644 --- a/libavutil/internal.h +++ b/libavutil/

[FFmpeg-devel] [PATCH 16/18] avutil/hwcontext_vulkan: Include hwcontext.h

2024-03-28 Thread Andreas Rheinhardt
struct Foo * declares a new type (namely struct Foo) if there is no declaration of struct Foo already visible in the current scope; otherwise it is just a pointer to an element of the already declared type "struct Foo". There is a gotcha with the first case: struct Foo is only declared in its scope

[FFmpeg-devel] [PATCH 17/18] avutil/internal: Move avpriv_set_systematic_pal2 decl to imgutils_internal.h

2024-03-28 Thread Andreas Rheinhardt
Signed-off-by: Andreas Rheinhardt --- libavcodec/bmpenc.c | 2 +- libavcodec/gif.c | 1 + libavcodec/pcxenc.c | 2 +- libavcodec/xwdenc.c | 1 + libavfilter/framepool.c | 1 + libavfilter/vf_scale.c| 1 + libavformat/av1.c | 1 +

[FFmpeg-devel] [PATCH 18/18] avcodec/arm/vp8: Don't discard const

2024-03-28 Thread Andreas Rheinhardt
Forgotten in 25e1986e6816fabf7f6065d090994f8ad050e540. Signed-off-by: Andreas Rheinhardt --- libavcodec/arm/vp8.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavcodec/arm/vp8.h b/libavcodec/arm/vp8.h index 7c59a7d63d..6f128ffebe 100644 --- a/libavcodec/arm/vp8.h +++ b

[FFmpeg-devel] [PATCH 2/4] avcodec/options_table: map IPT-C2, YCgCo-R variants in colorspace

2024-03-28 Thread Jan Ekström
--- libavcodec/options_table.h | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavcodec/options_table.h b/libavcodec/options_table.h index 7a2ef3474e..7a70fa7b6c 100644 --- a/libavcodec/options_table.h +++ b/libavcodec/options_table.h @@ -331,8 +331,11 @@ static const AVOption avcodec_opt

[FFmpeg-devel] [PATCH 1/4] avutil/pix{desc, fmt}: add new matrix coefficients from H.273 v3

2024-03-28 Thread Jan Ekström
* SMPTE ST 2128 IPT-C2 defines the coefficients utilized in DoVi Profile 5. Profile 5 can thus now be represented in VUI as {AVCOL_RANGE_JPEG, AVCOL_PRI_BT2020, AVCOL_TRC_SMPTE2084, AVCOL_SPC_IPT_C2, AVCHROMA_LOC_LEFT} (although other chroma sample locations are allowed). AVCOL_TRC_SMPTE20

[FFmpeg-devel] [PATCH 3/4] avfilter/{buffersrc, vf_setparams}: map IPT-C2, YCgCo-R variants

2024-03-28 Thread Jan Ekström
--- libavfilter/buffersrc.c| 3 +++ libavfilter/vf_setparams.c | 3 +++ 2 files changed, 6 insertions(+) diff --git a/libavfilter/buffersrc.c b/libavfilter/buffersrc.c index ddcd403785..680eedf25b 100644 --- a/libavfilter/buffersrc.c +++ b/libavfilter/buffersrc.c @@ -337,12 +337,15 @@ static

[FFmpeg-devel] [PATCH 4/4] avcodec/libx265: do not arbitrarily limit color values

2024-03-28 Thread Jan Ekström
Newer specifications such as H.273 have, and probably will further in the future add new values to these, so - instead of trying to update these limits - we should simply check if the values are not set to the value of "unspecified". This should allow newer avutil values such as IPT-C2 or YCgCo-R

[FFmpeg-devel] [PATCH] Revert "avformat/mov: ignore item boxes for animated heif"

2024-03-28 Thread James Almer
This reverts commit f6b7b473d456a6aa1c063c4261b17277e2c70ac0. The image in the item boxes and the animation in the trak box are not necessarely the same, so both should be exported. Signed-off-by: James Almer --- libavformat/mov.c | 44 1 file changed

[FFmpeg-devel] [PATCH v1] lavc/vaapi_encode: Add VAAPI version check for BLBRC

2024-03-28 Thread fei . w . wang-at-intel . com
From: Fei Wang Fix build fail when VAAPI version less than 0.39.2. Signed-off-by: Fei Wang --- libavcodec/vaapi_encode.c | 20 +--- 1 file changed, 17 insertions(+), 3 deletions(-) diff --git a/libavcodec/vaapi_encode.c b/libavcodec/vaapi_encode.c index 940f0678a5..c4b5411e68

[FFmpeg-devel] [PATCH 1/5] avcodec/ppc/hpeldsp_altivec: Fix left-shift of negative number

2024-03-28 Thread Andreas Rheinhardt
It is UB and affected e.g. the vp5 and vp61 FATE tests: https://fate.ffmpeg.org/report.cgi?time=20240327083327&slot=ppc-linux-gcc-13.2-ubsan-altivec-qemu Signed-off-by: Andreas Rheinhardt --- libavcodec/ppc/hpeldsp_altivec.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a

[FFmpeg-devel] [PATCH 2/5] fate/ffmpeg: Explicitly set pix fmt for sub2video tests

2024-03-28 Thread Andreas Rheinhardt
Otherwise the test would use bgra on little endian and argb on big endian. Signed-off-by: Andreas Rheinhardt --- tests/fate/ffmpeg.mak | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/fate/ffmpeg.mak b/tests/fate/ffmpeg.mak index 3c549b265e..fda3a29239 100644 --- a/t

[FFmpeg-devel] [PATCH 3/5] avcodec/pngdsp: Fix unaligned accesses, effective type violations

2024-03-28 Thread Andreas Rheinhardt
Affected the lscr fate-test (only visible on x86 if the SSE2 is disabled). Signed-off-by: Andreas Rheinhardt --- libavcodec/pngdsp.c | 25 +++-- 1 file changed, 19 insertions(+), 6 deletions(-) diff --git a/libavcodec/pngdsp.c b/libavcodec/pngdsp.c index 65916b1386..50ee96a6

[FFmpeg-devel] [PATCH 4/5] avfilter/vf_spp: Fix left-shift of negative value

2024-03-28 Thread Andreas Rheinhardt
Affected the vf-spp FATE-test (on x86 only when MMX is disabled). Signed-off-by: Andreas Rheinhardt --- libavfilter/vf_spp.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavfilter/vf_spp.c b/libavfilter/vf_spp.c index 5c6495612b..c8366ae319 100644 --- a/libavfilter/v

[FFmpeg-devel] [PATCH 5/5] avcodec/huffyuvencdsp: Fix load of misaligned values

2024-03-28 Thread Andreas Rheinhardt
Affected many ffvhuff FATE tests. Signed-off-by: Andreas Rheinhardt --- libavcodec/huffyuvencdsp.c | 32 1 file changed, 24 insertions(+), 8 deletions(-) diff --git a/libavcodec/huffyuvencdsp.c b/libavcodec/huffyuvencdsp.c index 36e8f6130b..27428635af 100644 ---

[FFmpeg-devel] [PATCH v2] avcodec/ac3: Implement float_to_fixed24 for aarch64 neon

2024-03-28 Thread Geoff Hill
Start porting AC-3 ARMv7 NEON algorithms over to aarch64. This one is low-hanging fruit since checkasm tests exist. Fixed the Makefile compared to v1. Tested on AWS Graviton2 (t4g.medium), GCC 12.3: $ tests/checkasm/checkasm --verbose --bench --test=ac3dsp ... NEON: - ac3dsp.float_to_fixed24 [

[FFmpeg-devel] [PATCH v2] lavc/libopenh264: Check for noopenh264

2024-03-28 Thread Akihiko Odaki
noopenh264 is a "fake implementation of the OpenH264 library we can link from regardless of the actual library being available": https://gitlab.com/freedesktop-sdk/noopenh264 A distributor may wish to link against openh264/noopenh264 and let the decoder and encoder work only if the actual library

<    1   2