glusterfs.git/xlators/protocol, branch v3.3.0qa22

nfs: avoid logging invalid fdctx in case of anonymous fds

2012-02-14T18:25:30+00:00

if get_fd_ctx fails (as in case of anonymous fds), overwhelming
amount of entries are seen in the nfs log, causing dd and other
heavy i/o operations to become unresponsive. this patch logs an
invalid fdctx only if it is not an anonymous fd.

Change-Id: I4e917d150d6a053af77d47a94a2f1c2633acadb5
BUG: 787365
Signed-off-by: Rajesh Amaravathi 
Reviewed-on: http://review.gluster.com/2747
Tested-by: Gluster Build System 
Reviewed-by: Amar Tumballi

protocol xdr: remove 'path<>'

2012-02-14T12:34:06+00:00

client was sending 'path' on wire, which gets ignored on server
side, and also doesn't get freed up, which causes memory leak.

also with not having path on wire, the xdr size on wire most of the
time can remain constant, which helps in allocating RDMA buffers.

Change-Id: Ie0d36a670be60b02fd1e925c6f977b1a71def5cd
BUG: 790298
Signed-off-by: Amar Tumballi 
Reviewed-on: http://review.gluster.com/2744
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

core: add an extra flag to readv()/writev() API

2012-02-14T09:09:02+00:00

needed to implement a proper handling of open flag alterations
using fcntl() on fd.

Change-Id: Ic280d5db6f1dc0418d5c439abb8db1d3ac21ced0
Signed-off-by: Amar Tumballi 
BUG: 782265
Reviewed-on: http://review.gluster.com/2723
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

protocol: code cleanup

2012-02-14T09:00:26+00:00

make dict serialize and unserialization code a macro

Change-Id: I459c77c6c1f54118c6c94390162670f4159b9690
BUG: 764890
Signed-off-by: Amar Tumballi 
Reviewed-on: http://review.gluster.com/2742
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

protocol/client: assign the right value to 'conf' before de-refing it

2012-02-07T12:21:44+00:00

variable assignment was done after it was actually getting de-referenced.
moved the assignment few lines up.

Change-Id: Id65e3e2d3dfe071e1c5b14c32488647070398ae4
BUG: 787117
Signed-off-by: Amar Tumballi 
Reviewed-on: http://review.gluster.com/2712
Tested-by: Gluster Build System 
Reviewed-by: Jeff Darcy

cli, protocol/server : improve validation for the option auth.(allow/reject)

2012-02-06T06:19:54+00:00

cli now checks validity of address list given for 'volume set auth.*'

Server xlator checks addresses supplied to auth.(allow/reject) option
including wildcards for correctness in case volfile is manually edited.

Original patch done by shylesh@gluster.com
Original patch  is at http://patches.gluster.com/patch/7566/

Change-Id: Icf52d6eeef64d6632b15aa90a379fadacdf74fef
BUG: 764197
Signed-off-by: Kaushal M 
Reviewed-on: http://review.gluster.com/306
Tested-by: Gluster Build System 
Reviewed-by: Jeff Darcy 
Reviewed-by: Vijay Bellur

protocol/client : prevent client from reconnecting when server

2012-02-03T15:41:51+00:00

authentication fails

This prevents the client from trying to reconnect on server
authentication failure. Reconnecting on authentcation failure causes hung
mounts on unauthorised clients. This patch fixes this problem.

Also, mount.glusterfs script unmounts mount-point on mount failure to
prevent hung mounts.

Change-Id: I5615074d27948077bad491a38cecae1b7f5159fb
BUG: 765240
Signed-off-by: Kaushal M 
Reviewed-on: http://review.gluster.com/398
Tested-by: Gluster Build System 
Reviewed-by: Amar Tumballi

cli: Extend "volume status" with statedump info

2012-01-27T12:20:04+00:00

This patch enhances and extends the "volume status" command with information
obtained from the statedump of the bricks of volumes.

Adds new status types : clients, inode, fd, mem, callpool
The new syntax of "volume status" is,
 #gluster volume status [all|{ []
                         [misc-details|clients|inode|fd|mem|callpool]}]

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

protocol/client: Pass the right arguments to CLIENT_GET_REMOTE_FD

2012-01-26T13:23:33+00:00

Change-Id: I04f984f20964650a38009bba7711d2757151ade5
BUG: 762935
Signed-off-by: Vijay Bellur 
Reviewed-on: http://review.gluster.com/2694
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

protocol/client: if the remote_fd is -1, then unwind instead of sending the call to server

2012-01-25T16:08:38+00:00

For calls with remote_fd set to -1, client xlator is sending the call to the
server which results in server not resolving it and thus fd being NULL. Locks
xlator when tries to get the inode context using the fd it segfaults. To avoid
it unwind the call in the client xlator if the remote_fd is -1.

Change-Id: Ic34a49fdf1012dd371f4b194703c0be74f29bda2
BUG: 784187
Signed-off-by: Raghavendra Bhat 
Reviewed-on: http://review.gluster.com/2684
Tested-by: Gluster Build System 
Reviewed-by: Jeff Darcy 
Reviewed-by: Krishnan Parthasarathi 
Reviewed-by: Anand Avati