glusterfs.git/glusterfsd/src/glusterfsd.c, branch v3.3.0beta4

glusterfsd: Make sure mountpoint is an absolute path

2012-05-09T00:00:21+00:00

If the mountpoint path given to glusterfs is not an absolute path, convert it to
an absolute path by concatenating it with the curren working directory.
This prevents cases, where in gluster cannot perform clean unmounts when mount
is done with a relative path.

Change-Id: Ie25add4e1dc59171e522c4244c79a6c148844ab3
BUG: 819466
Signed-off-by: Kaushal M 
Reviewed-on: http://review.gluster.com/3302
Tested-by: Gluster Build System 
Reviewed-by: Amar Tumballi 
Reviewed-by: Anand Avati

glusterd,glusterfsd : Perform proper cleanup of connections and socket files

2012-04-24T01:54:55+00:00

Perform proper disconnections and cleanup in glusterd on stopping nfs-server and
self-heal daemon processes to prevent glusterd from opening unneded unix domain
socket connections.

glusterfsd processes will cleanup the socket files properly in
cleanup_and_exit() to prevent junk socket files.

Also, fix rpcsvc_program_unregister() to preform the unregistering properly.

Change-Id: I1c7302c1166cf43feba1c7a813c3dc10169dc53a
BUG: 810089
Signed-off-by: Kaushal M 
Reviewed-on: http://review.gluster.com/3168
Tested-by: Gluster Build System 
Reviewed-by: Krishnan Parthasarathi

glusterfsd: mem-pool count of dict_t is adjusted

2012-04-11T23:48:25+00:00

increased dict_t pool count based on test results.

Change-Id: Ia5b24940b79a1489b00cca2993241c32be94bbec
Signed-off-by: Amar Tumballi 
BUG: 809034
Reviewed-on: http://review.gluster.com/3131
Tested-by: Gluster Build System 
Reviewed-by: Jeff Darcy 
Reviewed-by: Anand Avati

Revert "core: enable process to return the appropriate error code"

2012-03-22T11:19:35+00:00

This reverts commit 16717f4df7b0fbd3c255f5faf8a2744444231bb7

Change-Id: I65624db4d31e11e1bec66b07b352ce823ef069b9
Reviewed-on: http://review.gluster.com/3001
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

core: enable process to return the appropriate error code

2012-03-21T18:58:29+00:00

Credit of this patch goes to Avati  as I just did a
rebase to the lastest master.

Change-Id: I53b2adf90fd9808433bdc35b57dad0682bc90860
BUG: 762935
Signed-off-by: Amar Tumballi 
Reviewed-on: http://review.gluster.com/579
Tested-by: Gluster Build System 
Reviewed-by: Jeff Darcy 
Reviewed-by: Anand Avati

rpc-clnt: separate out connection_cleanup() from destroy()

2012-03-19T16:14:40+00:00

noticed that there are possibilities where one would like to do a
connection_cleanup() before destroying a RPC connection itself, also
current code is such that, rpc_clnt_connection_cleanup() does
rpc_clnt_ref() and unref(), creating a race window/double unref
possibilities in the code.

by separating out the functions, this race window/double fault can be
prevented.

Change-Id: I7ebd3392efa891232857b6db9108b0b19e40fc12
Signed-off-by: Amar Tumballi 
BUG: 802403
Reviewed-on: http://review.gluster.com/2979
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

core: bring a cmdline option to set memory-accounting

2012-03-15T10:33:18+00:00

currently this is implemented as a command line option, and not
as an easier translator option. this is because as of now, before
even the volume files are parsed, we would need memory accounting
enabled. there is scope for improving this behavior, but for now,
this approach solves the problem.

Also, this feature's major consumers are the testers who are
looking for leaks, hence option is hidden from usage output.

Change-Id: I09a5b13743ae43ff42c251989f921319e94cabe3
Signed-off-by: Amar Tumballi 
BUG: 799199
Reviewed-on: http://review.gluster.com/2856
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

core: dict allocations through mem-pool

2012-03-15T05:15:25+00:00

Change-Id: I7401639060957d437808779745a1e46c3f9f4585
Signed-off-by: Amar Tumballi 
BUG: 798503
Reviewed-on: http://review.gluster.com/2851
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

glusterfsd: fix some option definitions

2012-03-10T11:57:29+00:00

VOLUME_NAME used earlier in option definitions are confusing with
glusterd specific volumes. It should be translator name in volume
file.

Change-Id: Id4d86c8def65aa6ff1346ef49f8b20ea428de615
BUG: 788437
Signed-off-by: Amar Tumballi 
Reviewed-on: http://review.gluster.com/2789
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

glusterfsd: Don't reconfigure if process does not know volfile server

2012-03-06T07:10:48+00:00

Change-Id: I09e5527bb1496e0029d73f08dd7892d74df93084
BUG: 799882
Signed-off-by: Pranith Kumar K 
Reviewed-on: http://review.gluster.com/2865
Reviewed-by: Amar Tumballi 
Tested-by: Gluster Build System