glusterfs.git, branch v3.3.0qa12

build: Solaris build fixes

2011-09-27T12:44:41+00:00

Change-Id: Ic47d07a199ecbc42a141e391bbc75096e270e404
BUG: 3446
Reviewed-on: http://review.gluster.com/503
Tested-by: Gluster Build System 
Reviewed-by: Amar Tumballi

glusterd: Should not cleanup peerinfo too soon.

2011-09-27T12:44:09+00:00

friend_remove_cbk cleans up peerinfo and the unrefs the associated rpc_clnt
obj. When the cbk is run inside call_bail or saved_frames_unwind, we might end
up destroying the rpc_clnt and associated saved_frames_pool while we are still
using saved_frames to iterate through the frames.

Change-Id: Idf7768478a6d07a87c7faeac5b70e13bcacd2641
BUG: 3511
Reviewed-on: http://review.gluster.com/510
Tested-by: Gluster Build System 
Reviewed-by: Amar Tumballi 
Reviewed-by: Vijay Bellur

storage/posix : prevent unmount of underlying fs

2011-09-27T12:40:57+00:00

posix xlator now performs opendir () on the brick directory during init ().
This will prevent the underlying filesystem mounted to that directory from being
unmounted.

Change-Id: I02c190ab8a91abc4ab06959b36f50e0a3fa527ae
BUG: 3578
Reviewed-on: http://review.gluster.com/509
Reviewed-by: Amar Tumballi 
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

glusterd: support for resetting single option

2011-09-27T12:38:36+00:00

Allows resetting of only single options using 'volume reset' command.
New syntax of volume reset is: 'volume reset [option] [force]'.
Giving "all" as options or not specifying an option, causes all options
to be reset.

Change-Id: Ib9e220f326adeb1be1a774737a0b12c910012cea
BUG: 2980
Reviewed-on: http://review.gluster.com/450
Tested-by: Gluster Build System 
Reviewed-by: Amar Tumballi

cluster/distribute: fixed a spurious inode ref

2011-09-27T12:37:59+00:00

While bringing in support to open-fd migration, dht_local_init()
itself started doing 'loc_copy()'. I had left one case in dht_lookup()
where there is a extra loc_copy() on existing copied 'local->loc',
which would cause 2 inode_refs on a given inode, and only one
inode_unref() happens in dht_local_wipe().

Change-Id: Idd0375bdf9a6408db1e97e80389249ef8d802adb
BUG: 3590
Reviewed-on: http://review.gluster.com/504
Tested-by: Gluster Build System 
Reviewed-by: Shishir Gowda

glusterd: more checks before starting rebalance

2011-09-27T12:37:36+00:00

Checks if specified volume is of a distribute type and has more than one
brick before attempting to start rebalance.

Change-Id: I9a3405019e7af4a7d7e162b0dc054bf9a99364e7
BUG: 3561
Reviewed-on: http://review.gluster.com/501
Tested-by: Gluster Build System 
Reviewed-by: Amar Tumballi

glusted: while doing add-brick, handle distribute volume separately

2011-09-27T12:35:03+00:00

so the logic of different type of counts etc won't come into action,
which would fail the add-brick operation.

Change-Id: I5c8769d6d1c2433ffb084e5dce9c1e01678cff40
BUG: 3630
Reviewed-on: http://review.gluster.com/505
Tested-by: Gluster Build System 
Reviewed-by: Krishnan Parthasarathi

rpc: fix wrong wiping of state machine's state

2011-09-25T13:15:12+00:00

The macro __socket_proto_read() is used to read one sub-header in a fragment
after another. It also has to 'get out' of the state machine's
switch/case construct in two situations -

a) finished reading the full header

b) after a partial read, (and return in the same state at next pollin)

The 'reset' of the state should happen only after a full read of the header
and not if there was a partial read of the header.

Change-Id: I3650a83e1fae0f317cfd1b549835c72e39dc5253

glusterd: cleanup of volinfo '*_count' definitions

2011-09-23T13:48:32+00:00

earlier, sub_count was having different meaning depending on the
volume type.

now, for replica and stripe count, one can directly access the
'replica_count' or 'stripe_count' to get the corresponding
value from the volume info. 'sub_count' is preserved as is for backward
compatibility. there is a new variable 'dist_leaf_count' to get
info about how many bricks are present in one distribute sub volume.

Change-Id: I5ea1c8f9ae08f584cca63b91ba69035c7e4350ca
BUG: 3158
Reviewed-on: http://review.gluster.com/435
Reviewed-by: Krishnan Parthasarathi 
Tested-by: Gluster Build System

glusterd-volgen: prevent a crash when volume type is stripe-replicate

2011-09-23T13:31:23+00:00

when volgen tries to check for decommissioned nodes, a check for
stripe replicate volume was missed

Change-Id: Ie3aa97da2ec0d94fcf65f96bb4006b3fb54f59dc
BUG: 3616
Reviewed-on: http://review.gluster.com/494
Tested-by: Gluster Build System 
Reviewed-by: Krishnan Parthasarathi