acassis commented on PR #20012: URL: https://github.com/apache/nuttx/pull/20012#issuecomment-5470278413
> @acassis Regarding point #4, I understand the concern about riscv_hartid_to_cpuid() being used as an array index. Since the in-tree callers currently provide valid CPU mappings, would you recommend keeping the DEBUGASSERT as an invariant check, or is there a preferred way in NuttX to handle an invalid mapping here? I think it is better to prevent it in the release version too, not only on development/testing build -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
