sunriseos / SunriseOS

Horizon/NX kernel reimplementation
Apache License 2.0
232 stars 13 forks source link

Kernel: Use SeqCst for ARE_CPU_LOCALS_INIT #489

Open roblabla opened 5 years ago

roblabla commented 5 years ago

Relaxed is 100% broken in SMP context here.

roblabla commented 5 years ago

From @orycterope on discord

Since we set the atomic to true before we start the other cores, shouldn't Relaxed work?

I'm not 100% sure. I feel like it should work in practice, but I'm very unclear on whether this is considered legal or not.