Skip to content

Conversation

@assembler-0
Copy link
Owner

@assembler-0 assembler-0 commented Nov 2, 2025

Summary by CodeRabbit

Release Notes

  • Chores

    • Standardized include paths across the codebase for improved build consistency and maintainability.
    • Disabled ccache by default in build configuration.
    • Reorganized module directory structure in build system to reflect modular architecture.
  • New Features

    • Enhanced memory management system with per-CPU caching for improved allocation performance.

@assembler-0 assembler-0 merged commit c31fc77 into main Nov 2, 2025
1 of 2 checks passed
@coderabbitai
Copy link

coderabbitai bot commented Nov 2, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

The pull request systematically converts relative include paths throughout the codebase to project-rooted paths, normalizing header inclusion across drivers, filesystems, kernel modules, and utilities. Additionally, CMake configuration is updated to disable ccache by default, and include directories are restructured from a kernel-centric layout to a modular architecture-and-driver-oriented structure. The magazine heap allocator receives substantial enhancements with new public APIs and internal mechanisms for per-CPU caches and depot-based magazine management.

Poem

🐰 Hops of joy for paths made clear,
No more dots and slashes here!
Include roads now straight and true,
Cache disabled—new defaults brew.
Magazine heaps dance with grace,
Per-CPU caches find their place!


📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between fb87f81 and 4f6c3bc.

📒 Files selected for processing (36)
  • arch/x86_64/interrupts/Interrupts.c (1 hunks)
  • cmake/ccache.cmake (1 hunks)
  • cmake/source.cmake (1 hunks)
  • drivers/APIC/APIC.c (1 hunks)
  • drivers/ISA/ISA.c (1 hunks)
  • drivers/OPIC/OPIC.c (1 hunks)
  • drivers/Random.c (1 hunks)
  • drivers/Serial.c (1 hunks)
  • drivers/Vesa.c (1 hunks)
  • drivers/ethernet/intel/E1000.c (1 hunks)
  • drivers/ethernet/interface/Arp.c (1 hunks)
  • drivers/ethernet/interface/Arp.h (1 hunks)
  • drivers/ethernet/interface/Ip.c (1 hunks)
  • drivers/ethernet/realtek/RTL8139.c (1 hunks)
  • drivers/input/Keyboard.c (1 hunks)
  • drivers/storage/Ide.c (1 hunks)
  • drivers/usb/hid/USBKeyboard.h (1 hunks)
  • drivers/vmware/SVGAII.c (1 hunks)
  • drivers/xHCI/xHCI.c (1 hunks)
  • fs/CharDevice.c (1 hunks)
  • fs/EXT/Ext2.c (1 hunks)
  • fs/FsUtils.h (1 hunks)
  • fs/Iso9660.c (1 hunks)
  • fs/VFS.c (1 hunks)
  • fs/devfs/DevFS.c (1 hunks)
  • fs/devfs/DevFS.h (1 hunks)
  • fs/procfs/ProcFS.h (1 hunks)
  • kernel/core/Panic.c (1 hunks)
  • kernel/etc/Shell.c (2 hunks)
  • kernel/etc/VBEConsole.c (1 hunks)
  • kernel/execf/aout/AoutLoader.c (1 hunks)
  • kernel/execf/elf/ELFloader.c (1 hunks)
  • kernel/execf/pe/PEloader.c (1 hunks)
  • kernel/ipc/Ipc.c (1 hunks)
  • mm/dynamic/c/Magazine.c (2 hunks)
  • vfcompositor/app/GUIShell.c (1 hunks)

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants