Merge pull request #617 from Tetralux/dep-yield-processor

Rename `yield_processor` to `cpu_relax`
This commit is contained in:
gingerBill
2020-04-21 18:07:04 +01:00
committed by GitHub
2 changed files with 7 additions and 5 deletions
+5 -4
View File
@@ -1,8 +1,9 @@
package sync
foreign {
@(link_name="llvm.x86.sse2.pause")
yield_processor :: proc() ---
import "core:intrinsics"
cpu_relax :: inline proc() {
intrinsics.cpu_relax();
}
Ticket_Mutex :: struct {
@@ -18,7 +19,7 @@ ticket_mutex_init :: proc(m: ^Ticket_Mutex) {
ticket_mutex_lock :: inline proc(m: ^Ticket_Mutex) {
ticket := atomic_add(&m.ticket, 1, .Relaxed);
for ticket != m.serving {
yield_processor();
intrinsics.cpu_relax();
}
}
+2 -1
View File
@@ -2,6 +2,7 @@
package thread;
import "core:runtime"
import "core:intrinsics"
import "core:sync"
import "core:sys/unix"
@@ -125,7 +126,7 @@ join :: proc(t: ^Thread) {
if sync.atomic_swap(&t.already_joined, true, .Sequentially_Consistent) {
for {
if sync.atomic_load(&t.done, .Sequentially_Consistent) do return;
sync.yield_processor();
intrinsics.cpu_relax();
}
}