glusterfs.git/xlators/features/quota/src/quota.c, branch v3.8.5

quota: change the log level from warning to info

2016-08-24T10:52:41+00:00

This is a backport of http://review.gluster.org/#/c/14940/

> Change-Id: Icac7d87bb6c5ecc813528690e03c821bbe3789b5
> BUG: 1357463
> Signed-off-by: Manikandan Selvaganesh 
> Reviewed-on: http://review.gluster.org/14940
> CentOS-regression: Gluster Build System 
> Smoke: Gluster Build System 
> Reviewed-by: Prashanth Pai 
> NetBSD-regression: NetBSD Build System 
> Reviewed-by: Vijay Bellur 

Change-Id: I4c4afb0d32b3bce764a4eb88f8d403f4dad10d8f
BUG: 1368927
Signed-off-by: Manikandan Selvaganesh 
Reviewed-on: http://review.gluster.org/15229
Smoke: Gluster Build System 
CentOS-regression: Gluster Build System 
NetBSD-regression: NetBSD Build System 
Reviewed-by: Raghavendra G

quota : rectify quota-deem-statfs default settings

2016-06-15T11:22:40+00:00

Backport of http://review.gluster.org/#/c/14708/

When quota is enabled, quota-deem-statfs is on by default.
This is not updated in gluster v set help command. With, the
patch, the description of deem statfs command is updated.

> Change-Id: I75c47c4a0f0eb898b5e32efe7f0a6a558d7f8007
> BUG: 1345846
> Signed-off-by: Manikandan Selvaganesh 
> Reviewed-on: http://review.gluster.org/14708
> Smoke: Gluster Build System 
> NetBSD-regression: NetBSD Build System 
> CentOS-regression: Gluster Build System 
> Reviewed-by: Raghavendra G 

Change-Id: I75c47c4a0f0eb898b5e32efe7f0a6a558d7f8007
BUG: 1346134
Signed-off-by: Manikandan Selvaganesh 
Reviewed-on: http://review.gluster.org/14726
Smoke: Gluster Build System 
NetBSD-regression: NetBSD Build System 
CentOS-regression: Gluster Build System 
Reviewed-by: Raghavendra G

quota : fix null dereference issues in quota

2016-04-20T17:39:20+00:00

Change-Id: I3805b206077718da26adbeb8b29a53642e00886f
BUG: 1328696
Signed-off-by: Manikandan Selvaganesh 
Reviewed-on: http://review.gluster.org/14022
Smoke: Gluster Build System 
NetBSD-regression: NetBSD Build System 
CentOS-regression: Gluster Build System 
Reviewed-by: Jeff Darcy

quota: check inode limits only when new file/dir is created

2016-04-06T09:25:23+00:00

When a inode limit is full, writes to any existing file
fails with disk quota exceed even if usage limit is
not set or usage limit is not full.

BUG: 1323486
Change-Id: I9679fe26a2839ade0b1541fa7f0a2b71ac6dcc31
Signed-off-by: vmallika 
Reviewed-on: http://review.gluster.org/13911
Smoke: Gluster Build System 
NetBSD-regression: NetBSD Build System 
CentOS-regression: Gluster Build System 
Reviewed-by: Manikandan Selvaganesh

marker: build_ancestry in marker

2016-04-06T08:12:31+00:00

* quota-enforcer doesn't execute build_ancestry in the below
  code path
    1) Special client (PID < 0)
    2) unlink
    3) rename within the same directory
    4) link within the same directory

    In these cases, marker accounting can fail as parent not found.
    We need to build_ancestry in marker if it doesn't find parent
    during update txn

Change-Id: Idb7a2906500647baa6d183ba859b15e34769029c
BUG: 1320818
Signed-off-by: vmallika 
Reviewed-on: http://review.gluster.org/13857
CentOS-regression: Gluster Build System 
Reviewed-by: Raghavendra G 
Tested-by: Raghavendra G 
NetBSD-regression: NetBSD Build System 
Smoke: Gluster Build System

storage/posix: send proper iatt attributes for the root inode

2016-03-26T16:33:48+00:00

* changes in posix to send proper iatt attributes for the root directory
  when ancestry is built. Before posix was filling only the gfid and the
  inode type in the iatt structure keeping rest of the fields zeros. This
  was cached by posix-acl and used to send EACCES when some fops came on
  that object if the uid of the caller is same as the uid of the object on
  the disk.

* getting and setting inode_ctx in function 'posix_acl_ctx_get' is not atomic
  and can lead to memory leak when there are multiple looups for an
  inode at same time. This patch fix this problem

* Linking an inode in posix_build_ancestry, can cause a race in
  posix_acl.
  When parent inode is linked in posix_build_ancestry, and before
  it reaches posix_acl_readdirp_cbkc, reate/lookup can
  come on a leaf-inode, as parent-inode-ctx not yet updated
  in posix_acl_readdirp_cbk, create/lookup can fail
  with EACCESS. So do the inode linking in the quota xlator

Change-Id: I3101eefb65551cc4162c4ff2963be1b73deacd6d
BUG: 1320818
Signed-off-by: Raghavendra Bhat 
Reviewed-on: http://review.gluster.org/13730
Tested-by: Vijaikumar Mallikarjuna 
Smoke: Gluster Build System 
NetBSD-regression: NetBSD Build System 
CentOS-regression: Gluster Build System 
Reviewed-by: Raghavendra G

quota: limit xattr for subdir not healed on newly added bricks

2015-12-29T16:28:03+00:00

DHT after creating missing directory, tries to heal the
xattrs. This xattrs operation fails as INTERNAL FOP key was not set

Change-Id: I819d373cf7073da014143d9ada908228ddcd140c
BUG: 1294479
Signed-off-by: vmallika 
Reviewed-on: http://review.gluster.org/13100
Reviewed-by: Susant Palai 
Tested-by: Gluster Build System 
Reviewed-by: Raghavendra G 
Tested-by: NetBSD Build System

quota: add version to quota xattrs

2015-11-02T08:54:35+00:00

When a quota is disable and the clean-up process terminated
without completely cleaning-up the quota xattrs.
Now when quota is enabled again, this can mess-up the accounting

A version number is suffixed for all quota xattrs and this version
number is specific to marker xaltor, i.e when quota xattrs are
requested by quotad/client marker will remove the version suffix in the
key before sending the response

Change-Id: I1ca2c11460645edba0f6b68db70d476d8d26e1eb
BUG: 1272411
Signed-off-by: vmallika 
Reviewed-on: http://review.gluster.org/12386
Tested-by: NetBSD Build System 
Reviewed-by: Krishnan Parthasarathi 
Tested-by: Gluster Build System 
Reviewed-by: Manikandan Selvaganesh 
Reviewed-by: Raghavendra G

quota: fix crash in quota_fallocate

2015-10-09T04:00:12+00:00

list head was not initialized and brick
was crashing with fallocate.

This patch fixes the issue

Change-Id: I9757b88eab61054892f0fe3de63af2683cd4fef7
BUG: 1269754
Signed-off-by: vmallika 
Reviewed-on: http://review.gluster.org/12314
Reviewed-by: Manikandan Selvaganesh 
Tested-by: NetBSD Build System 
Reviewed-by: Raghavendra G

quota: use copy_frame when creating new frame during quota_check_limit

2015-10-07T04:04:53+00:00

DHT re-balance, sets frame root PID < 0 and quota_check_limit skips
enforcement if this PID is less than 0.

When creating new frame for quota_check_limit we need to use
copy_frame instead of create_frame, so that all auth information
are copied from original frame.

Change-Id: Ib3b4a3744f8b0d72a8bc32826f6edae836d6faed
BUG: 1267812
Signed-off-by: vmallika 
Reviewed-on: http://review.gluster.org/12265
Tested-by: NetBSD Build System 
Tested-by: Gluster Build System 
Reviewed-by: Raghavendra G