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.

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

Add check for EAGAIN in OS_DoSelect loop #858

Closed
skliper opened this issue Mar 10, 2021 · 0 comments · Fixed by #902 or #917
Closed

Add check for EAGAIN in OS_DoSelect loop #858

skliper opened this issue Mar 10, 2021 · 0 comments · Fixed by #902 or #917
Assignees
Milestone

Comments

@skliper
Copy link
Contributor

skliper commented Mar 10, 2021

Is your feature request related to a problem? Please describe.
On some other UNIX systems, select() can fail with the error EAGAIN if the system fails to allocate kernel-internal resources,
rather than ENOMEM as Linux does. POSIX specifies this error for poll(2), but not for select(). Portable programs may wish to
check for EAGAIN and loop, just as with EINTR.

} while (os_status < 0 && errno == EINTR);

Describe the solution you'd like
Add check for EAGAIN

Describe alternatives you've considered
None

Additional context
None

Requester Info
Jacob Hageman - NASA/GSFC, OSAL code review

jphickey added a commit to jphickey/osal that referenced this issue Mar 12, 2021
This is documented as a possible errno from select on
some systems, and the call should be repeated in that case.
@astrogeco astrogeco added posix and removed posix labels Mar 19, 2021
astrogeco added a commit that referenced this issue Mar 19, 2021
Fix #858, add check for EAGAIN in addition to EINTR
astrogeco added a commit to astrogeco/osal that referenced this issue Mar 22, 2021
Fix nasa#858, add check for EAGAIN in addition to EINTR
@skliper skliper added this to the 6.0.0 milestone Sep 24, 2021
jphickey added a commit to jphickey/osal that referenced this issue Aug 10, 2022
Use the OSAL-supplied typedef `osal_id_t` to store OSAL IDs,
along with OSAL-supplied conversion/cast routines when interfacing
this value with other modules.
jphickey pushed a commit to jphickey/osal that referenced this issue Aug 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants