RTEMS | Draft: bsps/leon3: Limit the interrupt map to 32 lines (!1514)

[Matteo Concas (@matteo.concas)]

Matteo Concas commented on a discussion on bsps/sparc/leon3/start/eirq.c: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1514#note_160107

 >  
 >  #ifdef LEON3_IRQAMP_IRQMAP
 >  /* Mapping from bus interrupt lines to IRQ(A)MP interrupt lines */
 > -rtems_vector_number LEON3_IrqCtrl_Mapping[ BSP_INTERRUPT_VECTOR_MAX_MAP + 1 ];
 > +rtems_vector_number LEON3_IrqCtrl_Mapping[ BSP_INTERRUPT_VECTOR_COUNT ];
 > +
 > +/*
 > + * A map field is eight bits wide.  A field value above the last interrupt
 > + * line of the IRQ(A)MP selects no interrupt line.  Line 0 is no interrupt.
 > + */
 > +static rtems_vector_number leon3_irqmap_line( uint32_t field )

Thanks for the update, I still think we should either assume the value to always be valid (and remove `leon3_irqmap_line()`) or consider an incorrect value as a fatal error.

-- 
View it on GitLab: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1514#note_160107
You're receiving this email because of your account on gitlab.rtems.org. Unsubscribe from this thread: https://gitlab.rtems.org/-/namespace/49/sent_notifications/5-8t8syiv4bouo1lcgb6lmdla24-1d/unsubscribe | Manage all notifications: https://gitlab.rtems.org/-/profile/notifications | Help: https://gitlab.rtems.org/help


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