From 21637fd3804d0bcda03ed8695357e8f26427d215 Mon Sep 17 00:00:00 2001 From: Till Straumann Date: Wed, 14 May 2008 22:33:32 +0000 Subject: 2008-05-14 Till Straumann * new-exceptions/bspsupport/irq.c: moved malloc/free outside of irq-protected critical section. --- c/src/lib/libcpu/powerpc/ChangeLog | 5 +++++ c/src/lib/libcpu/powerpc/new-exceptions/bspsupport/irq.c | 9 ++++++--- 2 files changed, 11 insertions(+), 3 deletions(-) (limited to 'c/src') diff --git a/c/src/lib/libcpu/powerpc/ChangeLog b/c/src/lib/libcpu/powerpc/ChangeLog index 360e3cba50..d53193ec07 100644 --- a/c/src/lib/libcpu/powerpc/ChangeLog +++ b/c/src/lib/libcpu/powerpc/ChangeLog @@ -1,3 +1,8 @@ +2008-05-14 Till Straumann + + * new-exceptions/bspsupport/irq.c: moved malloc/free + outside of irq-protected critical section. + 2008-04-24 Joel Sherrill * mpc8260/console-generic/console-generic.c, diff --git a/c/src/lib/libcpu/powerpc/new-exceptions/bspsupport/irq.c b/c/src/lib/libcpu/powerpc/new-exceptions/bspsupport/irq.c index d972ba9b7b..1b9dcafefe 100644 --- a/c/src/lib/libcpu/powerpc/new-exceptions/bspsupport/irq.c +++ b/c/src/lib/libcpu/powerpc/new-exceptions/bspsupport/irq.c @@ -86,16 +86,18 @@ int BSP_install_rtems_shared_irq_handler (const rtems_irq_connect_data* irq) return 0; } + /* pre-allocate memory outside of critical section */ + vchain = (rtems_irq_connect_data*)malloc(sizeof(rtems_irq_connect_data)); + rtems_interrupt_disable(level); if ( (int)rtems_hdl_tbl[irq->name].next_handler == -1 ) { rtems_interrupt_enable(level); printk("IRQ vector %d already connected to an unshared handler\n",irq->name); + free(vchain); return 0; } - vchain = (rtems_irq_connect_data*)malloc(sizeof(rtems_irq_connect_data)); - /* save off topmost handler */ vchain[0]= rtems_hdl_tbl[irq->name]; @@ -267,7 +269,6 @@ int BSP_remove_rtems_irq_handler (const rtems_irq_connect_data* irq) vchain = vchain->next_handler; rtems_hdl_tbl[irq->name]= *vchain; } - free(vchain); } /* Only disable at PIC if we removed the last handler */ @@ -282,6 +283,8 @@ int BSP_remove_rtems_irq_handler (const rtems_irq_connect_data* irq) rtems_interrupt_enable(level); + free(vchain); + return 1; } -- cgit v1.2.3