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/drivers/virtio: Fix missing lcpu.h include #925

Closed
wants to merge 1 commit into from

Conversation

andreittr
Copy link
Contributor

Description of changes

This change adds an #include of <uk/plat/lcpu.h> which provides ukplat_lcpu_disable_irq() and ukplat_lcpu_enable_irq(), that are used by virtio_blkdev_queue_cleanup_requests().

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): N/A
  • Platform(s): N/A
  • Application(s): N/A

Additional configuration

N/A

This change adds an #include of <uk/plat/lcpu.h> which provides
`ukplat_lcpu_disable_irq()` and `ukplat_lcpu_enable_irq()`, that are
used by `virtio_blkdev_queue_cleanup_requests()`.

Signed-off-by: Andrei Tatar <andrei@unikraft.io>
@andreittr andreittr requested a review from a team as a code owner June 2, 2023 10:19
@unikraft-bot unikraft-bot added area/plat Unikraft Patform lang/c Issues or PRs to do with C/C++ plat/driver plat/driver/virtio labels Jun 2, 2023
@razvand razvand assigned razvand and unassigned nderjung Aug 7, 2023
@razvand razvand removed request for a team and craciunoiuc August 7, 2023 08:54
@razvand razvand added this to the v0.14.0 (Prometheus) milestone Aug 7, 2023
Copy link
Contributor

@mariasfiraiala mariasfiraiala left a comment

Choose a reason for hiding this comment

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

Thanks for the fix, works good.

Reviewed-by: Maria Sfiraiala maria.sfiraiala@gmail.com

Copy link
Contributor

@RaduNichita RaduNichita left a comment

Choose a reason for hiding this comment

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

Works good also on my side.

Reviewed-by: Radu Nichita radunichita99@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

@unikraft-bot unikraft-bot added the ci/merged Merged by CI label Aug 9, 2023
@andreittr andreittr deleted the ttr/lcpu-fixinclude branch August 9, 2023 19:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/plat Unikraft Patform ci/merged Merged by CI lang/c Issues or PRs to do with C/C++ plat/driver/virtio plat/driver
Projects
Status: Done!
Development

Successfully merging this pull request may close these issues.

None yet

6 participants