Re: [PATCH] cifs: clear server inode number flag while autodisabling

Steve French <[email protected]>
Newsgroups gmane.linux.file-systems.cifs
Message-ID <[email protected]>
Merged into cifs-2.6.git

Will request upstream later today.

On Mon, Nov 16, 2009 at 7:42 AM, Suresh Jayaraman <[email protected]>wrote:

> On 11/16/2009 07:10 PM, Jeff Layton wrote:
> > On Mon, 16 Nov 2009 07:39:43 -0500
> > Jeff Layton <[email protected]> wrote:
> >
> >> On Mon, 16 Nov 2009 12:03:16 +0530
> >> Suresh Jayaraman <[email protected]> wrote:
> >>
> >>> Fix the commit ec06aedd44 that intended to turn off querying for server
> inode
> >>> numbers when server doesn't consistently support inode numbers.
> Presumably
> >>> the commit didn't actually clear the CIFS_MOUNT_SERVER_INUM flag,
> perhaps a
> >>> typo.
> >>>
> >>> Signed-off-by: Suresh Jayaraman <[email protected]>
> >>> Cc: Jeff Layton <[email protected]>
> >>> ---
> >>>  fs/cifs/misc.c |    2 +-
> >>>  1 files changed, 1 insertions(+), 1 deletions(-)
> >>>
> >>> diff --git a/fs/cifs/misc.c b/fs/cifs/misc.c
> >>> index 1e25efc..d27d4ec 100644
> >>> --- a/fs/cifs/misc.c
> >>> +++ b/fs/cifs/misc.c
> >>> @@ -720,7 +720,7 @@ void
> >>>  cifs_autodisable_serverino(struct cifs_sb_info *cifs_sb)
> >>>  {
> >>>     if (cifs_sb->mnt_cifs_flags & CIFS_MOUNT_SERVER_INUM) {
> >>> -           cifs_sb->mnt_cifs_flags &= CIFS_MOUNT_SERVER_INUM;
> >>> +           cifs_sb->mnt_cifs_flags &= ~CIFS_MOUNT_SERVER_INUM;
> >>>             cERROR(1, ("Autodisabling the use of server inode numbers
> on "
> >>>                        "%s. This server doesn't seem to support them "
> >>>                        "properly. Hardlinks will not be recognized on
> this "
> >>
> >> Doh! Boneheaded error...
> >>
> >> Acked-by: Jeff Layton <[email protected]>
> >
> >
> > This patch probably needs to go to 2.6.32 and -stable.
> >
>
> Ah, yes, please include -Cc stable.
>
> Cc: Stable <[email protected]>
>
>
> Thanks,
>
> --
> Suresh Jayaraman
>



-- 
Thanks,

Steve

_______________________________________________
linux-cifs-client mailing list
[email protected]
https://lists.samba.org/mailman/listinfo/linux-cifs-client
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.