Skip to content

sockets/posix: poll()/select() implementation should be factored out from sockets subsys into POSIX subsys #24966

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

Closed
Tracked by #51211 ...
pfalcon opened this issue May 5, 2020 · 0 comments · Fixed by #73978 or #76352
Assignees
Labels
area: POSIX POSIX API Library Enhancement Changes/Updates/Additions to existing features

Comments

@pfalcon
Copy link
Collaborator

pfalcon commented May 5, 2020

POSIX poll() was initially implemented in the scope of BSD Sockets API, and currently located in subsys/net/lib/sockets/ . It is however a generic POSIX facility, and was generalized to apply to arbitrary file descriptors (and objects represented by them). As we grow implementations of such objects (eventfd, unix domain sockets), dependency on networking sockets becomes a problem, as requires enabling unrelated config options and leads to pulling more code into user applications.

To address this, poll() (and by extension, select()) implementation should be migrated to lib/posix/.

@pfalcon pfalcon added area: POSIX POSIX API Library Maintainer At least 2 days before merge, maintainer review required labels May 5, 2020
@galak galak added the Enhancement Changes/Updates/Additions to existing features label May 21, 2020
@jgl-meta jgl-meta self-assigned this Jun 5, 2023
@cfriedt cfriedt added this to the v3.6.0 milestone Oct 13, 2023
@henrikbrixandersen henrikbrixandersen removed the Maintainer At least 2 days before merge, maintainer review required label Feb 6, 2024
@henrikbrixandersen henrikbrixandersen removed this from the v3.6.0 milestone Feb 6, 2024
@cfriedt cfriedt reopened this Jul 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: POSIX POSIX API Library Enhancement Changes/Updates/Additions to existing features
Projects
None yet
5 participants