glusterfs.git, branch v3.4.1qa1

glusterd: Allow bumping down a peer's op-version during probe

2013-09-10T16:09:18+00:00

  Backport of http://review.gluster.org/5715

Earlier, a peer running a higher op-version couldn't be probed into a
cluster running at a lower op-version. This created issues when trying
to expand an upgraded cluster. This patch changes this behaviour.

The cluster no longer rejects a peer being probed if its op-version is
higher than the cluster op-version. The peer will reduce its op-version
if it doesn't have any volumes. If the peer contains volumes and needs
to reduce its op-version, it fails the handshake and the probe fails.

BUG: 1005038
Change-Id: Iabe790a9f826a4ac63d379eeeba01efcfef01f4d
Signed-off-by: Kaushal M 
Reviewed-on: http://review.gluster.org/5834
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

cluster/dht: Ignore subvols with error in min-free-disk/inodes

2013-09-10T16:05:18+00:00

Currently when selecting a alternative subvolume when hashed
subvol has exceeded min-free-disk/inodes, we do not check if
layouts have errors (including decommissioning). This leads
to data being written to those subvolumes, and in case of
decommissioning, will lead to data loss.

BUG: 982919

> Original-Author: shishir gowda 
> Reviewed-on: http://review.gluster.org/5299

Change-Id: If301a86cf3ca5fad6529bd2e61382f9901663ba0
Signed-off-by: Amar Tumballi 
Reviewed-on: http://review.gluster.org/5888
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

rpc: fix typo which refers glibc macro

2013-09-10T08:25:23+00:00

A typo which read MAX_AUTH_BYTES instead of GF_MAX_AUTH_BYTES was
picking the value 400 instead of the larger 2048. This causes
failures when number of aux group ids is a large number.

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

gfapi: fix for not able to rename folders

2013-09-10T08:24:58+00:00

If oldname is being renamed to newname, we need to
check for type of newname only if newname exists.

Change-Id: I068a283f9ffe67fcd5e8754d6bf052a2339efbf2
BUG: 953694
Signed-off-by: Raghavendra Talur 
Reviewed-on: http://review.gluster.org/5886
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

gfapi: fix glfs_readlink()

2013-09-10T08:24:37+00:00

glfs_readlink() is supposed to memcpy the buffer pointer
returned by syncop_readlink(). Fix it.

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

gfapi: perform open() on pre-existing inode in glfs_creat()

2013-09-10T08:23:17+00:00

Performing syncop_create() on a pre-existing file can result
in opening the linkfile if DHT has a linkfile for the file.
This is because dht_create() will perform the op on the
hashed_subvol() and overwrite the layout in dht_create_cbk.

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

performance/write-behind: invoke request queue processing if

2013-09-10T08:22:49+00:00

we find fd marked bad while trying to fulfill lies.

* flush was queued behind some unfulfilled write.
* A previously wound write returned an error and hence fd was marked
  bad with corresponding error.
* wb_fulfill_head (invocation probably rooted in wb_flush), before
  winding checks for failures of previous writes and since there was a
  failure, calls wb_head_done without even winding one request in head.
* wb_head_done unrefs all the requests in list "head".
* since flush was last operation on fd (and most likely last operation
  on inode itself), no one invokes wb_process_queue and flush is stuck
  in request queue for eternity.

Change-Id: I3b5b114a1c401d477dd7ff64fb6119b43fda2d18
BUG: 988642
Signed-off-by: Raghavendra G 
Reviewed-on: http://review.gluster.org/5883
Tested-by: Gluster Build System

libgfapi : Fix for hardlink creation on SMB mount

2013-09-10T08:22:14+00:00

Previously the inode of the hard link was left uninitialised hence
glfs_loc_link() used to fail with invalid parameter.Since inode is
same for both the files in hard link creation,   updated the inode
field of hard link to be same as that of the original file.

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

glusterd: release big locks while doing mount

2013-09-10T08:21:14+00:00

Else things can deadlock in getspec v/s glusterd_do_mount()

Change-Id: Ie70b43916e495c1c8f93e4ed0836c2fb7b0e1f1d
BUG: 997576
Signed-off-by: Anand Avati 
Reviewed-on: http://review.gluster.org/5881
Reviewed-by: Krishnan Parthasarathi 
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

mount/fuse: save the basefd flags in the new fd

2013-09-10T08:20:58+00:00

Upon graph switch, the basefd's flags were not saved in the new fd created
for the new graph upon which all the further requests for the open file
would come. Thus posix was treating the fd as a read-only fd and was
denying the write on the fds.

Change-Id: I781b62b376a85d1a938c091559270c3f242f1a2a
BUG: 998352
Signed-off-by: Raghavendra Bhat 
Reviewed-on: http://review.gluster.org/5880
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur