Skip to content

Added an interface for managing physical memory.#58

Merged
frogrammer9 merged 3 commits intoOperacja-System:mainfrom
frogrammer9:pmem
Jan 27, 2026
Merged

Added an interface for managing physical memory.#58
frogrammer9 merged 3 commits intoOperacja-System:mainfrom
frogrammer9:pmem

Conversation

@frogrammer9
Copy link
Contributor

No description provided.

@frogrammer9 frogrammer9 force-pushed the pmem branch 2 times, most recently from 5d0483e to 7f116eb Compare December 17, 2025 17:21
@qbojj
Copy link
Collaborator

qbojj commented Dec 19, 2025

We should consider using sparse.
It will allow for preserving type info in pointers and prevent unintentional casting between them.
Furthermore it will allow for multiple address spaces (user vs kernel and physical vs virtual) and prevent unintentional dereferencing of pointers that are not safe to dereference in normal kernel flow (will need to create some utilities for checking range of pointers that we get from userspace).

Copy link
Collaborator

@Kamilosok Kamilosok left a comment

Choose a reason for hiding this comment

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

Some consistency and linguistic issues

@frogrammer9 frogrammer9 self-assigned this Jan 19, 2026
@frogrammer9 frogrammer9 added the enhancement New feature or request label Jan 19, 2026
@frogrammer9 frogrammer9 requested a review from Kamilosok January 24, 2026 13:12
Copy link
Collaborator

@Kamilosok Kamilosok left a comment

Choose a reason for hiding this comment

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

Two (important for me) nitpicks

@frogrammer9 frogrammer9 force-pushed the pmem branch 3 times, most recently from c51cf62 to edda4f8 Compare January 27, 2026 18:34
Copy link
Collaborator

@Kamilosok Kamilosok left a comment

Choose a reason for hiding this comment

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

All fixed ✔️

@frogrammer9 frogrammer9 merged commit 30e8b73 into Operacja-System:main Jan 27, 2026
7 checks passed
@frogrammer9 frogrammer9 deleted the pmem branch January 29, 2026 20:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants