Bug#1139233: fftw3: unconditional rdtime use on riscv64

[Adrian Bunk]

On Sun, Jun 07, 2026 at 03:25:36PM +0200, Aurelien Jarno wrote:
> Hi,
> 
> On 2026-06-07 16:15, Adrian Bunk wrote:
> > Source: fftw3
> > Version: 3.3.11-1
> > Severity: serious
> > Tags: patch
> > Forwarded: https://github.com/FFTW/fftw3/pull/406
> > X-Debbugs-Cc: debian-riscv at lists.debian.org
> > User: debian-riscv at lists.debian.org
> > Usertags: riscv64
> > Control: affects -1 src:sndfile-tools
> > 
> > https://ci.debian.net/packages/s/sndfile-tools/testing/riscv64/71828732/
> > 
> > ...
> > 177s autopkgtest [08:44:03]: test upstream-tests: [-----------------------
> > 184s valgrind /usr/bin/sndfile-generate-chirp : ok
> > 192s valgrind /usr/bin/sndfile-resample    : 1 errors, 0 bytes leaked
> > 196s valgrind /usr/bin/sndfile-resample    : 1 errors, 0 bytes leaked
> > 200s valgrind /usr/bin/sndfile-resample    : 1 errors, 0 bytes leaked
> > 204s valgrind /usr/bin/sndfile-spectrogram : tests/test-wrapper.sh: line 15:  1804 Illegal instruction        $valgrind $@ > $logfile 2>&1
> > 204s ok
> > 215s valgrind /usr/bin/sndfile-waveform    : 0 errors, 432 bytes leaked
> > 215s autopkgtest [08:44:41]: test upstream-tests: -----------------------]
> > 215s autopkgtest [08:44:41]: test upstream-tests:  - - - - - - - - - - results - - - - - - - - - -
> > 215s upstream-tests       FAIL stderr: tests/test-wrapper.sh: line 15:  1804 Illegal instruction        $valgrind $@ > $logfile 2>&1
> > ...
> > 
> > 
> > A fix is attached.
> 
> 
> Hmm, while the fix is correct for upstream, I do not think this is the 
> correct fix for Debian.

Debian would get an accepted upstream fix with the next upstream release.

> zicntr was part of the original rv64gc baseline 
> when the port was created before it got split out. I think we should 
> include it in our baseline, because anyway it is supported by all the 
> boards we target and widely used in the binaries we ship.

No objection from me to changing the baseline in gcc that way.

My patch will do the right thing in that case.

> The real issue here is that valgrind doesn't support it (also not in the 
> current upstream git), so probably that test should be disabled instead.

Yes, not running the sndfile-tools autopkgtest on riscv64 (as is already 
done on armhf) is an option.

> Regards
> Aurelien

cu
Adrian