Re: [GIT PULL] Ceph fixes for 5.1-rc7

Linus Torvalds
2019-04-25T18:25:14Z

On Thu, Apr 25, 2019 at 11:21 AM Al Viro <viro@zeniv.linux.org.uk> wrote:
>
> I really wonder if 76a495d666e5 (ceph: ensure d_name stability in
> ceph_dentry_hash()) makes any sense; OK, you have ->d_lock held
> over that, but what does it protect against? Sure, you'll get
> something that was valid while you held ->d_lock, but what good
> does it do to the callers? If they really have to care about
> races with d_move(), which value do they want?

I suspect they only care that the data they gather for the network
packet is coherent, and passes internal sanity tests. You get either
old or new, but at least you don't get "not NUL terminated because the
length didn't match the data".

Linus