This has been resolved by your 0.34.1 -> 0.35.1 update. Alex
On Sat, 1 Apr 2023 at 20:57, Khem Raj <raj.k...@gmail.com> wrote: > > I think mpv is failing to this upgrade, this issue talks about it > https://github.com/mpv-player/mpv/issues/9803 > https://errors.yoctoproject.org/Errors/Details/699341/ > This is a arm64/musl build. > > On Thu, Mar 30, 2023 at 10:26 PM Alexander Kanavin > <alex.kana...@gmail.com> wrote: > > > > Add a patch to gstreamer1.0-libav to address an API compatibility fail. > > > > Signed-off-by: Alexander Kanavin <a...@linutronix.de> > > --- > > ...c-stop-accessing-out-of-bounds-frame.patch | 89 --------------- > > ...c-stop-accessing-out-of-bounds-frame.patch | 108 ------------------ > > .../ffmpeg/ffmpeg/ffmpeg-fix-vulkan.patch | 34 ------ > > .../ffmpeg/{ffmpeg_5.1.2.bb => ffmpeg_6.0.bb} | 8 +- > > .../gstreamer1.0-libav/ffmpeg-6.0.patch | 49 ++++++++ > > .../gstreamer/gstreamer1.0-libav_1.22.0.bb | 4 +- > > 6 files changed, 54 insertions(+), 238 deletions(-) > > delete mode 100644 > > meta/recipes-multimedia/ffmpeg/ffmpeg/0001-avcodec-rpzaenc-stop-accessing-out-of-bounds-frame.patch > > delete mode 100644 > > meta/recipes-multimedia/ffmpeg/ffmpeg/0001-avcodec-smcenc-stop-accessing-out-of-bounds-frame.patch > > delete mode 100644 > > meta/recipes-multimedia/ffmpeg/ffmpeg/ffmpeg-fix-vulkan.patch > > rename meta/recipes-multimedia/ffmpeg/{ffmpeg_5.1.2.bb => ffmpeg_6.0.bb} > > (95%) > > create mode 100644 > > meta/recipes-multimedia/gstreamer/gstreamer1.0-libav/ffmpeg-6.0.patch > > > > diff --git > > a/meta/recipes-multimedia/ffmpeg/ffmpeg/0001-avcodec-rpzaenc-stop-accessing-out-of-bounds-frame.patch > > > > b/meta/recipes-multimedia/ffmpeg/ffmpeg/0001-avcodec-rpzaenc-stop-accessing-out-of-bounds-frame.patch > > deleted file mode 100644 > > index 2775a81cc86..00000000000 > > --- > > a/meta/recipes-multimedia/ffmpeg/ffmpeg/0001-avcodec-rpzaenc-stop-accessing-out-of-bounds-frame.patch > > +++ /dev/null > > @@ -1,89 +0,0 @@ > > -From 92f9b28ed84a77138105475beba16c146bdaf984 Mon Sep 17 00:00:00 2001 > > -From: Paul B Mahol <one...@gmail.com> > > -Date: Sat, 12 Nov 2022 16:12:00 +0100 > > -Subject: [PATCH] avcodec/rpzaenc: stop accessing out of bounds frame > > - > > -Upstream-Status: Backport > > [https://github.com/FFmpeg/FFmpeg/commit/92f9b28ed84a77138105475beba16c146bdaf984] > > - > > -Signed-off-by: <narpat.m...@windriver.com> > > - > > ---- > > - libavcodec/rpzaenc.c | 22 +++++++++++++++------- > > - 1 file changed, 15 insertions(+), 7 deletions(-) > > - > > -diff --git a/libavcodec/rpzaenc.c b/libavcodec/rpzaenc.c > > -index d710eb4f82..4ced9523e2 100644 > > ---- a/libavcodec/rpzaenc.c > > -+++ b/libavcodec/rpzaenc.c > > -@@ -205,7 +205,7 @@ static void get_max_component_diff(const BlockInfo > > *bi, const uint16_t *block_pt > > - > > - // loop thru and compare pixels > > - for (y = 0; y < bi->block_height; y++) { > > -- for (x = 0; x < bi->block_width; x++){ > > -+ for (x = 0; x < bi->block_width; x++) { > > - // TODO: optimize > > - min_r = FFMIN(R(block_ptr[x]), min_r); > > - min_g = FFMIN(G(block_ptr[x]), min_g); > > -@@ -278,7 +278,7 @@ static int leastsquares(const uint16_t *block_ptr, > > const BlockInfo *bi, > > - return -1; > > - > > - for (i = 0; i < bi->block_height; i++) { > > -- for (j = 0; j < bi->block_width; j++){ > > -+ for (j = 0; j < bi->block_width; j++) { > > - x = GET_CHAN(block_ptr[j], xchannel); > > - y = GET_CHAN(block_ptr[j], ychannel); > > - sumx += x; > > -@@ -325,7 +325,7 @@ static int calc_lsq_max_fit_error(const uint16_t > > *block_ptr, const BlockInfo *bi > > - int max_err = 0; > > - > > - for (i = 0; i < bi->block_height; i++) { > > -- for (j = 0; j < bi->block_width; j++){ > > -+ for (j = 0; j < bi->block_width; j++) { > > - int x_inc, lin_y, lin_x; > > - x = GET_CHAN(block_ptr[j], xchannel); > > - y = GET_CHAN(block_ptr[j], ychannel); > > -@@ -420,7 +420,9 @@ static void update_block_in_prev_frame(const uint16_t > > *src_pixels, > > - uint16_t *dest_pixels, > > - const BlockInfo *bi, int > > block_counter) > > - { > > -- for (int y = 0; y < 4; y++) { > > -+ const int y_size = FFMIN(4, bi->image_height - bi->row * 4); > > -+ > > -+ for (int y = 0; y < y_size; y++) { > > - memcpy(dest_pixels, src_pixels, 8); > > - dest_pixels += bi->rowstride; > > - src_pixels += bi->rowstride; > > -@@ -730,14 +732,15 @@ post_skip : > > - > > - if (err > s->sixteen_color_thresh) { // DO SIXTEEN COLOR BLOCK > > - uint16_t *row_ptr; > > -- int rgb555; > > -+ int y_size, rgb555; > > - > > - block_offset = get_block_info(&bi, block_counter); > > - > > - row_ptr = &src_pixels[block_offset]; > > -+ y_size = FFMIN(4, bi.image_height - bi.row * 4); > > - > > -- for (int y = 0; y < 4; y++) { > > -- for (int x = 0; x < 4; x++){ > > -+ for (int y = 0; y < y_size; y++) { > > -+ for (int x = 0; x < 4; x++) { > > - rgb555 = row_ptr[x] & ~0x8000; > > - > > - put_bits(&s->pb, 16, rgb555); > > -@@ -745,6 +748,11 @@ post_skip : > > - row_ptr += bi.rowstride; > > - } > > - > > -+ for (int y = y_size; y < 4; y++) { > > -+ for (int x = 0; x < 4; x++) > > -+ put_bits(&s->pb, 16, 0); > > -+ } > > -+ > > - block_counter++; > > - } else { // FOUR COLOR BLOCK > > - block_counter += encode_four_color_block(min_color, > > max_color, > > --- > > -2.34.1 > > - > > diff --git > > a/meta/recipes-multimedia/ffmpeg/ffmpeg/0001-avcodec-smcenc-stop-accessing-out-of-bounds-frame.patch > > > > b/meta/recipes-multimedia/ffmpeg/ffmpeg/0001-avcodec-smcenc-stop-accessing-out-of-bounds-frame.patch > > deleted file mode 100644 > > index 923fc6a9c1b..00000000000 > > --- > > a/meta/recipes-multimedia/ffmpeg/ffmpeg/0001-avcodec-smcenc-stop-accessing-out-of-bounds-frame.patch > > +++ /dev/null > > @@ -1,108 +0,0 @@ > > -From 13c13109759090b7f7182480d075e13b36ed8edd Mon Sep 17 00:00:00 2001 > > -From: Paul B Mahol <one...@gmail.com> > > -Date: Sat, 12 Nov 2022 15:19:21 +0100 > > -Subject: [PATCH] avcodec/smcenc: stop accessing out of bounds frame > > - > > -Upstream-Status: Backport > > [https://github.com/FFmpeg/FFmpeg/commit/13c13109759090b7f7182480d075e13b36ed8edd] > > - > > -Signed-off-by: <narpat.m...@windriver.com> > > - > > ---- > > - libavcodec/smcenc.c | 18 ++++++++++++++---- > > - 1 file changed, 14 insertions(+), 4 deletions(-) > > - > > -diff --git a/libavcodec/smcenc.c b/libavcodec/smcenc.c > > -index f3d26a4e8d..33549b8ab4 100644 > > ---- a/libavcodec/smcenc.c > > -+++ b/libavcodec/smcenc.c > > -@@ -61,6 +61,7 @@ typedef struct SMCContext { > > - { \ > > - row_ptr += stride * 4; \ > > - pixel_ptr = row_ptr; \ > > -+ cur_y += 4; \ > > - } \ > > - } \ > > - } > > -@@ -117,6 +118,7 @@ static void smc_encode_stream(SMCContext *s, const > > AVFrame *frame, > > - const uint8_t *prev_pixels = (const uint8_t *)s->prev_frame->data[0]; > > - uint8_t *distinct_values = s->distinct_values; > > - const uint8_t *pixel_ptr, *row_ptr; > > -+ const int height = frame->height; > > - const int width = frame->width; > > - uint8_t block_values[16]; > > - int block_counter = 0; > > -@@ -125,13 +127,14 @@ static void smc_encode_stream(SMCContext *s, const > > AVFrame *frame, > > - int color_octet_index = 0; > > - int color_table_index; /* indexes to color pair, quad, or octet > > tables */ > > - int total_blocks; > > -+ int cur_y = 0; > > - > > - memset(s->color_pairs, 0, sizeof(s->color_pairs)); > > - memset(s->color_quads, 0, sizeof(s->color_quads)); > > - memset(s->color_octets, 0, sizeof(s->color_octets)); > > - > > - /* Number of 4x4 blocks in frame. */ > > -- total_blocks = ((frame->width + 3) / 4) * ((frame->height + 3) / 4); > > -+ total_blocks = ((width + 3) / 4) * ((height + 3) / 4); > > - > > - pixel_ptr = row_ptr = src_pixels; > > - > > -@@ -145,11 +148,13 @@ static void smc_encode_stream(SMCContext *s, const > > AVFrame *frame, > > - int cache_index; > > - int distinct = 0; > > - int blocks = 0; > > -+ int frame_y = cur_y; > > - > > - while (prev_pixels && s->key_frame == 0 && block_counter + > > inter_skip_blocks < total_blocks) { > > -+ const int y_size = FFMIN(4, height - cur_y); > > - int compare = 0; > > - > > -- for (int y = 0; y < 4; y++) { > > -+ for (int y = 0; y < y_size; y++) { > > - const ptrdiff_t offset = pixel_ptr - src_pixels; > > - const uint8_t *prev_pixel_ptr = prev_pixels + offset; > > - > > -@@ -170,8 +175,10 @@ static void smc_encode_stream(SMCContext *s, const > > AVFrame *frame, > > - > > - pixel_ptr = xpixel_ptr; > > - row_ptr = xrow_ptr; > > -+ cur_y = frame_y; > > - > > - while (block_counter > 0 && block_counter + intra_skip_blocks < > > total_blocks) { > > -+ const int y_size = FFMIN(4, height - cur_y); > > - const ptrdiff_t offset = pixel_ptr - src_pixels; > > - const int sy = offset / stride; > > - const int sx = offset % stride; > > -@@ -180,7 +187,7 @@ static void smc_encode_stream(SMCContext *s, const > > AVFrame *frame, > > - const uint8_t *old_pixel_ptr = src_pixels + nx + ny * stride; > > - int compare = 0; > > - > > -- for (int y = 0; y < 4; y++) { > > -+ for (int y = 0; y < y_size; y++) { > > - compare |= memcmp(old_pixel_ptr + y * stride, pixel_ptr + > > y * stride, 4); > > - if (compare) > > - break; > > -@@ -197,9 +204,11 @@ static void smc_encode_stream(SMCContext *s, const > > AVFrame *frame, > > - > > - pixel_ptr = xpixel_ptr; > > - row_ptr = xrow_ptr; > > -+ cur_y = frame_y; > > - > > - while (block_counter + coded_blocks < total_blocks && > > coded_blocks < 256) { > > -- for (int y = 0; y < 4; y++) > > -+ const int y_size = FFMIN(4, height - cur_y); > > -+ for (int y = 0; y < y_size; y++) > > - memcpy(block_values + y * 4, pixel_ptr + y * stride, 4); > > - > > - qsort(block_values, 16, sizeof(block_values[0]), > > smc_cmp_values); > > -@@ -224,6 +233,7 @@ static void smc_encode_stream(SMCContext *s, const > > AVFrame *frame, > > - > > - pixel_ptr = xpixel_ptr; > > - row_ptr = xrow_ptr; > > -+ cur_y = frame_y; > > - > > - blocks = coded_blocks; > > - distinct = coded_distinct; > > --- > > -2.34.1 > > - > > diff --git a/meta/recipes-multimedia/ffmpeg/ffmpeg/ffmpeg-fix-vulkan.patch > > b/meta/recipes-multimedia/ffmpeg/ffmpeg/ffmpeg-fix-vulkan.patch > > deleted file mode 100644 > > index 95bd608a277..00000000000 > > --- a/meta/recipes-multimedia/ffmpeg/ffmpeg/ffmpeg-fix-vulkan.patch > > +++ /dev/null > > @@ -1,34 +0,0 @@ > > -From: Lynne <d...@lynne.ee> > > -Date: Sun, 25 Dec 2022 00:03:30 +0000 (+0100) > > -Subject: hwcontext_vulkan: remove optional encode/decode extensions from > > the list > > -X-Git-Url: > > http://git.ffmpeg.org/gitweb/ffmpeg.git/commitdiff_plain/eb0455d64690 > > - > > -hwcontext_vulkan: remove optional encode/decode extensions from the list > > - > > -They're not currently used, so they don't need to be there. > > -Vulkan stabilized the decode extensions less than a week ago, and their > > -name prefixes were changed from EXT to KHR. It's a bit too soon to be > > -depending on it, so rather than bumping, just remove these for now. > > - > > -Upstream-Status: Backport > > [https://git.ffmpeg.org/gitweb/ffmpeg.git/commitdiff_plain/eb0455d64690] > > ---- > > - > > -diff --git a/libavutil/hwcontext_vulkan.c b/libavutil/hwcontext_vulkan.c > > -index f1db1c7291..2a9b5f4aac 100644 > > ---- a/libavutil/hwcontext_vulkan.c > > -+++ b/libavutil/hwcontext_vulkan.c > > -@@ -358,14 +358,6 @@ static const VulkanOptExtension > > optional_device_exts[] = { > > - { VK_KHR_EXTERNAL_MEMORY_WIN32_EXTENSION_NAME, > > FF_VK_EXT_EXTERNAL_WIN32_MEMORY }, > > - { VK_KHR_EXTERNAL_SEMAPHORE_WIN32_EXTENSION_NAME, > > FF_VK_EXT_EXTERNAL_WIN32_SEM }, > > - #endif > > -- > > -- /* Video encoding/decoding */ > > -- { VK_KHR_VIDEO_QUEUE_EXTENSION_NAME, > > FF_VK_EXT_NO_FLAG }, > > -- { VK_KHR_VIDEO_DECODE_QUEUE_EXTENSION_NAME, > > FF_VK_EXT_NO_FLAG }, > > -- { VK_KHR_VIDEO_ENCODE_QUEUE_EXTENSION_NAME, > > FF_VK_EXT_NO_FLAG }, > > -- { VK_EXT_VIDEO_ENCODE_H264_EXTENSION_NAME, > > FF_VK_EXT_NO_FLAG }, > > -- { VK_EXT_VIDEO_DECODE_H264_EXTENSION_NAME, > > FF_VK_EXT_NO_FLAG }, > > -- { VK_EXT_VIDEO_DECODE_H265_EXTENSION_NAME, > > FF_VK_EXT_NO_FLAG }, > > - }; > > - > > - /* Converts return values to strings */ > > diff --git a/meta/recipes-multimedia/ffmpeg/ffmpeg_5.1.2.bb > > b/meta/recipes-multimedia/ffmpeg/ffmpeg_6.0.bb > > similarity index 95% > > rename from meta/recipes-multimedia/ffmpeg/ffmpeg_5.1.2.bb > > rename to meta/recipes-multimedia/ffmpeg/ffmpeg_6.0.bb > > index cccd9f65abf..e4a4a0effa1 100644 > > --- a/meta/recipes-multimedia/ffmpeg/ffmpeg_5.1.2.bb > > +++ b/meta/recipes-multimedia/ffmpeg/ffmpeg_6.0.bb > > @@ -22,13 +22,9 @@ LIC_FILES_CHKSUM = > > "file://COPYING.GPLv2;md5=b234ee4d69f5fce4486a80fdaf4a4263 \ > > > > file://COPYING.LGPLv2.1;md5=bd7a443320af8c812e4c18d1b79df004 \ > > > > file://COPYING.LGPLv3;md5=e6a600fd5e1d9cbde2d983680233ad02" > > > > -SRC_URI = "https://www.ffmpeg.org/releases/${BP}.tar.xz \ > > - > > file://0001-avcodec-rpzaenc-stop-accessing-out-of-bounds-frame.patch \ > > - > > file://0001-avcodec-smcenc-stop-accessing-out-of-bounds-frame.patch \ > > - file://ffmpeg-fix-vulkan.patch \ > > - " > > +SRC_URI = "https://www.ffmpeg.org/releases/${BP}.tar.xz"; > > > > -SRC_URI[sha256sum] = > > "619e706d662c8420859832ddc259cd4d4096a48a2ce1eefd052db9e440eef3dc" > > +SRC_URI[sha256sum] = > > "57be87c22d9b49c112b6d24bc67d42508660e6b718b3db89c44e47e289137082" > > > > # Build fails when thumb is enabled: > > https://bugzilla.yoctoproject.org/show_bug.cgi?id=7717 > > ARM_INSTRUCTION_SET:armv4 = "arm" > > diff --git > > a/meta/recipes-multimedia/gstreamer/gstreamer1.0-libav/ffmpeg-6.0.patch > > b/meta/recipes-multimedia/gstreamer/gstreamer1.0-libav/ffmpeg-6.0.patch > > new file mode 100644 > > index 00000000000..0a06540fb47 > > --- /dev/null > > +++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0-libav/ffmpeg-6.0.patch > > @@ -0,0 +1,49 @@ > > +From cde31d23c071ee93fae96331805f696856084254 Mon Sep 17 00:00:00 2001 > > +From: "U. Artie Eoff" <ullysses.a.e...@intel.com> > > +Date: Mon, 13 Feb 2023 17:02:01 -0500 > > +Subject: [PATCH] avviddec: change > > + AV_CODEC_CAP_AUTO_THREADS->AV_CODEC_CAP_OTHER_THREADS > > + > > +This fixes a compile error with recent upstream FFmpeg. > > + > > +The AV_CODEC_CAP_AUTO_THREADS was deprecated and renamed to > > +AV_CODEC_CAP_OTHER_THREADS in FFmpeg upstream commit > > +7d09579190de (lavc 58.132.100). > > + > > +The AV_CODEC_CAP_AUTO_THREADS was finally removed in FFmpeg upstream > > +commit 10c9a0874cb3 (lavc 59.63.100). > > + > > +Part-of: > > <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3964> > > + > > +Upstream-Status: Backport > > [https://gitlab.freedesktop.org/gstreamer/gstreamer/-/commit/cde31d23c071ee93fae96331805f696856084254?merge_request_iid=3964] > > +Signed-off-by: Alexander Kanavin <a...@linutronix.de> > > +--- > > + ext/libav/gstavviddec.c | 6 +++++- > > + 1 file changed, 5 insertions(+), 1 deletion(-) > > + > > +diff --git a/ext/libav/gstavviddec.c b/ext/libav/gstavviddec.c > > +index 43cea456ae8..6d7c4cd0de8 100644 > > +--- a/ext/libav/gstavviddec.c > > ++++ b/ext/libav/gstavviddec.c > > +@@ -35,6 +35,10 @@ > > + > > + GST_DEBUG_CATEGORY_STATIC (GST_CAT_PERFORMANCE); > > + > > ++#if LIBAVCODEC_VERSION_INT < AV_VERSION_INT(58,132,100) > > ++#define AV_CODEC_CAP_OTHER_THREADS AV_CODEC_CAP_AUTO_THREADS > > ++#endif > > ++ > > + #define GST_FFMPEG_VIDEO_CODEC_FRAME_FLAG_ALLOCATED (1<<15) > > + > > + #define MAX_TS_MASK 0xff > > +@@ -615,7 +619,7 @@ gst_ffmpegviddec_set_format (GstVideoDecoder * decoder, > > + if (ffmpegdec->max_threads == 0) { > > + /* When thread type is FF_THREAD_FRAME, extra latency is introduced > > equal > > + * to one frame per thread. We thus need to calculate the thread > > count ourselves */ > > +- if ((!(oclass->in_plugin->capabilities & AV_CODEC_CAP_AUTO_THREADS)) > > || > > ++ if ((!(oclass->in_plugin->capabilities & AV_CODEC_CAP_OTHER_THREADS)) > > || > > + (ffmpegdec->context->thread_type & FF_THREAD_FRAME)) > > + ffmpegdec->context->thread_count = > > + MIN (gst_ffmpeg_auto_max_threads (), 16); > > +-- > > +GitLab > > diff --git a/meta/recipes-multimedia/gstreamer/gstreamer1.0-libav_1.22.0.bb > > b/meta/recipes-multimedia/gstreamer/gstreamer1.0-libav_1.22.0.bb > > index 625a52ea555..d1dce6739dd 100644 > > --- a/meta/recipes-multimedia/gstreamer/gstreamer1.0-libav_1.22.0.bb > > +++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0-libav_1.22.0.bb > > @@ -11,7 +11,9 @@ LIC_FILES_CHKSUM = > > "file://COPYING;md5=69333daa044cb77e486cc36129f7a770 \ > > > > file://ext/libav/gstav.h;beginline=1;endline=18;md5=a752c35267d8276fd9ca3db6994fca9c > > \ > > " > > > > -SRC_URI = > > "https://gstreamer.freedesktop.org/src/gst-libav/gst-libav-${PV}.tar.xz"; > > +SRC_URI = > > "https://gstreamer.freedesktop.org/src/gst-libav/gst-libav-${PV}.tar.xz \ > > + file://ffmpeg-6.0.patch \ > > + " > > SRC_URI[sha256sum] = > > "0e48407b4905227a260213dbda84cba3812f0530fc7a75b43829102ef82810f1" > > > > S = "${WORKDIR}/gst-libav-${PV}" > > -- > > 2.30.2 > > > > > > > >
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#179930): https://lists.openembedded.org/g/openembedded-core/message/179930 Mute This Topic: https://lists.openembedded.org/mt/97967685/21656 Group Owner: openembedded-core+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
