-
-
Notifications
You must be signed in to change notification settings - Fork 68
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
dnsperf: net.h:133: perf_net_sockeq: Assertion `sock_b' failed. #208
Comments
I'm unable to hit this, although I hit another bug :D Possible to get a stack trace via gdb or something? |
Luckily I've reproduced it locally, even though build with Retested on commit 638e7e7.
|
This might or might not be the same thing as #216. |
- `dnsperf`: Fix DNS-OARC#208: - `recv_one()`: Fix handling errno, only store EAGAIN if no other error has been received - `do_recv()`: Don't break on error as it will count it as a received message
Those CFLAGS helped me replicate! Try the PR? |
It's probably incomplete. Even with the fix I can still see behavior described in #216. Also, occasionally dnsperf exists with
or
while sometimes it produces "normal looking" stats with all zeroes. Mainly the "badfd" error message is suspicious. All zeroes are also kind of weird as "all sockets reported failure" is what actually should happen. Prolonging the test length to 1 second to allow all RSTs to be processed does not change the behavior, just requires lot more cycles to get there. |
Well, I expected that. At least the assert is fixed. Will look at the badfd more. |
- `dnsperf`: Issue DNS-OARC#208: - `recv_one()`: Fix handling errno, only store EAGAIN if no other error has been received - `do_recv()`: Don't break on error as it will count it as a received message
- `net`: Fix DNS-OARC#208: Treat `EBADF` as `EAGAIN` for stateful connections, receive thread might read from a closed socket if send thread is reconnecting
@pspacek let's try again 😄 |
Version: 2.10.0, commit c1bef8b
Sometimes I've noticed crash in the
-m tcp
mode:Shortest reproducer I can come up with is:
where 127.0.0.1 does not listen on port 53.
/tmp/qlist
has justnet. SOA
and nothing else. Repeat it couple times and it should crash within a minute.Unfortunately the machine where this happens is in a farm and does not have debug build and all the other jazz needed to get proper traceback.
The text was updated successfully, but these errors were encountered: