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

plat/kvm/x86: Add dependency to ukbitops #1395

Merged
merged 1 commit into from May 21, 2024

Conversation

andreittr
Copy link
Contributor

Description of changes

This change adds a Kconfig dependency on ukbitops to the x86 KVM platform code, as it is needed by tscclock.c.

Prerequisite checklist

  • Read the contribution guidelines regarding submitting new changes to the project;
  • Tested your changes against relevant architectures and platforms;
  • Ran the checkpatch.uk on your commit series before opening this PR;
  • Updated relevant documentation.

Base target

  • Architecture(s): [e.g. x86_64 or N/A]
  • Platform(s): [e.g. kvm, xen or N/A]
  • Application(s): [e.g. app-python3 or N/A]

Additional configuration

N/A

This change adds a Kconfig dependency on ukbitops to the x86 KVM
platform code, as it is needed by tscclock.c.

Signed-off-by: Andrei Tatar <andrei@unikraft.io>
@andreittr andreittr requested a review from a team as a code owner April 24, 2024 10:37
@github-actions github-actions bot added area/plat Unikraft Patform plat/kvm Unikraft for KVM labels Apr 24, 2024
@razvand razvand requested review from Mihnea27, SerbanSo and eduardvintila and removed request for a team April 28, 2024 05:29
@razvand razvand self-assigned this Apr 28, 2024
@razvand razvand requested review from rares-miculescu and removed request for SerbanSo and eduardvintila April 28, 2024 05:29
@razvand razvand added this to the v0.17.0 (Calypso) milestone Apr 28, 2024
Copy link

@rares-miculescu rares-miculescu left a 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

Copy link

@Mihnea27 Mihnea27 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

Reviewed by: Mihnea Popeanga popeanga.mihnea@gmail.com

Copy link
Contributor

@razvand razvand left a 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

@razvand razvand changed the base branch from staging to staging-1395 May 21, 2024 20:35
@razvand razvand merged commit c7d1a3d into unikraft:staging-1395 May 21, 2024
11 of 14 checks passed
razvand pushed a commit that referenced this pull request May 21, 2024
This change adds a Kconfig dependency on ukbitops to the x86 KVM
platform code, as it is needed by tscclock.c.

Signed-off-by: Andrei Tatar <andrei@unikraft.io>
Reviewed-by: Rares Miculescu <miculescur@gmail.com>
Approved-by: Razvan Deaconescu <razvand@unikraft.io>
GitHub-Closes: #1395
@andreittr andreittr deleted the ttr/config-kvm-bitops branch May 24, 2024 12:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/plat Unikraft Patform plat/kvm Unikraft for KVM
Projects
Status: Done!
Development

Successfully merging this pull request may close these issues.

None yet

4 participants