Matheus Pecoraro commented on a discussion on 
cpukit/score/cpu/x86_64/include/rtems/score/cpu.h: 
https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/71#note_108066

 > +  }                                                         \
 >  }
 >  
 >  #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 */    \
 >  }
 >  
 >  #define _CPU_ISR_Flash(_level)                              \
 >  {                                                           \
 > -  amd64_enable_interrupts();                                \
 > +  if(!_level) {                                             \

Then I think the way it's currently done is fine for now. I just did the 
whitespace and syntax change Gedare had originally suggested.

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


_______________________________________________
bugs mailing list
[email protected]
http://lists.rtems.org/mailman/listinfo/bugs

Reply via email to