Skip to content

qa: add simultaneous playback/capture stress test scripts - #742

Closed
keyonjie wants to merge 1 commit into
thesofproject:masterfrom
keyonjie:sof-qa
Closed

keyonjie wants to merge 1 commit into
thesofproject:masterfrom
keyonjie:sof-qa

Conversation

@keyonjie

Copy link
Copy Markdown
Contributor

This will be helpful for simultaneous high stress test to find
concurrent and high workload stability issues.

Signed-off-by: Keyon Jie yang.jie@linux.intel.com

This will be helpful for simultaneous high stress test to find
concurrent and high workload stability issues.

Signed-off-by: Keyon Jie <yang.jie@linux.intel.com>
@@ -0,0 +1,30 @@
aplay -Dhw:0,1 -f dat -c 2 /dev/zero &

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Each test will need comments at the top describing what it does and any arguments it takes. Also SPDX style comment stating BSD licence. The file name here is not very meaningful either, this test does a duplex stream on 0,1 and a playback on 0,0 e.g. "test-duplex-plus-playback.sh" is probably a better name.

# Get its PID
PID_1=$!

arecord -Dhw:0,1 -f dat -c 2 /dev/null &

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The duplex PCM should be passed as an arg and so should playback PCM.

PID_3=$!

# Wait for 5 seconds
sleep 5

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sleep time should be an arg (it can be fractional)

@@ -0,0 +1,12 @@
i=0

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Needs comments, licence etc. Better name is "test-iterate.sh"

@@ -0,0 +1,12 @@
i=0
MAXLOOPS=1000

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

argument

else
exit 1
fi
sleep 1

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sleep should also be argument.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

and by default I would make it 0.1 seconds.

MAXLOOPS=1000
while [ $i -lt $MAXLOOPS ]; do
echo "iterate $i ..."
./sim.sh

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

test we run should be argument.

@lgirdwood

Copy link
Copy Markdown
Member

@keyonjie ping.

@keyonjie

keyonjie commented Dec 24, 2018 •

Copy link
Copy Markdown
Contributor Author

@lgirdwood I sent this PR only for sharing what I used for simultaneous pipelines stress test, it was not so formal and configurable yet.
Hi @keqiaozhang can you help address Liam's comments and send a new PR for it?

@mengdonglin mengdonglin added the enhancement New feature or request label Dec 25, 2018
@keyonjie

keyonjie commented Jan 4, 2019

Copy link
Copy Markdown
Contributor Author

Let me close this one, Keqiao will send new one later.

@keyonjie keyonjie closed this Jan 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants