Skip to content
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

delete 操作における期待されないコードパスを踏むバグ #33

Open
thawk105 opened this issue Oct 2, 2023 · 2 comments
Open

delete 操作における期待されないコードパスを踏むバグ #33

thawk105 opened this issue Oct 2, 2023 · 2 comments
Assignees
@thawk105 @ban-nobuhiro
Labels
enhancement New feature or request

Comments

@thawk105
Copy link
Contributor

thawk105 commented Oct 2, 2023 •
edited
Loading

問題の事象

・https://github.com/project-tsurugi/yakushima/actions/runs/6376901833/job/17304616012#step:6:842
delete 操作において、期待されないコードパスを踏んでいる。
・確率的に発生している。

TBD
@thawk105 thawk105 self-assigned this Oct 2, 2023
@thawk105 thawk105 added the bug Something isn't working label Oct 2, 2023
thawk105 added a commit that referenced this issue Oct 6, 2023
@thawk105
update to bypass unexpected code path.
4cbdb68 
#33
thawk105 added a commit that referenced this issue Oct 6, 2023
@thawk105
update to bypass unexpected code path.
898e161 
#33
@thawk105 thawk105 added enhancement New feature or request and removed bug Something isn't working labels Oct 6, 2023
@thawk105
Copy link
Contributor Author

thawk105 commented Oct 6, 2023

898e161#diff-045d0760c50a3f1f7c177337b8ac109d13435be1b714b4b37330f9a61f22828cR72-R78

期待されないメモリ状態を読んでいるが、問題が起きないようにロジックを変更した。
理想的にはこのパスを踏まないことだが、そのためにはコンパイラフェンス・メモリフェンスを適切に仕込むチューニングを必要とすると考える。

状況から本イシューラベルをバグから enhancement にし、作業優先度を下げる。

@ban-nobuhiro
Copy link
Contributor

見てみます

@ban-nobuhiro ban-nobuhiro self-assigned this May 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@thawk105 thawk105

@ban-nobuhiro ban-nobuhiro

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@thawk105 @ban-nobuhiro