glusterfs-afrv1.git, branch release-3.2

features/marker: Replacing -1 with GF_CLIENT_PID_GSYNCD as part of code cleanup.

2013-02-07T21:36:25+00:00

To Keep the code(rel-3.2) consistent with the master(rel-3.3) branch.

Change-Id: I6a052a35abf022d5551ec9b59db0ae19f81f326d
BUG: 769494
Signed-off-by: Mohammed Junaid 
Reviewed-on: http://review.gluster.org/2969
Tested-by: Gluster Build System 
Reviewed-by: Csaba Henk

Merged in the fedora spec file to bring glusterfs into compliance

2013-02-07T21:13:13+00:00

with FSH and Redhat/Fedora requirements.
Created a single spec that works for both 3.1 and 3.2 versions.

Releases will now require changing the source url in configure.ac
at the same time the release version is set.

OLDBUG: 2970
BUG: 764702

Change-Id: If9067d673f34b70ccfbc9b723019cca31044347a
Signed-off-by: Joe Julian 
Reviewed-on: http://review.gluster.org/702
Tested-by: Gluster Build System 
Reviewed-by: Kaleb KEITHLEY 
Reviewed-by: Anand Avati

mgmt/glusterd: check brickinfo for NULL before dereferencing it

2013-01-08T05:58:56+00:00

Change-Id: I59fed6adf13d6d72d79c986bf3578ca79b9756ac
BUG: 781953
Signed-off-by: Raghavendra Bhat 
Reviewed-on: http://review.gluster.org/2648
Tested-by: Gluster Build System 
Reviewed-by: Krishnan Parthasarathi

Most commands cause glusterd 3.2.7 to crash on Debian Wheezy

2012-07-20T23:13:10+00:00

When xlators/mgmt/glusterd/src/glusterd-rpc-ops.c is compiled with
-O2 by gcc-4.7.1, the compiler optimizes away all the rsp.foo = bar
statements because rsp goes out of scope at the end of each of
the blocks in the switch statement in glusterd-op_send_cli_response().

That is:
    void *cli_rsp = NULL;

    switch(foo) {
    case BAR:
    {
        struct bar rsp = {0,};
        /* init statements here are optimized away */
        rsp.x = 0;
        rsp.y = 1;
        cli_rsp = &rsp;
    }
    ...
    }
    /* don't expect cli_rsp to point at anything meaningful here */

This particular idiom hasn't been a problem thus far, e.g. even with
gcc-4.7.0 in Fedora17 also compiling with -O2. That not withstanding,
using this idiom has probably always been on shaky ground; semantically
it is correct that rsp goes out of scope at the end of the block.

Note: I have not surveyed the source to see whether this idiom appears
anywhere else.

Note also that glusterd-op_send_cli_response() has been rewritten around
the refactored gf_cli_rsp for 3.3.x and later and this particular bug is
not in 3.3.x and later releases.

BUG: 837684
Change-Id: I82f61ad2b9827c5b96af14b180a82c3ab350f559
Signed-off-by: Kaleb S. KEITHLEY 
Reviewed-on: http://review.gluster.com/3707
Tested-by: Gluster Build System 
Reviewed-by: Jeff Darcy 
Reviewed-by: Anand Avati

nfs: memory leak fixes

2012-07-13T01:05:58+00:00

* Fixes a leak of strdups in the parent_inode_loc_fill
  code path.

* Introduces nfs_forget() to cleanup the ctx(addr of
  inode_q), a major leak involving inode_q hereby fixed.

* Using the correct size to allocate fd_entry structures.

* Free hashmatch in call_state during call state wipe
  and reassignment of the same.

* Free cs->resolventry in the rename fop code path.

Change-Id: I4ff556eba27f0b56ef824c0a86ab3d393d0d38c1
BUG: 835059
Signed-off-by: Rajesh Amaravathi 
Reviewed-on: http://review.gluster.com/3641
Tested-by: Gluster Build System 
Reviewed-by: Kaleb KEITHLEY 
Reviewed-by: Anand Avati

cluster/afr: Avoid return value truncation

2012-06-10T17:20:27+00:00

Change-Id: I947316a0e1112bf3c7b7e21aefce96685a9d06db
BUG: 804606
Signed-off-by: Pranith Kumar K 
Reviewed-on: http://review.gluster.com/3543
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

cluster/afr: EIO should overwrite ENOENT in lookup

2012-06-10T17:20:00+00:00

In case if lookup decides there is a gfid-mismatch,
some enoents and self-heal cant remove the stale entry,
it tells lookup to unwind with EIO but since ENOENT
has more priority it is not over-written, this patch
fixes that case.

BUG: 765528
Change-Id: Ida03685629be8126365d660a16d8f9252f7ca030
Signed-off-by: Pranith Kumar K 
Reviewed-on: http://review.gluster.com/2675
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

cluster/afr: Handle self-heal of files with holes

2012-06-09T16:15:16+00:00

Change-Id: I7a560b3633151c9c1e5a787d1c808d8a396ee754
BUG: 765424
Signed-off-by: Pranith Kumar K 
Reviewed-on: http://review.gluster.com/3540
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

cluster/afr: Don't reset split-brain when data-self-heal is off

2012-06-06T10:39:31+00:00

Change-Id: Id73bdb84c3a1c0af6f0e2954514c3c07c65317bc
BUG: 804606
Signed-off-by: Pranith Kumar K 
Reviewed-on: http://review.gluster.com/3522
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

cluster/afr: Fix truncation of offset in 32-bit machines

2012-06-06T10:18:46+00:00

Thanks to the fix submitted by Ian Latter and Michael

Change-Id: I84a62a9183c70be6da055d23e4d60cc97cfcf531
BUG: 809982
Signed-off-by: Pranith Kumar K 
Reviewed-on: http://review.gluster.com/3521
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur