xyrisos / xyris Goto Github PK

They need to be moved out of <sys/sys.hpp> and into <sys/string.hpp>

Is your feature request related to a problem?
No.

Describe the feature request in detail
px_print_debug is going to be deprecated and the prefixes should be moved to defines a lá Linux.

Additional context
n/a

• Separate bitmap library
• Freeing pages
• Stress testing
• Detecting hardware memory limit

https://github.com/panix-os-dev-team/Panix-Rewrite/blob/4011da98bb7da3faaed666de2040e3614e9c9de7/Doxyfile#L218-L223

This issue was generated by todo based on a TODO comment in 4011da9. It's been assigned to @Kfeavel because they committed the code.

Is your feature request related to a problem?
A lot of the standard library functions implemented are not in the correct, standard header.

Describe the feature request in detail
All functions should move to the appropriate header. For information as to what goes where, reference Wikipedia.

Additional context
n/a

https://github.com/panix-os-dev-team/Panix-Rewrite/blob/db8febb98976cb91374edd3e5f23bce81abd066d/kernel/arch/i386/gdt.cpp#L68-L73

This issue was generated by todo based on a TODO comment in db8febb when #3 was merged. cc @panix-os-dev-team.

No clue why

Drivers should only need to include arch.hpp and not include specific architecture-based headers. Related to #54

It would be nice.

https://github.com/panix-os-dev-team/Panix-Rewrite/blob/db8febb98976cb91374edd3e5f23bce81abd066d/kernel/include/devices/smbios/smbios.hpp#L16-L21

This issue was generated by todo based on a TODO comment in db8febb when #3 was merged. cc @panix-os-dev-team.

OS Dev Wiki

• Bitmap Heap
• Linked List Bucket Heap

Expected behavior
When an ANSI reset is called in a string, the colors should reset to whatever was set as the default in the previous clear screen function call.

Actual behavior
The reset actually resets to white on black instead of whatever was set when clearing the TTY.

Steps to reproduce
Steps to reproduce the behavior:

1. Call clear TTY function with custom colors as foreground and back.
2. Update colors in a printf and then include an ANSI reset code.
3. Observe.

Screenshots
n/a

Additional context
n/a

Is your feature request related to a problem?
BIOS is outdated and UEFI will make our lives easier.

Describe the feature request in detail
BIOS sucks and UEFI will make our lives much easier for things like getting ACPI tables and native resolutions. Additionally, if we move to x86_64, UEFI will already boot the system into 64 bit mode, so we won't need to do that either.

Additional context
BootBoot only supports booting in 64 bit mode, so that might be an issue, or it might not be. Depends on how well Panix runs on x86_64 based systems since it's built for 32 bit architectures.

Draft:

Panix

Panix is a small, hobbyist kernel started by Keeton Feavel and further developed with the help of Micah Switzer.
Having now undergone three rewrites from the ground up, Panix has come a long way since its first start in 2018.

Design Philosophy

Panix is designed to be readable first and foremost. Having been developed at Cedarville University, the primary goal of Panix is to provide an opportunity for students (and anyone else interested) to learn the intricacies of kernel development. Function names are descriptive and structs are used when possible in order to help increase the legibility of code and provide an "easy-in" for kernel development.

Accomplishments

Panix is the subject of an independent study and the associated final report published by Keeton Feavel at Cedarville and used by Patrick Dudenhofer to help instruct students of CS-3320 (Operating Systems). The cedarville branch is an older, modified version of the Panix kernel used as a template for assignments designed to help concepts such as CPU ports and hardware communication. Eventually, once paging is fully implemented, another assignment will be developed to help explain paging and memory management.

How to Help

We are constantly looking for people to help contribute to Panix. If you would like to contribute driver code or a new feature, feel free to fork the repository and submit a pull request. If you are a student who would like to get involved, feel free to contact any of the project contributors.

https://github.com/panix-os-dev-team/Panix-Rewrite/blob/bc74f018f9c44ac087e8629351c27b42de6270a5/kernel/main.cpp#L49-L54

This issue was generated by todo based on a @todo comment in bc74f01 when #12 was merged. cc @panix-os-dev-team.

Is your feature request related to a problem?
The makefile we are currently using is pretty limited.

Describe the feature request in detail
Support for compiling x86_64 code should be supported.

Additional context
n/a

https://github.com/panix-os-dev-team/Panix-Rewrite/blob/bc74f018f9c44ac087e8629351c27b42de6270a5/kernel/arch/i386/flush.s#L7-L10

This issue was generated by todo based on a @todo comment in bc74f01 when #12 was merged. cc @panix-os-dev-team.

https://github.com/panix-os-dev-team/Panix-Rewrite/blob/db8febb98976cb91374edd3e5f23bce81abd066d/kernel/arch/i386/isr.cpp#L77-L82

This issue was generated by todo based on a TODO comment in db8febb when #3 was merged. cc @panix-os-dev-team.

https://github.com/panix-os-dev-team/Panix-Rewrite/blob/db8febb98976cb91374edd3e5f23bce81abd066d/kernel/arch/i386/gdt.cpp#L2-L7

This issue was generated by todo based on a TODO comment in db8febb when #3 was merged. cc @panix-os-dev-team.

@copyright Copyright Keeton Feavel et al © YEAR

@copyright Copyright the Panix Contributors © YEAR

Used for first fit memory allocation algorithm

This causes a triple fault.

Many of the headers are missing Doxygen comments for the functions. This should be resolved and any functions which are not thoroughly explained should also be documented.

https://github.com/panix-os-dev-team/Panix-Rewrite/blob/55856e3f8982294337105206b8303c26302252db/kernel/arch/x86/boot.s#L78-L43

This issue was generated by todo based on a @todo comment in 55856e3 when #20 was merged. cc @panix-os-dev-team.

#include <stdint.h>
#include <stdlib.h>
 
#if UINT32_MAX == UINTPTR_MAX
#define STACK_CHK_GUARD 0xe2dee396
#else
#define STACK_CHK_GUARD 0x595e9fbd94fda766
#endif
 
uintptr_t __stack_chk_guard = STACK_CHK_GUARD;
 
__attribute__((noreturn))
void __stack_chk_fail(void)
{
#if __STDC_HOSTED__
	abort();
#elif __is_myos_kernel
	panic("Stack smashing detected");
#endif
}

https://wiki.osdev.org/Stack_Smashing_Protector

https://github.com/panix-os-dev-team/Panix-Rewrite/blob/bc74f018f9c44ac087e8629351c27b42de6270a5/sysroot/usr/include/kernel/arch/i386/isr.hpp#L17-L20

This issue was generated by todo based on a @todo comment in bc74f01 when #12 was merged. cc @panix-os-dev-team.

https://github.com/panix-os-dev-team/Panix/blob/f62a2a5d22803f2e347badf7c5b92df5dda41d14/sysroot/usr/include/kernel/arch/x86/isr.hpp#L20-L24

This issue was generated by todo based on a @todo comment in f62a2a5 when #27 was merged. cc @panix-os-dev-team.

We need to remove magic numbers.

This allows us to load kernel modules in through Grub. Maybe something to look into if we try to move to hybrid / micro.
https://littleosbook.github.io/#the-road-to-user-mode

https://github.com/panix-os-dev-team/Panix-Rewrite/blob/55856e3f8982294337105206b8303c26302252db/kernel/arch/x86/timer.cpp#L44-L49

This issue was generated by todo based on a @todo comment in 55856e3 when #20 was merged. cc @panix-os-dev-team.

Is your feature request related to a problem?
panix.hpp is the problem. Everything includes it for no good reason other than including types and panic (which is not always required)

Describe the feature request in detail
panix.hpp was an awful idea. I'm sorry.

Additional context
No comment.

Is your feature request related to a problem?
Kernel panics are not able to be easily logged due to them being printed only to the VGA.

Describe the feature request in detail
Keeping track of kernel panics could help in the future where referencing previous register values might be helpful.

Additional context
n/a

https://github.com/panix-os-dev-team/Panix-Rewrite/blob/4011da98bb7da3faaed666de2040e3614e9c9de7/Doxyfile#L214-L219

This issue was generated by todo based on a TODO comment in 4011da9. It's been assigned to @Kfeavel because they committed the code.

RS232 Serial output works, but input is still unimplemented.

This will better reflect the fact that this is the third rewrite of Panix.

Is your feature request related to a problem?
A stack / queue library are needed for a variety of reasons. The primary reason right now being keeping track of ANSI escape codes passed to kprintf (and then onto putc).

Describe the feature request in detail
There need to be two separate libraries contained in the lib section of Panix. There should be a stack.cpp and queue.cpp along with their associated headers in the sysroot. Talk to @micahswitzer about unit testing the libraries.

Additional context
n/a

https://github.com/panix-os-dev-team/Panix-Rewrite/blob/db8febb98976cb91374edd3e5f23bce81abd066d/kernel/arch/i386/isr.cpp#L30-L35

This issue was generated by todo based on a TODO comment in db8febb when #3 was merged. cc @panix-os-dev-team.

Original source: https://www.gnu.org/software/grub/manual/multiboot/html_node/multiboot_002eh.html

This will eliminate the need to include an architecture specific header in drivers, such as the keyboard.

https://github.com/panix-os-dev-team/Panix/blob/f62a2a5d22803f2e347badf7c5b92df5dda41d14/kernel/main.cpp#L45-L50

This issue was generated by todo based on a @todo comment in f62a2a5 when #27 was merged. cc @panix-os-dev-team.

https://github.com/panix-os-dev-team/Panix-Rewrite/blob/bc74f018f9c44ac087e8629351c27b42de6270a5/sysroot/usr/include/kernel/arch/i386/isr.hpp#L35-L38

This issue was generated by todo based on a @todo comment in bc74f01 when #12 was merged. cc @panix-os-dev-team.