diff mbox series

[FFmpeg-devel,4/4] lavc/opusdsp: RISC-V V (512-bit) postfilter

Message ID 20221005161256.27612-4-remi@remlab.net
State New
Headers show
Series RISC-V V Opus postfilter | expand


Context Check Description
andriy/make_x86 success Make finished
andriy/make_fate_x86 success Make fate finished

Commit Message

Rémi Denis-Courmont Oct. 5, 2022, 4:12 p.m. UTC
This adds a variant of the postfilter for use with 512-bit vectors.
Half a vector is enough to perform the scalar product. Normally a whole
vector would be used anyhow. Indeed fractional multiplers are no faster
than the unit multipler.

But in this particular function, a full vector makes up 16 samples,
which would be loaded at each iteration of the outer loop. The minimum
guaranteed CELT postfilter period is only 15. Accounting for the edges,
we can only safely preload up to 13 samples.

The fractional multipler is thus used to cap the selected vector length
to a safe value of 8 elements or 256 bits.

Likewise, we have the 1024-bit variant with the quarter multipler. In
theory, a 2048-bit one would be possible with the eigth multipler, but
that length is not even defined in the specifications as of yet, nor is
it supported by any emulator - forget actual hardware.
 libavcodec/riscv/opusdsp_init.c |  8 ++++++++
 libavcodec/riscv/opusdsp_rvv.S  | 10 ++++++++++
 2 files changed, 18 insertions(+)
diff mbox series


diff --git a/libavcodec/riscv/opusdsp_init.c b/libavcodec/riscv/opusdsp_init.c
index e6f9505f77..d564cca50c 100644
--- a/libavcodec/riscv/opusdsp_init.c
+++ b/libavcodec/riscv/opusdsp_init.c
@@ -27,6 +27,8 @@ 
 void ff_opus_postfilter_rvv_128(float *data, int period, float *g, int len);
 void ff_opus_postfilter_rvv_256(float *data, int period, float *g, int len);
+void ff_opus_postfilter_rvv_512(float *data, int period, float *g, int len);
+void ff_opus_postfilter_rvv_1024(float *data, int period, float *g, int len);
 av_cold void ff_opus_dsp_init_riscv(OpusDSP *d)
@@ -41,6 +43,12 @@  av_cold void ff_opus_dsp_init_riscv(OpusDSP *d)
             case 32:
                 d->postfilter = ff_opus_postfilter_rvv_256;
+            case 64:
+                d->postfilter = ff_opus_postfilter_rvv_512;
+                break;
+            case 128:
+                d->postfilter = ff_opus_postfilter_rvv_512;
+                break;
diff --git a/libavcodec/riscv/opusdsp_rvv.S b/libavcodec/riscv/opusdsp_rvv.S
index 243c9a5e52..b3d23a9de5 100644
--- a/libavcodec/riscv/opusdsp_rvv.S
+++ b/libavcodec/riscv/opusdsp_rvv.S
@@ -25,6 +25,16 @@  func ff_opus_postfilter_rvv_128, zve32f
         j       1f
+func ff_opus_postfilter_rvv_512, zve32f
+        lvtypei a5, e32, mf2, ta, ma
+        j       1f
+func ff_opus_postfilter_rvv_1024, zve32f
+        lvtypei a5, e32, mf4, ta, ma
+        j       1f
 func ff_opus_postfilter_rvv_256, zve32f
         lvtypei a5, e32, m1, ta, ma