Dear Rioters,
The deadline for the 2014.05 Release Candidate was yesterday.
This is a list of issues that appear to be in need of attention along
with the people whose input is needed:
Martine:
https://github.com/RIOT-OS/RIOT/issues/1104
https://github.com/RIOT-OS/RIOT/issues/994
Christian:
https://github.com/RIOT-OS/RIOT/pull/1093
https://github.com/RIOT-OS/RIOT/pull/1064
https://github.com/RIOT-OS/RIOT/pull/977
https://github.com/RIOT-OS/RIOT/issues/751
Rene:
https://github.com/RIOT-OS/RIOT/pull/1083
https://github.com/RIOT-OS/RIOT/pull/764
Kaspar:
https://github.com/RIOT-OS/RIOT/issues/924
https://github.com/RIOT-OS/RIOT/issues/252
Please take the time to look into these issues and decide whether it
makes sense to untag them for this release.
I omitted issues that appear to be in a healthy condition.
Cheers, Ludwig
Heya,
here comes the update
Replace printf and puts with logging functions · Issue #994 · RIOT-OS/RIOT · GitHub
Untagged for this release.
Christian:
examples: added README.md to ccn-lite-client and relay by mehlis · Pull Request #1093 · RIOT-OS/RIOT · GitHub
Some open comments by Ludwig.
core: kernel: improved doxygen documentation by mehlis · Pull Request #977 · RIOT-OS/RIOT · GitHub
Almost done.
Rene:
tests: added a thread_sleep() at the end by OlegHahm · Pull Request #1083 · RIOT-OS/RIOT · GitHub
More or less done - Jenkins complains, but clicking on the link fails, too.
Can anyone enlighten me? Why does Jenkins still links to
stedten.imp.fu-berlin.de instead of ci.riot-os.org?
Initial x86 port: "qemu-i386" by Kijewski · Pull Request #764 · RIOT-OS/RIOT · GitHub
Needs rebase and splitting.
RIOT-OS:master ← Kijewski:msp-oneway-malloc
opened 02:36PM - 15 May 14 UTC
For MSP430 boards oneway-malloc is already used _if_ `malloc.h` was
included. Th… e problem is that `malloc.h` is not a standard header, even
though it is common. `stdlib.h` in the right place to look for
`malloc()` and friends.
This change removes this discrepancy. `malloc()` is just named like
that, without the leading underscore. The symbols now are weak, which
means that they won't override library functions if MSP's standard
library will provide these functions at some point. (Unlikely, since
using `malloc()` on tiny systems is less then optimal ...)
Needs a second ACK - will also enable merging of
RIOT-OS:master ← Kijewski:buildtest-insufficient-ram
opened 10:37AM - 22 Apr 14 UTC
Currently most blacklistings for examples and tests are done because the
board p… rovides too little RAM or ROM. Besides of the actual linking all
the compiling should nevertheless work just fine.
This PR adds the variable `BOARD_INSUFFICIENT_RAM` to tell the
`buildtest` to compile the code for a board, but omit the linking step.
Kaspar:
core/hwtimer: interrupt inconsistencies · Issue #924 · RIOT-OS/RIOT · GitHub
PR created by Ludwig:
RIOT-OS:master ← LudwigKnuepfer:issue-924
opened 01:51PM - 16 May 14 UTC
Before only the hardware timer's own interrupt was being disabled.
This led to a… race condition in the following scenario:
```
Thread1:
hwtimer_remove()
hwtimer_arch_disable_interrupt();
// INTERRUPT -> Thread2 (which has a higher priority than Thread1) gets scheduled
Thread2:
...
hwtimer_remove()
hwtimer_arch_disable_interrupt(); // hwtimer interrupt is already disabled
...
hwtimer_arch_enable_interrupt();
...
// yield | terminate -> Thread1 gets scheduled again
Thread1:
... // these instructions are being run with the hwtimer interrupt enabled
hwtimer_arch_enable_interrupt(); // hwtimer interrupt is already enabled
```
Fixes #924
Waiting for Kaspar (or someone else) to review.
Wrong `volatile`ness of thread pointers in sched.h · Issue #252 · RIOT-OS/RIOT · GitHub
Discussion between Kaspar and René ongoing.
The rest should be trivial as far as I can see.
If we manage to merge all this until Monday, we should be ready on Wednesday
for the release and have a BBQ on Thursday!?
Cheers,
Oleg
Hey folks!
Get ready for the final sprint!
Ludwig,
- can you acknowledge that #392
(https://github.com/RIOT-OS/RIOT/issues/392 ) can be untagged?
- can you ack https://github.com/RIOT-OS/RIOT/pull/1093 ?
Kaspar,
- can you review https://github.com/RIOT-OS/RIOT/pull/1193 ?
Christian,
- can you update https://github.com/RIOT-OS/RIOT/pull/977 according to
comments in #952 ?
- can you take a look at https://github.com/RIOT-OS/RIOT/pull/1208 and merge
at will?
Hauke,
- can you review https://github.com/RIOT-OS/RIOT/pull/1182 ?
René,
- is https://github.com/RIOT-OS/RIOT/pull/1216 ready to be merged?
If these PRs and related issues are handled, there are only two cleanup PRs to
be merged and René's X86 port to be ACK'ed and merged.
Go for it!
Cheers,
Oleg
Hi!
>Ludwig,
>- can you ack examples: added README.md to ccn-lite-client and relay by mehlis · Pull Request #1093 · RIOT-OS/RIOT · GitHub ?
I am still unsure what to do with the example after reading the README.
Maybe it's just me or it is required to know something about ccn to use it.
Maybe one can figure it out by actually running the example. While the
README certainly is an improvement I suggest someone else takes over and
reads it too.
Well, there should be at least two people on this mailing list with some
knowledge about CCN - I just chose one of them to take over the review.
Matthias or Emmanuel, do you have time today to take a look?
Cheers,
Oleg
