Skip to content

feat #81: print stack - #121

Merged
iamantoinev merged 9 commits into
mainfrom
print-stack
Sep 19, 2025
Merged

iamantoinev merged 9 commits into
mainfrom
print-stack

Conversation

@iamantoinev

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings September 19, 2025 13:17

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR implements stack tracing functionality for the kernel by adding a print_stack_trace() function that displays both stack contents and call chain information. The feature is integrated into the kernel panic handler to provide debugging information when the system crashes.

  • Added print_stack_trace() function that walks the stack frame and prints memory contents and return addresses
  • Integrated stack tracing into the kernel panic macro to automatically show stack information during crashes
  • Added test panic call in TTY handler to demonstrate the functionality

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

File Description
kernel/src/utils.c Implements the stack tracing functionality with x86 assembly
kernel/include/utils.h Header declaration for the stack trace function
kernel/include/kpanic.h Integrates stack tracing into the panic handler
kernel/src/tty/tty.c Adds test panic call to demonstrate stack tracing

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread kernel/src/utils.c
Comment thread kernel/src/utils.c Outdated
Comment thread kernel/src/tty/tty.c Outdated
@iamantoinev
iamantoinev requested a review from Copilot September 19, 2025 13:38

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread kernel/src/utils.c
Comment thread kernel/src/utils.c
iamantoinev and others added 2 commits September 19, 2025 15:40
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Antoine Verin <24255551+antoineverin@users.noreply.github.com>
@iamantoinev
iamantoinev enabled auto-merge (squash) September 19, 2025 13:41
@sonarqubecloud

Copy link
Copy Markdown

@iamantoinev
iamantoinev merged commit 7ffa674 into main Sep 19, 2025
4 checks passed
@iamantoinev
iamantoinev deleted the print-stack branch September 19, 2025 13:54
This was linked to issues Sep 19, 2025
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.

print stack strace print stack values of a function

3 participants