linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Miklos Szeredi <mszeredi@redhat.com>
To: linux-fsdevel@vger.kernel.org
Cc: linux-kernel@vger.kernel.org, Al Viro <viro@zeniv.linux.org.uk>,
	Eric Van Hensbergen <ericvh@gmail.com>,
	David Howells <dhowells@redhat.com>,
	Ilya Dryomov <idryomov@gmail.com>,
	Jan Harkes <jaharkes@cs.cmu.edu>,
	Tyler Hicks <tyhicks@canonical.com>,
	Boaz Harrosh <ooo@electrozaur.com>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	Oleg Drokin <oleg.drokin@intel.com>,
	Trond Myklebust <trond.myklebust@primarydata.com>,
	Mark Fasheh <mfasheh@suse.com>,
	Mike Marshall <hubcap@omnibond.com>,
	Alexei Starovoitov <ast@kernel.org>, Jan Kara <jack@suse.cz>,
	"Theodore Ts'o" <tytso@mit.edu>, Jaegeuk Kim <jaegeuk@kernel.org>,
	OGAWA Hirofumi <hirofumi@mail.parknet.co.jp>,
	Mikulas Patocka <mpatocka@redhat.com>,
	David Woodhouse <dwmw2@infradead.org>,
	Dave Kleikamp <shaggy@kernel.org>,
	Ryusuke Konishi <konishi.ryusuke@lab.ntt.co.jp>,
	Bob Copeland <me@bobcopeland.com>,
	Christoph Hellwig <hch@infradead.org>,
	Richard Weinberger <richard@nod.at>
Subject: [PATCH 0/7] vfs: finish rename -> rename2 conversion
Date: Tue, 23 Aug 2016 16:05:25 +0200	[thread overview]
Message-ID: <1471961132-1675-1-git-send-email-mszeredi@redhat.com> (raw)

This series gets rid of the duplicate interfaces for rename.  The only
difference between the two that the new, i_op->rename2() has an added flags
argument.  This needs to be handled in each instance.

For 21 local filesystems the RENAME_NOREPLACE flag can be supported "for
free", just by moving to the new interface and allowing this flag.  The
remaining 12 distributed filesystems are converted too, which only allow
zero flags.

Git tree is here.

  git://git.kernel.org/pub/scm/linux/kernel/git/mszeredi/vfs.git#rename2

Thanks,
Miklos
---

Miklos Szeredi (7):
  ncpfs: fix unused variable warning
  fs: support RENAME_NOREPLACE for local filesystems
  libfs: support RENAME_NOREPLACE in simple_rename()
  fs: make remaining filesystems use .rename2
  vfs: remove unused i_op->rename
  fs: rename "rename2" i_op to "rename"
  vfs: add note about i_op->rename changes to porting

 Documentation/filesystems/Locking           |  8 ++------
 Documentation/filesystems/directory-locking |  2 +-
 Documentation/filesystems/porting           |  4 ++++
 Documentation/filesystems/vfs.txt           |  9 ++-------
 drivers/staging/lustre/lustre/llite/namei.c |  8 ++++++--
 fs/9p/v9fs.h                                |  3 ++-
 fs/9p/vfs_inode.c                           |  6 +++++-
 fs/affs/affs.h                              |  3 ++-
 fs/affs/namei.c                             |  6 +++++-
 fs/afs/dir.c                                |  9 +++++++--
 fs/bad_inode.c                              |  2 +-
 fs/bfs/dir.c                                |  6 +++++-
 fs/btrfs/inode.c                            |  2 +-
 fs/cachefiles/namei.c                       |  3 +--
 fs/ceph/dir.c                               |  6 +++++-
 fs/cifs/cifsfs.c                            |  2 +-
 fs/coda/dir.c                               |  6 +++++-
 fs/debugfs/inode.c                          |  2 +-
 fs/ecryptfs/inode.c                         |  6 +++++-
 fs/exofs/namei.c                            |  8 ++++++--
 fs/ext2/namei.c                             |  6 +++++-
 fs/ext4/namei.c                             |  2 +-
 fs/f2fs/namei.c                             |  2 +-
 fs/fat/namei_msdos.c                        |  6 +++++-
 fs/fat/namei_vfat.c                         |  6 +++++-
 fs/fuse/dir.c                               |  2 +-
 fs/gfs2/inode.c                             |  2 +-
 fs/hfs/dir.c                                |  6 +++++-
 fs/hfsplus/dir.c                            |  6 +++++-
 fs/hostfs/hostfs_kern.c                     |  2 +-
 fs/hpfs/namei.c                             |  6 +++++-
 fs/jffs2/dir.c                              |  9 +++++++--
 fs/jfs/namei.c                              |  5 ++++-
 fs/kernfs/dir.c                             |  6 +++++-
 fs/libfs.c                                  |  6 +++++-
 fs/logfs/dir.c                              |  6 +++++-
 fs/minix/namei.c                            |  6 +++++-
 fs/namei.c                                  | 15 +++------------
 fs/ncpfs/dir.c                              | 11 +++++++----
 fs/nfs/dir.c                                |  6 +++++-
 fs/nfs/internal.h                           |  3 ++-
 fs/nilfs2/namei.c                           |  6 +++++-
 fs/ocfs2/namei.c                            |  6 +++++-
 fs/omfs/dir.c                               |  6 +++++-
 fs/orangefs/namei.c                         |  6 +++++-
 fs/overlayfs/dir.c                          |  2 +-
 fs/overlayfs/overlayfs.h                    |  4 ++--
 fs/reiserfs/namei.c                         |  6 +++++-
 fs/sysv/namei.c                             |  6 +++++-
 fs/ubifs/dir.c                              |  8 ++++++--
 fs/udf/namei.c                              |  6 +++++-
 fs/ufs/namei.c                              |  6 +++++-
 fs/xfs/xfs_iops.c                           |  4 ++--
 include/linux/fs.h                          |  5 ++---
 mm/shmem.c                                  |  2 +-
 security/tomoyo/realpath.c                  |  4 ++--
 56 files changed, 206 insertions(+), 91 deletions(-)

-- 
2.5.5

             reply	other threads:[~2016-08-23 14:11 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-08-23 14:05 Miklos Szeredi [this message]
2016-08-23 14:05 ` [PATCH 1/7] ncpfs: fix unused variable warning Miklos Szeredi
2016-08-23 14:05 ` [PATCH 2/7] fs: support RENAME_NOREPLACE for local filesystems Miklos Szeredi
2016-08-23 21:48   ` Richard Weinberger
2016-08-25 12:52   ` Bob Copeland
2016-09-08 13:55   ` Jan Kara
2016-08-23 14:05 ` [PATCH 3/7] libfs: support RENAME_NOREPLACE in simple_rename() Miklos Szeredi
2016-08-23 14:47   ` Greg Kroah-Hartman
2016-08-23 14:05 ` [PATCH 4/7] fs: make remaining filesystems use .rename2 Miklos Szeredi
2016-08-23 14:47   ` Greg Kroah-Hartman
2016-08-23 17:30     ` Mike Marshall
2016-08-23 16:24   ` Boaz Harrosh
2016-08-23 16:29     ` Boaz Harrosh
2016-08-23 14:05 ` [PATCH 5/7] vfs: remove unused i_op->rename Miklos Szeredi
2016-08-23 14:05 ` [PATCH 6/7] fs: rename "rename2" i_op to "rename" Miklos Szeredi
2016-08-23 14:05 ` [PATCH 7/7] vfs: add note about i_op->rename changes to porting Miklos Szeredi
2016-08-23 15:36 ` [PATCH 4/7] fs: make remaining filesystems use .rename2 David Howells

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=1471961132-1675-1-git-send-email-mszeredi@redhat.com \
    --to=mszeredi@redhat.com \
    --cc=ast@kernel.org \
    --cc=dhowells@redhat.com \
    --cc=dwmw2@infradead.org \
    --cc=ericvh@gmail.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=hch@infradead.org \
    --cc=hirofumi@mail.parknet.co.jp \
    --cc=hubcap@omnibond.com \
    --cc=idryomov@gmail.com \
    --cc=jack@suse.cz \
    --cc=jaegeuk@kernel.org \
    --cc=jaharkes@cs.cmu.edu \
    --cc=konishi.ryusuke@lab.ntt.co.jp \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=me@bobcopeland.com \
    --cc=mfasheh@suse.com \
    --cc=mpatocka@redhat.com \
    --cc=oleg.drokin@intel.com \
    --cc=ooo@electrozaur.com \
    --cc=richard@nod.at \
    --cc=shaggy@kernel.org \
    --cc=trond.myklebust@primarydata.com \
    --cc=tyhicks@canonical.com \
    --cc=tytso@mit.edu \
    --cc=viro@zeniv.linux.org.uk \
    /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).