RTEMS | x86_64: Fix ISR handler macros (!71)

Sebastian Huber (@sebhub) gitlab at rtems.org
Tue Jun 25 17:20:39 UTC 2024




Sebastian Huber started a new discussion on cpukit/score/cpu/x86_64/include/rtems/score/cpu.h: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/71#note_108067

 >  {                                                           \
 > -  amd64_enable_interrupts();                                \
 > -  _level = 0;                                               \
 > -  (void) _level; /* Prevent -Wunused-but-set-variable */    \
 > +  if (_level == 0) {                                        \
 > +    amd64_enable_interrupts();                              \
 > +  }                                                         \
 >  }
 >  
 >  #define _CPU_ISR_Disable(_level)                            \
 >  {                                                           \
 > -  amd64_enable_interrupts();                                \
 > -  _level = 1;                                               \
 > +  _level = _CPU_ISR_Get_level();                            \
 > +  amd64_disable_interrupts();                               \
 >    (void) _level; /* Prevent -Wunused-but-set-variable */    \

The warning suppression should be removed.

-- 
View it on GitLab: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/71#note_108067
You're receiving this email because of your account on gitlab.rtems.org.


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rtems.org/pipermail/bugs/attachments/20240625/6c1286fa/attachment-0001.htm>


More information about the bugs mailing list