glusterfs.git/libglusterfs/src, branch v3.3.0qa29

libglusterfs/dict: Add boundary conditions

2012-03-18T09:20:28+00:00

Functions modified:
1. data_to_int16()
2. data_to_uint16()

Change-Id: I6775ca5d39f519c24ca3a07ce41018472e238cdf
BUG: 769692
Signed-off-by: Harshavardhana 
Reviewed-on: http://review.gluster.com/2599
Tested-by: Gluster Build System 
Reviewed-by: Amar Tumballi 
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

Logs: Improved logs in lock/unlock execution path

2012-03-18T08:53:41+00:00

Statedump will now start showing the lk-owner of the stack.

Change-Id: I9f650ce9a8b528cd626c8bb595c1bd1050462c86
BUG: 803209
Signed-off-by: Pranith Kumar K 
Reviewed-on: http://review.gluster.com/2968
Tested-by: Gluster Build System 
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

mem-pool: more debug friendly logs by using gf_log_callingfn()

2012-03-18T07:02:57+00:00

Change-Id: I4d6e53ad8660314315f2ff3aa92ffbe707d6efe9
Signed-off-by: Amar Tumballi 
BUG: 803638
Reviewed-on: http://review.gluster.com/2951
Tested-by: Gluster Build System 
Reviewed-by: Jeff Darcy 
Reviewed-by: Anand Avati

core: bring a cmdline option to set memory-accounting

2012-03-15T10:33:18+00:00

currently this is implemented as a command line option, and not
as an easier translator option. this is because as of now, before
even the volume files are parsed, we would need memory accounting
enabled. there is scope for improving this behavior, but for now,
this approach solves the problem.

Also, this feature's major consumers are the testers who are
looking for leaks, hence option is hidden from usage output.

Change-Id: I09a5b13743ae43ff42c251989f921319e94cabe3
Signed-off-by: Amar Tumballi 
BUG: 799199
Reviewed-on: http://review.gluster.com/2856
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

core: dict allocations through mem-pool

2012-03-15T05:15:25+00:00

Change-Id: I7401639060957d437808779745a1e46c3f9f4585
Signed-off-by: Amar Tumballi 
BUG: 798503
Reviewed-on: http://review.gluster.com/2851
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

cluster/afr: handle sending NULL dentry name for inode link in self-heal-daemon

2012-03-14T10:53:35+00:00

* Without the dentry name, dentry cannot be created in inode_link, which
 leads to trying to access the null dentry to check if it is cyclic and
 thus segfault. So send the parent inode also NULL, which just returns
 the proper inode after assigning the gfid and type to the inode without
 trying to create dentry.

* Handle failures such as dentry_create returning NULL, in inode_link
  properly and return NULL in such cases.

* Increase the lru limit of inode table of self-heal-daemon to 2048

Change-Id: I7ae0e0e9be279d1694b6aafb5e054585e43f03ff
BUG: 801149
Signed-off-by: Raghavendra Bhat 
Reviewed-on: http://review.gluster.com/2893
Tested-by: Gluster Build System 
Reviewed-by: Amar Tumballi 
Reviewed-by: Vijay Bellur

core, cli : Changes to statedump

2012-03-14T09:57:49+00:00

This patch,
* Enables missing inodectx, fdctx dumpops for cli statedump command.
* Reintroduce changes introduced in statedump.c by 489a7a10 which were
  overwritten by db20a0f8.
* Change strncasecmp() in gf_proc_dump_parse_set_option() to strcasecmp()
  to properly parse similar options like fd-fdctx, inode-inodectx

Change-Id: I85ad854dce73048617b933d29cf360af9d0ed90a
BUG: 797788
Signed-off-by: Kaushal M 
Reviewed-on: http://review.gluster.com/2914
Tested-by: Gluster Build System 
Reviewed-by: Amar Tumballi

core: Made lkowner_unparse use 'type-safe' format-specifier

2012-03-12T14:59:50+00:00

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