glusterfs.git/libglusterfs, branch v3.3.0qa34

Replace GPLV3 MD5 with OpenSSL MD5

2012-04-04T23:13:21+00:00

Ric asked me to look at replacing the GPL licensed MD5 code with
something better, i.e. perhaps faster, and with a less restrictive
license, etc. So I took a couple hour holiday from working on
wrapping up the client_t and did this.

OpenSSL (nee SSLeay) is released under the OpenSSL license, a BSD/MIT
style license. OpenSSL (libcrypto.so) is used on Linux, OS X and *BSD,
Open Solaris, etc. IOW it's universally available on the platforms we
care about. It's written by Eric Young (eay), now at EMC/RSA, and I
can say from experience that the OpenSSL implementation of MD5 (at least)
is every bit as fast as RSA's proprietary implementation (primarily
because the implementations are very, very similar.) The last time I
surveyed MD5 implementations I found they're all pretty much the same
speed.

I changed the APIs (and ABIs) for the strong and weak checksums.
Strictly speaking I didn't need to do that. They're only called on
short strings of data, i.e. pathnames, so using int32_t and uint32_t
is ostensibly okay. My change is arguably a better, more general API
for this sort of thing. It's also what bit me when gerrit/jenkins
validation failed due to glusterfs segv-ing. (I didn't pay close enough
attention to the implementation of the weak checksum. But it forced me
to learn what gerrit/jenkins are doing and going forward I can do better
testing before submitting to gerrit.)

Now resubmitting with a BZ

Change-Id: I545fade1604e74fc68399894550229bd57a5e0df
BUG: 807718
Signed-off-by: Kaleb KEITHLEY 
Reviewed-on: http://review.gluster.com/3019
Tested-by: Gluster Build System 
Reviewed-by: Jeff Darcy

libglusterfs/run: POSIX compatible closing of child fds on non-Linux

2012-04-03T06:11:20+00:00

It just occurred to me that I realized there are POSIX features that let this be
done cleanly. However, For Linux we keep the proc based approach to get more
graceful strace logs.

Change-Id: I6b473271b2adfd85cef6612eb65d88e5d00f5943
BUG: 764294
Signed-off-by: Csaba Henk 
Reviewed-on: http://review.gluster.com/3068
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

libglusterfs/run: clean up properly also if program could not be spawned

2012-04-03T05:39:28+00:00

Change-Id: Ie81a6e1a99c1cb002653c0175bb3c5fcff9c8a4a
BUG: 808427
Reported-by: Krishnan Parthasarathi 
Signed-off-by: Csaba Henk 
Reviewed-on: http://review.gluster.com/3055
Tested-by: Gluster Build System 
Reviewed-by: Krishnan Parthasarathi

libglusterfs/run: test suite refinement

2012-04-03T05:38:49+00:00

If run is compiled as a standalone self-test program

(-DRUN_STANDALONE -DRUN_DO_TESTS)

add the possibility to pass an integer which makes
the program sleep that many seconds before exit
(0 forever).

This makes it possible to investigate post-test
state (eg. if there are leftover zombies).

Change-Id: Ia8d80a74eb83615ad50db2e17bb2701dc0d69070
BUG: 808427
Signed-off-by: Csaba Henk 
Reviewed-on: http://review.gluster.com/3067
Tested-by: Gluster Build System 
Reviewed-by: Krishnan Parthasarathi

glusterd: Include port in nfs status

2012-04-02T09:41:18+00:00

Change-Id: I679e491e801b694e8a0f93dd87cf540441dae927
BUG: 806877
Signed-off-by: Kaushal M 
Reviewed-on: http://review.gluster.com/3060
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

performance/quick-read: make unlink aware of open-behind behaviour of quick-read.

2012-03-28T17:40:41+00:00

quick-read should do an open of all the fds that are waiting for to be
opened behind before doing an unlink on a path.

Change-Id: I4c8d65c0511e8a37765ff57e30d99dd89b56c4fc
BUG: 765363
Signed-off-by: Raghavendra G 
Reviewed-on: http://review.gluster.com/506
Tested-by: Gluster Build System 
Reviewed-by: Shishir Gowda 
Reviewed-by: Vijay Bellur

cluster/dht : Acl fix for distribute directory selfheal

2012-03-26T22:13:43+00:00

Send acl xattrs, if present in the xattrs returned during lookup, during
directory self-heal.

Change-Id: I5337bbd3f3963aeed500a8a552e5f6713089b53e
BUG: 764787
Signed-off-by: Kaushal M 
Reviewed-on: http://review.gluster.com/737
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

libglusterfs/fd: ref fds while copying the fdentries.

2012-03-26T22:09:20+00:00

Change-Id: I8973ec5d9858adfbdd6efc7cd596cf5d5af0e3cb
BUG: 767862
Signed-off-by: Raghavendra G 
Reviewed-on: http://review.gluster.com/3006
Tested-by: Gluster Build System 
Reviewed-by: Anand Avati

core: adding extra data for fops

2012-03-22T23:40:27+00:00

with this change, the xlator APIs will have a dictionary as extra
argument, which is passed between all the layers. This can be
utilized for overloading in some of the operations.

Change-Id: I58a8186b3ef647650280e63f3e5e9b9de7827b40
Signed-off-by: Amar Tumballi 
BUG: 782265
Reviewed-on: http://review.gluster.com/2960
Tested-by: Gluster Build System 
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