On Monday 13 Feb 2017 01:25:29 Laurent Pinchart wrote: > On Saturday 11 Feb 2017 14:43:05 Kieran Bingham wrote: > > From: Kieran Bingham <[email protected]> > > > > Duplicate the functionality of vsp-unit-test-0001, to provide a set of > > work for the hardware to run, whilst we load the system using the > > 'stress' utility. > > > > Signed-off-by: Kieran Bingham <[email protected]> > > --- > > > > tests/vsp-unit-test-0022.sh | 55 ++++++++++++++++++++++++++++++++++++++++ > > 1 file changed, 55 insertions(+) > > create mode 100755 tests/vsp-unit-test-0022.sh > > > > diff --git a/tests/vsp-unit-test-0022.sh b/tests/vsp-unit-test-0022.sh > > new file mode 100755 > > index 000000000000..10230dffcc42 > > --- /dev/null > > +++ b/tests/vsp-unit-test-0022.sh > > @@ -0,0 +1,55 @@ > > +#!/bin/sh > > + > > +# > > +# Test active pipeline, with high load on CPU/Memory/IO using 'stress' > > +# > > +# Test WPF packing in RGB mode. Use a RPF -> WPF pipeline with a fixed > > ARGB32 > > +# format on the input and capture output frames in all RGB formats > > supported > > +# by the WPF. > > +# > > + > > +source vsp-lib.sh > > + > > +features="rpf.0 wpf.0" > > +formats="RGB332 ARGB555 XRGB555 RGB565 BGR24 RGB24 ABGR32 ARGB32 XBGR32 > > XRGB32" > > + > > +test_wpf_packing() { > > + test_start "WPF packing in $format during stress testing" > > + > > + pipe_configure rpf-wpf 0 0 > > + format_configure rpf-wpf 0 0 ARGB32 1024x768 $format > > + > > + vsp_runner rpf.0 & > > + vsp_runner wpf.0 > > + > > + local result=$(compare_frames) > > + > > + test_complete $result > > +} > > + > > +exists() { type -t "$1" > /dev/null 2>&1; } > > Let's keep formatting consistent and not inline functions on a single line. > Same comment for patch 2/2. > > > +test_main() { > > + local format > > + > > + if exists stress ; then > > + STRESS=`which stress` > > I don't think this is needed, you can just execute stress below, it will be > found from the $PATH. Same comment for patch 2/2. > > No need to resend, I'll fix this while applying. > > > + else > > + echo "$0: Stress test requires utility 'stress'" > > + test_complete skip
Additionally, as you haven't called test_start() yet, you shouldn't called test_complete(). > > + return > > + fi > > + > > + # Start stressing the system, as a background task > > + $STRESS --cpu 8 --io 4 --vm 2 --vm-bytes 128M & > > + > > + for format in $formats ; do > > + test_wpf_packing $format > > + done > > + > > + # Recover the system. Stress launches multiple PIDs, so it's best to: > > + killall -9 stress > > +} > > + > > +test_init $0 "$features" > > +test_run -- Regards, Laurent Pinchart
