Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
 help / color / mirror / Atom feed
* [FFmpeg-devel] [PATCH v7 01/18] swscale/graph: pass per-pass image pointers to setup()
@ 2025-06-20 13:15 Niklas Haas
  2025-06-20 13:15 ` [FFmpeg-devel] [PATCH v7 02/18] swscale/format: rename legacy format conversion table Niklas Haas
                   ` (17 more replies)
  0 siblings, 18 replies; 19+ messages in thread
From: Niklas Haas @ 2025-06-20 13:15 UTC (permalink / raw)
  To: ffmpeg-devel; +Cc: Niklas Haas

From: Niklas Haas <git@haasn.dev>

This behavior had no real justification and was just incredibly confusing,
since the in/out pointers passet to setup() did not match those passed to
run(), all for what is arguably an exception anyways (the palette setup).
---
 libswscale/graph.c | 10 +++++++---
 libswscale/graph.h |  2 --
 2 files changed, 7 insertions(+), 5 deletions(-)

diff --git a/libswscale/graph.c b/libswscale/graph.c
index dc7784aa49..2b26c068d1 100644
--- a/libswscale/graph.c
+++ b/libswscale/graph.c
@@ -169,6 +169,8 @@ static void free_legacy_swscale(void *priv)
 static void setup_legacy_swscale(const SwsImg *out, const SwsImg *in,
                                  const SwsPass *pass)
 {
+    const SwsGraph *graph = pass->graph;
+    const SwsImg *in_orig = &graph->exec.input;
     SwsContext *sws = pass->priv;
     SwsInternal *c = sws_internal(sws);
     if (sws->flags & SWS_BITEXACT && sws->dither == SWS_DITHER_ED && c->dither_error[0]) {
@@ -177,7 +179,7 @@ static void setup_legacy_swscale(const SwsImg *out, const SwsImg *in,
     }
 
     if (usePal(sws->src_format))
-        ff_update_palette(c, (const uint32_t *) in->data[1]);
+        ff_update_palette(c, (const uint32_t *) in_orig->data[1]);
 }
 
 static inline SwsContext *slice_ctx(const SwsPass *pass, int y)
@@ -712,8 +714,10 @@ void ff_sws_graph_run(SwsGraph *graph, uint8_t *const out_data[4],
     for (int i = 0; i < graph->num_passes; i++) {
         const SwsPass *pass = graph->passes[i];
         graph->exec.pass = pass;
-        if (pass->setup)
-            pass->setup(out, in, pass);
+        if (pass->setup) {
+            pass->setup(pass->output.fmt != AV_PIX_FMT_NONE ? &pass->output : out,
+                        pass->input ? &pass->input->output : in, pass);
+        }
         avpriv_slicethread_execute(graph->slicethread, pass->num_slices, 0);
     }
 }
diff --git a/libswscale/graph.h b/libswscale/graph.h
index 0630b31ce6..b829bac88c 100644
--- a/libswscale/graph.h
+++ b/libswscale/graph.h
@@ -92,8 +92,6 @@ struct SwsPass {
 
     /**
      * Called once from the main thread before running the filter. Optional.
-     * `out` and `in` always point to the main image input/output, regardless
-     * of `input` and `output` fields.
      */
     void (*setup)(const SwsImg *out, const SwsImg *in, const SwsPass *pass);
 
-- 
2.49.0

_______________________________________________
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".

^ permalink raw reply	[flat|nested] 19+ messages in thread

end of thread, other threads:[~2025-06-20 13:19 UTC | newest]

Thread overview: 19+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-06-20 13:15 [FFmpeg-devel] [PATCH v7 01/18] swscale/graph: pass per-pass image pointers to setup() Niklas Haas
2025-06-20 13:15 ` [FFmpeg-devel] [PATCH v7 02/18] swscale/format: rename legacy format conversion table Niklas Haas
2025-06-20 13:15 ` [FFmpeg-devel] [PATCH v7 03/18] swscale/format: add ff_fmt_clear() Niklas Haas
2025-06-20 13:15 ` [FFmpeg-devel] [PATCH v7 04/18] tests/checkasm: increase number of runs in between measurements Niklas Haas
2025-06-20 13:16 ` [FFmpeg-devel] [PATCH v7 05/18] tests/checkasm: generalize DEF_CHECKASM_CHECK_FUNC to floats Niklas Haas
2025-06-20 13:16 ` [FFmpeg-devel] [PATCH v7 06/18] swscale: add SWS_UNSTABLE flag Niklas Haas
2025-06-20 13:16 ` [FFmpeg-devel] [PATCH v7 07/18] swscale/ops: introduce new low level framework Niklas Haas
2025-06-20 13:16 ` [FFmpeg-devel] [PATCH v7 08/18] swscale/optimizer: add high-level ops optimizer Niklas Haas
2025-06-20 13:16 ` [FFmpeg-devel] [PATCH v7 09/18] swscale/ops_internal: add internal ops backend API Niklas Haas
2025-06-20 13:16 ` [FFmpeg-devel] [PATCH v7 10/18] swscale/ops: add dispatch layer Niklas Haas
2025-06-20 13:16 ` [FFmpeg-devel] [PATCH v7 11/18] swscale/optimizer: add packed shuffle solver Niklas Haas
2025-06-20 13:16 ` [FFmpeg-devel] [PATCH v7 12/18] swscale/ops_chain: add internal abstraction for kernel linking Niklas Haas
2025-06-20 13:16 ` [FFmpeg-devel] [PATCH v7 13/18] swscale/ops_backend: add reference backend basend on C templates Niklas Haas
2025-06-20 13:16 ` [FFmpeg-devel] [PATCH v7 14/18] swscale/ops_memcpy: add 'memcpy' backend for plane->plane copies Niklas Haas
2025-06-20 13:16 ` [FFmpeg-devel] [PATCH v7 15/18] swscale/x86: add SIMD backend Niklas Haas
2025-06-20 13:16 ` [FFmpeg-devel] [PATCH v7 16/18] tests/checkasm: add checkasm tests for swscale ops Niklas Haas
2025-06-20 13:16 ` [FFmpeg-devel] [PATCH v7 17/18] swscale/format: add new format decode/encode logic Niklas Haas
2025-06-20 13:16 ` [FFmpeg-devel] [PATCH v7 18/18] swscale/graph: allow experimental use of new format handler Niklas Haas
2025-06-20 13:17 ` [FFmpeg-devel] [PATCH v7 01/18] swscale/graph: pass per-pass image pointers to setup() Niklas Haas

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