search

XyrisOS / xyris

The Xyris Kernel
https://xyr.is
MIT License
153 stars 11 forks source link

Resolve "Refactor Memory Management" (Parse Bootloader Memory Map) #322

Closed Kfeavel closed 2 years ago

Kfeavel commented 2 years ago

Addresses part of #313

Closes #328 and #329 (because I shaved a few yaks in the process of working on this branch)

micahswitzer commented 2 years ago

I'm worried that removing size_t in many places is going to hurt portability to 64-bit systems whenever we decide to do that

Kfeavel commented 2 years ago

I'm worried that removing size_t in many places is going to hurt portability to 64-bit systems whenever we decide to do that

That's why I added the "%z" flag to printf. I'm going to put them back now that I realized it was relatively easy to add 64 bit support to printf. The formatting is a little bit different (%L vs %ll) but it does work.

Kfeavel commented 2 years ago

Merging since nothing is panicking and memory management will still be actively worked on anyways. Shouldn't be any stability issues, but it isn't going into stable so we'll still be able to correct anything.