glusterfs.git/xlators, branch v3.4.2qa4

mgmt/glusterd: Fix a memory leak in glusterd_is_local_addr()

2013-12-17T07:15:45+00:00

Change-Id: Id41d828e1cc56005f5e2a1e75b6d858703dd79c9
BUG: 1032122
Signed-off-by: Vijay Bellur 
Reviewed-on: http://review.gluster.org/6518
Reviewed-by: Lukáš Bezdička 
Tested-by: Gluster Build System

posix: if brick-uid or brick-gid is not specified, do not set

2013-12-17T07:15:24+00:00

Current code would set owner uid/gid explicitly to 0/0 on start
even if none was specified. Fix it.

Change-Id: I72dec9e79c51bd1eb3af5334c42b7c23b01d0258
BUG: 1040275
Signed-off-by: Anand Avati 
Reviewed-on: http://review.gluster.org/6476
Tested-by: Gluster Build System 
Tested-by: Lukáš Bezdička 
Reviewed-by: Krishnan Parthasarathi 
Reviewed-by: Vijay Bellur 
Reviewed-on: http://review.gluster.org/6519
Reviewed-by: Lukáš Bezdička

cluster/dht: Ignore decommissioned subvol in overlap optimization

2013-12-17T07:15:06+00:00

Change-Id: Ib727948c6e21b19fd509f258ff0aea1c5d1a84d1
BUG: 966845
Signed-off-by: shishir gowda 
Reviewed-on: http://review.gluster.org/5056
Reviewed-by: Amar Tumballi 
Reviewed-by: Jeff Darcy 
Tested-by: Gluster Build System 
Reviewed-on: http://review.gluster.org/6517
Reviewed-by: Shishir Gowda 
Reviewed-by: Vijay Bellur

cluster/dht: Fix anomaly check

2013-12-14T14:25:21+00:00

We were wrongly detecting holes/overlaps for already accounted
errors. Additionally, sort should also handle zero'ed out layout

Change-Id: Ic3d13e1d735b914f9acc01fe919bc90656baea48
BUG: 1003851
Signed-off-by: shishir gowda 
Reviewed-on: http://review.gluster.org/5762
Tested-by: Gluster Build System 
Reviewed-by: Amar Tumballi 
Reviewed-by: Anand Avati 
Reviewed-on: http://review.gluster.org/6469
Reviewed-by: Vijay Bellur

cluster/dht: Del GF_READDIR_SKIP_DIRS key from dict for first_up

2013-12-14T13:28:32+00:00

Currently, we sent GF_READDIR_SKIP_DIRS for all subvolumes if
first_subvol != first_up_subvolume.

Also first_up_subvolume can change with-in the life of a call and
cbk. Saving the first_up_subvol in dht_local for checks.

Back porting fix http://review.gluster.org/5577

BUG: 996474
Change-Id: I67b5bbe781e12812557b569b7d0a0beba4224159
Signed-off-by: shishir gowda 
Reviewed-on: http://review.gluster.org/6468
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

cluster/dht: Do migration inprog/complete check only if ENOENT

2013-12-14T07:37:13+00:00

Additionally, update op_errno to the lasted failure. If failures
found in complete_check, error returned would be EUCLEAN instead
of the right failure (in this case ENOENT)

Change-Id: Ib813867f4b817af651627b9ea07b0b09fa2b26ce
BUG: 966852
Original-author: shishir gowda 
Signed-off-by: Anand Avati 
Reviewed-on: http://review.gluster.org/6495
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

cluster/dht: set layout in inode ctx even if linkfile fails

2013-12-10T17:44:58+00:00

Creating linkfile could have failed, but we dont care about linkfile
for setting layout in the inode ctx (could be EEXIST etc.)

So ignore @inode in cbk and pick it up from local->loc.inode

Backporting  http://review.gluster.org/6319
BUG: 1032859

Change-Id: Ic95e303a4c060900d041820d4faa68d1c4685b6a
Original-author: Anand Avati 
Signed-off-by: shishir gowda 
Reviewed-on: http://review.gluster.org/6470
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

distribute: Rebalance should provide even disk space distribution

2013-12-10T12:14:57+00:00

Earlier disk space check had an issue which didn't
provide the needed functionality to avoid migration
when the destination had lesser available space,
scenario we need to avoid is stated below :

During rebalance `migrate-data` - Destination subvol experiences
a `reduction` in 'blocks' of free space, at the same time source
subvol gains certain 'blocks' of free space. A valid check is
necessary here to avoid errorneous move to destination where
the space could be scantily available.

This patch provides a proper fix in place by subtracting
necessary file blocks from destination and adding those blocks
to source.

backporting fix http://review.gluster.org/5961

BUG: 982919
Original-author: Harshavardhana 
Signed-off-by: shishir gowda 

Change-Id: If5808eaa89e66d7bcaeee7268fe3fe5b1b56f51d
Signed-off-by: shishir gowda 
Reviewed-on: http://review.gluster.org/6461
Reviewed-by: Harshavardhana 
Reviewed-by: Vijay Bellur 
Tested-by: Gluster Build System

cluster/afr: handle NULL check before strlen/strcmp in fgetxattr

2013-12-04T11:33:46+00:00

xattr name can legally be NULL. Handle that case without crashing.

Change-Id: Ie214cb05ccd52565dc247a9234ad83ae799d3866
BUG: 1036879
Signed-off-by: Anand Avati 
Reviewed-on: http://review.gluster.org/6420
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

glusterd: submit RPC requests without holding big lock

2013-12-04T08:35:23+00:00

If the endpoint of an RPC is not connected, the callback is called
synchronously within rpc_clnt_submit(). Since callbacks typically
hold the big lock, give up the big lock before calling rpc_clnt_submit
and acquire it freshly after the call.

Change-Id: Id89d8dd86c1a4012739ef4af7ea0935492b1a02b
BUG: 1037849
Signed-off-by: Anand Avati 
Reviewed-on: http://review.gluster.org/6414
Tested-by: Gluster Build System