[Bugs] [Bug 1230026] BVT: glusterd crashed and dumped during upgrade (on rhel7.1 server)

[bugzilla at redhat.com]

https://bugzilla.redhat.com/show_bug.cgi?id=1230026



--- Comment #2 from Anand Avati <aavati at redhat.com> ---
COMMIT: http://review.gluster.org/11155 committed in release-3.7 by Atin
Mukherjee (amukherj at redhat.com) 
------
commit 6724ae3b3becdf6be495e8b4f81f1fba302da69e
Author: anand <anekkunt at redhat.com>
Date:   Fri May 22 18:18:11 2015 +0530

    libglusterfs: Enabling the fini()  in cleanup_and_exit()

    Problem 1 : glusterd was crashing due to race between clean up thread and
rpc event thread.

    Scenario:
    As we can observed, X thread is in the process of exiting the process. It
has already
    run the exit handlers, which cleanup things that require cleaning up. This
includes
    liburcu resources. By the time Y thread  called rcu_bp_register(), the
liburcu resources
    have been cleaned up. rcu_bp_register() tries to access these non-existent
resources,
    which leads to the segmentation fault.

    Note1:
    Crash happen when the process is almost at the point of stopping(exiting),
it doesn't have any
    serious impact to functionality apart from creating the core dump file and
the log message.

    Fix .Do proper clean up before calling exit().

    Note2: Other xlator have clean up issues,so only glusterd clean up function
invoked.

    Note3: This patch also solve the selinux issue.

    Problem 2 : glusterd runs as rpm_script_t when it's executed from the rpm
scriptlet,files created
    in this context are set as rpm_script_t, so glusterd unable to access these
files when it runs
    in glusterd_t context.

    Fix: Fini clean up the files while glusterd exiting, so files are recreated
by glusterd while
    starting with proper SElinux context label.

    Backport of :
    >Change-Id: Idcfd087f51c18a729bdf44a146f9d294e2fca5e2
    >BUG: 1209461
    >Signed-off-by: anand <anekkunt at redhat.com>
    >Reviewed-on: http://review.gluster.org/10894
    >Tested-by: Gluster Build System <jenkins at build.gluster.com>
    >Reviewed-by: Atin Mukherjee <amukherj at redhat.com>
    >Tested-by: NetBSD Build System <jenkins at build.gluster.org>
    >Reviewed-by: Raghavendra Bhat <raghavendra at redhat.com>
    >Reviewed-by: Vijay Bellur <vbellur at redhat.com>

    Change-Id: I59579e675bd73d7a19f7b965bd2c3c0fcd95d241
    BUG: 1230026
    Signed-off-by: anand <anekkunt at redhat.com>
    Reviewed-on: http://review.gluster.org/11155
    Tested-by: Gluster Build System <jenkins at build.gluster.com>
    Tested-by: NetBSD Build System <jenkins at build.gluster.org>
    Reviewed-by: Atin Mukherjee <amukherj at redhat.com>

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.