-
Notifications
You must be signed in to change notification settings - Fork 103
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
Crash on sending_302_without_preparing unit test #1157
Comments
Unit tests are broken: just hit following crash on simple
|
The last Oops is reproduced on #1158 , so assign to @ikoveshnikov : probably the problem is already fixed in some recent PRs. |
firstly in DEFINE_TLS_TEST()->kernel_fpu_begin() and secondly in ttls_ecp_group_free()->ttls_bzero_safe()->kernel_fpu_begin(). The fix moves all the TLS unit tests to test_tls.c from tls/ and make each test responsible for calling kernel_fpu_{begin,end}(). The crypto routines can be split into 2 groups: called from process context of Tempesta FW initialization and called in run-time, softirq context. Only the second group must be called with saved FPU context. In fact, current crypto routines (covered by the test) don't use SIMD much and this is going to change in #1064.
Test crash #1157 (comment) is still here |
I couldn't reproduced the crash on unit tests neither with KASAN disabled nor with enabled. But there are some KASAN warnings and crashes. Start Tempesta with blank configuration:
Send a request and get a crash:
|
Just hit the test crash again on current master. I attach my kernel config for the issue reproducing.
|
Fix #1157, multiple bugs in sending_302 & sending_302_without_preparing tests
Run unit tests to reproduce the issue:
The text was updated successfully, but these errors were encountered: