On Thu, Aug 01, 2019 at 04:02:40PM +0200, Ondrej Mosnacek wrote: > rename_retry: > - spin_unlock(&this_parent->d_lock); > rcu_read_unlock(); > + spin_unlock(&this_parent->d_lock); > + if (lock_inode) > + inode_unlock(this_parent->d_inode); ... and while we are at it, what's to keep this_parent positive here, now that you've dropped ->d_lock on it? Or to prevent it becoming negative, then posiive _again_, with an unrelated inode?