RIOT-OS / Release-Specs

Specification for RIOT releases and corresponding test configurations
4 stars 21 forks source link

02-tests/task 03 Release 2016.07 - RC1 #24

Closed BytesGalore closed 8 years ago

BytesGalore commented 8 years ago

Task # 03

Perform a subset of tests (!= subset from Task #02) on samr21-xpro node:

list made manually by executing:

ls -1a | sed  -e 's/^/- [ ] /'

in RIOT/tests and removing unittests et al. from the list

BytesGalore commented 8 years ago

test/msg_send_receive fails on the samr21-xpro Is this already known?

kYc0o commented 8 years ago

Not to my knowledge. It's supposed to be a reference board so most of the code should work. How it fails?

BytesGalore commented 8 years ago

Basically the test state its failed, which results from the if-condition for the content received in the messages does not match [1]

[1] https://github.com/RIOT-OS/RIOT/blob/master/tests/msg_send_receive/main.c#L53

kYc0o commented 8 years ago

Humm... ok I'll try to reproduce.

BytesGalore commented 8 years ago

seems that something is messed up for the last iteration. I printed the values of counter and msg_resp.content.ptr:

# main(): This is RIOT! (Version: 2013.08-10643-gb5cb6-user01-LLED-E6410)
# Incremented counters to 1 and 1
# resp val: 1, counter: 1
# Incremented counters to 2 and 2
# resp val: 2, counter: 2
# Incremented counters to 3 and 3
# resp val: 3, counter: 3
# Incremented counters to 4 and 4
# resp val: 4, counter: 4
# Incremented counters to 5 and 5
# resp val: 5, counter: 5
# Incremented counters to 6 and 6
# resp val: 6, counter: 6
# Incremented counters to 7 and 7
# resp val: 7, counter: 7
# Incremented counters to 8 and 8
# resp val: 8, counter: 8
# Incremented counters to 9 and 9
# resp val: 9, counter: 9
# Incremented counters to 10 and 10
# resp val: 841, counter: 10
# Test failed.
kYc0o commented 8 years ago

Uhm... yeah I'm having the same problem. We need to open an issue for that and try to solve it. Else at least leave the issue open for the next release and solve it asap.

BytesGalore commented 8 years ago

ok, I created an issue https://github.com/RIOT-OS/RIOT/issues/5699 I assigned you, but feel free to reassign.

BytesGalore commented 8 years ago

alright next: test/posix_semaphore fails on performing the last test, i.e. TEST4. It seems the sem_timedwait(...) call doesn't wait the desired period.

BytesGalore commented 8 years ago

beside PRIu64 does not seem to work on the samr21-xpro, expected, e.g.: first: waited 1000134 usec actually got: first: waited lu usec

kYc0o commented 8 years ago

uhm... maybe it needs a special module as for floating point numbers?

miri64 commented 8 years ago

That's not floating points that's long integers ;-). Remember https://github.com/RIOT-OS/RIOT/issues/1891?

kYc0o commented 8 years ago

OK so it needs to load that module for samr21. Can it be done on the Makefile?

miri64 commented 8 years ago

There is no such mudule for long int at least not on the RIOT side. But we can hack something together... But since we decided yesterday to postpone this for next release I would propose to do so and do for now as we always did: ignore this kind of output for now :D

BytesGalore commented 8 years ago

Regarding the fail for test/posix_semaphore TEST4, I have a difference of ~265 us between the actual wait time vs the expected. I guess this corresponds with the timer drifts so basically this test succeeded :D