I've been going over the cap handling code with an aim toward simplifying the locking. There's one fix for a potential use-after-free race in here. This also eliminates a number of __acquires and __releases annotations by reorganizing the code, and adds some (hopefully helpful) comments. There should be no behavioral changes with this set. Jeff Layton (8): ceph: reorganize __send_cap for less spinlock abuse ceph: split up __finish_cap_flush ceph: add comments for handle_cap_flush_ack logic ceph: don't release i_ceph_lock in handle_cap_trunc ceph: don't take i_ceph_lock in handle_cap_import ceph: document what protects i_dirty_item and i_flushing_item ceph: fix potential race in ceph_check_caps ceph: throw a warning if we destroy session with mutex still locked fs/ceph/caps.c | 292 ++++++++++++++++++++++++------------------- fs/ceph/mds_client.c | 1 + fs/ceph/super.h | 4 +- 3 files changed, 170 insertions(+), 127 deletions(-) -- 2.25.1
