[RTEMS Project] #1815: _ISR_Nest_level should be protected with interrupts disabled in function _Watchdog_Insert

[RTEMS trac]

#1815: _ISR_Nest_level should be protected with interrupts disabled in function
_Watchdog_Insert
----------------------+----------------------------
 Reporter:  silvah15  |       Owner:  joel.sherrill
     Type:  defect    |      Status:  new
 Priority:  normal    |   Milestone:  5.0
Component:  cpukit    |     Version:  4.11
 Severity:  normal    |  Resolution:
 Keywords:            |
----------------------+----------------------------
Changes (by sebastian.huber):

 * milestone:  4.11 => 5.0


Old description:

> _ISR_Nest_level should be protected with interrupts disabled in function
> _Watchdog_Insert, because another interrupt can occur after
> _ISR_Nest_level is
> saved, causing inconsistency in the global variable.

New description:

 _ISR_Nest_level should be protected with interrupts disabled in function
 _Watchdog_Insert, because another interrupt can occur after
 _ISR_Nest_level is
 saved, causing inconsistency in the global variable.

--

Comment:

 I am not sure what the problem is.  Interrupts are nested, so the code
 looks all right to me.

 For SMP we have to re-write the watchdog stuff anyway and the
 _Watchdog_Sync_level and _Watchdog_Sync_count will go away.

--
Ticket URL: <http://devel.rtems.org/ticket/1815#comment:4>
RTEMS Project <http://www.rtems.org/>
RTEMS Project