glusterfs-afrv1.git/api, branch v3.4.1

gfapi: implement a minimial glfs_fini()

2013-09-19T21:28:50+00:00

At the very least, we should PARENT_DOWN on the currently active
graph and disconnect ourselves from glusterd.

Further cleanups underway.

Change-Id: I9276686a84b0975b5ce272b4cbec1b80920d5c5c
BUG: 1004519
Signed-off-by: Anand Avati 
Reviewed-on: http://review.gluster.org/5903
Tested-by: Gluster Build System

gfapi: fix for not able to rename folders

2013-09-10T08:24:58+00:00

If oldname is being renamed to newname, we need to
check for type of newname only if newname exists.

Change-Id: I068a283f9ffe67fcd5e8754d6bf052a2339efbf2
BUG: 953694
Signed-off-by: Raghavendra Talur 
Reviewed-on: http://review.gluster.org/5886
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

gfapi: fix glfs_readlink()

2013-09-10T08:24:37+00:00

glfs_readlink() is supposed to memcpy the buffer pointer
returned by syncop_readlink(). Fix it.

Change-Id: I5936b07abbd93cf02b354233dc60f6623e30a38b
BUG: 953694
Signed-off-by: Anand Avati 
Reviewed-on: http://review.gluster.org/5885
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

gfapi: perform open() on pre-existing inode in glfs_creat()

2013-09-10T08:23:17+00:00

Performing syncop_create() on a pre-existing file can result
in opening the linkfile if DHT has a linkfile for the file.
This is because dht_create() will perform the op on the
hashed_subvol() and overwrite the layout in dht_create_cbk.

Change-Id: I7a0db56921ec9fc3e278e0418db3b967f81e5598
BUG: 990410
Signed-off-by: Anand Avati 
Reviewed-on: http://review.gluster.org/5884
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

libgfapi : Fix for hardlink creation on SMB mount

2013-09-10T08:22:14+00:00

Previously the inode of the hard link was left uninitialised hence
glfs_loc_link() used to fail with invalid parameter.Since inode is
same for both the files in hard link creation,   updated the inode
field of hard link to be same as that of the original file.

Change-Id: Ifd6439867d982524a7b48bf3f6add6e844b3c4c1
BUG: 996063
Signed-off-by: Anand Avati 
Reviewed-on: http://review.gluster.org/5882
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

api: Fill loc.path and loc.gfid before syncop_open.

2013-09-10T08:20:05+00:00

syncop_open was crashing because of NULL dereference.
This fixes that.

Change-Id: I4bc48fac2a6c0b15c806cbbb4ae45c67891ab7ed
BUG: 1002577
Signed-off-by: Raghavendra Talur 
Reviewed-on: http://review.gluster.org/5877
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

api: Add null check for active_subvol in glfs_close.

2013-09-10T08:18:52+00:00

Problem:
In glfs_close, NULL value returned by glfs_active_subvol was not
being checked and was causing a crash.

Change-Id: I6eefa60e7b07dee251b98932b1d08a5c2981d3d7
BUG: 1002511
Signed-off-by: Raghavendra Talur 
Reviewed-on: http://review.gluster.org/5876
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

glusterfs, gfapi: fix symbol clash

2013-09-10T08:16:28+00:00

The callback structures in both protocol/client and glusterfsd,
gfapi used the same name for the actor table - gluster_cbk_actors.

CBKs are required only for the management connection, and the
actors of protocol/client are NOP functions. This supposed-to-be
NOP function dispatch tabble is actually ending up pointing to
the actor table of glusterfsd or gfapi.

These functions, even though set wrongly, are not even expected
to be called through the protocol/client callback path. Glusterd
however sends the FETCHSPEC (and other) notify callbacks to *all*
connected clients unconditionally, and there is a small period
of time when protocol/client is connected to glusterd for
PORTMAP query. If the FETCHSPEC callback notify is issued in
this window of time, we end up calling the wrong actor in the
client side resulting in a crash.

Change-Id: I605ff7df64c7faf4607369bbf275aedec28e1778
BUG: 1004091
Signed-off-by: Anand Avati 
Reviewed-on: http://review.gluster.org/5875
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

gfapi: apply an upper bound on nested symlink resolution

2013-09-10T08:16:09+00:00

In case of nested symlink resolution, implement an upper
bound on the number of such nested levels the resolver will
descend. This limit is arbitrary, and set to 2048 nested
levels.

Change-Id: I264e5bd60d317eda97f4e6f49bd5d8694f8664a9
BUG: 1004100
Signed-off-by: Anand Avati 
Reviewed-on: http://review.gluster.org/5874
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

gfapi: store the open/create flags in fd_t object

2013-09-10T08:15:07+00:00

The flags passed on to open and create calls were not being saved
in the fd_t object, hence the fd migration was failing.

Change-Id: I486bb818477fe4c393d64a711534a082162a0e53
BUG: 1005159
Signed-off-by: Poornima 
Reviewed-on: http://review.gluster.org/5873
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur