-
Notifications
You must be signed in to change notification settings - Fork 82
Fix: trace irq context for is_in_irq make correct decision #275
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
now , |
|
has probability in to deadlock for case 'test_socket_file' |
|
build_prs #275 |
|
Job is started, see https://github.com/vivoblueos/kernel/actions/runs/20804210922. |
|
❌ Job failed. Failed jobs: check_format (failure), build_and_check_boards (failure), see https://github.com/vivoblueos/kernel/actions/runs/20804210922. |
d0e7db7 to
ba13ad8
Compare
|
build_prs #275 |
|
Job is started, see https://github.com/vivoblueos/kernel/actions/runs/20804382293. |
|
❌ Job failed. Failed jobs: check_format (failure), build_and_check_boards (failure), see https://github.com/vivoblueos/kernel/actions/runs/20804382293. |
ba13ad8 to
b652ca2
Compare
|
build_prs #275 |
|
Job is started, see https://github.com/vivoblueos/kernel/actions/runs/20804578448. |
|
❌ Job failed. Failed jobs: check_format (failure), build_and_check_boards (failure), see https://github.com/vivoblueos/kernel/actions/runs/20804578448. |
b652ca2 to
1bc3a21
Compare
|
build_prs #275 |
|
Job is started, see https://github.com/vivoblueos/kernel/actions/runs/20804898480. |
|
❌ Job failed. Failed jobs: build_and_check_boards (failure), see https://github.com/vivoblueos/kernel/actions/runs/20804898480. |
1bc3a21 to
1afe094
Compare
|
build_prs #275 |
|
Job is started, see https://github.com/vivoblueos/kernel/actions/runs/20807505119. |
|
✅ All jobs completed successfully, see https://github.com/vivoblueos/kernel/actions/runs/20807505119. |
|
|
||
| #[no_mangle] | ||
| pub unsafe extern "C" fn uart0_handler() { | ||
| let _trace = IrqTrace::new(IrqNumber::new(37)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do not hard code this number.
| pub extern "C" fn handle_tick_increment() { | ||
| let _guard = DisableInterruptGuard::new(); | ||
| // riscv has its own irq trace in riscv.rs | ||
| #[cfg(not(any(target_board = "qemu_riscv64", target_board = "qemu_riscv32")))] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should wrap handle_tick_increment in a arch/board specific function.
1afe094 to
2c96b8b
Compare
No description provided.