fix bug

Author: tas0dev

src/boot/config.inc

@@ -1,5 +1,5 @@
 KERNEL_OFFSET EQU 0x10000       ; カーネルをロードするアドレス
-SECTOR_COUNT  EQU 40            ; 読み込むセクタの数
+SECTOR_COUNT  EQU 39            ; 読み込むセクタの数
 START_SECTOR  EQU 2             ; 開始するセクタ番号
 CYLINDER_NUM  EQU 0             ; シリンダ番号
 HEAD_NUM      EQU 0             ; ヘッド番号

src/kernel/mem/paging.c

@@ -3,17 +3,6 @@
 #include <mem/paging.h>
 #include <mem/map.h>
 #include <stddef.h>
-
-/* helpers: in current identity-map boot situation, phys==virt, so these are identity
- * conversions. If kernel later runs with higher-half mapping, update these helpers
- * to translate between physical and kernel virtual addresses accordingly.
- */
-static inline void *phys_to_virt(uint32_t phys) {
-        return (void *)phys;
-}
-static inline uint32_t virt_to_phys(void *virt) {
-        return (uint32_t)virt;
-}
 #include <console.h>
 #include <interrupt/irq.h>
 
@@ -37,7 +26,11 @@ void *alloc_page_table(void) {
         return frame;
 }
 
-// 1つの4KBページをマッピングする: phys -> virt（flags付き）。成功時0、失敗時-1
+/**
+ * @fn map_page
+ * @brief 物理アドレスphysを仮想アドレスvirtにflags属性でマッピングする
+ * @return 0: 成功 -1: 失敗
+ */
 int map_page(uint32_t phys, uint32_t virt, uint32_t flags) {
         if ((flags & PAGING_PRESENT) == 0) flags |= PAGING_PRESENT;
         uint32_t pd_idx = (virt >> 22) & 0x3FF;
@@ -59,7 +52,12 @@ int map_page(uint32_t phys, uint32_t virt, uint32_t flags) {
         return 0;
 }
 
-// 1つの4KBページをアンマップする。成功時0、未マップ時-1
+/**
+ * @fn unmap_page
+ * @brief 仮想アドレスvirtに対応するページをアンマップする
+ * @param virt アンマップする仮想アドレス
+ * @return 0: 成功 -1: 失敗
+ */
 int unmap_page(uint32_t virt) {
         uint32_t pd_idx = (virt >> 22) & 0x3FF;
         uint32_t pt_idx = (virt >> 12) & 0x3FF;
@@ -73,48 +71,32 @@ int unmap_page(uint32_t virt) {
         return 0;
 }
 
-// 指定されたMB数分のメモリを同一マッピングで初期化する
+/**
+ * @fn paging_init_identity
+ * @brief 最初のmap_mb MB分を同一マッピングで初期化する
+ * @param map_mb 同一マッピングするMB数
+ */
 void paging_init_identity(uint32_t map_mb) {
-        /* map_mb MB (rounded up to full pages) as identity mapping */
         uint32_t pages = ((map_mb * 1024u * 1024u) + 0xFFF) / 0x1000;
-        uint32_t pages_remaining = pages;
-
-        /* number of page tables needed (1024 entries per table -> 4MB each) */
-        uint32_t needed_pt = (pages + 1023) / 1024;
-
-        /* initialize page directory to not present */
-        for (uint32_t i = 0; i < 1024; ++i) page_directory[i] = 0x00000000;
 
-        for (uint32_t pt_index = 0; pt_index < needed_pt; ++pt_index) {
-                /* use first_table for pt_index==0 for static allocation */
-                uint32_t *pt = NULL;
-                if (pt_index == 0) {
-                        pt = first_table;
-                } else {
-                        void *alloc = alloc_page_table();
-                        if (!alloc) {
-                                printk("paging: failed to alloc page table for pt_index=%u\n", (unsigned)pt_index);
-                                break;
-                        }
-                        pt = (uint32_t *)alloc;
-                }
+        // 最初のテーブル: 最初の4MBを同一マッピング（1024エントリ）
+        for (uint32_t i = 0; i < 1024; ++i) {
+                first_table[i] = (i * 0x1000) | 3u; // present + rw
+        }
 
-                /* fill table entries for this PT */
-                for (uint32_t i = 0; i < 1024 && pages_remaining > 0; ++i, --pages_remaining) {
-                        uint32_t phys = (pt_index * 1024u + i) * 0x1000u;
-                        pt[i] = (phys & 0xFFFFF000u) | (PAGING_PRESENT | PAGING_RW);
-                }
+        // ディレクトリエントリ0はfirst_tableを指す
+        page_directory[0] = ((uint32_t)first_table) | 3u;
 
-                /* write PDE: PDE must contain physical address of pt */
-                uint32_t pt_phys = virt_to_phys((void *)pt);
-                page_directory[pt_index] = (pt_phys & 0xFFFFF000u) | (PAGING_PRESENT | PAGING_RW);
-        }
+        // 残りのPDEをnot presentにする
+        for (uint32_t i = 1; i < 1024; ++i) page_directory[i] = 0x00000000;
 
-        printk("paging: identity map initialized for %u MB (pages=%u, tables=%u)\n",
-               (unsigned)map_mb, (unsigned)pages, (unsigned)needed_pt);
+        printk("paging: identity map initialized for %u MB (pages=%u)\n", (unsigned)map_mb, (unsigned)pages);
 }
 
-// ページング機能を有効化する
+/**
+ * @fn paging_enable
+ * @brief ページングを有効化
+ */
 void paging_enable(void) {
         // CR3をロード
         asm volatile("mov %%eax, %%cr3" :: "a"((uint32_t)page_directory));
@@ -125,7 +107,6 @@ void paging_enable(void) {
         asm volatile("mov %%eax, %%cr0" :: "a" (cr0));
 }
 
-// ページフォルト発生時のハンドラ
 void page_fault_handler(uint32_t vec) {
         (void)vec;
         uint32_t fault_addr;