diff options
| author | Merlin Scholz <merlin@scholz.ruhr> | 2022-10-19 01:42:54 +0200 |
|---|---|---|
| committer | Merlin Scholz <merlin@scholz.ruhr> | 2022-10-19 01:42:54 +0200 |
| commit | 062b5ea39274e32d65f6369977a0b9142509e645 (patch) | |
| tree | 470e15cef450f4371afe7d2ee0573fbb3683f57b | |
| parent | 43541eb4960ae12af3a65f0fa389a5c8c13f67ab (diff) | |
Fix memory padding by replacing char with enum in log_action
| -rw-r--r-- | sys/sys/lockdoc.h | 7 | ||||
| -rw-r--r-- | sys/sys/lockdoc_event.h | 2 |
2 files changed, 5 insertions, 4 deletions
diff --git a/sys/sys/lockdoc.h b/sys/sys/lockdoc.h index b101b05bd4a..45c1aa7faeb 100644 --- a/sys/sys/lockdoc.h +++ b/sys/sys/lockdoc.h @@ -28,7 +28,8 @@ static inline void log_lock(int lock_op, const void* ptr, const char *file, int memset(&la_buffer,0,sizeof(la_buffer)); - // TODO fail shows 0x016c instead of 0x6c + //strncpy((char *)&la_buffer, "abcdefgh", LOG_CHAR_BUFFER_LEN); + // action la_buffer.action = LOCKDOC_LOCK_OP; @@ -41,11 +42,11 @@ static inline void log_lock(int lock_op, const void* ptr, const char *file, int // size only relevant for memory access // TODO type - //strncpy(la_buffer.type, ,LOG_CHAR_BUFFER_LEN); + strncpy(la_buffer.type, "dummy", LOG_CHAR_BUFFER_LEN); la_buffer.type[LOG_CHAR_BUFFER_LEN - 1] = '\0'; // TODO lock_member - strncpy(la_buffer.lock_member,"dummy",LOG_CHAR_BUFFER_LEN); + strncpy(la_buffer.lock_member, "dummy", LOG_CHAR_BUFFER_LEN); la_buffer.lock_member[LOG_CHAR_BUFFER_LEN - 1] = '\0'; diff --git a/sys/sys/lockdoc_event.h b/sys/sys/lockdoc_event.h index 6addbe5b3ab..8552cb9bd18 100644 --- a/sys/sys/lockdoc_event.h +++ b/sys/sys/lockdoc_event.h @@ -51,7 +51,7 @@ enum IRQ_SYNC { }; struct log_action { - char action; + enum LOCKDOC_OP action; uint32_t lock_op; uint32_t ptr; uint32_t size; |
