d/freebsd-dev
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

db_nextframe/amd64: remove TRAP_INTERRUPT frame type

Browse Source 
Besides the confusing name, this type is effectively unused.
In all cases where it could be set, the INTERRUPT type is set by the
earlier code.  The conditions for TRAP_INTERRUPT are a subset of the
conditions for INTERRUPT.

Reviewed by:	kib, markj
MFC after:	2 weeks
Differential Revision: https://reviews.freebsd.org/D22305
This commit is contained in:
Andriy Gapon 2019-11-11 17:11:49 +00:00
parent 467057fcd9
commit 2961e6efeb
Notes: svn2git 2020-12-20 02:59:44 +00:00 
svn path=/head/; revision=354619
1 changed files with 0 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
sys/amd64/amd64/db_trace.c
View File

@ -123,7 +123,6 @@ db_frame(struct db_variable *vp, db_expr_t *valuep, int op)
#define TRAP 1
#define INTERRUPT 2
#define SYSCALL 3
#define TRAP_INTERRUPT 5
static void db_nextframe(struct amd64_frame **, db_addr_t *, struct thread *);
static void db_print_stack_entry(const char *, db_addr_t, void *);
@ -218,13 +217,6 @@ db_nextframe(struct amd64_frame **fp, db_addr_t *ip, struct thread *td)
else if (strcmp(name, "Xint0x80_syscall") == 0)
frame_type = SYSCALL;
#endif
/* XXX: These are interrupts with trap frames. */
else if (strcmp(name, "Xtimerint") == 0 ||
strcmp(name, "Xcpustop") == 0 ||
strcmp(name, "Xcpususpend") == 0 ||
strcmp(name, "Xrendezvous") == 0 ||
strcmp(name, "Xipi_intr_bitmap_handler") == 0)
frame_type = TRAP_INTERRUPT;
}
/*
@ -256,7 +248,6 @@ db_nextframe(struct amd64_frame **fp, db_addr_t *ip, struct thread *td)
db_printf("--- syscall");
decode_syscall(tf->tf_rax, td);
break;
case TRAP_INTERRUPT:
case INTERRUPT:
db_printf("--- interrupt");
break;