glusterfs.git, branch v3.3.0qa18

cluster/afr: Set pargfid when missing

2011-12-22T13:18:11+00:00

client asserts for missing pargfid in case of unlink. So
Afr needs to make sure it is present in that fop.

Change-Id: Iea0ad65e1e7254c8df412942c52d5870e853aa51
BUG: 769055
Signed-off-by: Pranith Kumar K 
Reviewed-on: http://review.gluster.com/2495
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

libglusterfs: set inode path to be NULL in case of errors

2011-12-22T13:14:40+00:00

Some of the functions calling inode_path, __inode_path
are assuming the path to be set to NULL in case of errors.
Instead of fixing it in the calling functions, fixing it in
the __inode_path function.

Change-Id: I77736a2700d3c2c9732a536bcf2a398fe626d54e
BUG: 765430
Signed-off-by: Pranith Kumar K 
Reviewed-on: http://review.gluster.com/810
Tested-by: Gluster Build System 
Reviewed-by: Jeff Darcy 
Reviewed-by: Vijay Bellur

protocol/server: Do connection cleanup if reply fails

2011-12-22T13:13:08+00:00

We observed that after the first connection cleanup happens on
DISCONNECT the lock calls in transit are granted or added in
blocked locks queue. These locks were never cleaned up after that
because no unlock would come up on that connection. This would
leave references on that transport so it would never be destroyed.
Now, the connection cleanup happens whenever the reply
submission fails.

Also cleaned up the old code which is not used any more.

Change-Id: Ie4fe6f388ed18d9c907cf8ae06b0b7fd0601a660
BUG: 765430
Signed-off-by: Pranith Kumar K 
Reviewed-on: http://review.gluster.com/809
Tested-by: Gluster Build System 
Reviewed-by: Jeff Darcy 
Reviewed-by: Vijay Bellur

features/locks: Fix release of locks on disconnect

2011-12-22T13:11:25+00:00

When a disconnect comes on a transport, all the locks on that
transport should be cleared irrespective of entry presence.

Change-Id: I1c432ec1ddc8fe3f7bf236992f47437d0dacd6ae
BUG: 765430
Signed-off-by: Pranith Kumar K 
Reviewed-on: http://review.gluster.com/808
Tested-by: Gluster Build System 
Reviewed-by: Jeff Darcy 
Reviewed-by: Vijay Bellur

cluster/afr: Fix meta data lock range

2011-12-22T13:10:15+00:00

Change-Id: I7615f31309c6c8f5373e1ff0535d84396dfa1455
BUG: 765430
Signed-off-by: Pranith Kumar K 
Reviewed-on: http://review.gluster.com/807
Tested-by: Gluster Build System 
Reviewed-by: Jeff Darcy 
Reviewed-by: Vijay Bellur

transport/rdma: Add attr_timeout, attr_retry_cnt, attr_rnr_retry

2011-12-20T17:53:17+00:00

as configurable options

Change-Id: Ifc4710f149be35979746ddfbfb4181638601bc64
BUG: 766040
Signed-off-by: Harshavardhana 
Reviewed-on: http://review.gluster.com/766
Tested-by: Gluster Build System 
Reviewed-by: Jeff Darcy 
Reviewed-by: Vijay Bellur

glusterd: Fixed spurious checks in add_brick.

2011-12-20T05:42:30+00:00

Previous algo,
 - didn't allow volume type change from plain distribute
   to distribute-[stripe][-][replicate].
 - performed 'add-brick' on temporary volinfo without correct
   initialisations. This was superflous.

Change-Id: If5d5d78e59bd04685df9c27974f715df5f0b43f3
BUG: 765774
Signed-off-by: Krishnan Parthasarathi 
Reviewed-on: http://review.gluster.com/804
Tested-by: Gluster Build System 
Reviewed-by: Amar Tumballi

Fix local==NULL crash in wb_sync_cbk during disconnect.

2011-12-20T05:40:32+00:00

Change-Id: I26dc48a85756e189b1ef5cfef1658f9c2aed2157
BUG: 767359
Signed-off-by: Jeff Darcy 
Reviewed-on: http://review.gluster.com/784
Tested-by: Gluster Build System 
Reviewed-by: Amar Tumballi 
Reviewed-by: Vijay Bellur

cli: validate the volume set command properly

2011-12-19T16:58:50+00:00

For volume set command if after the volume name the key and the
value of the option are not given, then gracefully exit by
showing the proper usage of volume set, instead of sending the
request to glusterd, which makes glusterd crash.

Signed-off-by: Raghavendra Bhat 
Change-Id: I2f0d189a55663c7f47dddff35d4dc68fae16b755
BUG: 767591
Signed-off-by: Raghavendra Bhat 
Reviewed-on: http://review.gluster.com/797
Tested-by: Gluster Build System 
Reviewed-by: Krishnan Parthasarathi 
Reviewed-by: Jeff Darcy

cluster/stripe: copy the gfid from the dentry structure to loc

2011-12-19T16:58:16+00:00

In stripe_readdirp_cbk we used to send stat call on each of the entry
we have got from readdirp, by creating a new inode (but without
setting the gfid in loc). Protocol/client asserts if it cannot find
gfid in either loc or loc->inode. This patch copies the gfid of the
entry from the stat structure to loc.

Change-Id: I35228227e943f3b01fac720d6c587764c47e6869
BUG: 768840
Signed-off-by: Raghavendra Bhat 
Reviewed-on: http://review.gluster.com/806
Reviewed-by: Amar Tumballi 
Tested-by: Gluster Build System 
Reviewed-by: Jeff Darcy