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.

Sign up for GitHub

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

Jump to bottom

Socket::set_timeout doesn't work properly for the server side TCPClient instance #2435

Closed
tommikas opened this issue Aug 12, 2016 · 5 comments
Closed

Socket::set_timeout doesn't work properly for the server side TCPClient instance #2435

tommikas opened this issue Aug 12, 2016 · 5 comments
Labels
tracking type: bug

Comments

@tommikas
Copy link
Contributor

tommikas commented Aug 12, 2016 •
edited
Loading

  1. create, open, bind and set to listen a new TCPServer instance
  2. accept a new connection on a new TCPSocket instance
  3. set the TCPSocket to non-blocking mode
  4. call TCPSocket::recv

TCPSocket::recv blocks indefinitely if no data is pending or received.
@tommikas tommikas mentioned this issue Aug 22, 2016
Merged
@geky
Copy link
Contributor

geky commented Aug 22, 2016

Hi @tommikas, thanks for reporting this issue. It looks like this has been fixed by the unrelated pr from @c1728p9: #2502

Let us know if the problem still persists : )

@tommikas
Copy link
Contributor Author

http://jira.arm.com/browse/IOTSFW-2864

@sg-
Copy link
Contributor

sg- commented Aug 25, 2016

Please file these issues with a test case or example program or code snippet to reproduce.

Can you confirm if #2502 resolved this issue?

@ciarmcom
Copy link
Member

ARM Internal Ref: IOTMORF-442

@tommikas
Copy link
Contributor Author

tommikas commented Sep 1, 2016

#2502 fixed this. Closing.

@tommikas tommikas closed this as completed Sep 1, 2016
artokin pushed a commit to artokin/mbed-os that referenced this issue Sep 2, 2020
Squashed 'features/nanostack/sal-stack-nanostack/' changes from 48609ae…
9bea7f4 
…..91acece

91acece Remove test files
6568bc1 Merge branch 'release_internal' into release_external
4192cc8 Added configuration for RADIUS retry timer (ARMmbed#2438)
684b714 Added support for retries and multiple sockets to RADIUS client (ARMmbed#2426)
89e0ae0 WS: Restart auto CCA threshold after discovery (ARMmbed#2435)
dbb09b1 MAC/WS: Min possible Tack to 1ms and CCA interval to 2ms (ARMmbed#2434)
43b2ae2 Cca threshold test (ARMmbed#2436)
28108e1 Added device min sensitivity setting and stack information
7060c70 Cca threshold test (ARMmbed#2432)
640be71 WS: temporarily set default CCA threshold to -80 (ARMmbed#2431)
0a472ae WS: Calculate UFSI drift and trace (ARMmbed#2430)
61d3db8 Create APIs for DNS cache results
587add5 MAC: Validate TX time (ARMmbed#2429)
a1bfed4 Added typecast when computing max_timout from drift (ARMmbed#2428)
089fb3b Neighbour temporary entry update and Enhanced ACK tx update
70244f6 Wi-sun parameter and debug trace update
5752eae Created validate TX time handler (ARMmbed#2423)
022d61f Wi-sun Neighbour table update and DHCP new callback
857b41f Merge pull request ARMmbed#2421 from ARMmbed/update_from_mbed_os
1a9dd13 (via Mbed-OS)WS Management API missing include
4318f37 Calculate drift in critical state (ARMmbed#2419)
01a1909 FHSS WS: Do not use drift compensation with unpredictable linux timer (ARMmbed#2418)

git-subtree-dir: features/nanostack/sal-stack-nanostack
git-subtree-split: 91acece
artokin pushed a commit to artokin/mbed-os that referenced this issue Sep 7, 2020
Squashed 'features/nanostack/sal-stack-nanostack/' changes from 48609ae…
3d6d4d5 
…..d879e6d

d879e6d Merge branch 'release_internal' into release_external
eef9246 Fixed network border router timeout recovery and EAPOL relay address fix
bac7ca6 Changed RADIUS MTU and small fixes
a9f8b75 Addeed support for DHCP vendor data
d8f0003 DHCPv6 functionality update
7fe0423 Added DHCPv6 vendor data generation for DNS queries
639f9db FHSS: Changed retry backoffs when no BC schedule or TX slots (ARMmbed#2440)
91acece Remove test files
6568bc1 Merge branch 'release_internal' into release_external
4192cc8 Added configuration for RADIUS retry timer (ARMmbed#2438)
684b714 Added support for retries and multiple sockets to RADIUS client (ARMmbed#2426)
89e0ae0 WS: Restart auto CCA threshold after discovery (ARMmbed#2435)
dbb09b1 MAC/WS: Min possible Tack to 1ms and CCA interval to 2ms (ARMmbed#2434)
43b2ae2 Cca threshold test (ARMmbed#2436)
28108e1 Added device min sensitivity setting and stack information
7060c70 Cca threshold test (ARMmbed#2432)
640be71 WS: temporarily set default CCA threshold to -80 (ARMmbed#2431)
0a472ae WS: Calculate UFSI drift and trace (ARMmbed#2430)
61d3db8 Create APIs for DNS cache results
587add5 MAC: Validate TX time (ARMmbed#2429)
a1bfed4 Added typecast when computing max_timout from drift (ARMmbed#2428)
089fb3b Neighbour temporary entry update and Enhanced ACK tx update
70244f6 Wi-sun parameter and debug trace update
5752eae Created validate TX time handler (ARMmbed#2423)
022d61f Wi-sun Neighbour table update and DHCP new callback
857b41f Merge pull request ARMmbed#2421 from ARMmbed/update_from_mbed_os
1a9dd13 (via Mbed-OS)WS Management API missing include
4318f37 Calculate drift in critical state (ARMmbed#2419)
01a1909 FHSS WS: Do not use drift compensation with unpredictable linux timer (ARMmbed#2418)

git-subtree-dir: features/nanostack/sal-stack-nanostack
git-subtree-split: d879e6d
artokin pushed a commit to artokin/mbed-os that referenced this issue Sep 7, 2020
Squashed 'connectivity/nanostack/sal-stack-nanostack/' changes from 4…
9a4e8a5 
…8609ae..d879e6d

d879e6d Merge branch 'release_internal' into release_external
eef9246 Fixed network border router timeout recovery and EAPOL relay address fix
bac7ca6 Changed RADIUS MTU and small fixes
a9f8b75 Addeed support for DHCP vendor data
d8f0003 DHCPv6 functionality update
7fe0423 Added DHCPv6 vendor data generation for DNS queries
639f9db FHSS: Changed retry backoffs when no BC schedule or TX slots (ARMmbed#2440)
91acece Remove test files
6568bc1 Merge branch 'release_internal' into release_external
4192cc8 Added configuration for RADIUS retry timer (ARMmbed#2438)
684b714 Added support for retries and multiple sockets to RADIUS client (ARMmbed#2426)
89e0ae0 WS: Restart auto CCA threshold after discovery (ARMmbed#2435)
dbb09b1 MAC/WS: Min possible Tack to 1ms and CCA interval to 2ms (ARMmbed#2434)
43b2ae2 Cca threshold test (ARMmbed#2436)
28108e1 Added device min sensitivity setting and stack information
7060c70 Cca threshold test (ARMmbed#2432)
640be71 WS: temporarily set default CCA threshold to -80 (ARMmbed#2431)
0a472ae WS: Calculate UFSI drift and trace (ARMmbed#2430)
61d3db8 Create APIs for DNS cache results
587add5 MAC: Validate TX time (ARMmbed#2429)
a1bfed4 Added typecast when computing max_timout from drift (ARMmbed#2428)
089fb3b Neighbour temporary entry update and Enhanced ACK tx update
70244f6 Wi-sun parameter and debug trace update
5752eae Created validate TX time handler (ARMmbed#2423)
022d61f Wi-sun Neighbour table update and DHCP new callback
857b41f Merge pull request ARMmbed#2421 from ARMmbed/update_from_mbed_os
1a9dd13 (via Mbed-OS)WS Management API missing include
4318f37 Calculate drift in critical state (ARMmbed#2419)
01a1909 FHSS WS: Do not use drift compensation with unpredictable linux timer (ARMmbed#2418)

git-subtree-dir: connectivity/nanostack/sal-stack-nanostack
git-subtree-split: d879e6d
winneymj pushed a commit to winneymj/mbed-os that referenced this issue Sep 29, 2020
@mjwinney
Squashed 'connectivity/nanostack/sal-stack-nanostack/' changes from 4…
bc3ae1c 
…8609ae..d879e6d

d879e6d Merge branch 'release_internal' into release_external
eef9246 Fixed network border router timeout recovery and EAPOL relay address fix
bac7ca6 Changed RADIUS MTU and small fixes
a9f8b75 Addeed support for DHCP vendor data
d8f0003 DHCPv6 functionality update
7fe0423 Added DHCPv6 vendor data generation for DNS queries
639f9db FHSS: Changed retry backoffs when no BC schedule or TX slots (ARMmbed#2440)
91acece Remove test files
6568bc1 Merge branch 'release_internal' into release_external
4192cc8 Added configuration for RADIUS retry timer (ARMmbed#2438)
684b714 Added support for retries and multiple sockets to RADIUS client (ARMmbed#2426)
89e0ae0 WS: Restart auto CCA threshold after discovery (ARMmbed#2435)
dbb09b1 MAC/WS: Min possible Tack to 1ms and CCA interval to 2ms (ARMmbed#2434)
43b2ae2 Cca threshold test (ARMmbed#2436)
28108e1 Added device min sensitivity setting and stack information
7060c70 Cca threshold test (ARMmbed#2432)
640be71 WS: temporarily set default CCA threshold to -80 (ARMmbed#2431)
0a472ae WS: Calculate UFSI drift and trace (ARMmbed#2430)
61d3db8 Create APIs for DNS cache results
587add5 MAC: Validate TX time (ARMmbed#2429)
a1bfed4 Added typecast when computing max_timout from drift (ARMmbed#2428)
089fb3b Neighbour temporary entry update and Enhanced ACK tx update
70244f6 Wi-sun parameter and debug trace update
5752eae Created validate TX time handler (ARMmbed#2423)
022d61f Wi-sun Neighbour table update and DHCP new callback
857b41f Merge pull request ARMmbed#2421 from ARMmbed/update_from_mbed_os
1a9dd13 (via Mbed-OS)WS Management API missing include
4318f37 Calculate drift in critical state (ARMmbed#2419)
01a1909 FHSS WS: Do not use drift compensation with unpredictable linux timer (ARMmbed#2418)

git-subtree-dir: connectivity/nanostack/sal-stack-nanostack
git-subtree-split: d879e6d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
tracking type: bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@sg- @geky @ciarmcom @tommikas