[PATCH 2/5] Unionfs: Add missing unlock call in unionfs_file_release

Signed-off-by: Josef 'Jeff' Sipek [EMAIL PROTECTED] --- fs/unionfs/commonfops.c |6 -- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/fs/unionfs/commonfops.c b/fs/unionfs/commonfops.c index 6d87426..8527ac6 100644 --- a/fs/unionfs/commonfops.c +++ b/fs/unionfs/commonfops.c

[PATCH 09/16] Unionfs: mmap implementation

From: Yiannis Pericleous <[EMAIL PROTECTED]> Signed-off-by: Shaya Potter <[EMAIL PROTECTED]> Signed-off-by: Erez Zadok <[EMAIL PROTECTED]> Signed-off-by: Yiannis Pericleous <[EMAIL PROTECTED]> Signed-off-by: Josef 'Jeff' Sipek <[EMAIL PROTECTED]> --- fs/unionfs/Make

[PATCH 08/16] Unionfs: Remove unnecessary #define

From: Erez Zadok <[EMAIL PROTECTED]> UNIONFS_TMPNAM_LEN is used in only one place, and we have calculate the length of the string to begin with. Signed-off-by: Erez Zadok <[EMAIL PROTECTED]> Signed-off-by: Josef 'Jeff' Sipek <[EMAIL PROTECTED]> --- fs/unionfs/commonfops.c |

[PATCH 05/16] Unionfs: call statfs on lower file system properly

From: Erez Zadok <[EMAIL PROTECTED]> Get the correct lower dentry to use to statfs the first branch (always), Signed-off-by: Erez Zadok <[EMAIL PROTECTED]> Signed-off-by: Josef 'Jeff' Sipek <[EMAIL PROTECTED]> --- fs/unionfs/super.c | 15 ++- 1 files changed, 1

[PATCH 01/16] [PATCH] unionfs section mismatch

From: Randy Dunlap <[EMAIL PROTECTED]> Fix section marker in header file: WARNING: fs/unionfs/unionfs.o(.init.text+0x56): Section mismatch: reference to .exit.text:stop_sioq (between 'init_module' and 'init_sioq') Signed-off-by: Randy Dunlap <[EMAIL PROTECTED]> Signed-off-by:

[PATCH 07/16] Unionfs: Add standard copyright comment to include/linux/union_fs.h

From: Erez Zadok <[EMAIL PROTECTED]> Signed-off-by: Erez Zadok <[EMAIL PROTECTED]> Signed-off-by: Josef 'Jeff' Sipek <[EMAIL PROTECTED]> --- include/linux/union_fs.h | 11 +++ 1 files changed, 11 insertions(+), 0 deletions(-) diff --git a/include/linux/union_fs

[PATCH 03/16] Unionfs: Retry lookup for different silly-renamed files

enamed file name already existed, Unionfs would oops upon copyup to that temp name. Signed-off-by: Erez Zadok <[EMAIL PROTECTED]> Signed-off-by: Josef 'Jeff' Sipek <[EMAIL PROTECTED]> --- fs/unionfs/commonfops.c |6 -- 1 files changed, 4 insertions(+), 2 deletions(-) diff

[PATCH 10/16] Unionfs: merge find_new_branch_index and branch_id_to_idx into one function

From: Erez Zadok <[EMAIL PROTECTED]> Useful code cleanup and consolidation between the ODF code and non-ODF code. Signed-off-by: Erez Zadok <[EMAIL PROTECTED]> Signed-off-by: Josef 'Jeff' Sipek <[EMAIL PROTECTED]> --- fs/unionfs/commonfops.c | 35 +

[PATCH 06/16] MAINTAINERS: Add Erez Zadok as a maintainer of Unionfs

From: Erez Zadok <[EMAIL PROTECTED]> Signed-off-by: Erez Zadok <[EMAIL PROTECTED]> Signed-off-by: Josef 'Jeff' Sipek <[EMAIL PROTECTED]> --- MAINTAINERS |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index 905da

[PATCH 04/16] Unionfs: Set lower inodes correctly after branch management succeeds

From: Erez Zadok <[EMAIL PROTECTED]> Signed-off-by: Erez Zadok <[EMAIL PROTECTED]> Signed-off-by: Josef 'Jeff' Sipek <[EMAIL PROTECTED]> --- fs/unionfs/super.c | 50 +- 1 files changed, 45 insertions(+), 5 deletions(-) diff

[PATCH 02/16] Unionfs: Don't revalidate dropped dentries

From: Erez Zadok <[EMAIL PROTECTED]> This fixes a harmless but annoying message that unionfs prints if a dropped dentry is being revalidated, which could happen if you unlink open files. Signed-off-by: Erez Zadok <[EMAIL PROTECTED]> Signed-off-by: Josef 'Jeff' Sipek <[EMAIL PROT

[GIT PULL -mm] Unionfs cleanups, fixes, and mmap

Unionfs: Revalidate dentries passed to all inode/super operations Josef 'Jeff' Sipek (5): Unionfs: Cleanup new_dentry_private_data Unionfs: Change free_dentry_private_info to take a struct dentry Unionfs: Add BUG_ONs to unionfs_lower_* Unionfs: Change the semantics of sb

[PATCH 11/16] Unionfs: Revalidate dentries passed to all inode/super operations

ff-by: Josef 'Jeff' Sipek <[EMAIL PROTECTED]> --- fs/unionfs/inode.c | 71 --- fs/unionfs/rename.c | 13 +++-- fs/unionfs/super.c |9 ++- fs/unionfs/union.h | 13 - fs/unionfs/unlink.c | 15 --- fs/union

[PATCH 15/16] Unionfs: Change the semantics of sb info's rwsem

}_{,un}lock() to something more descriptive. Signed-off-by: Josef 'Jeff' Sipek <[EMAIL PROTECTED]> --- fs/unionfs/commonfops.c | 33 --- fs/unionfs/copyup.c | 10 --- fs/unionfs/dentry.c |7 + fs/unionfs/dirfops.c| 10 --- fs/u

[PATCH 14/16] Unionfs: Add BUG_ONs to unionfs_lower_*

Signed-off-by: Josef 'Jeff' Sipek <[EMAIL PROTECTED]> --- fs/unionfs/fanout.h | 35 +++ 1 files changed, 35 insertions(+), 0 deletions(-) diff --git a/fs/unionfs/fanout.h b/fs/unionfs/fanout.h index 0319835..d4933ce 100644 --- a/fs/unionfs/fanout.h ++

[PATCH 16/16] Unionfs: Remove superfluous check for NULL pointer

Since we use containers and the struct inode is _inside_ the unionfs_inode_info structure, UNIONFS_I will always (given a non-NULL inode pointer), return a valid non-NULL pointer. Signed-off-by: Josef 'Jeff' Sipek <[EMAIL PROTECTED]> --- fs/unionfs/fanout.h |8 +++- fs/unionfs/s

[PATCH 12/16] Unionfs: Cleanup new_dentry_private_data

Signed-off-by: Josef 'Jeff' Sipek <[EMAIL PROTECTED]> --- fs/unionfs/lookup.c | 96 +++--- fs/unionfs/union.h |1 + 2 files changed, 60 insertions(+), 37 deletions(-) diff --git a/fs/unionfs/lookup.c b/fs/unionfs/lookup.c index 758c813..2

[PATCH 13/16] Unionfs: Change free_dentry_private_info to take a struct dentry

This makes it more symmetric with new_dentry_private_info. Signed-off-by: Josef 'Jeff' Sipek <[EMAIL PROTECTED]> --- fs/unionfs/dentry.c |3 +-- fs/unionfs/lookup.c | 13 ++--- fs/unionfs/main.c |2 +- fs/unionfs/union.h |2 +- 4 files changed, 9 insertions(

[PATCH 13/16] Unionfs: Change free_dentry_private_info to take a struct dentry

This makes it more symmetric with new_dentry_private_info. Signed-off-by: Josef 'Jeff' Sipek [EMAIL PROTECTED] --- fs/unionfs/dentry.c |3 +-- fs/unionfs/lookup.c | 13 ++--- fs/unionfs/main.c |2 +- fs/unionfs/union.h |2 +- 4 files changed, 9 insertions(+), 11

[PATCH 12/16] Unionfs: Cleanup new_dentry_private_data

Signed-off-by: Josef 'Jeff' Sipek [EMAIL PROTECTED] --- fs/unionfs/lookup.c | 96 +++--- fs/unionfs/union.h |1 + 2 files changed, 60 insertions(+), 37 deletions(-) diff --git a/fs/unionfs/lookup.c b/fs/unionfs/lookup.c index 758c813..246a67a

[PATCH 16/16] Unionfs: Remove superfluous check for NULL pointer

Since we use containers and the struct inode is _inside_ the unionfs_inode_info structure, UNIONFS_I will always (given a non-NULL inode pointer), return a valid non-NULL pointer. Signed-off-by: Josef 'Jeff' Sipek [EMAIL PROTECTED] --- fs/unionfs/fanout.h |8 +++- fs/unionfs/super.c

[PATCH 14/16] Unionfs: Add BUG_ONs to unionfs_lower_*

Signed-off-by: Josef 'Jeff' Sipek [EMAIL PROTECTED] --- fs/unionfs/fanout.h | 35 +++ 1 files changed, 35 insertions(+), 0 deletions(-) diff --git a/fs/unionfs/fanout.h b/fs/unionfs/fanout.h index 0319835..d4933ce 100644 --- a/fs/unionfs/fanout.h +++ b/fs

[PATCH 11/16] Unionfs: Revalidate dentries passed to all inode/super operations

From: Erez Zadok [EMAIL PROTECTED] Be sure to properly revalidate all dentry chains passed to all inode and super_block operations. Remove the older BUG_ON test is_valid_dentry(). This should help improve cache-coherency. Signed-off-by: Erez Zadok [EMAIL PROTECTED] Signed-off-by: Josef 'Jeff

[PATCH 15/16] Unionfs: Change the semantics of sb info's rwsem

}_{,un}lock() to something more descriptive. Signed-off-by: Josef 'Jeff' Sipek [EMAIL PROTECTED] --- fs/unionfs/commonfops.c | 33 --- fs/unionfs/copyup.c | 10 --- fs/unionfs/dentry.c |7 + fs/unionfs/dirfops.c| 10 --- fs/unionfs/dirhelper.c

[PATCH 02/16] Unionfs: Don't revalidate dropped dentries

From: Erez Zadok [EMAIL PROTECTED] This fixes a harmless but annoying message that unionfs prints if a dropped dentry is being revalidated, which could happen if you unlink open files. Signed-off-by: Erez Zadok [EMAIL PROTECTED] Signed-off-by: Josef 'Jeff' Sipek [EMAIL PROTECTED] --- fs/unionfs

[GIT PULL -mm] Unionfs cleanups, fixes, and mmap

Unionfs: Revalidate dentries passed to all inode/super operations Josef 'Jeff' Sipek (5): Unionfs: Cleanup new_dentry_private_data Unionfs: Change free_dentry_private_info to take a struct dentry Unionfs: Add BUG_ONs to unionfs_lower_* Unionfs: Change the semantics of sb

[PATCH 06/16] MAINTAINERS: Add Erez Zadok as a maintainer of Unionfs

From: Erez Zadok [EMAIL PROTECTED] Signed-off-by: Erez Zadok [EMAIL PROTECTED] Signed-off-by: Josef 'Jeff' Sipek [EMAIL PROTECTED] --- MAINTAINERS |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index 905dab1..e09ad36 100644 --- a/MAINTAINERS

[PATCH 04/16] Unionfs: Set lower inodes correctly after branch management succeeds

From: Erez Zadok [EMAIL PROTECTED] Signed-off-by: Erez Zadok [EMAIL PROTECTED] Signed-off-by: Josef 'Jeff' Sipek [EMAIL PROTECTED] --- fs/unionfs/super.c | 50 +- 1 files changed, 45 insertions(+), 5 deletions(-) diff --git a/fs/unionfs/super.c

[PATCH 07/16] Unionfs: Add standard copyright comment to include/linux/union_fs.h

From: Erez Zadok [EMAIL PROTECTED] Signed-off-by: Erez Zadok [EMAIL PROTECTED] Signed-off-by: Josef 'Jeff' Sipek [EMAIL PROTECTED] --- include/linux/union_fs.h | 11 +++ 1 files changed, 11 insertions(+), 0 deletions(-) diff --git a/include/linux/union_fs.h b/include/linux/union_fs.h

[PATCH 03/16] Unionfs: Retry lookup for different silly-renamed files

file name already existed, Unionfs would oops upon copyup to that temp name. Signed-off-by: Erez Zadok [EMAIL PROTECTED] Signed-off-by: Josef 'Jeff' Sipek [EMAIL PROTECTED] --- fs/unionfs/commonfops.c |6 -- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/fs/unionfs

[PATCH 10/16] Unionfs: merge find_new_branch_index and branch_id_to_idx into one function

From: Erez Zadok [EMAIL PROTECTED] Useful code cleanup and consolidation between the ODF code and non-ODF code. Signed-off-by: Erez Zadok [EMAIL PROTECTED] Signed-off-by: Josef 'Jeff' Sipek [EMAIL PROTECTED] --- fs/unionfs/commonfops.c | 35 +-- fs/unionfs

[PATCH 01/16] [PATCH] unionfs section mismatch

From: Randy Dunlap [EMAIL PROTECTED] Fix section marker in header file: WARNING: fs/unionfs/unionfs.o(.init.text+0x56): Section mismatch: reference to .exit.text:stop_sioq (between 'init_module' and 'init_sioq') Signed-off-by: Randy Dunlap [EMAIL PROTECTED] Signed-off-by: Josef 'Jeff' Sipek

[PATCH 08/16] Unionfs: Remove unnecessary #define

From: Erez Zadok [EMAIL PROTECTED] UNIONFS_TMPNAM_LEN is used in only one place, and we have calculate the length of the string to begin with. Signed-off-by: Erez Zadok [EMAIL PROTECTED] Signed-off-by: Josef 'Jeff' Sipek [EMAIL PROTECTED] --- fs/unionfs/commonfops.c |2 +- fs/unionfs

[PATCH 05/16] Unionfs: call statfs on lower file system properly

From: Erez Zadok [EMAIL PROTECTED] Get the correct lower dentry to use to statfs the first branch (always), Signed-off-by: Erez Zadok [EMAIL PROTECTED] Signed-off-by: Josef 'Jeff' Sipek [EMAIL PROTECTED] --- fs/unionfs/super.c | 15 ++- 1 files changed, 10 insertions(+), 5

[PATCH 09/16] Unionfs: mmap implementation

From: Yiannis Pericleous [EMAIL PROTECTED] Signed-off-by: Shaya Potter [EMAIL PROTECTED] Signed-off-by: Erez Zadok [EMAIL PROTECTED] Signed-off-by: Yiannis Pericleous [EMAIL PROTECTED] Signed-off-by: Josef 'Jeff' Sipek [EMAIL PROTECTED] --- fs/unionfs/Makefile |2 +- fs/unionfs

[PATCH -mm] Unionfs: Fix lock leak in unionfs_ioctl

(This patch is already in the Unionfs git tree. This is just a heads up.) Signed-off-by: Josef 'Jeff' Sipek <[EMAIL PROTECTED]> --- fs/unionfs/commonfops.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/fs/unionfs/commonfops.c b/fs/unionfs/commonfops.c index 8

[PATCH -mm] Unionfs: Fix lock leak in unionfs_ioctl

(This patch is already in the Unionfs git tree. This is just a heads up.) Signed-off-by: Josef 'Jeff' Sipek [EMAIL PROTECTED] --- fs/unionfs/commonfops.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/fs/unionfs/commonfops.c b/fs/unionfs/commonfops.c index 83001aa

[PATCH 1/1] Documentation: Fix up docs still talking about i_sem

Signed-off-by: Josef 'Jeff' Sipek <[EMAIL PROTECTED]> --- Documentation/filesystems/directory-locking |5 +++-- Documentation/filesystems/porting |8 2 files changed, 7 insertions(+), 6 deletions(-) diff --git a/Documentation/filesystems/directory-lock

[PATCH 1/1] Documentation: Fix up docs still talking about i_sem

Signed-off-by: Josef 'Jeff' Sipek [EMAIL PROTECTED] --- Documentation/filesystems/directory-locking |5 +++-- Documentation/filesystems/porting |8 2 files changed, 7 insertions(+), 6 deletions(-) diff --git a/Documentation/filesystems/directory-locking b

[PATCH 5/5] fs: Remove path_walk export

Signed-off-by: Josef 'Jeff' Sipek <[EMAIL PROTECTED]> --- fs/namei.c|3 +-- include/linux/namei.h |1 - 2 files changed, 1 insertions(+), 3 deletions(-) diff --git a/fs/namei.c b/fs/namei.c index 50285a1..15f45ac 100644 --- a/fs/namei.c +++ b/fs/namei.c @@ -1024,7 +

[PATCH 3/5] nfsctl: Use vfs_path_lookup

use vfs_path_lookup instead of open-coding the necessary functionality. Signed-off-by: Josef 'Jeff' Sipek <[EMAIL PROTECTED]> Acked-by: NeilBrown <[EMAIL PROTECTED]> --- fs/nfsctl.c | 16 ++-- 1 files changed, 6 insertions(+), 10 deletions(-) diff --git a/fs/nfsctl.c b

[PATCH 1/5] fs: Introduce vfs_path_lookup

Signed-off-by: Josef 'Jeff' Sipek <[EMAIL PROTECTED]> --- fs/namei.c| 32 include/linux/namei.h |2 ++ 2 files changed, 34 insertions(+), 0 deletions(-) diff --git a/fs/namei.c b/fs/namei.c index 580162b..a30efbc 100644 --- a/fs/namei.c ++

[PATCH 2/5] sunrpc: Use vfs_path_lookup

use vfs_path_lookup instead of open-coding the necessary functionality. Signed-off-by: Josef 'Jeff' Sipek <[EMAIL PROTECTED]> Acked-by: Trond Myklebust <[EMAIL PROTECTED]> --- net/sunrpc/rpc_pipe.c | 16 +++- 1 files changed, 7 insertions(+), 9 deletions(-) diff --git

[PATCH 4/5] fs: Mark link_path_walk static

Signed-off-by: Josef 'Jeff' Sipek <[EMAIL PROTECTED]> --- fs/namei.c|4 +++- include/linux/namei.h |1 - 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/fs/namei.c b/fs/namei.c index a30efbc..50285a1 100644 --- a/fs/namei.c +++ b/fs/namei.c @@ -107,6

[PATCH 0/5] New path lookup function (V4)

p_create can be used on the nameidata structure to pass the create intent to the file system. Josef 'Jeff' Sipek. [1] http://lkml.org/lkml/2007/3/9/95 [2] http://lkml.org/lkml/2007/5/4/51 - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL

[PATCH 1/1] eCryptfs: Move ecryptfs docs into Documentation/filesystems/

Signed-off-by: Josef 'Jeff' Sipek <[EMAIL PROTECTED]> --- Documentation/ecryptfs.txt | 77 Documentation/filesystems/ecryptfs.txt | 77 2 files changed, 77 insertions(+), 77 deletions(-) delete mode

[PATCH 18/21] Unionfs: Remove defunct unionfs_put_inode super op

From: Erez Zadok <[EMAIL PROTECTED]> Removed old workaround code that was needed to get mmap working, is no longer needed with recent kernels. Signed-off-by: Erez Zadok <[EMAIL PROTECTED]> Signed-off-by: Josef 'Jeff' Sipek <[EMAIL PROTECTED]> --- fs/unio

[PATCH 13/21] Unionfs: Don't leak resources when copyup fails partially

Original-patch-by: Erez Zadok <[EMAIL PROTECTED]> Signed-off-by: Josef 'Jeff' Sipek <[EMAIL PROTECTED]> --- fs/unionfs/copyup.c | 24 ++-- 1 files changed, 22 insertions(+), 2 deletions(-) diff --git a/fs/unionfs/copyup.c b/fs/unionfs/copyup.c index 0975b6e..a8

[PATCH 03/21] Unionfs: Every printk should prefix with "unionfs: " consistently

From: Erez Zadok <[EMAIL PROTECTED]> Signed-off-by: Erez Zadok <[EMAIL PROTECTED]> Signed-off-by: Josef 'Jeff' Sipek <[EMAIL PROTECTED]> --- fs/unionfs/commonfops.c |7 --- fs/unionfs/copyup.c |2 +- fs/unionfs/dentry.c |8 +--- fs/unionfs/i

[PATCH 06/21] Unionfs: Added numerous comments

elling errors. Signed-off-by: Erez Zadok <[EMAIL PROTECTED]> Signed-off-by: Josef 'Jeff' Sipek <[EMAIL PROTECTED]> --- fs/unionfs/commonfops.c |7 --- fs/unionfs/copyup.c | 22 -- fs/unionfs/dentry.c |5 ++--- fs/unionfs/file.c |1 + f

[PATCH 05/21] Unionfs: Cleanup of strings and comments

From: Erez Zadok <[EMAIL PROTECTED]> Includes: - consistent style for multi-line comments - spell-check of all strings and comments Signed-off-by: Erez Zadok <[EMAIL PROTECTED]> Signed-off-by: Josef 'Jeff' Sipek <[EMAIL PROTECTED]> --- fs/stack.c

[PATCH 10/21] Unionfs: Move unionfs_query_file to commonfops.c

Moved unionfs_query_file closer to its one user in commonfops.c. Additionally, it can now become static, and branchman.c can be removed as it is empty. Signed-off-by: Josef 'Jeff' Sipek <[EMAIL PROTECTED]> --- fs/unionfs/Makefile |4 +- fs/unionfs/branchman.c

[PATCH 08/21] Unionfs: Rename Unionfs's double_lock_dentry to avoid confusion

From: Erez Zadok <[EMAIL PROTECTED]> To avoid potential confusion with a VFS function, rename our version of double_lock_dentry to unionfs_double_lock_dentry. Signed-off-by: Erez Zadok <[EMAIL PROTECTED]> Signed-off-by: Josef 'Jeff' Sipek <[EMAIL PROTECTED]> --- fs/union

[PATCH 11/21] Unionfs: Combine unionfs_write with __unionfs_write.

The __unionfs_write helper function was used only by unionfs_write, and there is really no reason why they should not be combined. Signed-off-by: Josef 'Jeff' Sipek <[EMAIL PROTECTED]> --- fs/unionfs/file.c | 30 ++ 1 files changed, 10 insertions(+), 20 del

[PATCH 12/21] Unionfs: Prefix external functions with 'extern' properly

From: Erez Zadok <[EMAIL PROTECTED]> Signed-off-by: Erez Zadok <[EMAIL PROTECTED]> Signed-off-by: Josef 'Jeff' Sipek <[EMAIL PROTECTED]> --- fs/unionfs/sioq.h |1 + fs/unionfs/union.h | 50 +- 2 files changed, 26 insertion

[PATCH 07/21] Unionfs: Consistent pointer declaration spacing

From: Erez Zadok <[EMAIL PROTECTED]> Change instances of "foo * var" to "foo *var" for consistency. Signed-off-by: Erez Zadok <[EMAIL PROTECTED]> Signed-off-by: Josef 'Jeff' Sipek <[EMAIL PROTECTED]> --- fs/unionfs/copyup.c |6 +++--- fs/unionfs/file

[PATCH 09/21] Unionfs: Rename our "do_rename" to __unionfs_rename

From: Erez Zadok <[EMAIL PROTECTED]> To avoid confusion with the VFS function do_rename, and to help ctags, rename our utility (static) function do_rename to __unionfs_rename. Signed-off-by: Erez Zadok <[EMAIL PROTECTED]> Signed-off-by: Josef 'Jeff' Sipek <[EMAIL PROTECTED]&g

[PATCH 16/21] Unionfs: Disallow setting leftmost branch to readonly

to use "remount,ro" if they want a readonly union). [jsipek: fixed up to apply cleanly] Signed-off-by: Erez Zadok <[EMAIL PROTECTED]> Signed-off-by: Josef 'Jeff' Sipek <[EMAIL PROTECTED]> --- fs/unionfs/super.c | 16 1 files changed, 12 insertions(+), 4 deletions

[PATCH 19/21] Unionfs: Actually catch bad use of unionfs_mnt{get,put}

Signed-off-by: Josef 'Jeff' Sipek <[EMAIL PROTECTED]> --- fs/unionfs/union.h | 34 +++--- 1 files changed, 7 insertions(+), 27 deletions(-) diff --git a/fs/unionfs/union.h b/fs/unionfs/union.h index 5376b76..335d579 100644 --- a/fs/unionfs/union.h +++ b/fs/u

[PATCH 17/21] Unionfs: Documentation update regarding overlapping branches and new lookup code

From: Erez Zadok <[EMAIL PROTECTED]> Added detailed comment and updated documentation to explain why overlapping branches are disallowed, and better explain the cache coherency issues. Signed-off-by: Erez Zadok <[EMAIL PROTECTED]> Signed-off-by: Josef 'Jeff' Sipek <[

[PATCH 15/21] Unionfs: Use krealloc instead of open-coding the functionality

Change the branch management code to use krealloc instead of playing tricks with kmalloc/memcpy/kfree. Signed-off-by: Josef 'Jeff' Sipek <[EMAIL PROTECTED]> --- fs/unionfs/super.c | 56 +--- 1 files changed, 14 insertions(+), 42 del

[PATCH 20/21] Unionfs: Removed a trailing whitespace

From: Yiannis Pericleous <[EMAIL PROTECTED]> Signed-off-by: Yiannis Pericleous <[EMAIL PROTECTED]> Signed-off-by: Josef 'Jeff' Sipek <[EMAIL PROTECTED]> --- fs/unionfs/super.c |6 -- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/fs/unionfs/super.c b/

[PATCH 14/21] Unionfs: Call realloc unconditionally

krealloc already checks if the new size is greater than the old size. Therefore, we can call realloc unconditionally - making the code simpler and cleaner. Signed-off-by: Josef 'Jeff' Sipek <[EMAIL PROTECTED]> --- fs/unionfs/lookup.c | 26 -- 1 files chan

[PATCH 21/21] Unionfs: Correctly decrement refcounts of mnt's upon branch management

From: Erez Zadok <[EMAIL PROTECTED]> The old logic was broken in one place, which another place tried to "fix" incorrectly. Also added detailed comments to explain the new/correct logic. Signed-off-by: Erez Zadok <[EMAIL PROTECTED]> Signed-off-by: Josef 'Jeff' Sipek <[

[PATCH 04/21] Unionfs: Add missing copyright notices

From: Erez Zadok <[EMAIL PROTECTED]> Signed-off-by: Erez Zadok <[EMAIL PROTECTED]> Signed-off-by: Josef 'Jeff' Sipek <[EMAIL PROTECTED]> --- fs/stack.c | 11 +++ fs/unionfs/sioq.c|5 - fs/unionfs/sioq.h| 13 + includ

[PATCH 01/21] Unionfs: Tiny documentation fixups

From: Erez Zadok <[EMAIL PROTECTED]> Signed-off-by: Erez Zadok <[EMAIL PROTECTED]> Signed-off-by: Josef 'Jeff' Sipek <[EMAIL PROTECTED]> --- Documentation/filesystems/unionfs/usage.txt |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/Documentation/f

[GIT PULL -mm] Unionfs cleanups and fixes

h "unionfs: " consistently Unionfs: Coding style fixes Unionfs: Tiny documentation fixups Thanks, Josef 'Jeff' Sipek <[EMAIL PROTECTED]> - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PRO

[GIT PULL -mm] Unionfs cleanups and fixes

: consistently Unionfs: Coding style fixes Unionfs: Tiny documentation fixups Thanks, Josef 'Jeff' Sipek [EMAIL PROTECTED] - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org

[PATCH 01/21] Unionfs: Tiny documentation fixups

From: Erez Zadok [EMAIL PROTECTED] Signed-off-by: Erez Zadok [EMAIL PROTECTED] Signed-off-by: Josef 'Jeff' Sipek [EMAIL PROTECTED] --- Documentation/filesystems/unionfs/usage.txt |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/Documentation/filesystems/unionfs

[PATCH 04/21] Unionfs: Add missing copyright notices

From: Erez Zadok [EMAIL PROTECTED] Signed-off-by: Erez Zadok [EMAIL PROTECTED] Signed-off-by: Josef 'Jeff' Sipek [EMAIL PROTECTED] --- fs/stack.c | 11 +++ fs/unionfs/sioq.c|5 - fs/unionfs/sioq.h| 13 + include/linux/fs_stack.h

[PATCH 21/21] Unionfs: Correctly decrement refcounts of mnt's upon branch management

From: Erez Zadok [EMAIL PROTECTED] The old logic was broken in one place, which another place tried to fix incorrectly. Also added detailed comments to explain the new/correct logic. Signed-off-by: Erez Zadok [EMAIL PROTECTED] Signed-off-by: Josef 'Jeff' Sipek [EMAIL PROTECTED] --- fs/unionfs

[PATCH 15/21] Unionfs: Use krealloc instead of open-coding the functionality

Change the branch management code to use krealloc instead of playing tricks with kmalloc/memcpy/kfree. Signed-off-by: Josef 'Jeff' Sipek [EMAIL PROTECTED] --- fs/unionfs/super.c | 56 +--- 1 files changed, 14 insertions(+), 42 deletions(-) diff

[PATCH 20/21] Unionfs: Removed a trailing whitespace

From: Yiannis Pericleous [EMAIL PROTECTED] Signed-off-by: Yiannis Pericleous [EMAIL PROTECTED] Signed-off-by: Josef 'Jeff' Sipek [EMAIL PROTECTED] --- fs/unionfs/super.c |6 -- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/fs/unionfs/super.c b/fs/unionfs/super.c index

[PATCH 14/21] Unionfs: Call realloc unconditionally

krealloc already checks if the new size is greater than the old size. Therefore, we can call realloc unconditionally - making the code simpler and cleaner. Signed-off-by: Josef 'Jeff' Sipek [EMAIL PROTECTED] --- fs/unionfs/lookup.c | 26 -- 1 files changed, 8 insertions

[PATCH 19/21] Unionfs: Actually catch bad use of unionfs_mnt{get,put}

Signed-off-by: Josef 'Jeff' Sipek [EMAIL PROTECTED] --- fs/unionfs/union.h | 34 +++--- 1 files changed, 7 insertions(+), 27 deletions(-) diff --git a/fs/unionfs/union.h b/fs/unionfs/union.h index 5376b76..335d579 100644 --- a/fs/unionfs/union.h +++ b/fs/unionfs

[PATCH 17/21] Unionfs: Documentation update regarding overlapping branches and new lookup code

From: Erez Zadok [EMAIL PROTECTED] Added detailed comment and updated documentation to explain why overlapping branches are disallowed, and better explain the cache coherency issues. Signed-off-by: Erez Zadok [EMAIL PROTECTED] Signed-off-by: Josef 'Jeff' Sipek [EMAIL PROTECTED

[PATCH 09/21] Unionfs: Rename our do_rename to __unionfs_rename

From: Erez Zadok [EMAIL PROTECTED] To avoid confusion with the VFS function do_rename, and to help ctags, rename our utility (static) function do_rename to __unionfs_rename. Signed-off-by: Erez Zadok [EMAIL PROTECTED] Signed-off-by: Josef 'Jeff' Sipek [EMAIL PROTECTED] --- fs/unionfs/rename.c

[PATCH 16/21] Unionfs: Disallow setting leftmost branch to readonly

,ro if they want a readonly union). [jsipek: fixed up to apply cleanly] Signed-off-by: Erez Zadok [EMAIL PROTECTED] Signed-off-by: Josef 'Jeff' Sipek [EMAIL PROTECTED] --- fs/unionfs/super.c | 16 1 files changed, 12 insertions(+), 4 deletions(-) diff --git a/fs/unionfs/super.c

[PATCH 07/21] Unionfs: Consistent pointer declaration spacing

From: Erez Zadok [EMAIL PROTECTED] Change instances of foo * var to foo *var for consistency. Signed-off-by: Erez Zadok [EMAIL PROTECTED] Signed-off-by: Josef 'Jeff' Sipek [EMAIL PROTECTED] --- fs/unionfs/copyup.c |6 +++--- fs/unionfs/file.c | 14 +++--- fs/unionfs/inode.c

[PATCH 11/21] Unionfs: Combine unionfs_write with __unionfs_write.

The __unionfs_write helper function was used only by unionfs_write, and there is really no reason why they should not be combined. Signed-off-by: Josef 'Jeff' Sipek [EMAIL PROTECTED] --- fs/unionfs/file.c | 30 ++ 1 files changed, 10 insertions(+), 20 deletions

[PATCH 12/21] Unionfs: Prefix external functions with 'extern' properly

From: Erez Zadok [EMAIL PROTECTED] Signed-off-by: Erez Zadok [EMAIL PROTECTED] Signed-off-by: Josef 'Jeff' Sipek [EMAIL PROTECTED] --- fs/unionfs/sioq.h |1 + fs/unionfs/union.h | 50 +- 2 files changed, 26 insertions(+), 25 deletions

[PATCH 10/21] Unionfs: Move unionfs_query_file to commonfops.c

Moved unionfs_query_file closer to its one user in commonfops.c. Additionally, it can now become static, and branchman.c can be removed as it is empty. Signed-off-by: Josef 'Jeff' Sipek [EMAIL PROTECTED] --- fs/unionfs/Makefile |4 +- fs/unionfs/branchman.c | 60

[PATCH 08/21] Unionfs: Rename Unionfs's double_lock_dentry to avoid confusion

From: Erez Zadok [EMAIL PROTECTED] To avoid potential confusion with a VFS function, rename our version of double_lock_dentry to unionfs_double_lock_dentry. Signed-off-by: Erez Zadok [EMAIL PROTECTED] Signed-off-by: Josef 'Jeff' Sipek [EMAIL PROTECTED] --- fs/unionfs/inode.c |2 +- fs

[PATCH 05/21] Unionfs: Cleanup of strings and comments

From: Erez Zadok [EMAIL PROTECTED] Includes: - consistent style for multi-line comments - spell-check of all strings and comments Signed-off-by: Erez Zadok [EMAIL PROTECTED] Signed-off-by: Josef 'Jeff' Sipek [EMAIL PROTECTED] --- fs/stack.c |6 -- fs

[PATCH 06/21] Unionfs: Added numerous comments

errors. Signed-off-by: Erez Zadok [EMAIL PROTECTED] Signed-off-by: Josef 'Jeff' Sipek [EMAIL PROTECTED] --- fs/unionfs/commonfops.c |7 --- fs/unionfs/copyup.c | 22 -- fs/unionfs/dentry.c |5 ++--- fs/unionfs/file.c |1 + fs/unionfs/lookup.c

[PATCH 03/21] Unionfs: Every printk should prefix with unionfs: consistently

From: Erez Zadok [EMAIL PROTECTED] Signed-off-by: Erez Zadok [EMAIL PROTECTED] Signed-off-by: Josef 'Jeff' Sipek [EMAIL PROTECTED] --- fs/unionfs/commonfops.c |7 --- fs/unionfs/copyup.c |2 +- fs/unionfs/dentry.c |8 +--- fs/unionfs/inode.c | 15

[PATCH 13/21] Unionfs: Don't leak resources when copyup fails partially

Original-patch-by: Erez Zadok [EMAIL PROTECTED] Signed-off-by: Josef 'Jeff' Sipek [EMAIL PROTECTED] --- fs/unionfs/copyup.c | 24 ++-- 1 files changed, 22 insertions(+), 2 deletions(-) diff --git a/fs/unionfs/copyup.c b/fs/unionfs/copyup.c index 0975b6e..a80ece6 100644

[PATCH 18/21] Unionfs: Remove defunct unionfs_put_inode super op

From: Erez Zadok [EMAIL PROTECTED] Removed old workaround code that was needed to get mmap working, is no longer needed with recent kernels. Signed-off-by: Erez Zadok [EMAIL PROTECTED] Signed-off-by: Josef 'Jeff' Sipek [EMAIL PROTECTED] --- fs/unionfs/super.c | 20 1

[PATCH 1/1] eCryptfs: Move ecryptfs docs into Documentation/filesystems/

Signed-off-by: Josef 'Jeff' Sipek [EMAIL PROTECTED] --- Documentation/ecryptfs.txt | 77 Documentation/filesystems/ecryptfs.txt | 77 2 files changed, 77 insertions(+), 77 deletions(-) delete mode 100644

[PATCH 0/5] New path lookup function (V4)

on the nameidata structure to pass the create intent to the file system. Josef 'Jeff' Sipek. [1] http://lkml.org/lkml/2007/3/9/95 [2] http://lkml.org/lkml/2007/5/4/51 - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info

[PATCH 2/5] sunrpc: Use vfs_path_lookup

use vfs_path_lookup instead of open-coding the necessary functionality. Signed-off-by: Josef 'Jeff' Sipek [EMAIL PROTECTED] Acked-by: Trond Myklebust [EMAIL PROTECTED] --- net/sunrpc/rpc_pipe.c | 16 +++- 1 files changed, 7 insertions(+), 9 deletions(-) diff --git a/net/sunrpc

[PATCH 4/5] fs: Mark link_path_walk static

Signed-off-by: Josef 'Jeff' Sipek [EMAIL PROTECTED] --- fs/namei.c|4 +++- include/linux/namei.h |1 - 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/fs/namei.c b/fs/namei.c index a30efbc..50285a1 100644 --- a/fs/namei.c +++ b/fs/namei.c @@ -107,6 +107,8

[PATCH 5/5] fs: Remove path_walk export

Signed-off-by: Josef 'Jeff' Sipek [EMAIL PROTECTED] --- fs/namei.c|3 +-- include/linux/namei.h |1 - 2 files changed, 1 insertions(+), 3 deletions(-) diff --git a/fs/namei.c b/fs/namei.c index 50285a1..15f45ac 100644 --- a/fs/namei.c +++ b/fs/namei.c @@ -1024,7 +1024,7

[PATCH 3/5] nfsctl: Use vfs_path_lookup

use vfs_path_lookup instead of open-coding the necessary functionality. Signed-off-by: Josef 'Jeff' Sipek [EMAIL PROTECTED] Acked-by: NeilBrown [EMAIL PROTECTED] --- fs/nfsctl.c | 16 ++-- 1 files changed, 6 insertions(+), 10 deletions(-) diff --git a/fs/nfsctl.c b/fs/nfsctl.c

[PATCH 1/5] fs: Introduce vfs_path_lookup

Signed-off-by: Josef 'Jeff' Sipek [EMAIL PROTECTED] --- fs/namei.c| 32 include/linux/namei.h |2 ++ 2 files changed, 34 insertions(+), 0 deletions(-) diff --git a/fs/namei.c b/fs/namei.c index 580162b..a30efbc 100644 --- a/fs/namei.c +++ b/fs

[PATCH 1/1] nfsctl: Use vfs_path_lookup

use vfs_path_lookup instead of open-coding the necessary functionality. Signed-off-by: Josef 'Jeff' Sipek <[EMAIL PROTECTED]> --- fs/nfsctl.c | 16 ++-- 1 files changed, 6 insertions(+), 10 deletions(-) diff --git a/fs/nfsctl.c b/fs/nfsctl.c index c043136..51f1b31 100644 --

[PATCH 4/5] fs: Mark link_path_walk static

Signed-off-by: Josef 'Jeff' Sipek <[EMAIL PROTECTED]> --- fs/namei.c|4 +++- include/linux/namei.h |1 - 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/fs/namei.c b/fs/namei.c index 090cce4..925c62e 100644 --- a/fs/namei.c +++ b/fs/namei.c @@ -108,6

[PATCH 2/2] fs: Use path_walk in do_path_lookup

Since, path_walk sets the total_link_count to 0, and calls link_path_walk, we can just call path_walk directly. Signed-off-by: Josef 'Jeff' Sipek <[EMAIL PROTECTED]> --- fs/namei.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/fs/namei.c b/fs/namei.c index 2

[PATCH 0/2] [TRIVIAL] Small cleanups for do_path_lookup (V2)

(For changes since V1, see the end of this email.) The following 2 patches are trivial cleanups to do_path_lookup in namei.c. Since these changes are trivial, they can go into 2.6.22-rc1 without any problems. Josef 'Jeff' Sipek (2): fs: Fix indentation in do_path_lookup fs: Use

[PATCH 1/2] fs: Fix indentation in do_path_lookup

Signed-off-by: Josef 'Jeff' Sipek <[EMAIL PROTECTED]> --- fs/namei.c |4 +--- 1 files changed, 1 insertions(+), 3 deletions(-) diff --git a/fs/namei.c b/fs/namei.c index 7a98676..2a5c232 100644 --- a/fs/namei.c +++ b/fs/namei.c @@ -1159,11 +1159,9 @@ static int fastcall do_path_look

<    1   2   3   4   5   >