glusterfs.git/xlators/cluster/stripe, branch v3.3.0qa39

core: coverity issues fixed

2012-04-23T21:52:57+00:00

this is not a complete set of issues getting fixed. Will
address other issues in another patch.

Change-Id: Ib01c7b11b205078cc4d0b3f11610751e32d14b69
Signed-off-by: Amar Tumballi 
BUG: 789278
Reviewed-on: http://review.gluster.com/3145
Tested-by: Gluster Build System 
Reviewed-by: Jeff Darcy

stripe: make sure we have complete set of subvolumes before making fop

2012-04-20T12:56:19+00:00

Change-Id: Ifc3b05183945a7ce3e25f8c777daaf3fd4aecff3
BUG: 810450
Signed-off-by: shishir gowda 
Reviewed-on: http://review.gluster.com/3190
Tested-by: Gluster Build System 
Reviewed-by: Amar Tumballi

locks, stripe: check if dict is NULL before refing it

2012-04-03T11:20:48+00:00

Change-Id: Ifab58504280ed92e4435f7805052745e56bef5ef
BUG: 808003
Signed-off-by: Raghavendra Bhat 
Reviewed-on: http://review.gluster.com/3073
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

cluster/stripe: minimum blk size log message

2012-03-30T07:42:44+00:00

Change-Id: I16ba2e17796cd91e89a6216d4981c09f1bcdc5cb
BUG: 807637
Signed-off-by: shylesh kumar 
Reviewed-on: http://review.gluster.com/3026
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

cluster/stripe- do not ref if dict is null

2012-03-28T09:59:51+00:00

Change-Id: Ifa4a5253482d837e2583de078e908b8c442f9dca
BUG: 807214
Signed-off-by: shylesh kumar 
Reviewed-on: http://review.gluster.com/3024
Tested-by: Gluster Build System 
Reviewed-by: Shishir Gowda

core: adding extra data for fops

2012-03-22T23:40:27+00:00

with this change, the xlator APIs will have a dictionary as extra
argument, which is passed between all the layers. This can be
utilized for overloading in some of the operations.

Change-Id: I58a8186b3ef647650280e63f3e5e9b9de7827b40
Signed-off-by: Amar Tumballi 
BUG: 782265
Reviewed-on: http://review.gluster.com/2960
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

cluster/stripe: fix {set/get}xattr query for dirs/symlink

2012-03-18T09:19:29+00:00

This patch fixes the following problems:
* ENOENT returned for getxattr (xtime) on symlinks
    Non-data entities are created only on the first subvolume but
    getxattr fop winds to all subvols. This results in all subvols
    except the first one to return ENOENT which is propogated down
    the stack.

* ENODATA returned for getxattr (xtime) on directory
    setxattr calls always wind to the first stripe subvolume.
    xtime getxattr invocation winds to all subvolumes but the
    xattr is present in only the first one, resulting in all
    subvols except the first one to return ENODATA.

Fix

  For symlinks getxattr now always winds to the first subvol and
  for directories setxattr sets the xattr on all subvols.

NOTE

   For directories the all-subvol-wind in setxattr is done
   only when request is from a special client (client-pid == -1)

Change-Id: I9236345ef319506770d2034b840ee4ac04704b21
BUG: 801394
Signed-off-by: Venky Shankar 
Reviewed-on: http://review.gluster.com/2948
Tested-by: Gluster Build System 
Reviewed-by: Shishir Gowda 
Reviewed-by: Anand Avati

mgmt/glusterd : volume set validation fixes

2012-03-18T07:52:40+00:00

This is the new version of the patch by Kaushik at review.gluster.com/699

The following new option types have been introduced:
 * GF_OPTION_TYPE_INTERNET_ADDRESS_LIST
 * GF_OPTION_TYPE_PRIORITY_LIST
 * GF_OPTION_TYPE_SIZE_LIST
and option types of several options in translators have been updated to use the
new types.

valid_internet_address(), valid_ipv4_address() & valid_ipv6_address() functions
has been updated for * wildcard matching. Previously used standalone wildcard
address checking functions have been removed.

Changes have been done to stripe translator to correctly set, update and use
stripe-blocksize. Also minimum value for block-size has been set to 16KB.

Change-Id: I2aa484ff695f6a915a8fc9a9f965cf0344f41d59
BUG: 765248
Signed-off-by: Kaushal M 
Reviewed-on: http://review.gluster.com/2899
Tested-by: Gluster Build System 
Reviewed-by: Shishir Gowda 
Reviewed-by: Anand Avati

cluster/stripe: Readdirp - send aggregated block_size in stat

2012-03-12T11:24:13+00:00

Change-Id: Iee83a355ab767acac89b11d0a5add8527fc387a7
BUG: 797308
Signed-off-by: shishir gowda 
Reviewed-on: http://review.gluster.com/2833
Tested-by: Gluster Build System 
Reviewed-by: Amar Tumballi

fops/removexattr: prevent users from removing glusterfs xattrs

2012-03-05T17:17:47+00:00

* Each xlator prevents the user from removing xlator-specific
  xattrs like trusted.gfid by handling it in respective removexattr
  functions.

* For xlators which did not define remove and fremovexattr,
  the functions have been implemented with appropriate checks.

xlator         |     fops-added
_______________|__________________________
               |
1. stripe      |     removexattr and fremovexattr
2. quota       |     removexattr and fremovexattr

Change-Id: I98e22109717978134378bc75b2eca83fefb2abba
BUG: 783525
Signed-off-by: Rajesh Amaravathi 
Reviewed-on: http://review.gluster.com/2836
Tested-by: Gluster Build System 
Reviewed-by: Amar Tumballi 
Reviewed-by: Jeff Darcy