I've revised my recent patch set based on feedback from Andreas. Here is my latest. Changes from last time are:
1. Removed the spin_trylock related info from the status sysfs file as per Andreas's feedback. Although valuable to have, I see his point. 2. Removed "gfs2: Add new trace point for glock ail management". I still think this would be useful to have, but perhaps only compiled in with a new debug kernel option. For now, leave it out. 3. Removed "gfs2: add memory barriers to gfs2_glock_remove_revoke". It is not needed. 4. Removed "gfs2: Only do glock put in gfs2_create_inode for free inodes". It was pushed to the repo separately. 5. Revised "gfs2: instrumentation wrt log_flush stuck" moving that code to a new function, which makes the err string contiguous for grep. 6. Added "gfs2: new slab for transactions" which is revised to fix bugs that were in the old version. 7. Added "gfs2: fix use-after-free on transaction ail lists" that fixes a problem that comes to light with the previous patch. 8. Patches 1/8 and 2/8 are omitted, since they were pushed separately. Bob Peterson (6): gfs2: Add new sysfs file for gfs2 status gfs2: print mapping->nrpages in glock dump for address space glocks gfs2: instrumentation wrt log_flush stuck gfs2: introduce new gfs2_glock_assert_withdraw gfs2: new slab for transactions gfs2: fix use-after-free on transaction ail lists fs/gfs2/glock.c | 32 +++++++++++++++--------- fs/gfs2/glock.h | 9 +++++++ fs/gfs2/glops.c | 2 ++ fs/gfs2/log.c | 54 ++++++++++++++++++++++++++++------------ fs/gfs2/main.c | 9 +++++++ fs/gfs2/sys.c | 65 +++++++++++++++++++++++++++++++++++++++++++++++++ fs/gfs2/trans.c | 22 ++++++++++++++--- fs/gfs2/trans.h | 1 + fs/gfs2/util.c | 1 + fs/gfs2/util.h | 1 + 10 files changed, 165 insertions(+), 31 deletions(-) -- 2.26.2
