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.
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
segmentation fault while serializing with 3.10.1 on x86_64/aarch64 with cpython 3.12 #479
Comments
Summary of user reported Note: I cannot reproduce this locally so I do not have a reproducer or trace available yet
https://github.com/home-assistant/core/blob/0c44051d2ab8a78d8b979aaf950f5dc1c5c5bfa4/homeassistant/helpers/entity_registry.py#L324
|
Got another
This one is a home assistant blue. |
So thats one is aarch64 |
Looks like this commit has an aarch64 and x86_64 only path 58a73f3#diff-abdceb17520cb5accf57f4d4eed04d97b84d89b341ce6c14071b9b4f79390c5cR115 |
Music Assistant reported the same issue and I think they went from 3.10.0 to 3.10.1 so that commit being the source tracks with the timeline |
Looks like the problem is probably fixed in 3.10.2. Thanks Unfortunately to looks like armv7 support is gone so we are stuck on 3.9.15 |
I tested 3.10.3 quite a bit today, and I couldn't repro the seg fault (I also couldn't repro it before on 3.10.1 though) Everything does still appear to be working as expected on armv7 and wheels build ok for it. |
We should get confirmation that the problem is fixed on May 29/30 from the next Home Assistant beta. |
Awaiting Home Assistant 2024.6.x beta to provide confirmation the seg fault issue is solved |
Ok, thanks for the report. I think this was the off-by-one fixed in 3.10.2. That also dropped arm7 wheels due the CI job breaking and not really being something I can fix. If someone if motivated enough to do a just cross-compile or cross-compile and test in QEMU job that would be welcome, I just can't. Please reopen if not fixed. |
Thanks for following up Home Assistant is successful building wheels for armv7 (and they work) on newer versions. I'll see about getting a PR together to add back armv7 later this week. |
The text was updated successfully, but these errors were encountered: