[FFmpeg-devel] [PATCH v4 0/7] avfilter/vf_bwdif: Add aarch64 neon functions
Martin Storsjö
martin at martin.st
Thu Jul 6 00:19:50 EEST 2023
On Tue, 4 Jul 2023, John Cox wrote:
> Also adds a filter_line3 method which on aarch64 neon yields approx 30%
> speedup over 2xfilter_line and a memcpy
>
> Differences from v3:
> Remove a few lines of neon in filter_line that should have been removed
> when copying from line3
>
> Sorry about the two patch sets in quick succession, but I think I've
> applied all the requested changes and I didn't want this mistake in the
> final patchset. (The mistake was benign - it just wasted a few cycles.)
>
> John Cox (7):
> tests/checkasm: Add test for vf_bwdif filter_intra
> avfilter/vf_bwdif: Add neon for filter_intra
> tests/checkasm: Add test for vf_bwdif filter_edge
> avfilter/vf_bwdif: Add neon for filter_edge
> avfilter/vf_bwdif: Add neon for filter_line
> avfilter/vf_bwdif: Add a filter_line3 method for optimisation
> avfilter/vf_bwdif: Add neon for filter_line3
I think this looks ok to me, so I'll go ahead and push it. The tests pass
on x86 too, msvc/aarch64, llvm-mingw/aarch64, macOS and linux.
Just a couple notes I didn't remember to mention before:
- Regarding the int parameters on the stack; as long as you do have the C
wrapper functions, you don't strictly need to have the same function
signature for the NEON function as for the actual DSP function. So if
you'd have wanted to have a different signature for the NEON function
(changing it to intptr_t), that'd worked too. But I do see the benefit of
keeping it identical to the DSP function interface.
- The way of making the the C function exported and calling that for the
tail is neat, but kinda unusual within ffmpeg. In most cases (except for
parts of swscale), we can just assume and rely on buffers being aligned
enough for the SIMD vector length of the current platform, and freely
overwrite a little into the padding at the end of the lines. Not sure if
this is the case here though.
(If it is, it's easy enough to remove those bits and make the C functions
static again as a follow-up.)
Also, checkasm coverage for >8bpp would be nice as mentioned, but if
someone wants to write asm for that, it should be doable to factorize the
new tests to run them for both 8 and 16 bpp.
That said, it looks ok enough to me so I'll push it.
// Martin
More information about the ffmpeg-devel
mailing list