Nicolas Chauvet
2 years ago
commit
4f292e56ed
@ -0,0 +1,149 @@
|
|||||||
|
From 55b24abf7219a0c6a6560187496e41fd60638552 Mon Sep 17 00:00:00 2001
|
||||||
|
From: Steve Lhomme <robux4@ycbcr.xyz>
|
||||||
|
Date: Fri, 18 Mar 2022 11:42:49 +0100
|
||||||
|
Subject: [PATCH 1/2] dav1d: fix compilation with (upcoming) dav1d 1.0
|
||||||
|
|
||||||
|
(cherry picked from commit dbf45cea2a8abdfbef897b8a71f3eb782bb1b712) (edited)
|
||||||
|
edited:
|
||||||
|
- 3.0 has the 128 pixels padding elsewhere
|
||||||
|
- 3.0 has an extra parameter for add_integer_with_range()
|
||||||
|
- 3.0 was setting i_extra_picture_buffers further down in the code
|
||||||
|
- 3.0 uses 16 threads max
|
||||||
|
|
||||||
|
Signed-off-by: Steve Lhomme <robux4@ycbcr.xyz>
|
||||||
|
---
|
||||||
|
modules/codec/dav1d.c | 22 +++++++++++++++++++++-
|
||||||
|
1 file changed, 21 insertions(+), 1 deletion(-)
|
||||||
|
|
||||||
|
diff --git a/modules/codec/dav1d.c b/modules/codec/dav1d.c
|
||||||
|
index 039165f52e..cfabbc27cb 100644
|
||||||
|
--- a/modules/codec/dav1d.c
|
||||||
|
+++ b/modules/codec/dav1d.c
|
||||||
|
@@ -63,10 +63,16 @@ vlc_module_begin ()
|
||||||
|
set_category(CAT_INPUT)
|
||||||
|
set_subcategory(SUBCAT_INPUT_VCODEC)
|
||||||
|
|
||||||
|
+#if DAV1D_API_VERSION_MAJOR >= 6
|
||||||
|
+ add_integer_with_range("dav1d-thread-frames", 0, 0, DAV1D_MAX_THREADS,
|
||||||
|
+ THREAD_FRAMES_TEXT, THREAD_FRAMES_LONGTEXT, false)
|
||||||
|
+ add_obsolete_string("dav1d-thread-tiles") // unused with dav1d 1.0
|
||||||
|
+#else
|
||||||
|
add_integer_with_range("dav1d-thread-frames", 0, 0, DAV1D_MAX_FRAME_THREADS,
|
||||||
|
THREAD_FRAMES_TEXT, THREAD_FRAMES_LONGTEXT, false)
|
||||||
|
add_integer_with_range("dav1d-thread-tiles", 0, 0, DAV1D_MAX_TILE_THREADS,
|
||||||
|
THREAD_TILES_TEXT, THREAD_TILES_LONGTEXT, false)
|
||||||
|
+#endif
|
||||||
|
vlc_module_end ()
|
||||||
|
|
||||||
|
/*****************************************************************************
|
||||||
|
@@ -294,6 +300,11 @@ static int OpenDecoder(vlc_object_t *p_this)
|
||||||
|
return VLC_ENOMEM;
|
||||||
|
|
||||||
|
dav1d_default_settings(&p_sys->s);
|
||||||
|
+#if DAV1D_API_VERSION_MAJOR >= 6
|
||||||
|
+ p_sys->s.n_threads = var_InheritInteger(p_this, "dav1d-thread-frames");
|
||||||
|
+ if (p_sys->s.n_threads == 0)
|
||||||
|
+ p_sys->s.n_threads = (i_core_count < 16) ? i_core_count : 16;
|
||||||
|
+#else
|
||||||
|
p_sys->s.n_tile_threads = var_InheritInteger(p_this, "dav1d-thread-tiles");
|
||||||
|
if (p_sys->s.n_tile_threads == 0)
|
||||||
|
p_sys->s.n_tile_threads =
|
||||||
|
@@ -303,6 +314,7 @@ static int OpenDecoder(vlc_object_t *p_this)
|
||||||
|
p_sys->s.n_frame_threads = var_InheritInteger(p_this, "dav1d-thread-frames");
|
||||||
|
if (p_sys->s.n_frame_threads == 0)
|
||||||
|
p_sys->s.n_frame_threads = (i_core_count < 16) ? i_core_count : 16;
|
||||||
|
+#endif
|
||||||
|
p_sys->s.allocator.cookie = dec;
|
||||||
|
p_sys->s.allocator.alloc_picture_callback = NewPicture;
|
||||||
|
p_sys->s.allocator.release_picture_callback = FreePicture;
|
||||||
|
@@ -313,12 +325,20 @@ static int OpenDecoder(vlc_object_t *p_this)
|
||||||
|
return VLC_EGENERIC;
|
||||||
|
}
|
||||||
|
|
||||||
|
+#if DAV1D_API_VERSION_MAJOR >= 6
|
||||||
|
+ msg_Dbg(p_this, "Using dav1d version %s with %d threads",
|
||||||
|
+ dav1d_version(), p_sys->s.n_threads);
|
||||||
|
+
|
||||||
|
+ dec->i_extra_picture_buffers = (p_sys->s.n_threads - 1);
|
||||||
|
+#else
|
||||||
|
msg_Dbg(p_this, "Using dav1d version %s with %d/%d frame/tile threads",
|
||||||
|
dav1d_version(), p_sys->s.n_frame_threads, p_sys->s.n_tile_threads);
|
||||||
|
|
||||||
|
+ dec->i_extra_picture_buffers = (p_sys->s.n_frame_threads - 1);
|
||||||
|
+#endif
|
||||||
|
+
|
||||||
|
dec->pf_decode = Decode;
|
||||||
|
dec->pf_flush = FlushDecoder;
|
||||||
|
- dec->i_extra_picture_buffers = (p_sys->s.n_frame_threads - 1);
|
||||||
|
|
||||||
|
dec->fmt_out.video.i_width = dec->fmt_in.video.i_width;
|
||||||
|
dec->fmt_out.video.i_height = dec->fmt_in.video.i_height;
|
||||||
|
--
|
||||||
|
2.36.1
|
||||||
|
|
||||||
|
|
||||||
|
From c95e5288ab2d222346b19552a462afe5159d1122 Mon Sep 17 00:00:00 2001
|
||||||
|
From: Steve Lhomme <robux4@ycbcr.xyz>
|
||||||
|
Date: Mon, 21 Mar 2022 15:53:52 +0100
|
||||||
|
Subject: [PATCH 2/2] dav1d: limit the number of extra frames needed by the
|
||||||
|
decoder
|
||||||
|
|
||||||
|
The i_extra_picture_buffers is used to add pictures to the pool that the core
|
||||||
|
will allocate. dav1d is actually using n_threads frames. And the core is
|
||||||
|
allocating 10 frames per default for AV1. So we need to add the missing ones.
|
||||||
|
|
||||||
|
(cherry picked from commit a32031dc0f5f32083fc54a21397bce732742ccbe) (rebased)
|
||||||
|
rebased:
|
||||||
|
- the code dav1d 1.0.0 in 3.0 uses different max versions
|
||||||
|
|
||||||
|
Signed-off-by: Steve Lhomme <robux4@ycbcr.xyz>
|
||||||
|
---
|
||||||
|
modules/codec/dav1d.c | 25 +++++++++++++++++++++++--
|
||||||
|
1 file changed, 23 insertions(+), 2 deletions(-)
|
||||||
|
|
||||||
|
diff --git a/modules/codec/dav1d.c b/modules/codec/dav1d.c
|
||||||
|
index cfabbc27cb..8a439ce4ff 100644
|
||||||
|
--- a/modules/codec/dav1d.c
|
||||||
|
+++ b/modules/codec/dav1d.c
|
||||||
|
@@ -304,7 +304,28 @@ static int OpenDecoder(vlc_object_t *p_this)
|
||||||
|
p_sys->s.n_threads = var_InheritInteger(p_this, "dav1d-thread-frames");
|
||||||
|
if (p_sys->s.n_threads == 0)
|
||||||
|
p_sys->s.n_threads = (i_core_count < 16) ? i_core_count : 16;
|
||||||
|
-#else
|
||||||
|
+
|
||||||
|
+#if DAV1D_API_VERSION_MAJOR > 6 || DAV1D_API_VERSION_MINOR >= 7
|
||||||
|
+ // after dav1d 1.0.0
|
||||||
|
+ p_sys->s.max_frame_delay = dav1d_get_frame_delay( &p_sys->s );
|
||||||
|
+#else // 1.0.0
|
||||||
|
+ // corresponds to c->n_fc when max_frame_delay is 0 in dav1d 1.0.0
|
||||||
|
+ static const uint8_t fc_lut[49] = {
|
||||||
|
+ 1, /* 1 */
|
||||||
|
+ 2, 2, 2, /* 2- 4 */
|
||||||
|
+ 3, 3, 3, 3, 3, /* 5- 9 */
|
||||||
|
+ 4, 4, 4, 4, 4, 4, 4, /* 10-16 */
|
||||||
|
+ 5, 5, 5, 5, 5, 5, 5, 5, 5, /* 17-25 */
|
||||||
|
+ 6, 6, 6, 6, 6, 6, 6, 6, 6, 6, 6, /* 26-36 */
|
||||||
|
+ 7, 7, 7, 7, 7, 7, 7, 7, 7, 7, 7, 7, 7, /* 37-49 */
|
||||||
|
+ };
|
||||||
|
+ if (p_sys->s.n_threads >= 50)
|
||||||
|
+ p_sys->s.max_frame_delay = 8;
|
||||||
|
+ else
|
||||||
|
+ p_sys->s.max_frame_delay = fc_lut[p_sys->s.n_threads - 1];
|
||||||
|
+#endif
|
||||||
|
+
|
||||||
|
+#else // before dav1d 1.0.0
|
||||||
|
p_sys->s.n_tile_threads = var_InheritInteger(p_this, "dav1d-thread-tiles");
|
||||||
|
if (p_sys->s.n_tile_threads == 0)
|
||||||
|
p_sys->s.n_tile_threads =
|
||||||
|
@@ -329,7 +350,7 @@ static int OpenDecoder(vlc_object_t *p_this)
|
||||||
|
msg_Dbg(p_this, "Using dav1d version %s with %d threads",
|
||||||
|
dav1d_version(), p_sys->s.n_threads);
|
||||||
|
|
||||||
|
- dec->i_extra_picture_buffers = (p_sys->s.n_threads - 1);
|
||||||
|
+ dec->i_extra_picture_buffers = p_sys->s.max_frame_delay;
|
||||||
|
#else
|
||||||
|
msg_Dbg(p_this, "Using dav1d version %s with %d/%d frame/tile threads",
|
||||||
|
dav1d_version(), p_sys->s.n_frame_threads, p_sys->s.n_tile_threads);
|
||||||
|
--
|
||||||
|
2.36.1
|
||||||
|
|
||||||
@ -1 +1 @@
|
|||||||
SHA512 (vlc-c4ab31d.tar.gz) = 1d3f5b61c6aa7b9f3f62badc153143c0ec4b86fbe0b807edc3101d951bc4b4c615dacc734eeb3376b9ff9e7a8299e2d11ca463ddcceadaa1cf9d5ea1b015eb76
|
SHA512 (vlc-e9eceae.tar.gz) = 164e9e96954415ed25b5b0256e50f588a6853223a859805609ba7a4ce2ab0f1cc0966926182e7fe74a1a6d7b63deacbe3aaf67316b41c33b024999fb45b5902e
|
||||||
|
|||||||
Loading…
Reference in new issue