linux-fsdevel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Jeff Layton <jlayton-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
To: smfrench-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org
Cc: linux-cifs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-fsdevel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: [PATCH 00/15] cifs: multiuser mount overhaul (try #4)
Date: Mon, 20 Sep 2010 16:01:29 -0700	[thread overview]
Message-ID: <1285023704-2159-1-git-send-email-jlayton@redhat.com> (raw)

This is the fourth attempt at a patchset to overhaul the multiuser CIFS
mount capability in the kernel. The main difference from the last is the
addition of a recurring workqueue job to close out idle tcons (Tree
Connects). The patchset has also been rebased on top of a very recent
pull of Steve's tree including the patches from Metze to implement
mfsymlinks.

Many people may be aware that CIFS has a MultiuserMount switch in
/proc/fs/cifs. This code basically works, but it has a number of
shortcomings that make it impractical to deploy. This patchset is
a redesign of that concept from scratch that allows CIFS to use multiple sessions
to the same server, establish them on an as-needed basis and tear them
down when they've been idle for a period of time.

Because we cannot prompt for passwords from the kernel, this code is
currently limited to using krb5 authentication. Igor Druzhinin has done
some work as a Google Summer of Code project to allow the stashing of
username/password in the kernel keyring. Once that code is ready, we'll
be able to extend this to other sec= types as well.

The patchset is large, but I've done my best to split it up into a
logical set of bisectable changes. Some of the patches touch the same
areas of code as earlier ones. This increases the amount of "churn" but
I think it makes a more logical sequence of changes.

At this point, I think this code is ready for merge. I'd like to see it
in 2.6.37. Comments and review are welcome.

Jeff Layton (15):
  cifs: add tcon field to cifsFileInfo struct
  cifs: make various routines use the cifsFileInfo->tcon pointer
  cifs: fix handling of signing with writepages
  cifs: add function to get a tcon from cifs_sb
  cifs: temporarily rename cifs_sb->tcon to ptcon to catch stragglers
  cifs: add cifs_sb_master_tcon and convert some callers to use it
  cifs: have cifs_new_fileinfo take a tcon arg
  cifs: add refcounted and timestamped container for holding tcons
  cifs: have cifsFileInfo hold a reference to a tlink rather than tcon
    pointer
  cifs: have find_readable/writable_file filter by fsuid
  cifs: fix cifs_show_options to show "username=" or "multiuser"
  cifs: add routines to build sessions and tcons on the fly
  cifs: on multiuser mount, set ownership to
    current_fsuid/current_fsgid
  cifs: add "multiuser" mount option
  cifs: implement recurring workqueue job to prune old tcons

 fs/cifs/cifs_dfs_ref.c |   21 ++--
 fs/cifs/cifs_fs_sb.h   |   11 +-
 fs/cifs/cifsacl.c      |   46 +++++--
 fs/cifs/cifsfs.c       |   22 ++--
 fs/cifs/cifsglob.h     |   40 ++++++
 fs/cifs/cifsproto.h    |    8 +-
 fs/cifs/connect.c      |  339 +++++++++++++++++++++++++++++++++++++++++++++++-
 fs/cifs/dir.c          |   97 ++++++++++-----
 fs/cifs/file.c         |  161 +++++++++++++-----------
 fs/cifs/fscache.c      |   13 +-
 fs/cifs/inode.c        |  213 +++++++++++++++++++++++--------
 fs/cifs/ioctl.c        |   17 +--
 fs/cifs/link.c         |   88 ++++++++-----
 fs/cifs/misc.c         |    2 +-
 fs/cifs/readdir.c      |   45 ++++---
 fs/cifs/xattr.c        |   60 ++++++---
 16 files changed, 888 insertions(+), 295 deletions(-)

-- 
1.7.2.3

             reply	other threads:[~2010-09-20 23:01 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-09-20 23:01 Jeff Layton [this message]
     [not found] ` <1285023704-2159-1-git-send-email-jlayton-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2010-09-20 23:01   ` [PATCH 01/15] cifs: add tcon field to cifsFileInfo struct Jeff Layton
2010-09-20 23:01   ` [PATCH 02/15] cifs: make various routines use the cifsFileInfo->tcon pointer Jeff Layton
2010-09-20 23:01   ` [PATCH 04/15] cifs: add function to get a tcon from cifs_sb Jeff Layton
2010-09-20 23:01   ` [PATCH 05/15] cifs: temporarily rename cifs_sb->tcon to ptcon to catch stragglers Jeff Layton
2010-09-20 23:01   ` [PATCH 07/15] cifs: have cifs_new_fileinfo take a tcon arg Jeff Layton
2010-09-20 23:01   ` [PATCH 08/15] cifs: add refcounted and timestamped container for holding tcons Jeff Layton
2010-09-20 23:01   ` [PATCH 09/15] cifs: have cifsFileInfo hold a reference to a tlink rather than tcon pointer Jeff Layton
     [not found]     ` <1285023704-2159-10-git-send-email-jlayton-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2010-09-23  6:08       ` Steve French
     [not found]         ` <AANLkTi=n=_R_WF8NizLo+zfcBEmzcQpBWWj=_GNEmua1-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2010-09-23 12:36           ` Jeff Layton
     [not found]             ` <20100923053623.09bd6f33-4QP7MXygkU+dMjc06nkz3ljfA9RmPOcC@public.gmane.org>
2010-09-23 12:43               ` Jeff Layton
2010-09-20 23:01   ` [PATCH 10/15] cifs: have find_readable/writable_file filter by fsuid Jeff Layton
2010-09-23  6:11     ` Steve French
2010-09-20 23:01   ` [PATCH 12/15] cifs: add routines to build sessions and tcons on the fly Jeff Layton
2010-09-20 23:01   ` [PATCH 13/15] cifs: on multiuser mount, set ownership to current_fsuid/current_fsgid Jeff Layton
     [not found]     ` <1285023704-2159-14-git-send-email-jlayton-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2010-09-23  6:19       ` Steve French
2010-09-23 12:38         ` Jeff Layton
2010-09-20 23:01   ` [PATCH 14/15] cifs: add "multiuser" mount option Jeff Layton
2010-09-20 23:01   ` [PATCH 15/15] cifs: implement recurring workqueue job to prune old tcons Jeff Layton
2010-09-20 23:01 ` [PATCH 03/15] cifs: fix handling of signing with writepages Jeff Layton
2010-09-20 23:01 ` [PATCH 06/15] cifs: add cifs_sb_master_tcon and convert some callers to use it Jeff Layton
2010-09-20 23:01 ` [PATCH 11/15] cifs: fix cifs_show_options to show "username=" or "multiuser" Jeff Layton
2010-09-23  6:12   ` Steve French
2010-10-05 22:00   ` 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=1285023704-2159-1-git-send-email-jlayton@redhat.com \
    --to=jlayton-h+wxahxf7alqt0dzr+alfa@public.gmane.org \
    --cc=linux-cifs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-fsdevel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=smfrench-Re5JQEeQqe8AvxtiuMwx3w@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).