Revert "Unionfs: Cleanup locking in new_dentry_private_data"

This reverts commit a48f757714d7e70878461c5393de787e1497a612.

Conflicts:

fs/unionfs/lookup.c

removed super_block * from odf_get_opaque args

Documentation: replace "odfforce" with "odf_keepcache"

lookup only up to opaque branch if looking up an opaque dir

bugfix: dont move to /odf/sr if same inode number is already there

odf_remove bug, check if unionfs dentry is not negative before trying to

read its i_count

changed the format string of inode nums to filenames to %lx

split odf.h into two .h files, odf.h and odf_internals.h

renamed dentry stack functions

added comment for odf_dentry_info struct

added a few FIXME comments

lock odf inode when calling the odf fs's get_parent

use snprintf when printing help in configfs

use 2 bytes for namelen in odf cache dir dirents

changed odf options length to PAGE_SIZE

filenames in odf/sr and odf/reclaim to be in hex, not decimal

removed duplicate odf_get_opaque and odf_set_opaque functions

removed opaque_branch_id field from unionfs_sb_info

cleanup thread to clean until both blocks and inodes are below threshhold

added new cleanup flags

cleaned up partial_lookup

Documentation: updated odf.txt to say that when a dentry is released from /odf/sr it is

unlinked

modified odf_release_sr to unlink instead of renaming to /odf/reclaim

dget odf root dentry on odf_read_super

use '\0' to null terminate a string

coding-style: aligned struct decl to span over multiple lines

removed unneeded functions for allocating/freeing odf_dentry_info's

cleanup: removed unecessary check at a return value

added KERN_WARNING to cleanup thread's print messages

coding-style: removed unecesary cast

coding_style: reformatted lines over 80 chars

fixed odf_parse_options to use strdup and strsep

look for odf_keepcache instead of odfforce option
fail if dirs specified with old odf

invariants: don't complain if no lower dentry of a deleted dentry

Signed-off-by: Erez Zadok <ezk@cs.sunysb.edu>

more (forgotten) cleanups to partial_lookup.

Signed-off-by: Erez Zadok <ezk@cs.sunysb.edu>

invariants: don't complain about directories with some NULL lower objects

Directories can have NULL lower objects in between start/end, but NOT if at
the start/end range.  We cannot verify that this dentry is a type=DIR,
because it may already be a negative dentry.  But if dbstart is greater than
dbend, we know that this couldn't have been a regular file: it had to have
been a directory.  In that case, don't complain about NULL lower objects in
in between start/end.

Signed-off-by: Erez Zadok <ezk@cs.sunysb.edu>

documentation: update of last modification date to odf.txt document

Signed-off-by: Erez Zadok <ezk@cs.sunysb.edu>

cleanup: rewrite unionfs_partial_lookup more cleanly

Signed-off-by: Erez Zadok <ezk@cs.sunysb.edu>

bug fix: prevent null mntget warning messages in unionfs_lookup_backend

Don't try to mntget a lower mnt at a point where the lower mnts don't yet
exist (the callers will have them later on).  Instead, get the lower mnt of
the sb->s_root.

Signed-off-by: Erez Zadok <ezk@cs.sunysb.edu>

documentation: expand comments to unionfs_lookup_backend

Signed-off-by: Erez Zadok <ezk@cs.sunysb.edu>
Conflicts:

fs/unionfs/lookup.c

cleanup: remove unnecessary call to inherit_mnt

Our unionfs_lookup doesn't need to call inherit_mnt: the lower mnts will be
correct here (but not if other methods such as unionfs_rename call our
lookup_backend helper function directly).

Signed-off-by: Erez Zadok <ezk@cs.sunysb.edu>

bug fix: don't warn if trying to mntput a "negative" union object

In unionfs_d_release, we may get negative dentries which don't have a lower
mnt.  In that case, don't print a warning message from unionfs_mntput about
trying to put a NULL mnt.

Signed-off-by: Erez Zadok <ezk@cs.sunysb.edu>

bug fix: get lower mnt from root dentry instead of lower dentry

While copying-up a file, the union object hasn't been fully formed yet, so
we don't have all the lower dentries/mnts to use (the caller will have them
later on).  Therefore, we should use the sb->s_root's lower mnt here.  This
fixes a "NULL mnt" warning message that came from unionfs_mntget.

Signed-off-by: Erez Zadok <ezk@cs.sunysb.edu>

coding-style: removed space before ')'

comment on why we cannot create entries in the odf (assign inode nums)

when creating the odf cached dir

one more comment on the odf_lookup code

comments for odf_lookup functions

removed ODF_LOOKUP_LINK flag and moved ODF_RMV_WH to the top of the flags

changed odf lookup flags values to hex and removed ODF_LOOKUP_LOCKED

changed unionfs_dentry_info to contain an odf_dentry_info instead of a pointer to one

modified odf_lookup functions to return a dentry instead of an

odf_dentry_info

Unionfs: allow users to override Makefile options in fs/unionfs/local.mk

Signed-off-by: Erez Zadok <ezk@cs.sunysb.edu>

remove trailing ':' when building dirs= options from odf sb file

removed warning when branch_id_to_idx fails to match requested id, and added a

comment for why this can be legal in the odf

bug fix: catch bad use of dirs= options (extra ':' separators)

Signed-off-by: Erez Zadok <ezk@cs.sunysb.edu>

unionfs: merge find_new_branch_index and branch_id_to_idx into one function

Useful code cleanup and consolidation between the ODF code and non-ODF code.

Signed-off-by: Erez Zadok <ezk@cs.sunysb.edu>
Conflicts:

fs/unionfs/commonfops.c
fs/unionfs/fanout.h

dont put ic dentry in odf_get_ic_dentry

modified odf_ic_cache_dentry not to use odf_lookup and to return a dentry instead of

an odf_dentry_info

unionfs: minor cleanup and better commenting in new_dentry_private_data

Signed-off-by: Erez Zadok <ezk@cs.sunysb.edu>
Conflicts:

fs/unionfs/lookup.c

Unionfs: Cleanup locking in new_dentry_private_data

Signed-off-by: Josef 'Jeff' Sipek <jsipek@cs.sunysb.edu>
Conflicts:

fs/unionfs/lookup.c

bugfix: prevent null-deref oops if lower f/s is NFS (mmap writes)

This is a workaround fora deficiency of the linux MM layer, which doesn't
allow clean coordination between upper and lower pages in stackable layers.
We prevent an oops, but the cost is that we're not able to implement
writepages cleanly, not can we call the lower file system's writepages.

Signed-off-by: Erez Zadok <ezk@cs.sunysb.edu>

changed unionfs_sb_info to contain a unionfs_sb_info instead of a pointer to one

removed the odf_sb_info field from the odf_dentry_info struct

removed the inum field from odf_dentry_info struct

changed the odf_sb_info structure to hold dentry pointers to the odf special dirs

instead of odf_dentry_info pointers

removed odf locks, added LOCK comments where we might still want to

lock something, ie when operating in odf/ic /sr and /reclaim

cleaner odf_is_new function

proper capitalization of unionfs

use sizeof() instead of strlen

print KERN_ERR is odf_read_super fails

remove the BUG_ON in odf_is_wh, for now because odf lookup is

called in odf_put_super before the whiteout file is created.

use LOOKUP_DIRECTORY instead of LOOKUP_FOLLOW

return EEXIST if a dir exists when constructing the odf hierarchy

removed unused function

added comments for odf special filenames constants and grouped them together

use only one version number for odf.

set version number to 2, since format has changed
since last release

cleanup: removed unneeded declarations

coding style: use branch_id function instead of accessing sb private

data directly

added BUG_ON to make sure dentry passed to get_parent is a dir

removed unnecessary call to unreg_configfs if init fails

coding style: align function arguments

coding style: use switch statement to avoid ugly if/else nesting

coding style: use void in a function taking no args

small fix: initialize 'res' to 0 to avoid gcc warning

cleanup: use krealloc instead of open-coding it with kmalloc/kfree/memcpy

Conflicts:

fs/unionfs/lookup.c

bugfix: prevent a NULL ptr deref if branch is nfs and using mmap writes

By some strange condition, the VFS can pass our unionfs_writepage a
writeback_control structure which, if passed as is to the lower ->writepage
-- here nfs_writepage, triggers a NULL ptr deref in NFS.  This fix works
around this issue, and results in a successfully written mmaped file.

Signed-off-by: Erez Zadok <ezk@cs.sunysb.edu>

bugfix: handle lower file systems that do not support writeable mappings

File systems such as jffs2 (tested in 2.6.22-rc22) do not have a
->writepage.  Make sure unionfs detects that conditions and returns EINVAL
when a user tries to mmap such a file with VM_WRITE.

Signed-off-by: Erez Zadok <ezk@cs.sunysb.edu>

cleanup thread to clean odf/sr as well on mount if odfforce is not set

set gid/uid to 0 in ofd_release_sr

cleanup: removed unnecesary braces

moved call of odf_release_sr from drop_inode to clear_inode

added undeclared variables to unionfs_remount

bug fix: set lower inodes correctly after branch management succeeds

Signed-off-by: Erez Zadok <ezk@cs.sunysb.edu>
Conflicts:

fs/unionfs/super.c

bug fix: prevent self deadlock with remount code in pivot_root scenarios

Signed-off-by: Erez Zadok <ezk@cs.sunysb.edu>

debugging: test for freed (0x5a) inodes

Signed-off-by: Erez Zadok <ezk@cs.sunysb.edu>

bugfix: statfs lower file system properly

Get the correct lower dentry to use to statfs the first branch (always), now
works with disk-based as well as network file systems.  Also document the
code better.

Signed-off-by: Erez Zadok <ezk@cs.sunysb.edu>
Conflicts:

fs/unionfs/super.c

Add Erez Zadok to maintainers list

add standard copyright comment to linux/union_fs.h

readdir bug: must pass the file position at the start of the dirent to filldir

documentation: added info about /odf/sr in odf.txt

when an inode is dropped, check if it has an entry in /odf/sr and move it to /odf/reclaim

silly rename (move to /odf/sr) files to be removed that have i_count > 1