From: Steve French <smfrench-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
To: Jeff Layton <jlayton-eUNUBHrolfbYtjvyW6yDsg@public.gmane.org>,
linux-fsdevel
<linux-fsdevel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>
Cc: Nakajima Akira
<nakajima.akira-o7dWnD6vFTHqq2nvvmkE/A@public.gmane.org>,
"linux-cifs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
<linux-cifs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>
Subject: Re: [PATCH] cifs: When "refer file directly", make new inode cache if "uniqueid is different"
Date: Tue, 7 Apr 2015 09:39:53 -0500 [thread overview]
Message-ID: <CAH2r5ms5wn1ii563gK=+=koeQEmy5gY=xTP+Y8_GGjyPJwznzQ@mail.gmail.com> (raw)
In-Reply-To: <20150407064551.36374c43-9yPaYZwiELC+kQycOl6kW4xkIHaj4LzF@public.gmane.org>
On Tue, Apr 7, 2015 at 5:45 AM, Jeff Layton <jlayton-eUNUBHrolfbYtjvyW6yDsg@public.gmane.org> wrote:
> On Wed, 24 Dec 2014 11:27:38 +0900
> Nakajima Akira <nakajima.akira-o7dWnD6vFTHqq2nvvmkE/A@public.gmane.org> wrote:
>
>> When refer file "directly" (e.g. ls -li <filename>),
>> if file is same name, old inode cache is used.
>> This causes that client shows wrong(old) inode number.
>> So this patch is that if uniqueid is different, return error.
>>
>> ## But this patch is applicable to when Server is UNIX.
>> ## When Server is Windows, we need another new patch.
>>
>>
>> Reproducible sample :
>> 1. create file 'a' at cifs client.
>> 2. rm 'a' and touch 'b a' at server.
>> 3. ls -li 'a' at client, then client shows wrong(old) inode number.
>>
>> Bug link:
>> https://bugzilla.kernel.org/show_bug.cgi?id=90021
>>
>>
>>
>> Signed-off-by: Nakajima Akira <nakajima.akira-o7dWnD6vFTHqq2nvvmkE/A@public.gmane.org>
>> diff -uprN -X linux-3.18-vanilla/Documentation/dontdiff linux-3.18-vanilla/fs/cifs/inode.c linux-3.18/fs/cifs/inode.c
>> --- linux-3.18-vanilla/fs/cifs/inode.c 2014-12-08 07:21:05.000000000 +0900
>> +++ linux-3.18/fs/cifs/inode.c 2014-12-19 11:07:59.127000000 +0900
>> @@ -402,9 +402,18 @@ int cifs_get_inode_info_unix(struct inod
>> rc = -ENOMEM;
>> } else {
>> /* we already have inode, update it */
>> +
>> + /* if uniqueid is different, return error */
>> + if (unlikely(cifs_sb->mnt_cifs_flags & CIFS_MOUNT_SERVER_INUM &&
>> + CIFS_I(*pinode)->uniqueid != fattr.cf_uniqueid)) {
>> + rc = -ENOENT;
>> + goto cgiiu_exit;
>> + }
>> +
>> cifs_fattr_to_inode(*pinode, &fattr);
>> }
>>
>> +cgiiu_exit:
>> return rc;
>> }
>>
>
> Returning ENOENT here seems like the wrong error to me. That path does
> exist, it just no longer refers to the same file as before.
>
> Maybe ESTALE would be better as it would allow the VFS layer
> to revalidate the dcache and invalidate the old dentry?
>
> --
> Jeff Layton <jlayton-eUNUBHrolfbYtjvyW6yDsg@public.gmane.org>
Similar to what Jeff mentioned, isn't the nfs_relavidate_inode path
roughly equivalent to what we want here (where nfs.ko returns ESTALE
on various cases where we detect an inode that doesn't match what we
expect)?
--
Thanks,
Steve
next parent reply other threads:[~2015-04-07 14:39 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <549A249A.3080000@nttcom.co.jp>
[not found] ` <20150407064551.36374c43@tlielax.poochiereds.net>
[not found] ` <20150407064551.36374c43-9yPaYZwiELC+kQycOl6kW4xkIHaj4LzF@public.gmane.org>
2015-04-07 14:39 ` Steve French [this message]
[not found] ` <5526335C.3030701@nttcom.co.jp>
[not found] ` <20150410091647.39fb6515@synchrony.poochiereds.net>
[not found] ` <CADT32eK6Zb7arYFyCPc+yt509P3pgg65bModxgNLyN1vb-a2-w@mail.gmail.com>
[not found] ` <20150413164235.2f4bd67b@synchrony.poochiereds.net>
[not found] ` <55374057.6010908@nttcom.co.jp>
2015-04-22 15:43 ` Fwd: [PATCH] cifs: When "refer file directly", make new inode cache if "uniqueid is different" Steve French
[not found] ` <55374057.6010908-o7dWnD6vFTHqq2nvvmkE/A@public.gmane.org>
2015-05-20 18:06 ` Steve French
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CAH2r5ms5wn1ii563gK=+=koeQEmy5gY=xTP+Y8_GGjyPJwznzQ@mail.gmail.com' \
--to=smfrench-re5jqeeqqe8avxtiumwx3w@public.gmane.org \
--cc=jlayton-eUNUBHrolfbYtjvyW6yDsg@public.gmane.org \
--cc=linux-cifs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linux-fsdevel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=nakajima.akira-o7dWnD6vFTHqq2nvvmkE/A@public.gmane.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).