On 6/6/23 17:05, Luís Henriques wrote:
Xiubo Li <xiubli@xxxxxxxxxx> writes:
On 6/6/23 14:25, Milind Changire wrote:
nit:
the commit title refers to ceph_open() but the code changes are
pertaining to ceph_lookup()
Good catch.
I couldn't remember why we didn't fix this before as I remembered I have found
this.
Yeah, it's really odd we didn't catch this before. I had to go look at
old email: this helper was initially used in ceph_atomic_open() only, but
then in v3 it was made more generic so that it could also be used in
ceph_lookup().
Xiubo, do you want me to send out a new version of this patch, or are you
OK simply updating the subject, using 'ceph_lookup' instead of
'ceph_open'?
No worry, I have fixed this in the 'testing' branch and I will send out
the v20 for this whole series later.
Thank
- Xiubo
Cheers,
