linux-fsdevel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Jeff Layton <jlayton@redhat.com>
To: linux-cifs-client@lists.samba.org, linux-fsdevel@vger.kernel.org
Subject: [PATCH 00/10] cifs: implement proper hardlink handling (try #5)
Date: Wed, 27 May 2009 16:33:57 -0400	[thread overview]
Message-ID: <1243456447-11176-1-git-send-email-jlayton@redhat.com> (raw)

This is the 5th attempt at a patchset to implement hardlink handling in
CIFS. The main differences from the last set are:

1) consolidated some of the patches to better help bisectability

2) redefined FILE_UNIX_INFO such that it has an embedded
FILE_UNIX_BASIC_INFO struct. This change makes it easier to use
cifs_unix_basic_to_fattr in the readdir codepath and eliminate
cifs_unix_info_to_fattr.

3) minor cleanups and fewer false deltas between patches

Aside from the unix readdir codepath changes listed above, the final
result from this set is pretty close to the result of the last set. This
one actually eliminates extra 100 lines or so too (mostly by changing
posix readdir to use the existing to_fattr routine instead of adding a
new one).

Christoph, since most of the changes here are stuff we've already
discussed, I went ahead and left your Reviewed-by: tag in place on the
ones that you've already reviewed. Let me know if you object to any of
the changes and we'll work it out.

Christoph Hellwig (2):
  cifs: reorganize get_cifs_acl
  cifs: clean up set_cifs_acl interfaces

Jeff Layton (8):
  cifs: have cifs_NTtimeToUnix take a little-endian arg
  cifs: make cnvrtDosUnixTm take a little-endian args and an offset
  cifs: rename cifs_iget to cifs_root_iget
  cifs: add new cifs_iget function and convert unix codepath to use it
  cifs: convert cifs_get_inode_info and non-posix readdir to use
    cifs_iget
  cifs: make serverino the default when mounting
  cifs: remove cifsInodeInfo->inUse counter
  cifs: remove "hardlink detection" from cifs_rename

 fs/cifs/cifsacl.c   |  200 +++++++------
 fs/cifs/cifsfs.c    |    3 +-
 fs/cifs/cifsfs.h    |   15 +-
 fs/cifs/cifsglob.h  |   28 ++-
 fs/cifs/cifspdu.h   |   14 +-
 fs/cifs/cifsproto.h |   21 +-
 fs/cifs/cifssmb.c   |    4 +-
 fs/cifs/connect.c   |    2 +
 fs/cifs/dir.c       |   22 +-
 fs/cifs/file.c      |    2 +-
 fs/cifs/inode.c     |  802 +++++++++++++++++++++------------------------------
 fs/cifs/netmisc.c   |   24 +-
 fs/cifs/readdir.c   |  517 ++++++++-------------------------
 13 files changed, 635 insertions(+), 1019 deletions(-)

             reply	other threads:[~2009-05-27 20:33 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-05-27 20:33 Jeff Layton [this message]
2009-05-27 20:33 ` [PATCH 01/10] cifs: have cifs_NTtimeToUnix take a little-endian arg Jeff Layton
2009-05-27 20:33 ` [PATCH 02/10] cifs: make cnvrtDosUnixTm take a little-endian args and an offset Jeff Layton
2009-05-27 20:34 ` [PATCH 03/10] cifs: reorganize get_cifs_acl Jeff Layton
2009-05-28 15:23   ` Shirish Pargaonkar
2009-05-27 20:34 ` [PATCH 04/10] cifs: clean up set_cifs_acl interfaces Jeff Layton
2009-05-28 15:23   ` [linux-cifs-client] " Shirish Pargaonkar
2009-05-27 20:34 ` [PATCH 05/10] cifs: rename cifs_iget to cifs_root_iget Jeff Layton
2009-05-27 20:34 ` [PATCH 06/10] cifs: add new cifs_iget function and convert unix codepath to use it Jeff Layton
2009-05-27 20:34 ` [PATCH 07/10] cifs: convert cifs_get_inode_info and non-posix readdir to use cifs_iget Jeff Layton
2009-05-27 20:34 ` [PATCH 08/10] cifs: make serverino the default when mounting Jeff Layton
2009-05-27 20:34 ` [PATCH 09/10] cifs: remove cifsInodeInfo->inUse counter Jeff Layton
2009-05-27 20:34 ` [PATCH 10/10] cifs: remove "hardlink detection" from cifs_rename Jeff Layton

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=1243456447-11176-1-git-send-email-jlayton@redhat.com \
    --to=jlayton@redhat.com \
    --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).