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-*: Fix missing/unneeded dependencies #1393
lib/posix-*: Fix missing/unneeded dependencies #1393
Conversation
This change adds a Kconfig dependency to ukatomic on several libraries that were written before the ukatomic split-off. This makes their using atomic operations explicit. Signed-off-by: Andrei Tatar <andrei@unikraft.io>
This change removes the Kconfig dependency on LIBUKLOCK as well as inclusion of <uk/mutex.h> as these were not used. Signed-off-by: Andrei Tatar <andrei@unikraft.io>
This change adds a conditional dependency on (no)libc, required because ukfile uses many libc-provided types. Signed-off-by: Andrei Tatar <andrei@unikraft.io>
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.
Looks good on my side.
Reviewed-by: Rares Miculescu miculescur@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.
Looks good to me
Reviewed-by: Robert Zamfir georobi.016@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.
Approved-by: Razvan Deaconescu razvand@unikraft.io
This change adds a Kconfig dependency to ukatomic on several libraries that were written before the ukatomic split-off. This makes their using atomic operations explicit. Signed-off-by: Andrei Tatar <andrei@unikraft.io> Reviewed-by: Robert Zamfir <georobi.016@gmail.com> Approved-by: Razvan Deaconescu <razvand@unikraft.io> GitHub-Closes: #1393
This change removes the Kconfig dependency on LIBUKLOCK as well as inclusion of <uk/mutex.h> as these were not used. Signed-off-by: Andrei Tatar <andrei@unikraft.io> Reviewed-by: Robert Zamfir <georobi.016@gmail.com> Approved-by: Razvan Deaconescu <razvand@unikraft.io> GitHub-Closes: #1393
This change adds a conditional dependency on (no)libc, required because ukfile uses many libc-provided types. Signed-off-by: Andrei Tatar <andrei@unikraft.io> Reviewed-by: Robert Zamfir <georobi.016@gmail.com> Approved-by: Razvan Deaconescu <razvand@unikraft.io> GitHub-Closes: #1393
Description of changes
This changeset fixes a number of missing or unneeded Kconfig dependencies. See individual commits for details.
Prerequisite checklist
checkpatch.uk
on your commit series before opening this PR;Base target
x86_64
or N/A]kvm
,xen
or N/A]app-python3
or N/A]Additional configuration
N/A