midgard/tabs
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Fix shift state and address BSS issue

Browse source
This commit is contained in:
Robbe Van Herck 2020-01-10 09:48:21 +01:00
parent 3abf49f881
commit 95c0371065
No known key found for this signature in database
GPG key ID: A66F76F7B81BD784
2 changed files with 6 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
docs/bootloader.md
View file

@ -63,6 +63,6 @@ As I don't quite get how segments work in real mode, there are most likely error
I reserve 32KiB of SFS reserved area for the ELF file of the kernel. This is currently for "historic reasons" (a.k.a. I'm too lazy to load the filesystem in the bootloader). The kernel then handles the filesystem. I reserve 32KiB of SFS reserved area for the ELF file of the kernel. This is currently for "historic reasons" (a.k.a. I'm too lazy to load the filesystem in the bootloader). The kernel then handles the filesystem.
### Second ELF program header does not get loaded ### BSS does not get cleared
This will most likely fix the issue of missing global strings. I do calculate the size of the BSS, so either I intialize every variable in a function or things like global variables that are false by default don't work. It does work in emulators because they initialize all memory to 0, but real computerss don't always do that.

6
kernel/drivers/keyboard/keyboard.c
View file

@ -10,8 +10,8 @@
#define KEY_QUEUE_EMPTY FIFO_QUEUE_EMPTY #define KEY_QUEUE_EMPTY FIFO_QUEUE_EMPTY
bool keyboard_shift_state = false; bool keyboard_shift_state;
bool keyboard_caps_state = false; bool keyboard_caps_state;
fifo* key_queue = NULL; fifo* key_queue = NULL;
@ -72,6 +72,8 @@ char getchar() {
} }
void keyboard_init() { void keyboard_init() {
keyboard_shift_state = false;
keyboard_caps_state = false;
key_queue = fifo_new(); key_queue = fifo_new();
uint8_t mask = inb(0x21); uint8_t mask = inb(0x21);