Sam Price started a new discussion on 
bsps/microblaze/microblaze_fpga/start/_interrupt_handler.S: 
https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/845#note_136228

 >      mfs r3, rmsr
 >      swi  r3, r1, MICROBLAZE_INTERRUPT_FRAME_MSR
 >  
 > -    /* Indicate unknown interrupt source */
 > -    braid _ISR_Handler
 > +    /* Indicate unknown interrupt source; 0xFF tells _ISR_Handler to query 
 > the INTC. */
 > +    /* Use addik so MSR flags stay unchanged before calling the handler. */
 > +    /* Keep the branch delay slot empty to avoid side effects. */
 >      addik r5, r0, 0xFF
 > +
 > +    /* braid has a delay slot; use a harmless nop there. */
 > +    braid _ISR_Handler
 > +    nop

This needs reverted.

d stands for delay.
the instruction after braid will happen prior to the branch.

-- 
View it on GitLab: 
https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/845#note_136228
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