feat: debug kernel stack #1176
Open
feat: debug kernel stack #1176
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Godones <[email protected]>
|
@fslongjin 请帮我看看这段代码实现有什么错误吗？感觉没什么错误，但是无法运行 |
具体是啥问题，贴一下报错？ |
|
|
是固定挂在这还是在启动阶段随机挂的 |
|
固定这个位置 |
|
具体位置是开启中断后，我打印了中断处理信息，发现发生了几次中断，然后就结束了 |
刚才看了一下，暂时没发现问题，我明天认真看看。 |
Signed-off-by: Godones <[email protected]>
fslongjin
reviewed
Jun 4, 2025
添加enable_kernel_wp函数来设置CR0的WP位，防止内核错误写入只读页面 Signed-off-by: longjin <[email protected]>
|
我为x86加了kernel wp的保护，现在这个机制能检测到栈溢出了。但是还存在以下问题：
|
Signed-off-by: longjin <[email protected]>
Restore accidentally deleted functions. Signed-off-by: Godones <[email protected]>
内核栈的检测只是用来debug使用，在内核稳定后应该使用原有的实现，所以分配多一倍的内存并没有太多的副作用。 |
|
我在想，异常处理程序里面不是有发生异常时的栈指针吗？那为啥不能traceback哈哈哈 |
|
使用unwind进行堆栈展开时，它的工作方式是从当前栈帧逐步向上展开的，它不能对异常处理函数进行展开 |
- 从default特性中移除kstack_protect - 为X86_64MMBootstrapInfo添加kernel_rodata_start字段 - 调整内核页标志对rodata区域的判断逻辑 Signed-off-by: longjin <[email protected]>
fslongjin
reviewed
Jun 10, 2025
kernel/src/mm/mod.rs
Outdated
Comment on lines
688
to
692
| // map集合中并不会主动设置该位，需要手动设置 | ||
| if vm_flags.contains(VmFlags::VM_WRITE) { | ||
| ret = ret.set_write(true) | ||
| } | ||
|
|
There was a problem hiding this comment.
这里的设置不对。会导致动态链接程序第二次加载的时候报错：
我检查了一下protection map，这个map的值貌似没问题？
https://code.dragonos.org.cn/xref/linux-6.6.21/arch/x86/mm/pgprot.c#8
Signed-off-by: longjin <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
目标: 检测内核栈溢出bug
实现: 在分配内核栈时手动在内核高位地址空间分配一个映射区间，同时保留一块未映射区域