-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
lib/posix-socket: Expose internal socket syscalls #1337
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All good!
Reviewed-by: Mihnea Firoiu mihneafiroiu0@gmail.com
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @andreittr, it looks good.
Reviewed-by: Radu Nichita radunichita99@gmail.com
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@andreittr should we also add the exposed symbols in exportsyms.uk
?
b7df93b
to
c156096
Compare
Good catch @StefanJum, thanks! Added & rebased to current staging. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed-by: Stefan Jumarea stefanjumarea02@gmail.com
This change exposes Unikraft-internal syscalls that create sockets. Both versions returning raw uk_files as well as opened file descriptors are provided. Signed-off-by: Andrei Tatar <andrei@unikraft.io>
c156096
to
3246b05
Compare
Force-push: rebased on current staging & patched to work with changes introduced in a922af7. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved-by: Razvan Deaconescu razvand@unikraft.io
This change exposes Unikraft-internal syscalls that create sockets. Both versions returning raw uk_files as well as opened file descriptors are provided. Signed-off-by: Andrei Tatar <andrei@unikraft.io> Reviewed-by: Mihnea Firoiu <mihneafiroiu0@gmail.com> Reviewed-by: Radu Nichita <radunichita99@gmail.com> Reviewed-by: Stefan Jumarea <stefanjumarea02@gmail.com> Approved-by: Razvan Deaconescu <razvand@unikraft.io> GitHub-Closes: #1337
Description of changes
This change exposes Unikraft-internal syscalls that create sockets. Both versions returning raw uk_files as well as opened file descriptors are provided.
Prerequisite checklist
checkpatch.uk
on your commit series before opening this PR;Base target
Additional configuration
N/A