New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix fast lock on windows (reassignment oldval) #956

Merged

shirok merged 1 commit into shirok:master from Hamayama:win_fastlock2

Oct 30, 2023

Contributor

Hamayama commented Oct 30, 2023

コミット 07ed42b (2023-10-28) で、Windows の fast lock (spin lock) が、
ScmAtomicVar を使うように変更されましたが、
このときにロックできなくなっていたため修正しました。
以下の test_0002_write で不具合を確認しました。
https://gist.github.com/Hamayama/fda48fd11d3583653edbbdd9d7d7e714
原因は、src/gauche/priv/atomicP.h で、
AO_compare_and_swap_full() が 3 パターン定義されていますが、
元々の libatomic_ops のもの以外は、
条件不成立時に oldval を現在の値で置き換えるという仕様があったためです。
この仕様のために、スピンの2周目で、条件が逆転していました。

＜関連 Issue＞


          Fix fast lock on windows (reassignment oldval)

1f0568b

shirok merged commit c9334bc into shirok:master

5 checks passed

Owner

shirok commented Oct 30, 2023

Good catch! Mahalo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment