Fixed L(ock) and U(nlock) indicators in cStateLockLog

This commit is contained in:
Klaus Schmidinger 2017-06-08 08:24:30 +02:00
parent 081e9eb45f
commit 38fed8c901
2 changed files with 7 additions and 5 deletions

View File

@ -9103,9 +9103,10 @@ Video Disk Recorder Revision History
- Fixed the locking sequence when switching between 'Now', 'Next' and 'Schedule' - Fixed the locking sequence when switching between 'Now', 'Next' and 'Schedule'
in the Schedules menu. in the Schedules menu.
2017-06-06: Version 2.3.7 2017-06-08: Version 2.3.7
- Fixed false positives when checking the locking sequence, in case of nested locks - Fixed false positives when checking the locking sequence, in case of nested locks
within the same thread. within the same thread.
- Fixed uninitialized variable SdWatchdog in vdr.c (reported by Frank Neumann). - Fixed uninitialized variable SdWatchdog in vdr.c (reported by Frank Neumann).
- Fixed the locking sequence when starting an instant recording. - Fixed the locking sequence when starting an instant recording.
- Fixed L(ock) and U(nlock) indicators in cStateLockLog.

View File

@ -4,7 +4,7 @@
* See the main source file 'vdr.c' for copyright information and * See the main source file 'vdr.c' for copyright information and
* how to reach the author. * how to reach the author.
* *
* $Id: thread.c 4.6 2017/06/07 12:31:31 kls Exp $ * $Id: thread.c 4.7 2017/06/08 08:22:10 kls Exp $
*/ */
#include "thread.h" #include "thread.h"
@ -560,6 +560,7 @@ cString cBackTrace::GetCaller(int Level, bool Mangled)
#define SLL_LENGTH 256 // the maximum length of log entries #define SLL_LENGTH 256 // the maximum length of log entries
#define SLL_MAX_LIST 9 // max. number of lists to log #define SLL_MAX_LIST 9 // max. number of lists to log
#define SLL_WRITE_FLAG 0x80000000 #define SLL_WRITE_FLAG 0x80000000
#define SLL_LOCK_FLAG 0x40000000
class cStateLockLog { class cStateLockLog {
private: private:
@ -599,10 +600,10 @@ void cStateLockLog::Dump(const char *Name, tThreadId ThreadId)
q += sprintf(q, "%5d", tid); q += sprintf(q, "%5d", tid);
int Flags = logFlags[logIndex]; int Flags = logFlags[logIndex];
bool Write = Flags & SLL_WRITE_FLAG; bool Write = Flags & SLL_WRITE_FLAG;
Flags &= ~SLL_WRITE_FLAG; bool Lock = Flags & SLL_LOCK_FLAG;
Flags &= ~(SLL_WRITE_FLAG | SLL_LOCK_FLAG);
int Changed = LastFlags ^ Flags; int Changed = LastFlags ^ Flags;
LastFlags = Flags; LastFlags = Flags;
bool Lock = (Flags & Changed) != 0;
for (int i = 0; i <= SLL_MAX_LIST; i++) { for (int i = 0; i <= SLL_MAX_LIST; i++) {
char c = '-'; char c = '-';
int b = 1 << i; int b = 1 << i;
@ -659,7 +660,7 @@ void cStateLockLog::Check(const char *Name, bool Lock, bool Write)
else if (--logCounter[Index][n] == 0) else if (--logCounter[Index][n] == 0)
flags[Index] &= ~b; flags[Index] &= ~b;
logThreadIds[logIndex] = ThreadId; logThreadIds[logIndex] = ThreadId;
logFlags[logIndex] = flags[Index] | (Write ? SLL_WRITE_FLAG : 0); logFlags[logIndex] = flags[Index] | (Write ? SLL_WRITE_FLAG : 0) | (Lock ? SLL_LOCK_FLAG : 0);
#ifdef DEBUG_LOCKCALL #ifdef DEBUG_LOCKCALL
strn0cpy(logCaller[logIndex], cBackTrace::GetCaller(Lock ? 3 : 5, true), SLL_LENGTH); strn0cpy(logCaller[logIndex], cBackTrace::GetCaller(Lock ? 3 : 5, true), SLL_LENGTH);
#endif #endif