Sebastian Huber started a new discussion on bsps/sparc/leon3/start/eirq.c: 
https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1078#note_143476

 >      }
 >    } else {
 > -    rtems_interrupt_lock_context lock_context;
 > -    uint32_t ipend;
 > -
 > -    /*
 > -     * This is a very dangerous operation and should only be used for test
 > -     * software.  We may accidentally clear the pending state set by
 > -     * peripherals with this read-modify-write operation.
 > -     */
 > -    LEON3_IRQCTRL_ACQUIRE(&lock_context);
 > -    ipend = grlib_load_32(&regs->ipend);
 > -    ipend |= bit;
 > -    grlib_store_32(&regs->ipend, ipend);
 > -    LEON3_IRQCTRL_RELEASE(&lock_context);
 > +    #if defined(RTEMS_TEST)

No, there should be definitely no usage of RTEMS_TEST or whatever in the code 
base. This change breaks the validation tests. How did you test this?

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