From: Jamie Lokier <jamie@shareable.org>
To: Jeff Layton <jlayton@samba.org>
Cc: linux-cifs-client@lists.samba.org, linux-fsdevel@vger.kernel.org
Subject: Re: [PATCH 00/15] cifs: implement multisession mounts (RFC)
Date: Wed, 31 Mar 2010 13:18:54 +0100 [thread overview]
Message-ID: <20100331121854.GA1456@shareable.org> (raw)
In-Reply-To: <1269978677-6817-1-git-send-email-jlayton@samba.org>
Jeff Layton wrote:
> To be honest, I have doubts anyway about whether we should take this
> patch into what seems to be considered a legacy codebase these days.
> It's rather difficult to bolt this functionality onto the existing CIFS
> codebase. It's probably more suited to putting into the newer SMB2
> codebase. That said, it'll probably be just as difficult to do this
> there since most of that code was copied from CIFS.
Legacy?
Maybe only in environments where everyone is running only Windows
servers, all of them less than 2 years old :-) (I've yet to see such
an environment, btw).
The stable version of Samba (3.x) supports only SMB1.
Samba 4, which does support SMB2, is "not yet in a state where it can
replace existing production deployments"; the Ubuntu package
description says "experimental, should not be used in production".
In fact when I wanted to deploy an SMB2 service from Linux recently
(to get better file link semantics on a Windows client), I gave up on
it, it was too disruptive to replace Samba 3 with Samba 4.
This per-user patchset you've produced sounds quite useful, thank you.
Please don't think you are targetting only a few horribly outdated
environments with it :-)
-- Jamie
next prev parent reply other threads:[~2010-03-31 12:18 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-03-30 19:51 [PATCH 00/15] cifs: implement multisession mounts (RFC) Jeff Layton
2010-03-30 19:51 ` [PATCH 01/15] cifs: remove unused list_head from cifs_sb_info Jeff Layton
2010-03-30 19:51 ` [PATCH 02/15] cifs: add function to get a tcon from cifs_sb Jeff Layton
2010-03-30 19:51 ` [PATCH 03/15] cifs: track local_nls in volume info Jeff Layton
2010-03-30 19:51 ` [PATCH 04/15] cifs: move SMB session creation code into separate function Jeff Layton
2010-03-30 19:51 ` [PATCH 05/15] cifs: move tcon find/create " Jeff Layton
2010-03-30 19:51 ` [PATCH 06/15] cifs: add tcon field to cifsFileInfo struct Jeff Layton
2010-03-30 19:51 ` [PATCH 07/15] cifs: make various routines use the cifsFileInfo->tcon pointer Jeff Layton
2010-03-30 19:51 ` [PATCH 08/15] cifs: have find_readable/writable_file filter by fsuid Jeff Layton
2010-03-30 19:51 ` [PATCH 09/15] cifs: fix cifs_show_options to show "username=" or "multises" Jeff Layton
2010-03-30 19:51 ` [PATCH 10/15] cifs: move secType to the SMB session Jeff Layton
2010-03-30 19:51 ` [PATCH 11/15] cifs: have cifs_new_fileinfo take a tcon arg Jeff Layton
2010-03-30 19:51 ` [PATCH 12/15] cifs: build sessions and tcons on the fly Jeff Layton
2010-03-30 19:51 ` [PATCH 13/15] cifs: on multises mount, set ownership to current_fsuid/current_fsgid Jeff Layton
2010-03-30 19:51 ` [PATCH 14/15] cifs: add "multises" mount option Jeff Layton
2010-03-30 19:51 ` [PATCH 15/15] cifs: temporary hack -- set secType to Kerberos Jeff Layton
2010-03-31 12:18 ` Jamie Lokier [this message]
2010-03-31 13:42 ` [PATCH 00/15] cifs: implement multisession mounts (RFC) Jeff Layton
2010-03-31 16:43 ` Jeremy Allison
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=20100331121854.GA1456@shareable.org \
--to=jamie@shareable.org \
--cc=jlayton@samba.org \
--cc=linux-cifs-client@lists.samba.org \
--cc=linux-fsdevel@vger.kernel.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).