-
Notifications
You must be signed in to change notification settings - Fork 214
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
32-bit support (with Multiboot2) #70
Comments
I hope this is togglable with a feature flag. Just saying, I'm
personally not in favor of this idea.
…On 7/31/19, bjorn3 ***@***.***> wrote:
cc phil-opp/blog_os#403 (comment).
@Mendess2526 let's continue the discussion here.
This will need src/stage_3.s to be adapted to stay in protected mode and to
setup non-PAE page tables. It will then need at least changes to the rust
part to handle non-PAE page tables.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
#70
--
Signed,
Ethin D. Probst
|
Yeah it would need to be a toggleable feature. We should support the multiboot2 protocol for this |
Shouldnt it just work when it is compiled for |
Yeah that’s a good point. I think in the mean time it might be good to add some check in the build script that it’s being built for x86_64 (if that is even possible), otherwise error out and point to this issue |
This should be possible through the |
72: Configurable kernel stack size, better non-x86_64 errors r=phil-opp a=64 Allows the configuration of the kernel stack size through an environment variable. Also makes the errors a bit nicer if you try to use this crate from a non-x86_64 architecture (only works if you build it as a library), as discussed in #70. Co-authored-by: Matt Taylor <mstaveleytaylor@gmail.com>
72: Configurable kernel stack size, better non-x86_64 errors r=phil-opp a=64 Allows the configuration of the kernel stack size through an environment variable. Also makes the errors a bit nicer if you try to use this crate from a non-x86_64 architecture (only works if you build it as a library), as discussed in #70. Co-authored-by: Matt Taylor <mstaveleytaylor@gmail.com>
Personally I don't think we should support 32-bit at all. The only reason I can see this being required is running this on old hardware, and you have VMs for that. Now that you can chainload the bootloader (via grub, for example), you can (technically) run the system on "real" (virtual, but technically real) hardware. Its not physically real, but virtualized hardware, with a few changes here and there, is based off of at least a model of old hardware (i.e. the VMware BIOS implementation uses Phoenix BIOS, I think). 32-bit support overcomplicates a lot of things and would just bloat the code to unreasonable proportions. |
cc phil-opp/blog_os#403 (comment).
@Mendess2526 let's continue the discussion here.
This will need src/stage_3.s to be adapted to stay in protected mode and to setup non-PAE page tables. It will then need at least changes to the rust part to handle non-PAE page tables.
The text was updated successfully, but these errors were encountered: