test mostly compares runtime of 64 bit vs 32 bit s16ne-by-volume multiplication Signed-off-by: Peter Meerwald <pmeerw at pmeerw.net> --- src/.gitignore | 1 + src/Makefile.am | 8 ++- src/tests/mult-s16-test.c | 145 +++++++++++++++++++++++++++++++++++++++++++++ 3 files changed, 153 insertions(+), 1 deletion(-) create mode 100644 src/tests/mult-s16-test.c diff --git a/src/.gitignore b/src/.gitignore index ad8773b..a09e268 100644 --- a/src/.gitignore +++ b/src/.gitignore @@ -71,3 +71,4 @@ thread-test usergroup-test utf8-test volume-test +mult-s16-test diff --git a/src/Makefile.am b/src/Makefile.am index d479194..0a2fbd2 100644 --- a/src/Makefile.am +++ b/src/Makefile.am @@ -245,7 +245,8 @@ TESTS_default = \ mix-test \ proplist-test \ cpu-test \ - lock-autospawn-test + lock-autospawn-test \ + mult-s16-test TESTS_norun = \ mcalign-test \ @@ -503,6 +504,11 @@ cpu_test_LDADD = $(AM_LDADD) libpulsecore- at PA_MAJORMINOR@.la libpulse.la libpuls cpu_test_CFLAGS = $(AM_CFLAGS) $(LIBCHECK_CFLAGS) cpu_test_LDFLAGS = $(AM_LDFLAGS) $(BINLDFLAGS) $(LIBCHECK_LIBS) +mult_s16_test_SOURCES = tests/mult-s16-test.c +mult_s16_test_LDADD = $(AM_LDADD) libpulsecore- at PA_MAJORMINOR@.la libpulse.la libpulsecommon- at PA_MAJORMINOR@.la +mult_s16_test_CFLAGS = $(AM_CFLAGS) $(LIBCHECK_CFLAGS) +mult_s16_test_LDFLAGS = $(AM_LDFLAGS) $(BINLDFLAGS) $(LIBCHECK_LIBS) + rtstutter_SOURCES = tests/rtstutter.c rtstutter_LDADD = $(AM_LDADD) libpulsecore- at PA_MAJORMINOR@.la libpulse.la libpulsecommon- at PA_MAJORMINOR@.la rtstutter_CFLAGS = $(AM_CFLAGS) diff --git a/src/tests/mult-s16-test.c b/src/tests/mult-s16-test.c new file mode 100644 index 0000000..15ed8f2 --- /dev/null +++ b/src/tests/mult-s16-test.c @@ -0,0 +1,145 @@ +/*** + This file is part of PulseAudio. + + PulseAudio is free software; you can redistribute it and/or modify + it under the terms of the GNU Lesser General Public License as published + by the Free Software Foundation; either version 2.1 of the License, + or (at your option) any later version. + + PulseAudio is distributed in the hope that it will be useful, but + WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + General Public License for more details. + + You should have received a copy of the GNU Lesser General Public License + along with PulseAudio; if not, write to the Free Software + Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 + USA. +***/ + +#ifdef HAVE_CONFIG_H +#include <config.h> +#endif + +#include <check.h> +#include <unistd.h> +#include <stdlib.h> +#include <math.h> + +#include <pulse/rtclock.h> +#include <pulsecore/random.h> +#include <pulsecore/macro.h> + +#define PA_CPU_TEST_RUN_START(l, t1, t2) \ +{ \ + int _j, _k; \ + int _times = (t1), _times2 = (t2); \ + pa_usec_t _start, _stop; \ + pa_usec_t _min = INT_MAX, _max = 0; \ + double _s1 = 0, _s2 = 0; \ + const char *_label = (l); \ + \ + for (_k = 0; _k < _times2; _k++) { \ + _start = pa_rtclock_now(); \ + for (_j = 0; _j < _times; _j++) + +#define PA_CPU_TEST_RUN_STOP \ + _stop = pa_rtclock_now(); \ + \ + if (_min > (_stop - _start)) _min = _stop - _start; \ + if (_max < (_stop - _start)) _max = _stop - _start; \ + _s1 += _stop - _start; \ + _s2 += (_stop - _start) * (_stop - _start); \ + } \ + pa_log_debug("%s: %llu usec (avg: %g, min = %llu, max = %llu, stddev = %g).", _label, \ + (long long unsigned int)_s1, \ + ((double)_s1 / _times2), \ + (long long unsigned int)_min, \ + (long long unsigned int)_max, \ + sqrt(_times2 * _s2 - _s1 * _s1) / _times2); \ +} + +static inline int32_t pa_mult_s16_volume_32(int16_t v, int32_t cv) { + /* Multiplying the 32 bit volume factor with the + * 16 bit sample might result in an 48 bit value. We + * want to do without 64 bit integers and hence do + * the multiplication independently for the HI and + * LO part of the volume. */ + int32_t hi = cv >> 16; + int32_t lo = cv & 0xFFFF; + return ((v * lo) >> 16) + (v * hi); +} + +static inline int32_t pa_mult_s16_volume_64(int16_t v, int32_t cv) { + /* Multiply with 64 bit integers on 64 bit platforms */ + return (v * (int64_t) cv) >> 16; +} + +#define SAMPLES 1028 +#define TIMES 10000 +#define TIMES2 100 + +START_TEST (mult_s16_test) { + int16_t samples[SAMPLES]; + int32_t volumes[SAMPLES]; + int32_t sum1 = 0, sum2 = 0; + int i; + + pa_random(samples, sizeof(samples)); + pa_random(volumes, sizeof(volumes)); + + for (i = 0; i < SAMPLES; i++) { + int32_t a = pa_mult_s16_volume_32(samples[i], volumes[i]); + int32_t b = pa_mult_s16_volume_64(samples[i], volumes[i]); + + if (a != b) { + pa_log_debug("%d: %d != %d", i, a, b); + fail(); + } + } + + PA_CPU_TEST_RUN_START("32 bit mult", TIMES, TIMES2) { + for (i = 0; i < SAMPLES; i++) { + sum1 += pa_mult_s16_volume_32(samples[i], volumes[i]); + } + } PA_CPU_TEST_RUN_STOP + + PA_CPU_TEST_RUN_START("64 bit mult", TIMES, TIMES2) { + for (i = 0; i < SAMPLES; i++) + sum2 += pa_mult_s16_volume_64(samples[i], volumes[i]); + } PA_CPU_TEST_RUN_STOP + + fail_unless(sum1 == sum2); +} +END_TEST + +int main(int argc, char *argv[]) { + int failed = 0; + Suite *s; + TCase *tc; + SRunner *sr; + + if (!getenv("MAKE_CHECK")) + pa_log_set_level(PA_LOG_DEBUG); + +#if __WORDSIZE == 64 || ((ULONG_MAX) > (UINT_MAX)) + pa_log_debug("This seems to be 64-bit code."); +#elif __WORDSIZE == 32 + pa_log_debug("This seems to be 32-bit code."); +#else + pa_log_debug("Don't know if this is 32- or 64-bit code."); +#endif + + s = suite_create("Mult-s16"); + tc = tcase_create("mult-s16"); + tcase_add_test(tc, mult_s16_test); + tcase_set_timeout(tc, 120); + suite_add_tcase(s, tc); + + sr = srunner_create(s); + srunner_run_all(sr, CK_NORMAL); + failed = srunner_ntests_failed(sr); + srunner_free(sr); + + return (failed == 0) ? EXIT_SUCCESS : EXIT_FAILURE; +} -- 1.7.9.5