glusterfs.git, branch v3.2.5qa4

protocol/client: print remote operation failure logs in error mode instead if info

2011-10-28T12:00:18+00:00

Change-Id: Ic4cd16618925656a12049f28f9b30bcac2242d4b
BUG: 2346
Reviewed-on: http://review.gluster.com/621
Tested-by: Gluster Build System 
Reviewed-by: Amar Tumballi 
Reviewed-by: Vijay Bellur

glusterd: Removing delayed moving of op sm for stop vol/remove brick op.

2011-10-28T11:58:18+00:00

Earlier we would wait until brick disconnect or 5s whichever was earlier,
before we move op sm from brick op stage to commit stage. This involves a race
where both the above mentioned events can happen 'concurrently' and result in
double free of the event context.

Change-Id: Ia369994448e71a052679d2373b8ab9177d56a541
BUG: 3700
Reviewed-on: http://review.gluster.com/626
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

pump: Change crawl to accommodate afr_lookup changes

2011-10-28T11:54:08+00:00

Change-Id: I895574dd6fa411784eb5282c799ccf3ff7c65625
BUG: 3747
Reviewed-on: http://review.gluster.com/635
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

cluster/afr: Take gfid-req from xattr_req only if inode->gfid is null

2011-10-28T11:52:25+00:00

Afr needs to send the xattr_req without gfid so instead of modifying
the xattr_req that comes from parent xlators take a copy and then
modify.

Change-Id: I7ee9e941eb931d67f3c89feb664a257edd01803b
BUG: 3747
Reviewed-on: http://review.gluster.com/634
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

mount/fuse: set gfid in lookup after a failed revalidate lookup

2011-10-28T11:50:21+00:00

Change-Id: I1904aa63d9365ebda3e979449454ac08db85d93d
BUG: 3747
Reviewed-on: http://review.gluster.com/633
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

check the return value of inode_path for less than or equal to zero

2011-10-28T11:14:56+00:00

Change-Id: I9bbdfe79664c1339b66819a6c7ea4b7698beb5c6
BUG: 3757
Reviewed-on: http://review.gluster.com/640
Tested-by: Gluster Build System 
Reviewed-by: Amar Tumballi

Improves the 'profile info' output. Makes it more self explanatory.

2011-10-27T10:48:35+00:00

Output of profile info was not self explanatory.

Change-Id: Iaab0ce055b7e0548645a0a77921e608497c15e43
BUG: 3028
Testcase: run 'profile info' and see if all the fields and columns are self explanatory.
Reviewed-on: http://review.gluster.com/620
Tested-by: Gluster Build System 
Reviewed-by: Amar Tumballi 
Reviewed-by: Vijay Bellur

locks: perform inode_ref of pl_update_refkeeper outside locked region

2011-10-27T10:47:54+00:00

Performing inode_ref inside the critical section of pl_update_refkeeper
(around pl_inode_t->mutex) causes a deadlock with inode_table_t->lock.
The other thread (process state dump) holds the inode_table_t->lock first
to loop over inodes and then while dumping an inode's lock context, tries
to acquire pl_inode_t->mutex thereby completing the deadlock.

The fix is to perform inode_ref outside the critical section in
pl_update_refkeeper

Change-Id: I689ab4a9b46b36287740279fdec6159182c86119
BUG: 3753
Reviewed-on: http://review.gluster.com/629
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

features/marker-quota: Donot log when inode or inode-ctx is NULL.

2011-10-27T09:57:34+00:00

When running rm -rf from mount point, quota-updation process may endup in a state where the
inode is already removed from the gluster inode table, but the updation has process is still in
progress. This is not a error condition because the updation is done asynchronously.

Change-Id: I2d3aad716bb1c971115004a70ca76e2dd5088e44
BUG: 3609
Reviewed-on: http://review.gluster.com/603
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

glusterd: Fail volume 'modify' operations when rb is ongoing.

2011-10-24T13:17:34+00:00

add-brick, stop-volume, remove-brick are the operations that are explicitly
'failed' when attempted while replace-brick is in progress.

Change-Id: Ia8034177e03a9ab10f9e8d5c8bc417427a1218e3
BUG: 3738
Reviewed-on: http://review.gluster.com/631
Tested-by: Gluster Build System 
Reviewed-by: Amar Tumballi 
Reviewed-by: Vijay Bellur