[pulseaudio-discuss] [PATCHv2 05/10] tests: Add remap test code to cpu-test

Tanu Kaskinen tanuk at iki.fi
Thu Jan 31 22:26:30 PST 2013


On Wed, 2013-01-30 at 11:04 +0100, Peter Meerwald wrote:
> From: Peter Meerwald <p.meerwald at bct-electronic.com>
> 
> v2 (comments by Paul Menzel):
> * generate test samples from -1..1, -0x8000..0x7fff
> * check all output samples (not just half of them)
> 
> the idea is to compare the output of the C (reference) implementation
> against the output of the optimized code path; currently, there are MMX
> and SSE implementation for the mono-to-stereo remapper for s16 and float
> sample formats
> 
> Signed-off-by: Peter Meerwald <p.meerwald at bct-electronic.com>
> Cc: Paul Menzel <paulepanter at users.sourceforge.net>
> ---
>  src/tests/cpu-test.c |  227 ++++++++++++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 227 insertions(+)

Thanks! I'll apply this soon (along with patches 1-4, the rest I haven't
yet looked at). A couple of comments:

> + static void run_remap_test_mono_stereo_float(pa_remap_t *remap,
> +    pa_do_remap_func_t func, pa_do_remap_func_t orig_func,
> +    int align, pa_bool_t correct, pa_bool_t perf) {

Wrapped function argument lists are usually formatted like this:

static void run_remap_test_mono_stereo_float(
        pa_remap_t *remap,
        pa_do_remap_func_t func,
        pa_do_remap_func_t orig_func,
        int align,
        pa_bool_t correct,
        pa_bool_t perf) {

> +START_TEST (remap_sse_test) {

The test is for SSE2, so I'll change the test name to remap_sse2_test.

-- 
Tanu



More information about the pulseaudio-discuss mailing list