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

Add support for x2apic mode on amd64 #208

Merged
merged 1 commit into from
Oct 10, 2024
Merged

Add support for x2apic mode on amd64 #208

merged 1 commit into from
Oct 10, 2024

Conversation

dbittman
Copy link
Contributor

This PR adds support for the X2 mode of the advanced programmed interrupt controller on AMD64 machines. This is an extension to the original APIC, but on some CPU models, seems to be required. It makes programming the interrupt controller faster, too, since it allows for access to the registers via MSRs instead of MMIO.

@dbittman dbittman added enhancement New feature or request kernel Internal kernel issues labels Sep 24, 2024
@dbittman dbittman self-assigned this Sep 24, 2024
@dbittman dbittman requested a review from gvnn3 as a code owner September 24, 2024 18:39
@dbittman dbittman requested a review from PandaZ3D October 9, 2024 21:39
Copy link
Contributor

@PandaZ3D PandaZ3D left a comment

Choose a reason for hiding this comment

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

LGTM! I only left a question below.

Copy link
Contributor

Choose a reason for hiding this comment

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

Wouldn't it be better to prepend \r\n in the log macros?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh, probably, but I actually have a plan to fix this at a higher level, but that'll be in some future "console I/O" PR...

@dbittman dbittman merged commit 1340567 into main Oct 10, 2024
1 check passed
@dbittman dbittman deleted the dbittman-x2apic branch October 10, 2024 19:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request kernel Internal kernel issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants