Author: mckusick
Date: Mon May 27 06:22:43 2019
New Revision: 348300
URL: https://svnweb.freebsd.org/changeset/base/348300
Log:
Add function name and line number debugging information to softupdates
worklist structures to help track their movement between work lists.
No functional change to the operation of soft updates intended.
Modified:
head/sys/ufs/ffs/ffs_softdep.c
head/sys/ufs/ffs/softdep.h
Modified: head/sys/ufs/ffs/ffs_softdep.c
==============================================================================
--- head/sys/ufs/ffs/ffs_softdep.c Mon May 27 04:20:31 2019
(r348299)
+++ head/sys/ufs/ffs/ffs_softdep.c Mon May 27 06:22:43 2019
(r348300)
@@ -1011,42 +1011,59 @@ MTX_SYSINIT(softdep_lock, &lk, "Global Softdep Lock",
#define WORKLIST_REMOVE_UNLOCKED WORKLIST_REMOVE
#else /* DEBUG */
-static void worklist_insert(struct workhead *, struct worklist *, int);
-static void worklist_remove(struct worklist *, int);
+static void worklist_insert(struct workhead *, struct worklist *, int,
+ const char *, int);
+static void worklist_remove(struct worklist *, int, const char *, int);
-#define WORKLIST_INSERT(head, item) worklist_insert(head, item, 1)
-#define WORKLIST_INSERT_UNLOCKED(head, item) worklist_insert(head, item, 0)
-#define WORKLIST_REMOVE(item) worklist_remove(item, 1)
-#define WORKLIST_REMOVE_UNLOCKED(item) worklist_remove(item, 0)
+#define WORKLIST_INSERT(head, item) \
+ worklist_insert(head, item, 1, __func__, __LINE__)
+#define WORKLIST_INSERT_UNLOCKED(head, item)\
+ worklist_insert(head, item, 0, __func__, __LINE__)
+#define WORKLIST_REMOVE(item)\
+ worklist_remove(item, 1, __func__, __LINE__)
+#define WORKLIST_REMOVE_UNLOCKED(item)\
+ worklist_remove(item, 0, __func__, __LINE__)
static void
-worklist_insert(head, item, locked)
+worklist_insert(head, item, locked, func, line)
struct workhead *head;
struct worklist *item;
int locked;
+ const char *func;
+ int line;
{
if (locked)
LOCK_OWNED(VFSTOUFS(item->wk_mp));
if (item->wk_state & ONWORKLIST)
- panic("worklist_insert: %p %s(0x%X) already on list",
- item, TYPENAME(item->wk_type), item->wk_state);
+ panic("worklist_insert: %p %s(0x%X) already on list, "
+ "added in function %s at line %d",
+ item, TYPENAME(item->wk_type), item->wk_state,
+ item->wk_func, item->wk_line);
item->wk_state |= ONWORKLIST;
+ item->wk_func = func;
+ item->wk_line = line;
LIST_INSERT_HEAD(head, item, wk_list);
}
static void
-worklist_remove(item, locked)
+worklist_remove(item, locked, func, line)
struct worklist *item;
int locked;
+ const char *func;
+ int line;
{
if (locked)
LOCK_OWNED(VFSTOUFS(item->wk_mp));
if ((item->wk_state & ONWORKLIST) == 0)
- panic("worklist_remove: %p %s(0x%X) not on list",
- item, TYPENAME(item->wk_type), item->wk_state);
+ panic("worklist_remove: %p %s(0x%X) not on list, "
+ "removed in function %s at line %d",
+ item, TYPENAME(item->wk_type), item->wk_state,
+ item->wk_func, item->wk_line);
item->wk_state &= ~ONWORKLIST;
+ item->wk_func = func;
+ item->wk_line = line;
LIST_REMOVE(item, wk_list);
}
#endif /* DEBUG */
@@ -1172,8 +1189,10 @@ workitem_free(item, type)
#ifdef DEBUG
if (item->wk_state & ONWORKLIST)
- panic("workitem_free: %s(0x%X) still on list",
- TYPENAME(item->wk_type), item->wk_state);
+ panic("workitem_free: %s(0x%X) still on list, "
+ "added in function %s at line %d",
+ TYPENAME(item->wk_type), item->wk_state,
+ item->wk_func, item->wk_line);
if (item->wk_type != type && type != D_NEWBLK)
panic("workitem_free: type mismatch %s != %s",
TYPENAME(item->wk_type), TYPENAME(type));
Modified: head/sys/ufs/ffs/softdep.h
==============================================================================
--- head/sys/ufs/ffs/softdep.h Mon May 27 04:20:31 2019 (r348299)
+++ head/sys/ufs/ffs/softdep.h Mon May 27 06:22:43 2019 (r348300)
@@ -213,6 +213,10 @@ struct worklist {
struct mount *wk_mp; /* Mount we live in */
unsigned int wk_type:8, /* type of request */
wk_state:24; /* state flags */
+#ifdef DEBUG
+ const char *wk_func; /* func where added / removed */
+ int wk_line; /* line where added / removed */
+#endif
};
#define WK_DATA(wk) ((void *)(wk))
#define WK_PAGEDEP(wk) ((struct pagedep *)(wk))
_______________________________________________
svn-src-head@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"
