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

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

iobuf: add a counter to keep track of request misses

2012-03-12T12:32:06+00:00

requests can be denied if the size of the request is higher than the
available sizes.

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

protocol/client: Register a timer(grace-timer) conditionally.

2012-03-10T13:27:56+00:00

A grace timer is registered on a disconnect, but a reconnect timer sends a
connect request after every 3sec and if the server is down, the client protocol
receives disconnect and a grace timer will be registered which on timeout will
increase the lk-version value. Its enough to register the grace timer once after
the disconnect and later just ignore other psuedo disconnects.

Change-Id: I36a153aa86b350d87fe50d014ee0297f558a7fb6
BUG: 795386
Signed-off-by: Mohammed Junaid 
Reviewed-on: http://review.gluster.com/2906
Tested-by: Gluster Build System 
Reviewed-by: Amar Tumballi 
Reviewed-by: Vijay Bellur

syncop: Make syntask scalable

2012-03-10T06:19:32+00:00

At the moment, synctask uses task->frame to perform
all the syncops, this will lead to high-memory usage if the task
crawls millions of directories. i.e millions of STACK_WINDS/UNWINDS.
To prevent this, in each task a new stack is created to perform
the fops which is reset after every syncop.

Change-Id: I53c262ec348be9b1d91af73da01f1c217f31ce6e
BUG: 798907
Signed-off-by: Pranith Kumar K 
Reviewed-on: http://review.gluster.com/2850
Tested-by: Gluster Build System 
Reviewed-by: Jeff Darcy

features/marker: GFID file handle based backend related fixes

2012-03-08T05:12:41+00:00

* make sure loc->gfid is filled.

Change-Id: I5304e650d9fc181ee1f3b65be7f2ff3847d9722e
BUG: 790389
Signed-off-by: Raghavendra G 
Reviewed-on: http://review.gluster.com/2888
Tested-by: Gluster Build System 
Reviewed-by: Raghavendra Bhat 
Reviewed-by: Vijay Bellur

libglusterfs/fd: fixed fd_anonymous() leak

2012-03-07T18:39:50+00:00

fd was getting leaked whenever there was a overlapping operations,
which caused memory leak, and process fd leaks, which made most of
the operations on NFS mount of a replicate volume not work. With
the fix, things are back to normal.

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

protocol/client: Added lk_ctx info in fdctx dump

2012-03-07T18:17:41+00:00

- Added a brief explanation as to why we can't use gf_log
  when in statedump.

- Removed gf_log messages from client_priv_dump since
  it can cause a 'deadlock' - See statedump.c for explanation

- Added try-lock based accessors for fd_lk_list for dump purposes.

Change-Id: I1d755a4ef2c568acf22fb8c4ab0a33a4f5fd07b4
BUG: 789858
Signed-off-by: Krishnan Parthasarathi 
Reviewed-on: http://review.gluster.com/2882
Tested-by: Gluster Build System 
Reviewed-by: Amar Tumballi 
Reviewed-by: Vijay Bellur

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