[FFmpeg-devel] [PATCH] lavc/aacpsdsp: fix R-V V stereo interpolate
Rémi Denis-Courmont
remi at remlab.net
Sun Dec 17 17:19:15 EET 2023
The penultimate loop iteration could pick any vl such that:
vlenb/4 < vl <= vlenb/2
Thus if the total length is not a multiple of vlenb/2, the vfadd.vf
on the penultimate iteration would yield corrupt values for the last
iteration.
To avoid this, force vl = vlen/2 until the last iteration. Unfortunately
this latent bug is not reproducible with either hardware or QEMU as of now.
---
libavcodec/riscv/aacpsdsp_rvv.S | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/libavcodec/riscv/aacpsdsp_rvv.S b/libavcodec/riscv/aacpsdsp_rvv.S
index f46b35fe91..a79d7d7818 100644
--- a/libavcodec/riscv/aacpsdsp_rvv.S
+++ b/libavcodec/riscv/aacpsdsp_rvv.S
@@ -234,7 +234,8 @@ func ff_ps_stereo_interpolate_rvv, zve32f
vfmacc.vf v22, ft3, v24
fmul.s ft3, ft3, ft4
1:
- vsetvli t0, a4, e32, m2, ta, ma
+ min t0, t0, a4
+ vsetvli zero, t0, e32, m2, ta, ma
vlseg2e32.v v0, (a0) // v0:l_re, v2:l_im
sub a4, a4, t0
vlseg2e32.v v4, (a1) // v4:r_re, v6:r_im
--
2.43.0
More information about the ffmpeg-devel
mailing list