* Added more information when kernel crashes

* Updated the Github Actions

Author: Arun007coder

.github/workflows/makefile.yml

@@ -22,7 +22,7 @@ jobs:
       shell: bash
 
     - name: Create toolchain
-      run: ./src/scripts/make_toolchain.sh --parallel_build -s ~/sysroot download_i686_elf_tools
+      run: ./src/scripts/make_toolchain.sh --parallel_build -s ~/sysroot
       shell: bash
 
     - name: Compile kernel

Makefile

@@ -11,7 +11,9 @@ OSTOOL_DIR=~/sysroot/usr/bin/
 
 CC=i686-elf-gcc
 CCVERSION = $(shell echo $(CC) $(shell $(PREFIX)$(CC) --version | grep $(CC) | sed 's/^.* //g'))
-CFLAGS= -I$(INCLUDEDIR) -I/usr/include -nostdlib -DKERNEL_COMPILER="\"$(CCVERSION)\"" -lgcc -fno-builtin -fno-exceptions -fno-leading-underscore -ffreestanding -Wall -Wpedantic -ggdb -O0 -D__ENABLE_DEBUG_SYMBOL_LOADING__=1 -D__COMPOSITOR_LOW_END__=1
+CFLAGS=
+KCFLAGS= -I$(INCLUDEDIR) -I/usr/include -nostdlib -DKERNEL_COMPILER="\"$(CCVERSION)\"" -lgcc -fno-builtin -fno-exceptions -fno-leading-underscore -ffreestanding -Wall -Wpedantic -ggdb -O0 -D__ENABLE_DEBUG_SYMBOL_LOADING__=1 -D__COMPOSITOR_LOW_END__=0 $(CFLAGS)
+
 
 CXX=$(TOOLDIR)/i686-elf-g++
 CXXFLAGS=
@@ -171,7 +173,7 @@ $(ISOFILE): $(IMAGEFILE) $(EXECUTABLE)
 
 %.o : %.c
 	@echo '[CC] $@'
-	@$(PREFIX)/$(CC) $(CFLAGS) -c -o $@ $<
+	@$(PREFIX)/$(CC) $(KCFLAGS) -c -o $@ $<
 
 %.o : %.s
 	@echo '[GAS] $@'

docs/README.md

@@ -14,7 +14,7 @@ Kernel Version
 
 
 ## 🧱 Kernel
-The current kernel version for SectorOS-RW4 is v6.23.06.2NR.<br>
+The current kernel version for SectorOS-RW4 is v6.23.06.3NR.<br>
 
 ## Screenshots
 <img src="./assets/Screenshot_01.png"></img>

docs/changelog.txt

@@ -235,4 +235,8 @@
 * Updated the sleep routine
 * Added entries to mount syscall
 * Implemented a function to show message in compositor
-* Added function to access memory mapped IO
+* Added function to access memory mapped IO
+
+22-06-2023: 07:56 PM IST v6.23.06.3NR
+* Added more information when kernel crashes
+* Updated the Github Actions

src/include/system.h

@@ -43,7 +43,7 @@ typedef uint32_t uintptr_t;
 * RELN: index of current release in the current month
 * STATUS: [PR]:Prerelease, [AL]:alpha, [NR]:Normal release
 */
-#define KERNEL_VERSION "6.23.06.2NR"
+#define KERNEL_VERSION "6.23.06.3NR"
 #define KERNEL_VERSION_CODENAME "Mystic Mango"
 
 #define KERNEL_ENABLED_OPTIONS "\b"

src/interrupts/exception.c

@@ -2,6 +2,7 @@
 #include "vga_text.h"
 #include "debug.h"
 #include "process.h"
+#include "logdisk.h"
 
 int isLessMSG = 0;
 
@@ -91,6 +92,7 @@ void exception_handler(registers_t cps)
         else
         {
             text_chcolor(VGA_RED, VGA_BLACK);
+            logdisk_change_policy(LOG_OFF);
             printf("KERNEL PANIC. EXCEPTION OCCURRED %d: %s\n", cps.ino, exception_messages[cps.ino]);
             printf("Extended stack pointer = 0x%06x\n", cps.esp);
             printf("Extended instruction pointer  = 0x%06x\n", cps.eip);
@@ -106,6 +108,20 @@ void exception_handler(registers_t cps)
             printf("ESI = 0x%06x\n", cps.esi);
             printf("EDI = 0x%06x\n", cps.edi);
             printf("EBP = 0x%06x\n", cps.ebp);
+            ldprintf("KERNEL", LOG_ERR, "KERNEL PANIC. EXCEPTION OCCURRED %d: %s", cps.ino, exception_messages[cps.ino]);
+            ldprintf("KERNEL", LOG_ERR, "Extended stack pointer = 0x%06x", cps.esp);
+            ldprintf("KERNEL", LOG_ERR, "Extended instruction pointer  = 0x%06x", cps.eip);
+            ldprintf("KERNEL", LOG_ERR, "Code segment selector = 0x%06x", cps.cs);
+            ldprintf("KERNEL", LOG_ERR, "Extended flags = 0x%06x", cps.eflags);
+            ldprintf("KERNEL", LOG_ERR, "Error code = 0x%06x", cps.ecode);
+            ldprintf("KERNEL", LOG_ERR, "Registers:");
+            ldprintf("KERNEL", LOG_ERR, "EAX = 0x%06x", cps.eax);
+            ldprintf("KERNEL", LOG_ERR, "EBX = 0x%06x", cps.ebx);
+            ldprintf("KERNEL", LOG_ERR, "ECX = 0x%06x", cps.ecx);
+            ldprintf("KERNEL", LOG_ERR, "EDX = 0x%06x", cps.edx);
+            ldprintf("KERNEL", LOG_ERR, "ESI = 0x%06x", cps.esi);
+            ldprintf("KERNEL", LOG_ERR, "EDI = 0x%06x", cps.edi);
+            ldprintf("KERNEL", LOG_ERR, "EBP = 0x%06x", cps.ebp);
             serialprintf("KERNEL PANIC. EXCEPTION OCCURRED %d: %s\n", cps.ino, exception_messages[cps.ino]);
             serialprintf("Extended stack pointer = 0x%06x\n", cps.esp);
             serialprintf("Extended instruction pointer  = 0x%06x\n", cps.eip);

src/memory/paging.c

@@ -2,6 +2,7 @@
 #include "kheap.h"
 #include "debug.h"
 #include "elf_loader.h"
+#include "logdisk.h"
 
 extern void * heap_start, * heap_end, * heap_max, * heap_curr;
 extern bool kheap_enabled;
@@ -303,6 +304,8 @@ page_table_t* copy_page_table(page_directory_t * src_page_dir, page_directory_t
     return table;
 }
 
+char page_hnd_error[1024];
+
 void page_fault_handler(registers_t* regs)
 {
     printf("Page fault:\n");
@@ -324,6 +327,15 @@ void page_fault_handler(registers_t* regs)
     if(inst_fetch) printf("Instruction fetch ");
     printf("]\n");
 
+    strcpy(page_hnd_error, "Possible causes: [ ");
+    if(!present) strcat(page_hnd_error, "Page not present ");
+    if(rw) strcat(page_hnd_error, "Page is read only ");
+    if(user) strcat(page_hnd_error, "Page is read only ");
+    if(reserved) strcat(page_hnd_error, "Overwrote reserved bits ");
+    if(inst_fetch) strcat(page_hnd_error, "Instruction fetch ");
+    strcat(page_hnd_error, "]");
+    ldprintf("Page manager", LOG_ERR, page_hnd_error);
+
     serialprintf("Possible causes: [ ");
     if(!present) serialprintf("Page not present ");
     if(rw) serialprintf("Page is read only ");
@@ -337,6 +349,9 @@ void page_fault_handler(registers_t* regs)
     printf("Faulting address: 0x%x\n", faulting_addr);
     printf("Faulting virtual address: 0x%x : {%s+0x%x}\n", faulting_vaddr, vsym.name, vsym.offset);
 
+    ldprintf("Page manager", LOG_ERR, "Faulting address: 0x%x", faulting_addr);
+    ldprintf("Page manager", LOG_ERR, "Faulting virtual address: 0x%x : {%s+0x%x}", faulting_vaddr, vsym.name, vsym.offset);
+
     serialprintf("Faulting address: 0x%x\n", faulting_addr);
     serialprintf("Faulting virtual address: 0x%x : {%s+0x%x}\n", faulting_vaddr, vsym.name, vsym.offset);
 

src/scripts/i686-elf-tools.sh

@@ -77,7 +77,7 @@ echo "PARALLEL         = ${PARALLEL}"
 function main {
 
     installPackages
-    installMXE
+    # installMXE
 
     if [[ $ENV_ONLY == true ]]; then
         echoColor "Successfully installed build environment. Exiting as 'env' only was specified"