Skip to content

[NTOS:KE/x64] Implement initial IPI code#6130

Merged
tkreuzer merged 1 commit intoreactos:masterfrom
tkreuzer:pr/ntos-x64-ipi
Dec 18, 2023
Merged

[NTOS:KE/x64] Implement initial IPI code#6130
tkreuzer merged 1 commit intoreactos:masterfrom
tkreuzer:pr/ntos-x64-ipi

Conversation

@tkreuzer
Copy link
Copy Markdown
Contributor

@tkreuzer tkreuzer commented Dec 7, 2023

Purpose

This PR implements the basic x64 kernel IPI code. For now it only supports APC, DPC and freeze IPIs, which will be made use of soon. DPC for SMP scheduling and freeze for SMP debugging. It doesn't yet support generic function IPIs, which are a bit more complex and will come later.

@tkreuzer tkreuzer added kernel&hal Code changes to the ntoskrnl and HAL SMP Multiprocessor support. labels Dec 7, 2023
@tkreuzer tkreuzer added this to the SMP bringup milestone Dec 7, 2023
@tkreuzer tkreuzer self-assigned this Dec 7, 2023
@tkreuzer tkreuzer requested a review from HeisSpiter as a code owner December 7, 2023 18:03
Copy link
Copy Markdown
Contributor

@DarkFire01 DarkFire01 left a comment

Choose a reason for hiding this comment

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

Cool :) I'll keep the IPI code for i386 arch specific for now too like we discussed too!

Copy link
Copy Markdown
Contributor

@HBelusca HBelusca left a comment

Choose a reason for hiding this comment

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

Minor suggestions, otherwise lgtm.

extern KSPIN_LOCK KiReverseStallIpiLock;

/* PRIVATE FUNCTIONS *********************************************************/
#ifndef _M_AMD64
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
#ifndef _M_AMD64
#ifndef _M_AMD64

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Fixed

return Status;
}

#endif
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
#endif
#endif // !_M_AMD64

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Fixed

@tkreuzer tkreuzer merged commit 1831bc6 into reactos:master Dec 18, 2023
@tkreuzer tkreuzer deleted the pr/ntos-x64-ipi branch December 18, 2023 21:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kernel&hal Code changes to the ntoskrnl and HAL SMP Multiprocessor support.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants