Wayne Thornton commented on a discussion on cpukit/dhrl/dhrl.c: 
https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1193#note_150866

 > +    RTEMS_DEFAULT_ATTRIBUTES,
 > +    &ctx->worker_a_id
 > +  );
 > +  if ( sc != RTEMS_SUCCESSFUL ) {
 > +    return sc;
 > +  }
 > +
 > +  CPU_ZERO( &cpuset );
 > +  CPU_SET( core_a, &cpuset );
 > +  sc = rtems_task_set_affinity(
 > +    ctx->worker_a_id,
 > +    sizeof( cpu_set_t ),
 > +    &cpuset
 > +  );
 > +  if ( sc != RTEMS_SUCCESSFUL ) {
 > +    return sc;

Understood. I've removed the `goto` labels entirely and unrolled the cleanup 
calls directly into each failure condition block so that resources are still 
properly freed. Will push in next commit.

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