linux-toradex.git/fs/ceph/mds_client.c, branch v3.2

ceph: add missing spin_unlock at ceph_mdsc_build_path()

2011-12-13T19:59:53+00:00

one of the paths was missing spin_unlock

Signed-off-by: Yehuda Sadeh

ceph: use i_ceph_lock instead of i_lock

2011-12-07T18:46:44+00:00

We have been using i_lock to protect all kinds of data structures in the
ceph_inode_info struct, including lists of inodes that we need to iterate
over while avoiding races with inode destruction.  That requires grabbing
a reference to the inode with the list lock protected, but igrab() now
takes i_lock to check the inode flags.

Changing the list lock ordering would be a painful process.

However, using a ceph-specific i_ceph_lock in the ceph inode instead of
i_lock is a simple mechanical change and avoids the ordering constraints
imposed by igrab().

Reported-by: Amon Ott 
Signed-off-by: Sage Weil

ceph/mds_client.c: quiet sparse noise

2011-11-06T04:10:11+00:00

Quiet the following sparse noise:

warning: symbol 'get_nonsnap_parent' was not declared. Should it be static?
warning: symbol 'done_closing_sessions' was not declared. Should it be static?

Local functions don't need external visability. Make them static.

Signed-off-by: H Hartley Sweeten 
Cc: Sage Weil 
Signed-off-by: Sage Weil

ceph: use new D_COMPLETE dentry flag

2011-11-06T04:10:10+00:00

We used to use a flag on the directory inode to track whether the dcache
contents for a directory were a complete cached copy.  Switch to a dentry
flag CEPH_D_COMPLETE that is safely updated by ->d_prune().

Signed-off-by: Sage Weil

libceph: don't complain on msgpool alloc failures

2011-10-25T23:10:15+00:00

The pool allocation failures are masked by the pool; there is no need to
spam the console about them.  (That's the whole point of having the pool
in the first place.)

Mark msg allocations whose failure is safely handled as such.

Signed-off-by: Sage Weil

ceph: fix encoding of ino only (not relative) paths

2011-08-15T20:03:56+00:00

A 'path' consists of a starting ino and relative component.  Encode even
when there is no relative component.  This is primarily needed by the
NFS reexport code.

Signed-off-by: Sage Weil

ceph: document unlocked d_parent accesses

2011-07-26T18:31:26+00:00

For the most part we don't care about racing with rename when directing
MDS requests; either the old or new parent is fine.  Document that, and
do some minor cleanup.

Reviewed-by: Yehuda Sadeh 
Signed-off-by: Sage Weil

ceph: explicitly reference rename old_dentry parent dir in request

2011-07-26T18:31:14+00:00

We carry a pin on the parent directory for the rename source and dest
dentries.  For the source it's r_locked_dir; we need to explicitly
reference the old_dentry parent as well, since the dentry's d_parent may
change between when the request was created and pinned and when it is
freed.

Reviewed-by: Yehuda Sadeh 
Signed-off-by: Sage Weil

ceph: avoid d_parent in ceph_dentry_hash; fix ceph_encode_fh() hashing bug

2011-07-26T18:30:55+00:00

Have caller pass in a safely-obtained reference to the parent directory
for calculating a dentry's hash valud.

While we're here, simpify the flow through ceph_encode_fh() so that there
is a single exit point and cleanup.

Also fix a bug with the dentry hash calculation: calculate the hash for the
dentry we were given, not its parent.

Reviewed-by: Yehuda Sadeh 
Signed-off-by: Sage Weil

ceph: ignore lease mask

2011-07-26T18:28:25+00:00

The lease mask is no longer used (and it changed a while back).  Instead,
use a non-zero duration to indicate that there is a lease being issued.

Reviewed-by: Yehuda Sadeh 
Signed-off-by: Sage Weil