Skip to content

Cleaned up stress tests. #829

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
supercooldave wants to merge 2 commits into
base: development
Choose a base branch
from
Open

Cleaned up stress tests. #829

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
dbc7960
Cleaned up stress tests.
Jun 22, 2017
2d4449b
Tidied up IGNORED_FILES.grep
Jul 8, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
42 changes: 34 additions & 8 deletions src/tests/IGNORED_FILES.grep
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,13 +1,6 @@
# This file support line comments.
# Note that the # must appear first on the line
# Note that the comment character # must appear in the first position on a line

#Pedsim util modules should not be considered tests
stress/pedsim/Ped_util/Agent_passive
stress/pedsim/Ped_util/Global_funs
stress/pedsim/Ped_util/IO
stress/pedsim/Ped_util/Quad_tree
stress/pedsim/Ped_util/Regions
stress/pedsim/Ped_util/XML
#
# Modules used by 'modules' tests
# Should not be considered as tests
Expand Down Expand Up @@ -58,8 +51,27 @@ encore/stream/StreamIO
# Module used by 'stress' tests
# Should not be considered as a test
#
stress/savina/4.fjthrput/ThroughputActor
stress/savina/4.fjthrput/ThroughputConfig

stress/savina/5.fjcreate/ForkJoinActor
stress/savina/5.fjcreate/ForkJoinConfig

stress/savina/7.Chameneos/List

stress/savina/9.concdict/ArrayList
stress/savina/9.concdict/Hashmap
stress/savina/9.concdict/More

stress/savina/11.BndBuffer/Consumer
stress/savina/11.BndBuffer/HackyQueue
stress/savina/11.BndBuffer/Manager
stress/savina/11.BndBuffer/Producer
stress/savina/11.BndBuffer/RandomNewSyntax
stress/savina/11.BndBuffer/Util

stress/savina/12.DiningPhilosophers/Philosophers

stress/savina/13.SleepingBarber/Barber
stress/savina/13.SleepingBarber/MainTestBoundedFifo
stress/savina/13.SleepingBarber/CounterActor
Expand All @@ -69,6 +81,20 @@ stress/savina/13.SleepingBarber/FiFoList

stress/savina/14.CigaretteSmokers/CigaretteSmokers

stress/savina/21.ParallelQuickSort/ArrayList

stress/savina/29.PiPrecision/BigDecimal
stress/savina/29.PiPrecision/PiPrecisionConfig

# Pedsim util modules should not be considered tests
#
stress/pedsim/Ped_util/Agent_passive
stress/pedsim/Ped_util/Global_funs
stress/pedsim/Ped_util/IO
stress/pedsim/Ped_util/Quad_tree
stress/pedsim/Ped_util/Regions
stress/pedsim/Ped_util/XML

# Module used by 'big' tests are not tests
#
stress/big/input-data/ExampleText
Expand Down
4 changes: 3 additions & 1 deletion src/tests/stress/savina/11.BndBuffer/Makefile
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,8 +1,10 @@

main:
encorec -O3 main.enc -I ../../modules/standard
../../../../../release/encorec -O3 main.enc

all: main

run: main
/usr/bin/time ./main

Copy link
Copy Markdown
Contributor

@kikofernandez kikofernandez Jul 6, 2017

Choose a reason for hiding this comment

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

Is this Makefile needed by the CI?

Copy link
Copy Markdown
Author

@supercooldave supercooldave Jul 6, 2017

Choose a reason for hiding this comment

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

Yes, because there is no other way to pass the -O3 flag to the compiler.
(Whether the -O3 flag is really required is a question for the BoundedBuffer implementor.)
The reason for tidying up the Makefile was to reduce errors in the output, and increase faith in make stress.

clean: