diff options
| author | Visa Hankala <visa@cvs.openbsd.org> | 2019-03-23 05:47:24 +0000 |
|---|---|---|
| committer | Visa Hankala <visa@cvs.openbsd.org> | 2019-03-23 05:47:24 +0000 |
| commit | f521f9e1b2f1715477b2c105eb48c1f25315e953 (patch) | |
| tree | e82fafdc3546960380cc7e5354d69f36b67f6ade /sys/arch/mips64/include | |
| parent | 805f6dc04087ae26b5b0db8f107a6685a3362df3 (diff) | |
Use the debugger mutex for `ddb_mp_mutex'. This should prevent a race
that could leave `ddb_mp_mutex' locked if one CPU incremented
`db_active' while another CPU was in the critical section. When the race
hit, the debugger was unable to resume execution or switch between CPUs.
Race analyzed by patrick@
OK mpi@ patrick@
Diffstat (limited to 'sys/arch/mips64/include')
| -rw-r--r-- | sys/arch/mips64/include/db_machdep.h | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/sys/arch/mips64/include/db_machdep.h b/sys/arch/mips64/include/db_machdep.h index 7367d00a7be..fc40211ad82 100644 --- a/sys/arch/mips64/include/db_machdep.h +++ b/sys/arch/mips64/include/db_machdep.h @@ -1,4 +1,4 @@ -/* $OpenBSD: db_machdep.h,v 1.17 2016/04/27 11:10:48 mpi Exp $ */ +/* $OpenBSD: db_machdep.h,v 1.18 2019/03/23 05:47:23 visa Exp $ */ /* * Copyright (c) 1998-2003 Opsycon AB (www.opsycon.se) @@ -70,8 +70,6 @@ void db_stopcpu(int); int dbmd_print_insn(uint32_t, db_addr_t, int (*)(const char *, ...)); -extern struct mutex ddb_mp_mutex; - #define DDB_STATE_NOT_RUNNING 0 #define DDB_STATE_RUNNING 1 #define DDB_STATE_EXITING 2 |