diff options
author | Bjoern B. Brandenburg <bbb@cs.unc.edu> | 2010-01-29 19:25:26 -0500 |
---|---|---|
committer | Andrea Bastoni <bastoni@cs.unc.edu> | 2010-05-29 17:25:50 -0400 |
commit | 5e987d486c0f89d615d134512938fc1198b3ca67 (patch) | |
tree | b781dfcce3dcfe37e8c0a18e694bc58165165ac2 /litmus/ctrldev.c | |
parent | 37b840336a1663a5ce62d663a702d9afefd56d23 (diff) |
Bugfix: clear LITMUS^RT state on fork completely
When a real-time task forks, then its LITMUS^RT-specific fields should be cleared,
because we don't want real-time tasks to spawn new real-time tasks that bypass
the plugin's admission control (if any).
This was broken in three ways:
1) kernel/fork.c did not erase all of tsk->rt_param, only the first few bytes due to
a wrong size argument to memset().
2) It should have been calling litmus_fork() instead anyway.
3) litmus_fork() was _also_ not clearing all of tsk->rt_param, due to another size
argument bug.
Interestingly, 1) and 2) can be traced back to the 2007->2008 port,
whereas 3) was added by Mitchell much later on (to dead code, no less).
I'm really surprised that this never blew up before.
Diffstat (limited to 'litmus/ctrldev.c')
0 files changed, 0 insertions, 0 deletions