Gedare Bloom started a new discussion on cpukit/dhrl/dhrl.c: 
https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1193#note_150598

 > +    next_epoch,
 > +    memory_order_release
 > +  );
 > +
 > +  rtems_barrier_wait( ctx->start_barrier, RTEMS_NO_TIMEOUT );
 > +
 > +  /* Spin utilizing the architecture-specific _CPU_Pause_speculation 
 > instruction hook */
 > +  while (
 > +    atomic_load_explicit( &ctx->winning_data_ptr, memory_order_acquire ) ==
 > +    (uintptr_t) NULL
 > +  ) {
 > +    _CPU_Pause_speculation();
 > +  }
 > +
 > +  winning_result = (volatile void *)
 > +    atomic_load_explicit( &ctx->winning_data_ptr, memory_order_acquire );

This could be simplified.

```
while ( winning_result == NULL ) {
  winning_result = atomic_load_explicit(...);
  _CPU_Pause_speculation();
}
```

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