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
Workaround for division by zero crash #385
Conversation
Also, the author of this PR knows very little about C++, DPDK, and BESS... we ask reviewers to please be gentle :) |
b826989
to
08135fd
Compare
this workaround has been running on 3 deployments for more than 24 hours, no crash has been reported since then. I'm planning to merge this soon, after I figure out why CI is failing. |
pls look at the comments given. |
@amarsri28 I replied to your comments on #376. It looks like you don't have any explicit concern with this PR, so I'm going ahead and merge this. |
retest this please |
This reverts commit 103b198.
We don't know yet the root cause of #376. The crash has been observed multiple times per day on a few production deployments. This is a workaround to prevent the crash while we investigate the root cause.
We don't know how to reproduce the issue, but we know it happens once in a while (tens of minutes to hours after starting bessd). As a consequence, we assume this workaround will be hit only a few times, causing only a few packets to skip the meter and be marked as green. We can better asses the frequency by looking at the new log message.