* [FFmpeg-devel] [PATCH 1/2] lavfi: remove scale_vulkan filter
@ 2023-06-20 17:32 Lynne
2023-06-20 17:35 ` Nicolas George
2023-06-20 21:58 ` Niklas Haas
0 siblings, 2 replies; 4+ messages in thread
From: Lynne @ 2023-06-20 17:32 UTC (permalink / raw)
To: Ffmpeg Devel
[-- Attachment #1: Type: text/plain, Size: 105 bytes --]
libplacebo is better in every way for anything involving scaling or format conversions
Patch attached.
[-- Attachment #2: 0001-lavfi-remove-scale_vulkan-filter.patch --]
[-- Type: text/x-diff, Size: 19470 bytes --]
From 146e753f3e618cb986c2649f0776f66e99098dea Mon Sep 17 00:00:00 2001
From: Lynne <dev@lynne.ee>
Date: Tue, 20 Jun 2023 18:56:26 +0200
Subject: [PATCH 1/2] lavfi: remove scale_vulkan filter
libplacebo is better in every way for anything involving scaling or format conversions
---
configure | 1 -
libavfilter/Makefile | 1 -
libavfilter/allfilters.c | 1 -
libavfilter/vf_scale_vulkan.c | 419 ----------------------------------
4 files changed, 422 deletions(-)
delete mode 100644 libavfilter/vf_scale_vulkan.c
diff --git a/configure b/configure
index ed9efad985..81ec58bacd 100755
--- a/configure
+++ b/configure
@@ -3831,7 +3831,6 @@ zmq_filter_deps="libzmq"
zoompan_filter_deps="swscale"
zscale_filter_deps="libzimg const_nan"
scale_vaapi_filter_deps="vaapi"
-scale_vulkan_filter_deps="vulkan spirv_compiler"
vpp_qsv_filter_deps="libmfx"
vpp_qsv_filter_select="qsvvpp"
xfade_opencl_filter_deps="opencl"
diff --git a/libavfilter/Makefile b/libavfilter/Makefile
index 9b7813575a..62e98bbb09 100644
--- a/libavfilter/Makefile
+++ b/libavfilter/Makefile
@@ -457,7 +457,6 @@ OBJS-$(CONFIG_SCALE_CUDA_FILTER) += vf_scale_cuda.o scale_eval.o \
OBJS-$(CONFIG_SCALE_NPP_FILTER) += vf_scale_npp.o scale_eval.o
OBJS-$(CONFIG_SCALE_QSV_FILTER) += vf_vpp_qsv.o
OBJS-$(CONFIG_SCALE_VAAPI_FILTER) += vf_scale_vaapi.o scale_eval.o vaapi_vpp.o
-OBJS-$(CONFIG_SCALE_VULKAN_FILTER) += vf_scale_vulkan.o vulkan.o vulkan_filter.o
OBJS-$(CONFIG_SCALE2REF_FILTER) += vf_scale.o scale_eval.o
OBJS-$(CONFIG_SCALE2REF_NPP_FILTER) += vf_scale_npp.o scale_eval.o
OBJS-$(CONFIG_SCDET_FILTER) += vf_scdet.o
diff --git a/libavfilter/allfilters.c b/libavfilter/allfilters.c
index 9a7fadc58d..eeb014248b 100644
--- a/libavfilter/allfilters.c
+++ b/libavfilter/allfilters.c
@@ -427,7 +427,6 @@ extern const AVFilter ff_vf_scale_cuda;
extern const AVFilter ff_vf_scale_npp;
extern const AVFilter ff_vf_scale_qsv;
extern const AVFilter ff_vf_scale_vaapi;
-extern const AVFilter ff_vf_scale_vulkan;
extern const AVFilter ff_vf_scale2ref;
extern const AVFilter ff_vf_scale2ref_npp;
extern const AVFilter ff_vf_scdet;
diff --git a/libavfilter/vf_scale_vulkan.c b/libavfilter/vf_scale_vulkan.c
deleted file mode 100644
index 3029cf2b42..0000000000
--- a/libavfilter/vf_scale_vulkan.c
+++ /dev/null
@@ -1,419 +0,0 @@
-/*
- * Copyright (c) Lynne
- *
- * 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 "libavutil/random_seed.h"
-#include "libavutil/opt.h"
-#include "vulkan_filter.h"
-#include "vulkan_spirv.h"
-#include "scale_eval.h"
-#include "internal.h"
-#include "colorspace.h"
-
-enum ScalerFunc {
- F_BILINEAR = 0,
- F_NEAREST,
-
- F_NB,
-};
-
-typedef struct ScaleVulkanContext {
- FFVulkanContext vkctx;
-
- int initialized;
- FFVulkanPipeline pl;
- FFVkExecPool e;
- FFVkQueueFamilyCtx qf;
- FFVkSPIRVShader shd;
- VkSampler sampler;
-
- /* Push constants / options */
- struct {
- float yuv_matrix[4][4];
- } opts;
-
- char *out_format_string;
- char *w_expr;
- char *h_expr;
-
- enum ScalerFunc scaler;
- enum AVColorRange out_range;
-} ScaleVulkanContext;
-
-static const char scale_bilinear[] = {
- C(0, vec4 scale_bilinear(int idx, ivec2 pos, vec2 crop_range, vec2 crop_off))
- C(0, { )
- C(1, vec2 npos = (vec2(pos) + 0.5f) / imageSize(output_img[idx]); )
- C(1, npos *= crop_range; /* Reduce the range */ )
- C(1, npos += crop_off; /* Offset the start */ )
- C(1, return texture(input_img[idx], npos); )
- C(0, } )
-};
-
-static const char rgb2yuv[] = {
- C(0, vec4 rgb2yuv(vec4 src, int fullrange) )
- C(0, { )
- C(1, src *= yuv_matrix; )
- C(1, if (fullrange == 1) { )
- C(2, src += vec4(0.0, 0.5, 0.5, 0.0); )
- C(1, } else { )
- C(2, src *= vec4(219.0 / 255.0, 224.0 / 255.0, 224.0 / 255.0, 1.0); )
- C(2, src += vec4(16.0 / 255.0, 128.0 / 255.0, 128.0 / 255.0, 0.0); )
- C(1, } )
- C(1, return src; )
- C(0, } )
-};
-
-static const char write_nv12[] = {
- C(0, void write_nv12(vec4 src, ivec2 pos) )
- C(0, { )
- C(1, imageStore(output_img[0], pos, vec4(src.r, 0.0, 0.0, 0.0)); )
- C(1, pos /= ivec2(2); )
- C(1, imageStore(output_img[1], pos, vec4(src.g, src.b, 0.0, 0.0)); )
- C(0, } )
-};
-
-static const char write_420[] = {
- C(0, void write_420(vec4 src, ivec2 pos) )
- C(0, { )
- C(1, imageStore(output_img[0], pos, vec4(src.r, 0.0, 0.0, 0.0)); )
- C(1, pos /= ivec2(2); )
- C(1, imageStore(output_img[1], pos, vec4(src.g, 0.0, 0.0, 0.0)); )
- C(1, imageStore(output_img[2], pos, vec4(src.b, 0.0, 0.0, 0.0)); )
- C(0, } )
-};
-
-static const char write_444[] = {
- C(0, void write_444(vec4 src, ivec2 pos) )
- C(0, { )
- C(1, imageStore(output_img[0], pos, vec4(src.r, 0.0, 0.0, 0.0)); )
- C(1, imageStore(output_img[1], pos, vec4(src.g, 0.0, 0.0, 0.0)); )
- C(1, imageStore(output_img[2], pos, vec4(src.b, 0.0, 0.0, 0.0)); )
- C(0, } )
-};
-
-static av_cold int init_filter(AVFilterContext *ctx, AVFrame *in)
-{
- int err;
- uint8_t *spv_data;
- size_t spv_len;
- void *spv_opaque = NULL;
- VkFilter sampler_mode;
- ScaleVulkanContext *s = ctx->priv;
- FFVulkanContext *vkctx = &s->vkctx;
- FFVkSPIRVShader *shd = &s->shd;
- FFVkSPIRVCompiler *spv;
- FFVulkanDescriptorSetBinding *desc;
-
- int crop_x = in->crop_left;
- int crop_y = in->crop_top;
- int crop_w = in->width - (in->crop_left + in->crop_right);
- int crop_h = in->height - (in->crop_top + in->crop_bottom);
- int in_planes = av_pix_fmt_count_planes(s->vkctx.input_format);
-
- switch (s->scaler) {
- case F_NEAREST:
- sampler_mode = VK_FILTER_NEAREST;
- break;
- case F_BILINEAR:
- sampler_mode = VK_FILTER_LINEAR;
- break;
- };
-
- spv = ff_vk_spirv_init();
- if (!spv) {
- av_log(ctx, AV_LOG_ERROR, "Unable to initialize SPIR-V compiler!\n");
- return AVERROR_EXTERNAL;
- }
-
- ff_vk_qf_init(vkctx, &s->qf, VK_QUEUE_COMPUTE_BIT);
- RET(ff_vk_exec_pool_init(vkctx, &s->qf, &s->e, s->qf.nb_queues*4, 0, 0, 0, NULL));
- RET(ff_vk_init_sampler(vkctx, &s->sampler, 0, sampler_mode));
- RET(ff_vk_shader_init(&s->pl, &s->shd, "scale_compute",
- VK_SHADER_STAGE_COMPUTE_BIT, 0));
-
- ff_vk_shader_set_compute_sizes(&s->shd, 32, 32, 1);
-
- GLSLC(0, layout(push_constant, std430) uniform pushConstants { );
- GLSLC(1, mat4 yuv_matrix; );
- GLSLC(0, }; );
- GLSLC(0, );
-
- ff_vk_add_push_constant(&s->pl, 0, sizeof(s->opts),
- VK_SHADER_STAGE_COMPUTE_BIT);
-
- desc = (FFVulkanDescriptorSetBinding []) {
- {
- .name = "input_img",
- .type = VK_DESCRIPTOR_TYPE_COMBINED_IMAGE_SAMPLER,
- .dimensions = 2,
- .elems = in_planes,
- .stages = VK_SHADER_STAGE_COMPUTE_BIT,
- .samplers = DUP_SAMPLER(s->sampler),
- },
- {
- .name = "output_img",
- .type = VK_DESCRIPTOR_TYPE_STORAGE_IMAGE,
- .mem_layout = ff_vk_shader_rep_fmt(s->vkctx.output_format),
- .mem_quali = "writeonly",
- .dimensions = 2,
- .elems = av_pix_fmt_count_planes(s->vkctx.output_format),
- .stages = VK_SHADER_STAGE_COMPUTE_BIT,
- },
- };
-
- RET(ff_vk_pipeline_descriptor_set_add(vkctx, &s->pl, shd, desc, 2, 0, 0));
-
- GLSLD( scale_bilinear );
-
- if (s->vkctx.output_format != s->vkctx.input_format) {
- GLSLD( rgb2yuv );
- }
-
- switch (s->vkctx.output_format) {
- case AV_PIX_FMT_NV12: GLSLD(write_nv12); break;
- case AV_PIX_FMT_YUV420P: GLSLD( write_420); break;
- case AV_PIX_FMT_YUV444P: GLSLD( write_444); break;
- default: break;
- }
-
- GLSLC(0, void main() );
- GLSLC(0, { );
- GLSLC(1, ivec2 size; );
- GLSLC(1, ivec2 pos = ivec2(gl_GlobalInvocationID.xy); );
- GLSLF(1, vec2 in_d = vec2(%i, %i); ,in->width, in->height);
- GLSLF(1, vec2 c_r = vec2(%i, %i) / in_d; ,crop_w, crop_h);
- GLSLF(1, vec2 c_o = vec2(%i, %i) / in_d; ,crop_x,crop_y);
- GLSLC(0, );
-
- if (s->vkctx.output_format == s->vkctx.input_format) {
- for (int i = 0; i < desc[i].elems; i++) {
- GLSLF(1, size = imageSize(output_img[%i]); ,i);
- GLSLC(1, if (IS_WITHIN(pos, size)) { );
- switch (s->scaler) {
- case F_NEAREST:
- case F_BILINEAR:
- GLSLF(2, vec4 res = scale_bilinear(%i, pos, c_r, c_o); ,i);
- GLSLF(2, imageStore(output_img[%i], pos, res); ,i);
- break;
- };
- GLSLC(1, } );
- }
- } else {
- GLSLC(1, vec4 res = scale_bilinear(0, pos, c_r, c_o); );
- GLSLF(1, res = rgb2yuv(res, %i); ,s->out_range == AVCOL_RANGE_JPEG);
- switch (s->vkctx.output_format) {
- case AV_PIX_FMT_NV12: GLSLC(1, write_nv12(res, pos); ); break;
- case AV_PIX_FMT_YUV420P: GLSLC(1, write_420(res, pos); ); break;
- case AV_PIX_FMT_YUV444P: GLSLC(1, write_444(res, pos); ); break;
- default: return AVERROR(EINVAL);
- }
- }
-
- GLSLC(0, } );
-
- if (s->vkctx.output_format != s->vkctx.input_format) {
- const AVLumaCoefficients *lcoeffs;
- double tmp_mat[3][3];
-
- lcoeffs = av_csp_luma_coeffs_from_avcsp(in->colorspace);
- if (!lcoeffs) {
- av_log(ctx, AV_LOG_ERROR, "Unsupported colorspace\n");
- return AVERROR(EINVAL);
- }
-
- ff_fill_rgb2yuv_table(lcoeffs, tmp_mat);
-
- for (int y = 0; y < 3; y++)
- for (int x = 0; x < 3; x++)
- s->opts.yuv_matrix[x][y] = tmp_mat[x][y];
- s->opts.yuv_matrix[3][3] = 1.0;
- }
-
- RET(spv->compile_shader(spv, ctx, shd, &spv_data, &spv_len, "main",
- &spv_opaque));
- RET(ff_vk_shader_create(vkctx, shd, spv_data, spv_len, "main"));
-
- RET(ff_vk_init_compute_pipeline(vkctx, &s->pl, shd));
- RET(ff_vk_exec_pipeline_register(vkctx, &s->e, &s->pl));
-
- s->initialized = 1;
-
- return 0;
-
-fail:
- if (spv_opaque)
- spv->free_shader(spv, &spv_opaque);
- if (spv)
- spv->uninit(&spv);
-
- return err;
-}
-
-static int scale_vulkan_filter_frame(AVFilterLink *link, AVFrame *in)
-{
- int err;
- AVFilterContext *ctx = link->dst;
- ScaleVulkanContext *s = ctx->priv;
- AVFilterLink *outlink = ctx->outputs[0];
-
- AVFrame *out = ff_get_video_buffer(outlink, outlink->w, outlink->h);
- if (!out) {
- err = AVERROR(ENOMEM);
- goto fail;
- }
-
- if (!s->initialized)
- RET(init_filter(ctx, in));
-
- RET(ff_vk_filter_process_simple(&s->vkctx, &s->e, &s->pl, out, in,
- s->sampler, &s->opts, sizeof(s->opts)));
-
- err = av_frame_copy_props(out, in);
- if (err < 0)
- goto fail;
-
- if (s->out_range != AVCOL_RANGE_UNSPECIFIED)
- out->color_range = s->out_range;
- if (s->vkctx.output_format != s->vkctx.input_format)
- out->chroma_location = AVCHROMA_LOC_TOPLEFT;
-
- av_frame_free(&in);
-
- return ff_filter_frame(outlink, out);
-
-fail:
- av_frame_free(&in);
- av_frame_free(&out);
- return err;
-}
-
-static int scale_vulkan_config_output(AVFilterLink *outlink)
-{
- int err;
- AVFilterContext *avctx = outlink->src;
- ScaleVulkanContext *s = avctx->priv;
- FFVulkanContext *vkctx = &s->vkctx;
- AVFilterLink *inlink = outlink->src->inputs[0];
-
- err = ff_scale_eval_dimensions(s, s->w_expr, s->h_expr, inlink, outlink,
- &vkctx->output_width,
- &vkctx->output_height);
- if (err < 0)
- return err;
-
- if (s->out_format_string) {
- s->vkctx.output_format = av_get_pix_fmt(s->out_format_string);
- if (s->vkctx.output_format == AV_PIX_FMT_NONE) {
- av_log(avctx, AV_LOG_ERROR, "Invalid output format.\n");
- return AVERROR(EINVAL);
- }
- } else {
- s->vkctx.output_format = s->vkctx.input_format;
- }
-
- if (s->vkctx.output_format != s->vkctx.input_format) {
- if (!ff_vk_mt_is_np_rgb(s->vkctx.input_format)) {
- av_log(avctx, AV_LOG_ERROR, "Unsupported input format for conversion\n");
- return AVERROR(EINVAL);
- }
- if (s->vkctx.output_format != AV_PIX_FMT_NV12 &&
- s->vkctx.output_format != AV_PIX_FMT_YUV420P &&
- s->vkctx.output_format != AV_PIX_FMT_YUV444P) {
- av_log(avctx, AV_LOG_ERROR, "Unsupported output format\n");
- return AVERROR(EINVAL);
- }
- } else if (s->out_range != AVCOL_RANGE_UNSPECIFIED) {
- av_log(avctx, AV_LOG_ERROR, "Cannot change range without converting format\n");
- return AVERROR(EINVAL);
- }
-
- return ff_vk_filter_config_output(outlink);
-}
-
-static void scale_vulkan_uninit(AVFilterContext *avctx)
-{
- ScaleVulkanContext *s = avctx->priv;
- FFVulkanContext *vkctx = &s->vkctx;
- FFVulkanFunctions *vk = &vkctx->vkfn;
-
- ff_vk_exec_pool_free(vkctx, &s->e);
- ff_vk_pipeline_free(vkctx, &s->pl);
- ff_vk_shader_free(vkctx, &s->shd);
-
- if (s->sampler)
- vk->DestroySampler(vkctx->hwctx->act_dev, s->sampler,
- vkctx->hwctx->alloc);
-
- ff_vk_uninit(&s->vkctx);
-
- s->initialized = 0;
-}
-
-#define OFFSET(x) offsetof(ScaleVulkanContext, x)
-#define FLAGS (AV_OPT_FLAG_FILTERING_PARAM | AV_OPT_FLAG_VIDEO_PARAM)
-static const AVOption scale_vulkan_options[] = {
- { "w", "Output video width", OFFSET(w_expr), AV_OPT_TYPE_STRING, {.str = "iw"}, .flags = FLAGS },
- { "h", "Output video height", OFFSET(h_expr), AV_OPT_TYPE_STRING, {.str = "ih"}, .flags = FLAGS },
- { "scaler", "Scaler function", OFFSET(scaler), AV_OPT_TYPE_INT, {.i64 = F_BILINEAR}, 0, F_NB, .flags = FLAGS, "scaler" },
- { "bilinear", "Bilinear interpolation (fastest)", 0, AV_OPT_TYPE_CONST, {.i64 = F_BILINEAR}, 0, 0, .flags = FLAGS, "scaler" },
- { "nearest", "Nearest (useful for pixel art)", 0, AV_OPT_TYPE_CONST, {.i64 = F_NEAREST}, 0, 0, .flags = FLAGS, "scaler" },
- { "format", "Output video format (software format of hardware frames)", OFFSET(out_format_string), AV_OPT_TYPE_STRING, .flags = FLAGS },
- { "out_range", "Output colour range (from 0 to 2) (default 0)", OFFSET(out_range), AV_OPT_TYPE_INT, {.i64 = AVCOL_RANGE_UNSPECIFIED}, AVCOL_RANGE_UNSPECIFIED, AVCOL_RANGE_JPEG, .flags = FLAGS, "range" },
- { "full", "Full range", 0, AV_OPT_TYPE_CONST, { .i64 = AVCOL_RANGE_JPEG }, 0, 0, FLAGS, "range" },
- { "limited", "Limited range", 0, AV_OPT_TYPE_CONST, { .i64 = AVCOL_RANGE_MPEG }, 0, 0, FLAGS, "range" },
- { "jpeg", "Full range", 0, AV_OPT_TYPE_CONST, { .i64 = AVCOL_RANGE_JPEG }, 0, 0, FLAGS, "range" },
- { "mpeg", "Limited range", 0, AV_OPT_TYPE_CONST, { .i64 = AVCOL_RANGE_MPEG }, 0, 0, FLAGS, "range" },
- { "tv", "Limited range", 0, AV_OPT_TYPE_CONST, { .i64 = AVCOL_RANGE_MPEG }, 0, 0, FLAGS, "range" },
- { "pc", "Full range", 0, AV_OPT_TYPE_CONST, { .i64 = AVCOL_RANGE_JPEG }, 0, 0, FLAGS, "range" },
- { NULL },
-};
-
-AVFILTER_DEFINE_CLASS(scale_vulkan);
-
-static const AVFilterPad scale_vulkan_inputs[] = {
- {
- .name = "default",
- .type = AVMEDIA_TYPE_VIDEO,
- .filter_frame = &scale_vulkan_filter_frame,
- .config_props = &ff_vk_filter_config_input,
- },
-};
-
-static const AVFilterPad scale_vulkan_outputs[] = {
- {
- .name = "default",
- .type = AVMEDIA_TYPE_VIDEO,
- .config_props = &scale_vulkan_config_output,
- },
-};
-
-const AVFilter ff_vf_scale_vulkan = {
- .name = "scale_vulkan",
- .description = NULL_IF_CONFIG_SMALL("Scale Vulkan frames"),
- .priv_size = sizeof(ScaleVulkanContext),
- .init = &ff_vk_filter_init,
- .uninit = &scale_vulkan_uninit,
- FILTER_INPUTS(scale_vulkan_inputs),
- FILTER_OUTPUTS(scale_vulkan_outputs),
- FILTER_SINGLE_PIXFMT(AV_PIX_FMT_VULKAN),
- .priv_class = &scale_vulkan_class,
- .flags_internal = FF_FILTER_FLAG_HWFRAME_AWARE,
- .flags = AVFILTER_FLAG_HWDEVICE,
-};
--
2.40.1
[-- Attachment #3: Type: text/plain, Size: 251 bytes --]
_______________________________________________
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] 4+ messages in thread
* Re: [FFmpeg-devel] [PATCH 1/2] lavfi: remove scale_vulkan filter
2023-06-20 17:32 [FFmpeg-devel] [PATCH 1/2] lavfi: remove scale_vulkan filter Lynne
@ 2023-06-20 17:35 ` Nicolas George
2023-06-20 21:58 ` Niklas Haas
1 sibling, 0 replies; 4+ messages in thread
From: Nicolas George @ 2023-06-20 17:35 UTC (permalink / raw)
To: FFmpeg development discussions and patches
[-- Attachment #1.1: Type: text/plain, Size: 283 bytes --]
Lynne (12023-06-20):
> libplacebo is better in every way for anything involving scaling or format conversions
Is libplacebo considered a system library?
Are scaling and format conversions not considered essential parts of
FFmpeg's task?
Regards,
--
Nicolas George
[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
[-- Attachment #2: Type: text/plain, Size: 251 bytes --]
_______________________________________________
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] 4+ messages in thread
* Re: [FFmpeg-devel] [PATCH 1/2] lavfi: remove scale_vulkan filter
2023-06-20 17:32 [FFmpeg-devel] [PATCH 1/2] lavfi: remove scale_vulkan filter Lynne
2023-06-20 17:35 ` Nicolas George
@ 2023-06-20 21:58 ` Niklas Haas
2023-06-20 22:12 ` Lynne
1 sibling, 1 reply; 4+ messages in thread
From: Niklas Haas @ 2023-06-20 21:58 UTC (permalink / raw)
To: Ffmpeg Devel
On Tue, 20 Jun 2023 19:32:28 +0200 Lynne <dev@lynne.ee> wrote:
> libplacebo is better in every way for anything involving scaling or format conversions
Hi,
vf_libplacebo always goes through internal RGB conversion, even for no-op.
scale_vulkan (ditto overlay_vulkan) can be more efficient for simple YCbCr
frames.
I would rather replace scale_vulkan's implementation by pl_scale_* calls if you
want to go this route.
_______________________________________________
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] 4+ messages in thread
* Re: [FFmpeg-devel] [PATCH 1/2] lavfi: remove scale_vulkan filter
2023-06-20 21:58 ` Niklas Haas
@ 2023-06-20 22:12 ` Lynne
0 siblings, 0 replies; 4+ messages in thread
From: Lynne @ 2023-06-20 22:12 UTC (permalink / raw)
To: FFmpeg development discussions and patches
Jun 20, 2023, 23:58 by ffmpeg@haasn.xyz:
> On Tue, 20 Jun 2023 19:32:28 +0200 Lynne <dev@lynne.ee> wrote:
>
>> libplacebo is better in every way for anything involving scaling or format conversions
>>
>
> Hi,
>
> vf_libplacebo always goes through internal RGB conversion, even for no-op.
> scale_vulkan (ditto overlay_vulkan) can be more efficient for simple YCbCr
> frames.
>
> I would rather replace scale_vulkan's implementation by pl_scale_* calls if you
> want to go this route.
>
I suppose I'll fix them up a little and leave them in, with
a note in the docs.
overlay_vulkan does work, but scale_vulkan is currently
broken for yuv420p (444 works oddly).
_______________________________________________
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] 4+ messages in thread
end of thread, other threads:[~2023-06-20 22:12 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-06-20 17:32 [FFmpeg-devel] [PATCH 1/2] lavfi: remove scale_vulkan filter Lynne
2023-06-20 17:35 ` Nicolas George
2023-06-20 21:58 ` Niklas Haas
2023-06-20 22:12 ` Lynne
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