glusterfs.git/xlators/features, branch v3.2.5qa3

locks: perform inode_ref of pl_update_refkeeper outside locked region

2011-10-27T10:47:54+00:00

Performing inode_ref inside the critical section of pl_update_refkeeper
(around pl_inode_t->mutex) causes a deadlock with inode_table_t->lock.
The other thread (process state dump) holds the inode_table_t->lock first
to loop over inodes and then while dumping an inode's lock context, tries
to acquire pl_inode_t->mutex thereby completing the deadlock.

The fix is to perform inode_ref outside the critical section in
pl_update_refkeeper

Change-Id: I689ab4a9b46b36287740279fdec6159182c86119
BUG: 3753
Reviewed-on: http://review.gluster.com/629
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

features/marker-quota: Donot log when inode or inode-ctx is NULL.

2011-10-27T09:57:34+00:00

When running rm -rf from mount point, quota-updation process may endup in a state where the
inode is already removed from the gluster inode table, but the updation has process is still in
progress. This is not a error condition because the updation is done asynchronously.

Change-Id: I2d3aad716bb1c971115004a70ca76e2dd5088e44
BUG: 3609
Reviewed-on: http://review.gluster.com/603
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

features/marker-quota: Return error when inode-contribution not present in inode_ctx.

2011-09-30T07:13:18+00:00

Also, create the inode_contribution node even before checking for dirty and xattr not present in mq_inspect_directory_xattr function.

Change-Id: Ie5b6e6ba8d5abcfc6aeb9b192720ac10aac1319e
BUG: 3624
Reviewed-on: http://review.gluster.com/538
Tested-by: Gluster Build System 
Reviewed-by: Raghavendra G

Revert "cluster/afr: eager locking of FD writes"

2011-09-22T15:42:01+00:00

This reverts commit 81456ec2dfb312ae60c5c4e6f960a3cbf8aaaa4c.

Change-Id: Id03335117f5137f5d09781850bf4fba6eca0f73d
Reviewed-on: http://review.gluster.com/492
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

features/marker: Use appropriate loc struct to do removexattr on newpath after rename.

2011-09-20T08:41:01+00:00

Change-Id: Idb31e845bc876f46b476d8fa769d67d8db89e4a1
BUG: 3493
Reviewed-on: http://review.gluster.com/389
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

features/quota: explicitly create xattrs in marker_create_cbk

2011-09-20T06:09:32+00:00

 - the earlier approach of creating quota related xattrs through
   side-effect of updating size and contribution values won't work,
   since when no contribution xattr is present, the updation process
   treats contribution value as zero and hence will be equal to size
   of freshly created files

Change-Id: If9b2063b1ac3a4cf50d3fe2c81e907bc8eccb677
BUG: 3531
Reviewed-on: http://review.gluster.com/383
Tested-by: Gluster Build System 
Reviewed-by: Mohammed Junaid

features/quota: implement mknod fop.

2011-09-20T06:09:18+00:00

Change-Id: If8f2a0bb635160ee78f35787ee9f8a4db87ae8ac
BUG: 3531
Reviewed-on: http://review.gluster.com/382
Tested-by: Gluster Build System 
Reviewed-by: Mohammed Junaid

features/marker-quota: Perform xattr related operations with root permissions in rename fop.

2011-09-12T12:23:06+00:00

Change-Id: Id9ac1ecdd9753377c9eb24464f51dcbdc0cd2821
BUG: 3194
Reviewed-on: http://review.gluster.com/366
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

features/marker-quota: Prefix the function names with mq (marker-quota).

2011-09-12T10:30:52+00:00

This is to fix to bug marker translator and quota translator cannot co-exist in same process.

Change-Id: Ie0419e1a66d300e163d27b21441ad8d8139b6097
BUG: 30203020
Reviewed-on: http://review.gluster.com/393
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

memleak fixes for marker

2011-09-12T04:14:21+00:00

Change-Id: I33623f874fbf3a3c491322877ed54e99d3cfc623
BUG: 2784
Reviewed-on: http://review.gluster.com/298
Tested-by: Gluster Build System 
Reviewed-by: Amar Tumballi 
Reviewed-by: Vijay Bellur