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

mu index v 1.10.0 has a segmentation fault ~15% of time on macOS 13.3 #2469

Closed
powellb opened this issue Apr 5, 2023 · 5 comments
Closed

mu index v 1.10.0 has a segmentation fault ~15% of time on macOS 13.3 #2469

powellb opened this issue Apr 5, 2023 · 5 comments
Labels
bug mu needinfo

Comments

@powellb
Copy link

powellb commented Apr 5, 2023

Running mu index from command-line works 75-85% of the time. The other times it crashes with a segmentation fault.

To Reproduce
Repeated calls to mu index

Environment
macOS 13.3, homebrew installed mu, emacs v. 29.0.60 (however, the issue is not in mu4e)

Checklist
This issue never occurred prior to updating to v1.10.0

The core dump is attached.

core.txt
@djcb djcb mentioned this issue Apr 6, 2023
Closed
@djcb
Copy link
Owner

djcb commented Apr 6, 2023

Ah, we've seen this, but never with a symbolic backtrace before, so this should help. Thanks!

@djcb djcb removed the new label Apr 6, 2023
djcb added a commit that referenced this issue Apr 8, 2023
@djcb
mu-logger: Add lock for file-logger and stress test
2aa0d20 
Add a lock to the file logger and add a unit-test to stress test it.
(It does fail without the lock, and passed with it).

Should help for: #2469.
@djcb
Copy link
Owner

djcb commented Apr 8, 2023

I don't have the MacOS setup but I've pushed a candidate fix to master that might help for this.

djcb added a commit that referenced this issue Apr 9, 2023
@djcb
mu-logger: Add lock for file-logger and stress test
e29dc2d 
Add a lock to the file logger and add a unit-test to stress test it.
(It does fail without the lock, and passed with it).

Should help for: #2469.
@djcb djcb added the needinfo label Apr 9, 2023
@djcb
Copy link
Owner

djcb commented Apr 9, 2023

v1.10.2 contains a possible fix for this; can your retry? Thanks.

@powellb
Copy link
Author

powellb commented Apr 9, 2023

Thank you. I have updated, and I just ran a shell-script to execute mu index 100 times. Not a single failure (as mentioned before, I would have had 15-20 failures prior).

It seems that this issue is closed.

@djcb
Copy link
Owner

djcb commented Apr 10, 2023

Thanks! Okay, I'm closing this.

@djcb djcb closed this as completed Apr 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug mu needinfo
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@powellb @djcb