From: Dennis Mungai <dmngaie@gmail.com> To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org> Subject: Re: [FFmpeg-devel] [PATCH 8/9] avfilter/scale_amf: Add AMF HW scaler & color converter Date: Wed, 14 Feb 2024 19:26:32 +0300 Message-ID: <CAKKYfmFoYf9OkmjCkHgpohmT3653NV5MBgYEZcmxiz3B9x323Q@mail.gmail.com> (raw) In-Reply-To: <CADnG-DSgTDkXaWcz=Efdf5o=zni8C21soQs5aArwsfnzbSwu+w@mail.gmail.com> On Wed, 14 Feb 2024, 18:28 Evgeny Pavlov, <lucenticus@gmail.com> wrote: > On Wed, Feb 14, 2024 at 4:08 PM Timo Rothenpieler <timo@rothenpieler.org> > wrote: > > > On 14/02/2024 02:55, Dmitrii Ovchinnikov wrote: > > > From: Evgeny Pavlov <lucenticus@gmail.com> > > > > > > This commit adds two AMF filters: scale_amf & scale_amf_hq. > > > Both filters are using AMF hardware acceleration. > > > scale_amf supports simple scaling algorithms & color conversion. > > > scale_amf_hq supports advanced scaling algorithms & might be used > > > for upscaling only. > > > > Haven't looked at the patch yet, but can't this be one filter, and it > > picks the best possible method depending on options/inputs/whatever? > > _______________________________________________ > > 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". > > > > AMF has 2 separate components for color conversion + simple scaling > (VideoConverter) and for advanced scaling (HQScaler). > We've got a recommendation from the AMD AMF team to implement these > components as separate ffmpeg filters. > Still, this should be a single VIdeo Post-Processing (VPP) -style filter, exposing these scaling and video post processing options as tunables therein. A perfect example of such an implementation that excels in such an abstraction is intel's vpp_qsv filter, from which multiple compute, color space conversion methods, tonemapping, etc are made available through tunables. Another benefit of such an abstraction would be that re-using this filter on other GPU derivatives of the discrete silicon, eg in smaller IGPs sharing these offload blocks would be auto-detecting logic to ensure that even with defaults, the filter chains run. Taking another example from Intel, they have a full H/W path for low power encode and Post-Processing that can be automatically toggled on by specific filter options without user intervention, guaranteeing runtime safety for the same command(s) even on newer GPUs. Food for thought. > _______________________________________________ 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".
next prev parent reply other threads:[~2024-02-14 16:26 UTC|newest] Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top 2024-02-14 1:55 [FFmpeg-devel] [PATCH 1/9] libavutil: add hwcontext_amf Dmitrii Ovchinnikov 2024-02-14 1:55 ` [FFmpeg-devel] [PATCH 2/9] libavcodec: add amfdec Dmitrii Ovchinnikov 2024-02-14 23:41 ` Mark Thompson 2024-02-14 1:55 ` [FFmpeg-devel] [PATCH 3/9] avcodec/amfenc: Fixes the color information in the output Dmitrii Ovchinnikov 2024-02-14 1:55 ` [FFmpeg-devel] [PATCH 4/9] avcodec/amfenc: HDR metadata Dmitrii Ovchinnikov 2024-02-14 1:55 ` [FFmpeg-devel] [PATCH 5/9] avcodec/amfenc: add 10 bit encoding in av1_amf Dmitrii Ovchinnikov 2024-02-14 1:55 ` [FFmpeg-devel] [PATCH 6/9] avcodec/amfenc: add smart access video option Dmitrii Ovchinnikov 2024-02-14 1:55 ` [FFmpeg-devel] [PATCH 7/9] libavcodec/amfenc: redesign to use hwcontext_amf Dmitrii Ovchinnikov 2024-02-14 1:55 ` [FFmpeg-devel] [PATCH 8/9] avfilter/scale_amf: Add AMF HW scaler & color converter Dmitrii Ovchinnikov 2024-02-14 15:08 ` Timo Rothenpieler 2024-02-14 15:27 ` Evgeny Pavlov 2024-02-14 16:26 ` Dennis Mungai [this message] 2024-02-19 11:18 ` Evgeny Pavlov 2024-02-19 14:43 ` Dennis Mungai 2024-02-14 1:55 ` [FFmpeg-devel] [PATCH 9/9] doc/filters: Add documentation for AMF filters Dmitrii Ovchinnikov 2024-02-14 2:56 ` [FFmpeg-devel] [PATCH 1/9] libavutil: add hwcontext_amf James Almer 2024-02-14 16:48 ` Dmitrii Ovchinnikov 2024-02-14 20:56 ` Mark Thompson
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=CAKKYfmFoYf9OkmjCkHgpohmT3653NV5MBgYEZcmxiz3B9x323Q@mail.gmail.com \ --to=dmngaie@gmail.com \ --cc=ffmpeg-devel@ffmpeg.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
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