Two changes that would be useful: - enable PT_*FPREGS, Mark has already done the job. - hide PT_STEP since hardware support seems missing (the spec only talks about single stepping support in "Debug mode", which is not "Machine" or "Supervisor" mode). Since we don't emulate it (like mips64 for example) it doesn't make sense to provide the define. I doubt that hiding PT_STEP won't magically fix devel/gdb runtime support though.
Thoughts? ok? Index: ptrace.h =================================================================== RCS file: /d/cvs/src/sys/arch/riscv64/include/ptrace.h,v retrieving revision 1.2 diff -u -p -p -u -r1.2 ptrace.h --- ptrace.h 12 May 2021 01:20:52 -0000 1.2 +++ ptrace.h 3 Sep 2021 12:25:54 -0000 @@ -16,10 +16,10 @@ * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE. */ +#if 0 #define PT_STEP (PT_FIRSTMACH + 0) +#endif #define PT_GETREGS (PT_FIRSTMACH + 1) #define PT_SETREGS (PT_FIRSTMACH + 2) -#if 0 // XXX ptrace fpreg support #define PT_GETFPREGS (PT_FIRSTMACH + 3) #define PT_SETFPREGS (PT_FIRSTMACH + 4) -#endif -- jca | PGP : 0x1524E7EE / 5135 92C1 AD36 5293 2BDF DDCC 0DFA 74AE 1524 E7EE
