Skip to content
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.

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

Re-write of timedelta.icomp to produce proper average overrun figure #76

Open
wants to merge 6 commits into
base: ck-integration
Choose a base branch
from
Open

Conversation

ArcEye
Copy link

@ArcEye ArcEye commented Oct 5, 2015

SMP_SAFE version also

New script latency-test-plus
Will produce servo only test by default, but with average overrun display
Takes parameters to set thread speeds and will log results to file path
eg. latency-test-plus --base 25us --servo 1ms --logfile /tmp/latency.log

New sims in example configs, which display a latency test whilst operating
Hopefully provide a more realistic test

No v2 versions of latency-test,
they would be redundant when version preference introduced

Signed-off-by: Mick [email protected]

@ArcEye
Copy link
Author

ArcEye commented Oct 5, 2015

I have finished gnawing this bone now!

ArcEye added 2 commits October 7, 2015 08:44
SMP_SAFE version also

New script latency-test-plus
Will produce servo only test by default, but with average overrun display
Takes parameters to set thread speeds and will log results to file path
eg. `latency-test-plus --base 25us --servo 1ms --logfile /tmp/latency.log`

Hacked version of halsampler for use with logging

New sims in example configs, which display a latency test whilst operating
Hopefully provide a more realistic test

No v2 versions of latency-test,
they would be redundant when version preference introduced

Signed-off-by: Mick <[email protected]>
Declare *ip as unused in function definitions from instcomp
to prevent compiler warnings where not explicitly used thereafter

`struct inst_data *ip __attribute__((unused)) = arg;`

Signed-off-by: Mick <[email protected]>
@ArcEye
Copy link
Author

ArcEye commented Oct 7, 2015

Plus minor edit to instcomp
Not sure if this threw unused variable warning previously, or if it started with gcc 4.9, which seems to be much pickier.

Changed to explain instantiable component loading via newinst
and show alternative pin naming if newinst used

Signed-off-by: Mick <[email protected]>
Add __attribute__((unused)) to *ip in functions generation,
to prevent compiler warnings for components that make no use of it

Signed-off-by: Mick <[email protected]>
@ArcEye
Copy link
Author

ArcEye commented Nov 4, 2015

Last 3 just catch up with changes to instcomp in master and remove some compiler warnings

instcomp and comp fatal error messages changed to begin 'Error :'
to enable them to be spotted more easily amoungst general
build output

Signed-off-by: Mick <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant