glusterfs.git/glusterfsd/src/glusterfsd-mgmt.c, branch v3.4.2qa4

glusterfsd: fix small memory leaks in glusterfsd-mgmt.c

2013-12-02T04:38:04+00:00

E.g. In glusterfs_volfile_fetch(), req.xdata.xdata_val is allocated
in dict_allocate_and_serialize() but not freed after mgmt_submit_request().

A survey of dict_allocate_and_serialize/_submit_request in
glusterfsd-mgmt.c shows no consistent pattern of freeing the xdata_val
and also the dict, which is a little disturbing. (Yes, clearly not
every place this occurs needs to be freed the same way.)

Change-Id: I8f31179ae12c05c9b5406d8e9e28110fcbfac1c7
BUG: 1036102
Signed-off-by: Kaleb S. KEITHLEY 
Reviewed-on: http://review.gluster.org/6381
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

glusterfsd: Use dynamic volfile buffer

2013-07-31T07:38:40+00:00

Backport of
 fa78792 glusterfsd: Use dynamic volfile buffer
from the master branch.

Glusterfsd used a fixed buffer to store volfiles fetched via getspec.
This caused problems with large volfiles. Changining this to a dynamic
buffer allows large volfiles to be loaded in memory.

BUG: 986100
Change-Id: I9ec86c09a00bc84fef7a0da05bbeb6b3e07d5146
Signed-off-by: Kaushal M 
Reviewed-on: http://review.gluster.org/5431
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

glusterd,glusterfsd,libgfapi: Client op-version

2013-02-06T20:17:06+00:00

This patch introduces op-version support for glusterfs clients.
Now, a client sends its supported op-versions during the volfile fetch request
and glusterd will return the volfile only if the client can support the current
op-version of the cluster.

Change-Id: Iab1f1f1706802962bcf27058657c44e8a344d2f6
BUG: 907311
Signed-off-by: Kaushal M 
Reviewed-on: http://review.gluster.org/4247
Tested-by: Gluster Build System 
Reviewed-by: Amar Tumballi 
Reviewed-by: Anand Avati

Fix xdr_to_generic success check

2012-12-10T05:59:56+00:00

This patch fixes the success check for xdr_to_generic function across the
codebase.

Also, cleans up the brick_op actors table in glusterfsd-mgmt.c to make sure that
the actors are called directly by rpcsvc.

Change-Id: I3086585f30c44f69f1bc83665f89e30025f76d3a
BUG: 884452
Signed-off-by: Kaushal M 
Reviewed-on: http://review.gluster.org/4278
Reviewed-by: Pranith Kumar Karampuri 
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

BD Backend: CLI commands to create/delete image

2012-11-29T17:39:53+00:00

Cli commands  added to create/delete a LV device.

The following command creates lv in a given vg.
$ gluster bd create :/ 

The following command deletes lv in a given vg.
$ gluster bd delete :/

BUG: 805138
Change-Id: Ie4e100eca14e2ee32cf2bb4dd064b17230d673bf
Signed-off-by: M. Mohan Kumar 
Reviewed-on: http://review.gluster.org/3718
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

glusterfsd-mgmt: make brick-ops work in synctask

2012-11-19T08:35:07+00:00

this way, the main epoll thread is freed up to serve other requests.

Change-Id: Iecf68aceb8bd84433d0f9fcd98d96c734a4cc134
BUG: 842955
Signed-off-by: Amar Tumballi 
Reviewed-on: http://review.gluster.org/4096
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

Fixed some general typing errors.

2012-09-27T18:20:20+00:00

Eg: changed recieved to received

Change-Id: I360fcb99c97c8a0222e373fee20ea2fccfb938db
BUG: 860543
Signed-off-by: Varun Shastry 
Reviewed-on: http://review.gluster.org/3998
Tested-by: Gluster Build System 
Reviewed-by: Jeff Darcy 
Reviewed-by: Vijay Bellur

core: enable process to return the appropriate error code

2012-09-22T03:43:05+00:00

Setup a pipe() in glusterfs_ctx_t to communicate with the fork'ed
child the exit status and return it to the shell.

Change-Id: Ibbaa581d45acb24d5141e43ae848cae29312d95f
BUG: 762935
Signed-off-by: Anand Avati 
Reviewed-on: http://review.gluster.org/3836
Tested-by: Gluster Build System 
Reviewed-by: Amar Tumballi

All: License message change

2012-09-13T20:19:37+00:00

License message changed for server-side, dual license GPLV2 and LGPLv3+.

Change-Id: Ia9e53061b9d2df3b3ef3bc9778dceff77db46a09
BUG: 852318
Signed-off-by: Varun Shastry 
Reviewed-on: http://review.gluster.org/3940
Tested-by: Gluster Build System 
Reviewed-by: Kaleb KEITHLEY 
Reviewed-by: Anand Avati