On Wed, Mar 15, 2023 at 02:33:48PM +0530, Kirtikumar Anandrao Ramchandani wrote: > While I am going through the code at the moment, I think there is one more > issue. It probably can't just compare "old_dir" and "new_dir", since those > are just pointers to structs. So, both addresses can be completely > different, and still represent the same folder, yes? No, they can not. We should never have different in-core instances of struct inode representing the same on-disk object - otherwise all locking goes to hell, for example.