Skip to content

[FEATURE] use --fatal-errors for aplay/arecord tests #1120

Open

Description

@plbossart

Is your feature request related to a problem? Please describe.
When we reach an xrun, we should stop sof-test scripts. xruns need to be recoverable, but they should never happen. we don't want to miss these issues because the recovery quicks-in and hides problems.

Describe the solution you'd like
Add --fatal-errors to aplay/arecord command-line options.

Example run with --test-position (needs aplay patch to dump the last result)

Status(R/W) (standalone avail=36 delay=924):
  state       : RUNNING
  trigger_time: 2045.504937
  tstamp      : 2190.754602
  delay       : 924
  avail       : 36
  avail_max   : 444
Status(R/W) (standalone avail=44 delay=912):
  state       : RUNNING
  trigger_time: 2045.504937
  tstamp      : 2190.754852
  delay       : 912
  avail       : 48
  avail_max   : 48
underrun!!! (at least 471.161 ms long)
Status:
  state       : XRUN
  trigger_time: 2190.786234
  tstamp      : 2191.257392
  delay       : 0
  avail       : 1412
  avail_max   : 1412
aplay: xrun:1690: fatal underrun: Success

Describe alternatives you've considered
none that I can think of

Additional context
Not sure if we want to add this in sof-test proper, or add this in the environment variables with "APLAY_OPTS" that are already supported.

cc:

Activity

  1. plbossart commented on Oct 27, 2023

    @plbossart
    MemberAuthor
  2. plbossart commented on Aug 15, 2024

    @plbossart
    MemberAuthor

    btw this PR alsa-project/alsa-utils#242 was merged so most likely we can "just" add this --fatal-errors option to tests?

  3. marc-hb commented on Aug 15, 2024

    @marc-hb
    Collaborator

    Yes, work in "progress": b4523e0

  4. marc-hb commented on Aug 15, 2024

    @marc-hb
    Collaborator

    btw this PR alsa-project/alsa-utils#242 was merged

    We don't have a way to upgrade ALSA on test devices AFAIK

  5. marc-hb commented on Mar 19, 2025

    @marc-hb
    Collaborator

    I lost track of why this is not a duplicate of

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions