Merge request https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1084 was reviewed by Xinhong Hu
-- Xinhong Hu commented on a discussion: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1084#note_144367 ACK, understood. I will not resolve review threads unless requested. -- Xinhong Hu commented on a discussion: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1084#note_144368 Commits have been squashed and the MR was force-pushed. -- Xinhong Hu commented on a discussion on testsuites/psxtests/psxmsgq_prio_wakeup/init.c: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1084#note_144369 > + goto cleanup; > + } > + usleep( 10000 ); I have updated my test to use semaphore-based synchronization instead of usleep(), ensuring threads deterministically block before mq_send(). This removes timing dependencies and aligns with the approach discussed above. -- Xinhong Hu commented on a discussion on cpukit/posix/src/mqueueopen.c: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1084#note_144370 > + char *name; > + Status_Control status; > + CORE_message_queue_Disciplines discipline; @gedare Thank you for the guidance. I have manually adjusted the formatting based on your feedback, and the CI pipeline is now passing. If there are any remaining issues or if you still prefer that I use `clang-format` for consistency, please let me know—I’m happy to set it up and apply it. -- View it on GitLab: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1084 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
