lkml.org 
[lkml]   [2012]   [Mar]   [31]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [git pull] vfs part 2
    On Sat, Mar 31, 2012 at 12:48:22PM -0700, Linus Torvalds wrote:
    > On Sat, Mar 31, 2012 at 12:39 PM, Al Viro <viro@zeniv.linux.org.uk> wrote:
    > >
    > > Variants:
    > > ? ? ? ?1) drop that commit, fold obvious fix into it, leave it for for-next
    > > once -rc1 is there; ask you to pull the branch with that sucker excluded
    > > ? ? ? ?2) fold the fix into commit, ask you to pull the resulting branch
    > > ? ? ? ?3) add fix to the end of existing branch, ask to pull
    >
    > Let's go for (2). I'll take a new look after the pull request, but no
    > guarantees that I'll then pull it.
    >
    > Btw, I liked the do_lookup cleanups, but nicer commit logs would have
    > been good. Especially the first lines. The whole
    >
    > untangling do_lookup(), part 1
    > untangling do_lookup(), part 2
    > untangling do_lookup(), part 3
    > untangling do_lookup(), part 4
    > untangling do_lookup(), part 5
    > untangling do_lookup(), part 6
    > untangling do_lookup(), part 7
    > untangling do_lookup(), part 8
    > untangling do_lookup(), part 9
    >
    > in shortlogs (and gitk!) does turn me off. So since you have to redo
    > the series anyway, can I ask you to perhaps try to make it a bit more
    > descriptive?

    OK... Keep in mind that this is a splitup of what used to be a monolithic
    patch from Miklos, so this series is basically "how do I convince myself
    that his patch is correct (or find a hole in it)". No holes found and
    I like the resulting cleanup, so in it went...

    Anyway, commit messages prettied up a bit, fix folded and the whole thing
    is pushed to the same place - i.e.

    git://git.kernel.org/pub/scm/linux/kernel/git/viro/vfs.git for-linus

    Shortlog:
    Al Viro (18):
    selinuxfs: merge dentry allocation into sel_make_dir()
    aio: merge aio_cancel_all() with wait_for_all_aios()
    aio: take final put_ioctx() into callers of io_destroy()
    pstore: trim pstore_get_inode()
    mtdchar: kill persistently held vfsmount
    get rid of pointless includes of ext2_fs.h
    new helper: ext2_image_size()
    migrate ext2_fs.h guts to fs/ext2/ext2.h
    ext3: move headers to fs/ext3/
    untangling do_lookup() - isolate !dentry stuff from the rest of it.
    untangling do_lookup() - expand the area under ->i_mutex
    untangling do_lookup() - eliminate a loop.
    untangling do_lookup() - get rid of need_reval in !dentry case
    untangling do_lookup() - massage !dentry case towards __lookup_hash()
    untangling do_lookup() - merge failure exits in !dentry case
    untangling do_lookup() - merge d_alloc_and_lookup() callers
    untangling do_lookup() - switch to calling __lookup_hash()
    untangling do_lookup() - take __lookup_hash()-calling case out of line.

    J. Bruce Fields (1):
    vfs: fix out-of-date dentry_unhash() comment

    Miklos Szeredi (4):
    vfs: fix d_need_lookup/d_revalidate order in do_lookup
    vfs: don't revalidate just looked up dentry
    vfs: move MAY_EXEC check from __lookup_hash()
    vfs: split __lookup_hash

    Thierry Reding (1):
    ext2: No longer export ext2_fs.h to user space

    Diffstat:
    arch/blackfin/kernel/setup.c | 7 +-
    drivers/mtd/mtdchar.c | 53 +--
    fs/aio.c | 32 +-
    fs/ext2/ext2.h | 631 +++++++++++++++++++++++++++++
    fs/ext2/xattr_security.c | 5 +-
    fs/ext2/xattr_trusted.c | 5 +-
    fs/ext2/xip.c | 2 -
    fs/ext3/acl.c | 8 +-
    fs/ext3/balloc.c | 10 +-
    fs/ext3/bitmap.c | 4 +-
    fs/ext3/dir.c | 7 +-
    include/linux/ext3_fs.h => fs/ext3/ext3.h | 488 +++++++++++++++++++----
    fs/ext3/ext3_jbd.c | 2 +-
    fs/ext3/file.c | 6 +-
    fs/ext3/fsync.c | 8 +-
    fs/ext3/hash.c | 4 +-
    fs/ext3/ialloc.c | 13 +-
    fs/ext3/inode.c | 12 +-
    fs/ext3/ioctl.c | 7 +-
    fs/ext3/namei.c | 14 +-
    fs/ext3/resize.c | 5 +-
    fs/ext3/super.c | 18 +-
    fs/ext3/symlink.c | 4 +-
    fs/ext3/xattr.c | 7 +-
    fs/ext3/xattr_security.c | 6 +-
    fs/ext3/xattr_trusted.c | 6 +-
    fs/ext3/xattr_user.c | 5 +-
    fs/gfs2/file.c | 1 -
    fs/namei.c | 195 ++++------
    fs/ocfs2/ioctl.c | 2 -
    fs/pstore/inode.c | 26 +-
    include/linux/Kbuild | 1 -
    include/linux/ext2_fs.h | 569 +-------------------------
    include/linux/ext2_fs_sb.h | 126 ------
    include/linux/ext3_fs_i.h | 151 -------
    include/linux/ext3_fs_sb.h | 91 -----
    include/linux/ext3_jbd.h | 229 -----------
    init/do_mounts_initrd.c | 1 -
    init/do_mounts_rd.c | 9 +-
    security/selinux/hooks.c | 9 +-
    security/selinux/selinuxfs.c | 110 ++---
    41 files changed, 1249 insertions(+), 1640 deletions(-)
    rename include/linux/ext3_fs.h => fs/ext3/ext3.h (67%)
    delete mode 100644 include/linux/ext2_fs_sb.h
    delete mode 100644 include/linux/ext3_fs_i.h
    delete mode 100644 include/linux/ext3_fs_sb.h
    delete mode 100644 include/linux/ext3_jbd.h


    \
     
     \ /
      Last update: 2012-03-31 22:11    [W:0.028 / U:209.860 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site