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.

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

fastd occasionally segfaults in Gluon 2020.1.x #2029

Closed
kpanic23 opened this issue May 22, 2020 · 5 comments
Closed

fastd occasionally segfaults in Gluon 2020.1.x #2029

kpanic23 opened this issue May 22, 2020 · 5 comments
Assignees
Labels
0. type: bug This is a bug 0. type: regression 1. severity: blocker This issue/pr is required for the next release 9. meta: upstream issue Issue pertains to an upstream project
Milestone

Comments

@kpanic23
Copy link
Contributor

Bug report

What is the problem?
fastd crashes with segfault, causing the node to lose gateway connection.

kern.info kernel: [80946.000734] fastd[5014]: segfault at 10 ip 00007f3c229823e9 sp 00007ffd2ce68f60 error 6 in libc.so[7f3c2297f000+49000]

What is the expected behaviour?

  • How do you think it should work instead?
    fastd running without crashing :o)

  • Did it work like that before?
    Yes, up until 2019.1.2.

Gluon Version:
2020.1.2

Site Configuration:
https://github.com/ff3l/site-ff3l

Custom patches:
none

@mweinelt mweinelt added this to the 2020.2 milestone May 22, 2020
@mweinelt mweinelt added 0. type: bug This is a bug 9. meta: upstream issue Issue pertains to an upstream project labels May 22, 2020
@neocturne neocturne self-assigned this May 22, 2020
@neocturne
Copy link
Member

The fastd-debug branch of https://github.com/NeoRaider/gluon now contains a patch that should fix the issue (and add a few sanity checks to catch problems as soon as possible).

This will likely not be the fix that will eventually make it into upstream musl, as it introduces unneeded memory barriers in hot code paths even for completely single-threaded programs, but as a temporary fix to ensure correctness it should be sufficient.

@neocturne
Copy link
Member

I have replaced the fix with a better one that doesn't affect single-threaded programs.

@neocturne
Copy link
Member

And another update, this time with the full patch series proposed by Rich Felker. Please test!

@rotanid rotanid added 0. type: regression 1. severity: blocker This issue/pr is required for the next release labels May 25, 2020
@oszilloskop
Copy link
Contributor

Almost two days with two Futros with patched firmware there were no more fastd crashes.

@neocturne
Copy link
Member

Fixed with c3be82f and backported to v2020.1.x.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0. type: bug This is a bug 0. type: regression 1. severity: blocker This issue/pr is required for the next release 9. meta: upstream issue Issue pertains to an upstream project
Projects
None yet
Development

No branches or pull requests

5 participants