All of lore.kernel.org
 help / color / mirror / Atom feed
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

  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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.