diff options
| author | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2015-11-11 12:05:22 +0100 |
|---|---|---|
| committer | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2015-11-12 08:21:45 +0100 |
| commit | 610909fa85dde9278625ce839b98972fa336d06f (patch) | |
| tree | 83f6ca23652ffff781269d947f78a79083d3e4c7 | |
| parent | Fix interrupt epilogue for ARMv7-AR and PowerPC (diff) | |
| download | rtems-610909fa85dde9278625ce839b98972fa336d06f.tar.bz2 | |
irq-server: Fix race condition on SMP systems
| -rw-r--r-- | c/src/lib/libbsp/shared/src/irq-server.c | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/c/src/lib/libbsp/shared/src/irq-server.c b/c/src/lib/libbsp/shared/src/irq-server.c index 3537493ac0..7dee1dc596 100644 --- a/c/src/lib/libbsp/shared/src/irq-server.c +++ b/c/src/lib/libbsp/shared/src/irq-server.c @@ -57,20 +57,21 @@ static unsigned bsp_interrupt_server_errors; static void bsp_interrupt_server_trigger(void *arg) { + rtems_interrupt_lock_context lock_context; bsp_interrupt_server_entry *e = arg; bsp_interrupt_vector_disable(e->vector); - if (rtems_chain_is_node_off_chain(&e->node)) { - rtems_interrupt_lock_context lock_context; + rtems_interrupt_lock_acquire(&bsp_interrupt_server_lock, &lock_context); - rtems_interrupt_lock_acquire(&bsp_interrupt_server_lock, &lock_context); + if (rtems_chain_is_node_off_chain(&e->node)) { rtems_chain_append_unprotected(&bsp_interrupt_server_chain, &e->node); - rtems_interrupt_lock_release(&bsp_interrupt_server_lock, &lock_context); } else { ++bsp_interrupt_server_errors; } + rtems_interrupt_lock_release(&bsp_interrupt_server_lock, &lock_context); + rtems_event_system_send(bsp_interrupt_server_id, RTEMS_EVENT_SYSTEM_SERVER); } |