linux-toradex.git/fs/gfs2/dir.c, branch v3.0.29

GFS2: When adding a new dir entry, inc link count if it is a subdir

2011-05-09T15:43:53+00:00

This adds an increment of the link count when we add a new directory
entry, if that entry is itself a directory. This means that we no
longer need separate code to perform this operation.

Now that both adding and removing directory entries automatically
update the parent directory's link count if required, that makes
the code shorter and simpler than before.

Signed-off-by: Steven Whitehouse

GFS2: Make gfs2_dir_del update link count when required

2011-05-09T15:42:37+00:00

When we remove an entry from a directory, we can save ourselves
some trouble if we know the type of the entry in question, since
if it is itself a directory, we can update the link count of the
parent at the same time as removing the directory entry.

In addition this patch also merges the rmdir and unlink code which
was almost identical anyway. This eliminates the calls to remove
the . and .. directory entries on each rmdir (not needed since the
directory will be deallocated, anyway) which was the only thing preventing
passing the dentry to gfs2_dir_del(). The passing of the dentry
rather than just the name allows us to figure out the type of the entry
which is being removed, and thus adjust the link count when required.

Signed-off-by: Steven Whitehouse

GFS2: move function foreach_leaf to gfs2_dir_exhash_dealloc

2011-04-20T07:54:44+00:00

The previous patches made function gfs2_dir_exhash_dealloc do nothing
but call function foreach_leaf.  This patch simplifies the code by
moving the entire function foreach_leaf into gfs2_dir_exhash_dealloc.

Signed-off-by: Bob Peterson 
Signed-off-by: Steven Whitehouse

GFS2: pass leaf_bh into leaf_dealloc

2011-04-20T07:54:26+00:00

Function foreach_leaf used to look up the leaf block address and get
a buffer_head.  Then it would call leaf_dealloc which did the same
lookup.  This patch combines the two operations by making foreach_leaf
pass the leaf bh to leaf_dealloc.

Signed-off-by: Bob Peterson 
Signed-off-by: Steven Whitehouse

GFS2: Combine transaction from gfs2_dir_exhash_dealloc

2011-04-20T07:53:56+00:00

At the end of function gfs2_dir_exhash_dealloc, it was setting the dinode
type to "file" to prevent directory corruption in case of a crash.
It was doing so in its own journal transaction.  This patch makes the
change occur when the last call is make to leaf_dealloc, since it needs
to rewrite the directory dinode at that time anyway.

Signed-off-by: Bob Peterson 
Signed-off-by: Steven Whitehouse

GFS2: remove *leaf_call_t and simplify leaf_dealloc

2011-04-20T07:53:35+00:00

Since foreach_leaf is only called with leaf_dealloc as its only possible
call function, we can simplify the code by making it call leaf_dealloc
directly.  This simplifies the code and eliminates the need for
leaf_call_t, the generic call method.  This is a first small step in
simplifying the directory leaf deallocation code.

Signed-off-by: Bob Peterson 
Signed-off-by: Steven Whitehouse

GFS2: filesystem hang caused by incorrect lock order

2011-04-18T14:23:50+00:00

This patch fixes a deadlock in GFS2 where two processes are trying
to reclaim an unlinked dinode:
One holds the inode glock and calls gfs2_lookup_by_inum trying to look
up the inode, which it can't, due to I_FREEING.  The other has set
I_FREEING from vfs and is at the beginning of gfs2_delete_inode
waiting for the glock, which is held by the first.  The solution is to
add a new non_block parameter to the gfs2_iget function that causes it
to return -ENOENT if the inode is being freed.

Signed-off-by: Bob Peterson 
Signed-off-by: Steven Whitehouse

GFS2: Make . and .. qstrs constant

2010-09-20T10:21:09+00:00

Rather than calculating the qstrs for . and .. each time
we need them, its better to keep a constant version of
these and just refer to them when required.

Signed-off-by: Steven Whitehouse 
Reviewed-by: Christoph Hellwig

GFS2: Remove i_disksize

2010-09-20T10:18:29+00:00

With the update of the truncate code, ip->i_disksize and
inode->i_size are merely copies of each other. This means
we can remove ip->i_disksize and use inode->i_size exclusively
reducing the size of a GFS2 inode by 8 bytes.

Signed-off-by: Steven Whitehouse

GFS2: remove dependency on __GFP_NOFAIL

2010-07-29T08:37:18+00:00

The k[mc]allocs in dr_split_leaf() and dir_double_exhash() are failable,
so remove __GFP_NOFAIL from their masks.

Cc: Bob Peterson 
Signed-off-by: David Rientjes 
Signed-off-by: Steven Whitehouse