-
Notifications
You must be signed in to change notification settings - Fork 3k
LWIP - fix recv blocking send on accepted sockets #2502
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
Conversation
When a socket is created via accept set the mode to nonblocking - a timeout of 1ms. This allows send and recv to occur at the same time.
/morph test |
@mbed-bot: TEST HOST_OSES=ALL |
[Build 824] |
Result: FAILUREYour command has finished executing! Here's what you wrote!
Outputmbed Build Number: 675 Test failed! |
/morph test |
Result: FAILUREYour command has finished executing! Here's what you wrote!
Outputmbed Build Number: 677 Test failed! |
/morph test |
Result: FAILUREYour command has finished executing! Here's what you wrote!
Outputmbed Build Number: 678 Test failed! |
I think this should fix issue #2435 as well. Edit: based on a quick test, it does. |
/morph test |
LGTM |
Result: FAILUREYour command has finished executing! Here's what you wrote!
Outputmbed Build Number: 680 Test failed! |
/morph test |
Result: FAILUREYour command has finished executing! Here's what you wrote!
Outputmbed Build Number: 682 Test failed! |
/morph test |
Result: FAILUREYour command has finished executing! Here's what you wrote!
Outputmbed Build Number: 688 Test failed! |
/morph test |
Result: SUCCESSYour command has finished executing! Here's what you wrote!
Outputmbed Build Number: 691 All builds and test passed! |
/morph test |
Result: FAILUREYour command has finished executing! Here's what you wrote!
Outputmbed Build Number: 692 Test failed! |
/morph test |
Result: FAILUREYour command has finished executing! Here's what you wrote!
Outputmbed Build Number: 693 Test failed! |
One of the failures was due to a small bug in the nist test. This should be fixed over in this pr #2529 |
/morph test |
1 similar comment
/morph test |
Result: FAILUREYour command has finished executing! Here's what you wrote!
Outputmbed Build Number: 702 Test failed! |
Result: SUCCESSYour command has finished executing! Here's what you wrote!
Outputmbed Build Number: 703 All builds and test passed! |
/morph test |
Result: FAILUREYour command has finished executing! Here's what you wrote!
Outputmbed Build Number: 704 Test failed! |
/morph test |
Result: FAILUREYour command has finished executing! Here's what you wrote!
Outputmbed Build Number: 706 Test failed! |
When a socket is created via accept set the mode to nonblocking -
a timeout of 1ms. This allows send and recv to occur at the same time.