glusterfs.git/xlators/features, branch v3.6.5

features/quota: prevent statfs frame-loss when an error happens during

2015-08-20T09:04:42+00:00

ancestry building.

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

We do quota_build_ancestry in function 'quota_get_limit_dir',
suppose if quota_build_ancestry fails, then we don't have a
frame saved to continue the statfs FOP and client can hang.

> Change-Id: I92e25c1510d09444b9d4810afdb6b2a69dcd92c0
> BUG: 1178619
> Signed-off-by: Raghavendra G 
> Signed-off-by: vmallika 
> Reviewed-on: http://review.gluster.org/9380
> Tested-by: Gluster Build System 

Change-Id: Ia25cf738250fdc2c766f96c26e3c31093d534aba
BUG: 1247959
Signed-off-by: vmallika 
Reviewed-on: http://review.gluster.org/11790
Tested-by: Gluster Build System 
Reviewed-by: Raghavendra Bhat 
Reviewed-by: Raghavendra G

quota: fix mem leak in quota enforcer

2015-08-18T10:39:05+00:00

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

Do inode_unref on parent

> Change-Id: I21d82eb8716dd73aa2dc291b3ae8506e4fb4ea8b
> BUG: 1207735
> Signed-off-by: vmallika 
> Reviewed-on: http://review.gluster.org/11526
> Reviewed-by: Krishnan Parthasarathi 
> Tested-by: Gluster Build System 
> Tested-by: NetBSD Build System 
> Reviewed-by: Raghavendra G 
> Tested-by: Raghavendra G 

Change-Id: I8f103ea96e6853d48c934b791ff4d95b226e4453
BUG: 1247964
Signed-off-by: vmallika 
Reviewed-on: http://review.gluster.org/11793
Tested-by: Gluster Build System 
Reviewed-by: Raghavendra Bhat

quota: Validate NULL inode from the entries received in readdirp_cbk

2015-05-21T09:17:54+00:00

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

> In quota readdirp_cbk, inode ctx filled for the all entries
> received.
> In marker readdirp_cbk, files/directories are inspected for
> dirty

> There is no guarantee that entry->inode is populated.
> If entry->inode is NULL, this needs to be treated as readdir
>
> Change-Id: Id2d17bb89e4770845ce1f13d73abc2b3c5826c06
> BUG: 1215550
> Signed-off-by: vmallika 
> Reviewed-on: http://review.gluster.org/10416
> Tested-by: NetBSD Build System
> Tested-by: Gluster Build System 
> Reviewed-by: Raghavendra G 
> Tested-by: Raghavendra G 

Change-Id: Iace5a53efac58ab5a0a82bcfddfff6fb2a5b5068
BUG: 1217423
Signed-off-by: vmallika 
Reviewed-on: http://review.gluster.org/10470
Tested-by: Gluster Build System 
Reviewed-by: Raghavendra G 
Reviewed-by: Raghavendra Bhat

features/glupy: fix file locations

2015-05-19T13:00:23+00:00

Something about the reconfiguration of our test machines to accommodate
packaging changes in master had the side effect of breaking 3.6
regression tests.  This patch, based on 10616 in master, brings the 3.6
paths in line with master, so if everything's installed correctly for
master than it will work for 3.6 as well.

Change-Id: Icc64a32b6c7e95ff84235be7bef5914c69dae8d0
Signed-off-by: Jeff Darcy 
Reviewed-on: http://review.gluster.org/10617
Reviewed-by: Kaleb KEITHLEY 
Tested-by: Gluster Build System 
Reviewed-by: Raghavendra Bhat

quota : Fix smoke test failure.

2015-04-16T06:28:59+00:00

Functions check_ancestory_2 and quota_build_ancestry related
to quota feature was not declared in the header file.
This patch fixes the failure caused because of the mentioned
reason.

Change-Id: I967ee8ea943e2e7a54718e21245d1f63eb4d70e9
BUG: 1165938
Signed-off-by: Sachin Pandit 
Reviewed-on: http://review.gluster.org/10243
Reviewed-by: Raghavendra Talur 
Reviewed-by: mohammed rafi  kc 
Reviewed-by: Raghavendra Bhat 
Tested-by: Raghavendra Bhat 
Reviewed-by: Vijaikumar Mallikarjuna

Quota: Build ancestry in the lookup

2015-03-25T08:54:43+00:00

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

> Marker can fail or can account incorrect numbers when it doesn't find a
> ancestry for a inode.
>
> Solution:
> Current build_ancestry is done only on demand in the write/create FOPs
> in quota enforcer.
> It is good to do this in the quota_lookup as well.
>
> Change-Id: I8aaf5b3e05a3ca51e7ab1eaa1b636a90f659a872
> BUG: 1184885
> Signed-off-by: vmallika 
> Reviewed-on: http://review.gluster.org/9478
> Tested-by: Gluster Build System 
> Reviewed-by: Raghavendra Bhat 
> Reviewed-by: Vijay Bellur 

Change-Id: I57d3f801996da7194f5290067ff367888994786d
BUG: 1203648
Signed-off-by: vmallika 
Reviewed-on: http://review.gluster.org/9943
Tested-by: Gluster Build System 
Reviewed-by: Raghavendra Bhat

features/marker: log message clean up

2015-03-04T12:23:47+00:00

1. Changed log messages to be more appropriate.
2. Changed loglevel of failures in fop_cbks to be recorded as TRACE.
   Logging of failures at higher loglevels is unessential in non-endpoint
 translators.
3. Removed a log message related to memory allocation failure.

BUG: 1188066
Change-Id: I63c560c3bbd12706357fb3f696378c1a1e1efb44
Signed-off-by: Vijay Bellur 
Reviewed-on: http://review.gluster.org/8168
Tested-by: Gluster Build System 
Reviewed-by: Kaleb KEITHLEY 
Reviewed-by: Raghavendra G 
Tested-by: Raghavendra G 
Reviewed-on: http://review.gluster.org/9525
Reviewed-by: Raghavendra Bhat

features/marker: do not call inode_path on the inode not yet linked

2015-02-03T13:19:40+00:00

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

> * in readdirp callbak marker is calling inode_path on the inodes that
>   are not yet linked to the inode table.
>
> Change-Id: I7f5db29c6a7e778272044f60f8e73c60574df3a9
> BUG: 1176393
> Signed-off-by: Raghavendra Bhat 
> Reviewed-on: http://review.gluster.org/9320
> Tested-by: Gluster Build System 
> Reviewed-by: Raghavendra G 
> Tested-by: Raghavendra G 

Change-Id: Ibcfabe479ae6fd07a94ce80532fe1971d242974d
BUG: 1174170
Signed-off-by: vmallika 
Reviewed-on: http://review.gluster.org/9509
Tested-by: Gluster Build System 
Reviewed-by: Raghavendra Bhat

avoid memory leaks in snapview-server

2015-01-09T09:32:48+00:00

* snapview-server in readdirp, creates the inode for entries with names "." and
  ".." for each readdirp operation without creating dentries leading to
  memleak. It should have avoided creation of inodes for those entries

> Change-Id: I3b2025fd10872fcc3303d0becec764ffd4e37601
> BUG: 1179663
> Signed-off-by: Raghavendra Bhat 
> Reviewed-on: http://review.gluster.org/9404
> Tested-by: Gluster Build System 
> Reviewed-by: Vijay Bellur 
> Signed-off-by: Raghavendra Bhat 

Change-Id: Iea6c53c8db8b82189ef6c4a84dd3814349628ca3
BUG: 1180411
Reviewed-on: http://review.gluster.org/9426
Tested-by: Gluster Build System 
Reviewed-by: Sachin Pandit 
Reviewed-by: Raghavendra Bhat

features/uss: Perform NULL check on @name in svc_getxattr

2015-01-09T05:00:40+00:00

        Backport of: http://review.gluster.org/9378

LISTXATTR fop is internally converted into a GETXATTR with
the "name" parameter set to NULL. In svc_getxattr(), a listxattr
was causing a crash because of a NULL pointer dereference on @name.

FIX:
Add the necessary NULL check.

Change-Id: Ifb4ca2a45ecc78c384e92822403eaf2f1b573798
BUG: 1180070
Signed-off-by: Krutika Dhananjay 
Reviewed-on: http://review.gluster.org/9417
Tested-by: Gluster Build System 
Reviewed-by: Raghavendra Bhat