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

gfapi: fix read past EOF in glfs_preadv()

2013-06-20T08:46:28+00:00

When syncop_preadv() returns 0, we are returning @size as-is
(without modifying user passed buffer). This results in
junk data being read when read offset is at or beyond EOF.

Change-Id: I8aa9c957359f9a52833b944b452f501b6f5538d3
BUG: 953694
Signed-off-by: Anand Avati 
Reviewed-on: http://review.gluster.org/5237
Reviewed-by: Raghavendra Talur 
Reviewed-by: José Rivera 
Tested-by: José Rivera 
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

api: Fix api.so and libgfapi.so build time dependency

2013-06-08T21:54:04+00:00

Not all versions of autoconf handle dependencies expressed
as "A: B". Use the xxx_DEPENDENCIES variable to make it
version independent

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

gfapi: add new API glfs_dup() to copy a file descriptor

2013-06-08T21:53:48+00:00

Duplicate a glfs_fd file descriptor. Inherit the internal
fd_t with a reference.

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

api/Makefile: link api.so against libgfapi (and dependencies)

2013-06-08T21:53:26+00:00

Samba opens vfs_gluster.so with dlopen(..,RTLD_LOCAL). This makes
the symobls in the shared module and the library it exclusively
links against (specifically gfapi) not resolvable from the .so
files which are opened in turn within (like xlator .so files)
unless explicitly linked against the dependent libraries.

This is not otherwise required in case of glusterfsd as the main
executable symbols are always available for resolution from
dlopen()'ed shared objects, making Samba's vfs_glusterfs
requirement unique, but luckily not incompatible.

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

gfapi: handle graph switch (cwd, fds, locks)

2013-06-08T21:52:52+00:00

- Migrate open fds to new graph
- Migrate locks held in open fd to new fd
- Refresh CWD, so resolution of relative paths happens on refreshed
  inode (on new graph).

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

gfapi: ignore NULL and -1 parameters in glfs_set_logging()

2013-06-08T21:52:35+00:00

Permit changing either logfile or level by specifying the other
as -1 or NULL respectively.

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

gfapi: provide stable st_dev in stat structure

2013-06-08T21:51:38+00:00

Samba depends on st_dev being unique and stable for different
filesystems (it uses st_dev+st_ino as key to store metadata of
a file).

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

gfapi: link inodes in relevant entry FOPs

2013-06-08T21:51:04+00:00

Do not let inode linking to happen only in lookup(). While
that works, it is inefficient.

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

gfapi: optimistic resolution with ESTALE detection

2013-06-08T21:50:20+00:00

Enhance the path resolver to be optimistic. Instead of performing
a wasteful ->lookup() fop for every component in every API call,
build in speculation logic to consider the previous resolution
result temporarily, and if it results in ESTALE either later in
the path resolution or in the FOP, then retry resolution with
->lookup() at every level and retry the FOP.

This is the same optimistic resolution/retry logic Linux kernel
resolver is proposing to use.

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

gfapi: add new API glfs_set_xlator_option()

2013-06-08T21:49:42+00:00

This is the equivalent of specifying --xlator-option command line
argument to glusterfsd.

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