littlekernel / lk

LK embedded kernel
MIT License
3.11k stars 613 forks source link

[lib][debugcommands] option to input physical address for debug comm… #359

Closed vamshi51 closed 7 months ago

vamshi51 commented 1 year ago

…ands

dw/mw and the sister commands take virtual address as input for display or modify. Option "-p" to the command signifies the address input is physical, and the address translation is handled by the command.

Newly added option is available with WITH_KERNEL_VM, usage as follows

] dw 0xffffffff05100000 4 0xffffffff05100000: 0000dead ] dw 0x5100000 4 -p 0xffffffff05100000: 0000dead

vamshi51 commented 1 year ago

@travisg , do you see anything pending on this pull-request?

travisg commented 1 year ago

Hmm, trying to figure out what I commented on. I really hate the github review UI, hard to remember what I even left comments on...

travisg commented 1 year ago

It wont merge cleanly now apparently, can you rebase your change on top of the master branch?

vamshi51 commented 1 year ago

@travisg could you please merge it.

travisg commented 7 months ago

merged!