glusterfs.git, branch v3.6.2beta2

mgmt/glusterd: do not restart nfs server when snapshot is deactivated

2015-01-09T09:35:02+00:00

> Change-Id: Ie5eaa2beb4446640b22873f91e17da90d1cd8fad
> BUG: 1174625
> Signed-off-by: Raghavendra Bhat 
> Reviewed-on: http://review.gluster.org/9280
> Tested-by: Gluster Build System 
> Reviewed-by: Kaushal M 
> Signed-off-by: Raghavendra Bhat 

Change-Id: Ic5049f919fb444b45b3372a3b486183ed46d60f8
BUG: 1180404
Reviewed-on: http://review.gluster.org/9425
Tested-by: Gluster Build System 
Reviewed-by: Kaushal M 
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

performance/readdir-ahead: indicate EOF for readdirp.

2015-01-08T09:42:32+00:00

posix xlator sends op_errno as ENOENT and op_ret as 0, to indicate readdir has
been completed. readdir-ahead should send that op_errno that it has saved in the
fd context, when it serves the readdir requests. Otherwise some xlators sitting
above performance xlators such as snapview-client, which checks for end of
readdir operation by checking op_ret to 0 and op_errno to ENOENT will not be
able to identify end of readdir.

Change-Id: Ib0835136c61cb1e0d7df933226c479c7db703a71
BUG: 1175753
Signed-off-by: Raghavendra Bhat 
Reviewed-on: http://review.gluster.org/9283
Reviewed-by: Raghavendra G 
Tested-by: Gluster Build System 
Reviewed-by: Brian Foster 
Reviewed-by: Vijay Bellur 
Signed-off-by: Sachin Pandit 
Reviewed-on: http://review.gluster.org/9348

uss/gluster: In SVS, check for entry-point in dict only if inode_ctx is not available.

2015-01-08T09:42:13+00:00

Change-Id: I990487003b712bf4aed8f54291417965f301655e
BUG: 1175752
Signed-off-by: vmallika 
Reviewed-on: http://review.gluster.org/9265
Tested-by: Gluster Build System 
Reviewed-by: Sachin Pandit 
Reviewed-by: Rajesh Joseph 
Reviewed-by: Krishnan Parthasarathi 
Tested-by: Krishnan Parthasarathi 
Signed-off-by: Sachin Pandit 
Reviewed-on: http://review.gluster.org/9347
Reviewed-by: Raghavendra Bhat

features/snapview-client: handle fsync fop.

2015-01-08T09:41:52+00:00

Change-Id: Idaf46bd7497266af837789b09a0c62698f56ee4e
BUG: 1175749
Signed-off-by: Raghavendra Bhat 
Reviewed-on: http://review.gluster.org/9258
Tested-by: Gluster Build System 
Reviewed-by: Sachin Pandit 
Reviewed-by: Vijay Bellur 
Signed-off-by: Sachin Pandit 
Reviewed-on: http://review.gluster.org/9345

features/snapview-client: handle readdir requests differently for samba.

2015-01-08T09:41:20+00:00

* For samba export, the entry point is also added to the readdir response.

Change-Id: I825c017e0f16db1f1890bb56e086f36e6558a1c2
BUG: 1175742
Signed-off-by: Raghavendra Bhat 
Reviewed-on: http://review.gluster.org/9218
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur 
Signed-off-by: Sachin Pandit 
Reviewed-on: http://review.gluster.org/9344

gfapi: new inode created in glfs_resolve_component even if inode is in inode table

2015-01-08T09:40:33+00:00

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

problem: USS allows split-brain file to be accessed while main volume
         gives I/O error.

cause:
        AFR detects split-brain on lookup. It stores this information in
        inode context. open and readv fop checks this flag from inode context.
        open and readv fop fails if split-brain flag is set for the file.

        USS uses gfapi to access snapshot volume. During open call
        gfapi internally calls glfs_resolve_component. glfs_resolve_component
        generates a new inode even if inode is present for the file.
        Because of which afr_lookup acts on a new inode which does not
        contain the split-brain flag.

Change-Id: I56233e1436036783eaf6012034842e52122e981e
Signed-off-by: Rajesh Joseph 
Signed-off-by: Raghavendra Talur 
Reviewed-on: http://review.gluster.org/9415
Tested-by: Gluster Build System 
Reviewed-by: Raghavendra Bhat

gfapi: Change lookup logic for "/.." and "/." .

2015-01-08T09:40:12+00:00

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

We should handle /.. and /. in unique way,
otherwise we may end up setting glusterfs xattrs
on parent of the brick dir.

Change-Id: Id317cd625436734f496852b8513f3e9dc1c4fa4e
BUG: 1179658
Signed-off-by: Raghavendra Talur 
Reviewed-on: http://review.gluster.org/9403
Tested-by: Gluster Build System 
Reviewed-by: soumya k 
Reviewed-by: Poornima G 
Reviewed-by: Raghavendra Bhat

glusterfs.spec.in: synch with fedora dist-git glusterfs.spec

2015-01-08T09:38:30+00:00

Minor diffs between the two.

Change-Id: Ic7fda269063ecd4b00b3c9c4a0c31c69e2b967ba
BUG: 1146524
Signed-off-by: Kaleb S. KEITHLEY 
Reviewed-on: http://review.gluster.org/8854
Tested-by: Gluster Build System 
Reviewed-by: Humble Devassy Chirammal 
Reviewed-by: Raghavendra Bhat