Message ID | 20170830144514.10979-1-paul.kocialkowski@linux.intel.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Quoting Paul Kocialkowski (2017-08-30 15:45:14) > This introduces a selftest for the audio library. > > It consists of generating a signal from a list of frequencies and > ensuring that the integrity checking function does detect these > frequencies (and only these frequencies). > > Signed-off-by: Paul Kocialkowski <paul.kocialkowski@linux.intel.com> > --- > lib/tests/Makefile.am | 2 +- > lib/tests/Makefile.sources | 5 ++++ > lib/tests/igt_audio.c | 57 ++++++++++++++++++++++++++++++++++++++++++++++ > 3 files changed, 63 insertions(+), 1 deletion(-) > create mode 100644 lib/tests/igt_audio.c > > diff --git a/lib/tests/Makefile.am b/lib/tests/Makefile.am > index 5d14194a..b1caa628 100644 > --- a/lib/tests/Makefile.am > +++ b/lib/tests/Makefile.am > @@ -14,7 +14,7 @@ AM_CFLAGS = $(DRM_CFLAGS) $(CWARNFLAGS) $(DEBUG_CFLAGS) \ > -DIGT_DATADIR=\""$(abs_srcdir)"\" \ > $(NULL) > > -LDADD = ../libintel_tools.la $(PCIACCESS_LIBS) $(DRM_LIBS) $(LIBUNWIND_LIBS) $(TIMER_LIBS) > +LDADD = ../libintel_tools.la $(PCIACCESS_LIBS) $(DRM_LIBS) $(GSL_CFLAGS) $(LIBUNWIND_LIBS) $(TIMER_LIBS) Let's play spot the odd one out. -Chris
On Wed, 2017-08-30 at 17:45 +0300, Paul Kocialkowski wrote: > This introduces a selftest for the audio library. > > It consists of generating a signal from a list of frequencies and > ensuring that the integrity checking function does detect these > frequencies (and only these frequencies). > > Signed-off-by: Paul Kocialkowski <paul.kocialkowski@linux.intel.com> > --- > lib/tests/Makefile.am | 2 +- > lib/tests/Makefile.sources | 5 ++++ > lib/tests/igt_audio.c | 57 > ++++++++++++++++++++++++++++++++++++++++++++++ > 3 files changed, 63 insertions(+), 1 deletion(-) > create mode 100644 lib/tests/igt_audio.c > > diff --git a/lib/tests/Makefile.am b/lib/tests/Makefile.am > index 5d14194a..b1caa628 100644 > --- a/lib/tests/Makefile.am > +++ b/lib/tests/Makefile.am > @@ -14,7 +14,7 @@ AM_CFLAGS = $(DRM_CFLAGS) $(CWARNFLAGS) $(DEBUG_CFLAGS) \ > -DIGT_DATADIR=\""$(abs_srcdir)"\" \ > $(NULL) > > -LDADD = ../libintel_tools.la $(PCIACCESS_LIBS) $(DRM_LIBS) $(LIBUNWIND_LIBS) > $(TIMER_LIBS) > +LDADD = ../libintel_tools.la $(PCIACCESS_LIBS) $(DRM_LIBS) $(GSL_CFLAGS) > $(LIBUNWIND_LIBS) $(TIMER_LIBS) > > LDADD += $(CAIRO_LIBS) $(LIBUDEV_LIBS) $(GLIB_LIBS) -lm > AM_CFLAGS += $(CAIRO_CFLAGS) $(LIBUDEV_CFLAGS) $(GLIB_CFLAGS) > diff --git a/lib/tests/Makefile.sources b/lib/tests/Makefile.sources > index 8d1a8dea..eb702844 100644 > --- a/lib/tests/Makefile.sources > +++ b/lib/tests/Makefile.sources > @@ -18,6 +18,11 @@ check_prog_list = \ > igt_can_fail_simple \ > $(NULL) > > +#if HAVE_GSL > +check_prog_list += \ > + igt_audio > +#endif > + > TESTS = \ > $(check_prog_list) \ > $(check_script_list) \ > diff --git a/lib/tests/igt_audio.c b/lib/tests/igt_audio.c > new file mode 100644 > index 00000000..2354d5a0 > --- /dev/null > +++ b/lib/tests/igt_audio.c > @@ -0,0 +1,57 @@ > +/* > + * Copyright © 2017 Intel Corporation > + * > + * Permission is hereby granted, free of charge, to any person obtaining a > + * copy of this software and associated documentation files (the "Software"), > + * to deal in the Software without restriction, including without limitation > + * the rights to use, copy, modify, merge, publish, distribute, sublicense, > + * and/or sell copies of the Software, and to permit persons to whom the > + * Software is furnished to do so, subject to the following conditions: > + * > + * The above copyright notice and this permission notice (including the next > + * paragraph) shall be included in all copies or substantial portions of the > + * Software. > + * > + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR > + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, > + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL > + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER > + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING > + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER > DEALINGS > + * IN THE SOFTWARE. > + * > + */ > + > +#include "igt_core.h" > +#include "igt_audio.h" > + > +static int test_frequencies[] = { > + 300, > + 600, > + 1200, > + 80000, > + 10000, > +}; > + > +static int test_frequencies_count = sizeof(test_frequencies) / sizeof(int); > + > +igt_simple_main > +{ > + short buffer[2 * 1024]; And just specify the size here explicitly, no need to present it in units of 1024 > + struct audio_signal *signal; > + int i; > + > + signal = audio_signal_init(2, 44800); > + igt_assert(signal); > + > + for (i = 0; i < test_frequencies_count; i++) Just use ARRAY_SIZE(test_frequencies), no need for test_frequencies_count > + audio_signal_add_frequency(signal, test_frequencies[i]); > + > + audio_signal_synthesize(signal); > + audio_signal_fill(signal, buffer, 1024); > + > + igt_assert(audio_signal_detect(signal, 2, 44800, buffer, 1024)); > + > + audio_signal_clean(signal); > + free(signal); > +}
On Wed, 2017-08-30 at 16:44 +0100, Chris Wilson wrote: > Quoting Paul Kocialkowski (2017-08-30 15:45:14) > > This introduces a selftest for the audio library. > > > > It consists of generating a signal from a list of frequencies and > > ensuring that the integrity checking function does detect these > > frequencies (and only these frequencies). > > > > Signed-off-by: Paul Kocialkowski <paul.kocialkowski@linux.intel.com> > > --- > > lib/tests/Makefile.am | 2 +- > > lib/tests/Makefile.sources | 5 ++++ > > lib/tests/igt_audio.c | 57 > > ++++++++++++++++++++++++++++++++++++++++++++++ > > 3 files changed, 63 insertions(+), 1 deletion(-) > > create mode 100644 lib/tests/igt_audio.c > > > > diff --git a/lib/tests/Makefile.am b/lib/tests/Makefile.am > > index 5d14194a..b1caa628 100644 > > --- a/lib/tests/Makefile.am > > +++ b/lib/tests/Makefile.am > > @@ -14,7 +14,7 @@ AM_CFLAGS = $(DRM_CFLAGS) $(CWARNFLAGS) > > $(DEBUG_CFLAGS) \ > > -DIGT_DATADIR=\""$(abs_srcdir)"\" \ > > $(NULL) > > > > -LDADD = ../libintel_tools.la $(PCIACCESS_LIBS) $(DRM_LIBS) > > $(LIBUNWIND_LIBS) $(TIMER_LIBS) > > +LDADD = ../libintel_tools.la $(PCIACCESS_LIBS) $(DRM_LIBS) > > $(GSL_CFLAGS) $(LIBUNWIND_LIBS) $(TIMER_LIBS) > > Let's play spot the odd one out. Haha, sorry about that, I went over it way too quickly. Thanks catching it!
Hi, On Wed, 2017-08-30 at 14:53 -0400, Lyude Paul wrote: > On Wed, 2017-08-30 at 17:45 +0300, Paul Kocialkowski wrote: > > This introduces a selftest for the audio library. > > > > It consists of generating a signal from a list of frequencies and > > ensuring that the integrity checking function does detect these > > frequencies (and only these frequencies). > > > > Signed-off-by: Paul Kocialkowski <paul.kocialkowski@linux.intel.com> > > --- > > lib/tests/Makefile.am | 2 +- > > lib/tests/Makefile.sources | 5 ++++ > > lib/tests/igt_audio.c | 57 > > ++++++++++++++++++++++++++++++++++++++++++++++ > > 3 files changed, 63 insertions(+), 1 deletion(-) > > create mode 100644 lib/tests/igt_audio.c > > > > diff --git a/lib/tests/Makefile.am b/lib/tests/Makefile.am > > index 5d14194a..b1caa628 100644 > > --- a/lib/tests/Makefile.am > > +++ b/lib/tests/Makefile.am > > @@ -14,7 +14,7 @@ AM_CFLAGS = $(DRM_CFLAGS) $(CWARNFLAGS) > > $(DEBUG_CFLAGS) \ > > -DIGT_DATADIR=\""$(abs_srcdir)"\" \ > > $(NULL) > > > > -LDADD = ../libintel_tools.la $(PCIACCESS_LIBS) $(DRM_LIBS) > > $(LIBUNWIND_LIBS) > > $(TIMER_LIBS) > > +LDADD = ../libintel_tools.la $(PCIACCESS_LIBS) $(DRM_LIBS) > > $(GSL_CFLAGS) > > $(LIBUNWIND_LIBS) $(TIMER_LIBS) > > > > LDADD += $(CAIRO_LIBS) $(LIBUDEV_LIBS) $(GLIB_LIBS) -lm > > AM_CFLAGS += $(CAIRO_CFLAGS) $(LIBUDEV_CFLAGS) $(GLIB_CFLAGS) > > diff --git a/lib/tests/Makefile.sources b/lib/tests/Makefile.sources > > index 8d1a8dea..eb702844 100644 > > --- a/lib/tests/Makefile.sources > > +++ b/lib/tests/Makefile.sources > > @@ -18,6 +18,11 @@ check_prog_list = \ > > igt_can_fail_simple \ > > $(NULL) > > > > +#if HAVE_GSL > > +check_prog_list += \ > > + igt_audio > > +#endif > > + > > TESTS = \ > > $(check_prog_list) \ > > $(check_script_list) \ > > diff --git a/lib/tests/igt_audio.c b/lib/tests/igt_audio.c > > new file mode 100644 > > index 00000000..2354d5a0 > > --- /dev/null > > +++ b/lib/tests/igt_audio.c > > @@ -0,0 +1,57 @@ > > +/* > > + * Copyright © 2017 Intel Corporation > > + * > > + * Permission is hereby granted, free of charge, to any person > > obtaining a > > + * copy of this software and associated documentation files (the > > "Software"), > > + * to deal in the Software without restriction, including without > > limitation > > + * the rights to use, copy, modify, merge, publish, distribute, > > sublicense, > > + * and/or sell copies of the Software, and to permit persons to > > whom the > > + * Software is furnished to do so, subject to the following > > conditions: > > + * > > + * The above copyright notice and this permission notice (including > > the next > > + * paragraph) shall be included in all copies or substantial > > portions of the > > + * Software. > > + * > > + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, > > EXPRESS OR > > + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF > > MERCHANTABILITY, > > + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO > > EVENT SHALL > > + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, > > DAMAGES OR OTHER > > + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, > > ARISING > > + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR > > OTHER > > DEALINGS > > + * IN THE SOFTWARE. > > + * > > + */ > > + > > +#include "igt_core.h" > > +#include "igt_audio.h" > > + > > +static int test_frequencies[] = { > > + 300, > > + 600, > > + 1200, > > + 80000, > > + 10000, > > +}; > > + > > +static int test_frequencies_count = sizeof(test_frequencies) / > > sizeof(int); > > + > > +igt_simple_main > > +{ > > + short buffer[2 * 1024]; > > And just specify the size here explicitly, no need to present it in > units of 1024 Sure, will do in v2. > > + struct audio_signal *signal; > > + int i; > > + > > + signal = audio_signal_init(2, 44800); > > + igt_assert(signal); > > + > > + for (i = 0; i < test_frequencies_count; i++) > > Just use ARRAY_SIZE(test_frequencies), no need for > test_frequencies_count Good point. Thanks for the review! > > + audio_signal_add_frequency(signal, > > test_frequencies[i]); > > + > > + audio_signal_synthesize(signal); > > + audio_signal_fill(signal, buffer, 1024); > > + > > + igt_assert(audio_signal_detect(signal, 2, 44800, buffer, > > 1024)); > > + > > + audio_signal_clean(signal); > > + free(signal); > > +}
diff --git a/lib/tests/Makefile.am b/lib/tests/Makefile.am index 5d14194a..b1caa628 100644 --- a/lib/tests/Makefile.am +++ b/lib/tests/Makefile.am @@ -14,7 +14,7 @@ AM_CFLAGS = $(DRM_CFLAGS) $(CWARNFLAGS) $(DEBUG_CFLAGS) \ -DIGT_DATADIR=\""$(abs_srcdir)"\" \ $(NULL) -LDADD = ../libintel_tools.la $(PCIACCESS_LIBS) $(DRM_LIBS) $(LIBUNWIND_LIBS) $(TIMER_LIBS) +LDADD = ../libintel_tools.la $(PCIACCESS_LIBS) $(DRM_LIBS) $(GSL_CFLAGS) $(LIBUNWIND_LIBS) $(TIMER_LIBS) LDADD += $(CAIRO_LIBS) $(LIBUDEV_LIBS) $(GLIB_LIBS) -lm AM_CFLAGS += $(CAIRO_CFLAGS) $(LIBUDEV_CFLAGS) $(GLIB_CFLAGS) diff --git a/lib/tests/Makefile.sources b/lib/tests/Makefile.sources index 8d1a8dea..eb702844 100644 --- a/lib/tests/Makefile.sources +++ b/lib/tests/Makefile.sources @@ -18,6 +18,11 @@ check_prog_list = \ igt_can_fail_simple \ $(NULL) +#if HAVE_GSL +check_prog_list += \ + igt_audio +#endif + TESTS = \ $(check_prog_list) \ $(check_script_list) \ diff --git a/lib/tests/igt_audio.c b/lib/tests/igt_audio.c new file mode 100644 index 00000000..2354d5a0 --- /dev/null +++ b/lib/tests/igt_audio.c @@ -0,0 +1,57 @@ +/* + * Copyright © 2017 Intel Corporation + * + * Permission is hereby granted, free of charge, to any person obtaining a + * copy of this software and associated documentation files (the "Software"), + * to deal in the Software without restriction, including without limitation + * the rights to use, copy, modify, merge, publish, distribute, sublicense, + * and/or sell copies of the Software, and to permit persons to whom the + * Software is furnished to do so, subject to the following conditions: + * + * The above copyright notice and this permission notice (including the next + * paragraph) shall be included in all copies or substantial portions of the + * Software. + * + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS + * IN THE SOFTWARE. + * + */ + +#include "igt_core.h" +#include "igt_audio.h" + +static int test_frequencies[] = { + 300, + 600, + 1200, + 80000, + 10000, +}; + +static int test_frequencies_count = sizeof(test_frequencies) / sizeof(int); + +igt_simple_main +{ + short buffer[2 * 1024]; + struct audio_signal *signal; + int i; + + signal = audio_signal_init(2, 44800); + igt_assert(signal); + + for (i = 0; i < test_frequencies_count; i++) + audio_signal_add_frequency(signal, test_frequencies[i]); + + audio_signal_synthesize(signal); + audio_signal_fill(signal, buffer, 1024); + + igt_assert(audio_signal_detect(signal, 2, 44800, buffer, 1024)); + + audio_signal_clean(signal); + free(signal); +}
This introduces a selftest for the audio library. It consists of generating a signal from a list of frequencies and ensuring that the integrity checking function does detect these frequencies (and only these frequencies). Signed-off-by: Paul Kocialkowski <paul.kocialkowski@linux.intel.com> --- lib/tests/Makefile.am | 2 +- lib/tests/Makefile.sources | 5 ++++ lib/tests/igt_audio.c | 57 ++++++++++++++++++++++++++++++++++++++++++++++ 3 files changed, 63 insertions(+), 1 deletion(-) create mode 100644 lib/tests/igt_audio.c