* Re: [FFmpeg-devel] [PATCH 3/3] scale_qsv frame crop support
[not found] ` <20240910181057.43453-3-koushd@gmail.com>
@ 2025-01-22 2:07 ` Koushik Dutta
2025-01-22 2:14 ` Soft Works
0 siblings, 1 reply; 2+ messages in thread
From: Koushik Dutta @ 2025-01-22 2:07 UTC (permalink / raw)
To: ffmpeg-devel
Is anyone reviewing patches for qsv? I have two pending patches for
the qsv scale/crop filters. I submitted similar patches for cuda,
videotoolbox, and vulkan already, which were committed.
Koush
On Tue, Sep 10, 2024 at 11:11 AM Koushik Dutta <koushd@gmail.com> wrote:
>
> The crop filter has no effect on scale_qsv:
>
> -vf crop=100:100,scale_qsv=300x300
>
> Hardware frames (AV_PIX_FMT_FLAG_HWACCEL) are expected to use the crop_* properties,
> as seen in the implementation vf_crop.c.
>
> This patch is slightly different from the previously submitted patches
> since qsv supports frame crop via the vpp_qsv filter. If no explicit crop
> is detected via that filter, AVFrame.crop_* will be used instead.
> Removal of vpp_qsv's crop argument may be warranted.
> ---
> libavfilter/qsvvpp.c | 10 ++++++++++
> 1 file changed, 10 insertions(+)
>
> diff --git a/libavfilter/qsvvpp.c b/libavfilter/qsvvpp.c
> index 0818ada117..05ce387bc3 100644
> --- a/libavfilter/qsvvpp.c
> +++ b/libavfilter/qsvvpp.c
> @@ -467,6 +467,16 @@ static QSVFrame *submit_frame(QSVVPPContext *s, AVFilterLink *inlink, AVFrame *p
> else if (qsv_frame->frame->repeat_pict == 4)
> qsv_frame->surface.Info.PicStruct |= MFX_PICSTRUCT_FRAME_TRIPLING;
>
> + // if crop arguments are not present from the vpp_qsv filter, use the provided AVFrame
> + // crop_* members instead.
> + if (!qsv_frame->surface.Info.CropX && !qsv_frame->surface.Info.CropY
> + && qsv_frame->surface.Info.CropW == picref->width && qsv_frame->surface.Info.CropH == picref->height) {
> + qsv_frame->surface.Info.CropW = (mfxU16)((picref->width - picref->crop_right) - picref->crop_left);
> + qsv_frame->surface.Info.CropH = (mfxU16)((picref->height - picref->crop_bottom) - picref->crop_top);
> + qsv_frame->surface.Info.CropX = (mfxU16)picref->crop_left;
> + qsv_frame->surface.Info.CropY = (mfxU16)picref->crop_top;
> + }
> +
> return qsv_frame;
> }
>
> --
> 2.39.3 (Apple Git-146)
>
_______________________________________________
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] 2+ messages in thread