Bug 1004519 - SMB:smbd crashes while doing volume operations
Summary: SMB:smbd crashes while doing volume operations
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: libgfapi
Version: mainline
Hardware: Unspecified
OS: Unspecified
urgent
unspecified
Target Milestone: ---
Assignee: GlusterFS Bugs list
QA Contact: Sudhir D
URL:
Whiteboard:
Depends On: 1003584
Blocks: 1004091
TreeView+ depends on / blocked
 
Reported: 2013-09-04 20:16 UTC by Raghavendra Talur
Modified: 2014-04-17 13:14 UTC (History)
6 users (show)

Fixed In Version: glusterfs-3.4.3
Doc Type: Bug Fix
Doc Text:
Clone Of: 1003584
Environment:
Last Closed: 2014-04-17 13:14:16 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

Comment 1 Anand Avati 2013-09-05 18:17:13 UTC
REVIEW: http://review.gluster.org/5790 (parser: make the parser thread safe.) posted (#3) for review on master by Anand Avati (avati)

Comment 2 Anand Avati 2013-09-07 00:54:54 UTC
COMMIT: http://review.gluster.org/5790 committed in master by Anand Avati (avati) 
------
commit 2347a1a7594dafc1e6a74784287efe84963fe4ea
Author: Anand Avati <avati>
Date:   Wed Aug 28 18:38:26 2013 -0700

    parser: make the parser thread safe.
    
    The volfile parser thread safe by guarding the parsing phase
    in a mutex. Thread safety becomes a problem when there are multiple
    glfs_t objects created by gfapi and all of them potentially parse
    the respective volfiles at the same time.
    
    Change-Id: I4376019c4956994b72397ab36e6ac3ce849797ec
    BUG: 1004519
    Signed-off-by: Anand Avati <avati>
    Reviewed-on: http://review.gluster.org/5790
    Reviewed-by: Raghavendra G <rgowdapp>
    Reviewed-by: Raghavendra Bhat <raghavendra>
    Tested-by: Gluster Build System <jenkins.com>
    Reviewed-by: Anand Avati <avati>

Comment 3 Anand Avati 2013-09-10 00:50:42 UTC
REVIEW: http://review.gluster.org/5872 (parser: make the parser thread safe.) posted (#1) for review on release-3.4 by Anand Avati (avati)

Comment 4 Anand Avati 2013-09-10 01:06:07 UTC
REVIEW: http://review.gluster.org/5872 (parser: make the parser thread safe.) posted (#2) for review on release-3.4 by Anand Avati (avati)

Comment 5 Anand Avati 2013-09-10 08:14:46 UTC
COMMIT: http://review.gluster.org/5872 committed in release-3.4 by Vijay Bellur (vbellur) 
------
commit a95bf63d063b3e2345936d53410b515b7198f2ac
Author: Anand Avati <avati>
Date:   Wed Aug 28 18:38:26 2013 -0700

    parser: make the parser thread safe.
    
    The volfile parser thread safe by guarding the parsing phase
    in a mutex. Thread safety becomes a problem when there are multiple
    glfs_t objects created by gfapi and all of them potentially parse
    the respective volfiles at the same time.
    
    Change-Id: I4376019c4956994b72397ab36e6ac3ce849797ec
    BUG: 1004519
    Signed-off-by: Anand Avati <avati>
    Reviewed-on: http://review.gluster.org/5872
    Tested-by: Gluster Build System <jenkins.com>
    Reviewed-by: Vijay Bellur <vbellur>

Comment 6 Anand Avati 2013-09-13 04:24:06 UTC
REVIEW: http://review.gluster.org/5788 (gfapi: implement a minimial glfs_fini()) posted (#2) for review on master by Anand Avati (avati)

Comment 7 Anand Avati 2013-09-13 04:24:39 UTC
REVIEW: http://review.gluster.org/5903 (gfapi: implement a minimial glfs_fini()) posted (#1) for review on release-3.4 by Anand Avati (avati)

Comment 8 Anand Avati 2013-09-14 19:56:54 UTC
REVIEW: http://review.gluster.org/5788 (gfapi: implement a minimial glfs_fini()) posted (#3) for review on master by Anand Avati (avati)

Comment 9 Anand Avati 2013-09-14 19:59:06 UTC
REVIEW: http://review.gluster.org/5903 (gfapi: implement a minimial glfs_fini()) posted (#2) for review on release-3.4 by Anand Avati (avati)

Comment 10 Anand Avati 2013-09-14 22:47:49 UTC
REVIEW: http://review.gluster.org/5903 (gfapi: implement a minimial glfs_fini()) posted (#3) for review on release-3.4 by Anand Avati (avati)

Comment 11 Anand Avati 2013-09-17 18:43:09 UTC
COMMIT: http://review.gluster.org/5788 committed in master by Anand Avati (avati) 
------
commit d7f7510f218e656cf2ba3bd821b80c14ab53846a
Author: Anand Avati <avati>
Date:   Wed Aug 28 18:05:03 2013 -0700

    gfapi: implement a minimial glfs_fini()
    
    At the very least, we should PARENT_DOWN on the currently active
    graph and disconnect ourselves from glusterd.
    
    Further cleanups underway.
    
    Change-Id: I9276686a84b0975b5ce272b4cbec1b80920d5c5c
    BUG: 1004519
    Signed-off-by: Anand Avati <avati>
    Reviewed-on: http://review.gluster.org/5788
    Tested-by: Gluster Build System <jenkins.com>
    Reviewed-by: Raghavendra Bhat <raghavendra>
    Reviewed-by: Amar Tumballi <amarts>

Comment 12 Anand Avati 2013-09-19 21:29:34 UTC
COMMIT: http://review.gluster.org/5903 committed in release-3.4 by Anand Avati (avati) 
------
commit 35b09786463629352ba10657a7888bc7da918474
Author: Anand Avati <avati>
Date:   Wed Aug 28 18:05:03 2013 -0700

    gfapi: implement a minimial glfs_fini()
    
    At the very least, we should PARENT_DOWN on the currently active
    graph and disconnect ourselves from glusterd.
    
    Further cleanups underway.
    
    Change-Id: I9276686a84b0975b5ce272b4cbec1b80920d5c5c
    BUG: 1004519
    Signed-off-by: Anand Avati <avati>
    Reviewed-on: http://review.gluster.org/5903
    Tested-by: Gluster Build System <jenkins.com>

Comment 13 Anand Avati 2013-10-15 19:29:21 UTC
REVIEW: http://review.gluster.org/6092 (gfapi: fix return value of glfs_fini) posted (#1) for review on master by Krishnan Parthasarathi (kparthas)

Comment 14 Anand Avati 2013-10-15 19:40:05 UTC
REVIEW: http://review.gluster.org/6092 (gfapi: fix return value of glfs_fini) posted (#2) for review on master by Krishnan Parthasarathi (kparthas)

Comment 15 Anand Avati 2013-10-15 19:45:26 UTC
REVIEW: http://review.gluster.org/6093 (gfapi: fix return value of glfs_fini) posted (#1) for review on release-3.4 by Krishnan Parthasarathi (kparthas)

Comment 16 Anand Avati 2013-10-17 19:49:35 UTC
COMMIT: http://review.gluster.org/6092 committed in master by Anand Avati (avati) 
------
commit dfa08fe05843b5da077ddbe059d744ce3864044e
Author: Krishnan Parthasarathi <kparthas>
Date:   Wed Oct 16 00:56:01 2013 +0530

    gfapi: fix return value of glfs_fini
    
    Change-Id: I5b90c7602334226a978bbdae2f9516e8701b403f
    BUG: 1004519
    Signed-off-by: Krishnan Parthasarathi <kparthas>
    Reviewed-on: http://review.gluster.org/6092
    Reviewed-by: Shyamsundar Ranganathan <srangana>
    Tested-by: Gluster Build System <jenkins.com>
    Reviewed-by: Anand Avati <avati>

Comment 17 Anand Avati 2013-10-30 22:22:02 UTC
COMMIT: http://review.gluster.org/6093 committed in release-3.4 by Anand Avati (avati) 
------
commit 56be81f0e06b6158da86328d293fbb09d711791d
Author: Krishnan Parthasarathi <kparthas>
Date:   Wed Oct 16 00:56:01 2013 +0530

    gfapi: fix return value of glfs_fini
    
    Change-Id: I5b90c7602334226a978bbdae2f9516e8701b403f
    BUG: 1004519
    Signed-off-by: Krishnan Parthasarathi <kparthas>
    Reviewed-on: http://review.gluster.org/6093
    Tested-by: Gluster Build System <jenkins.com>
    Reviewed-by: Anand Avati <avati>

Comment 18 Niels de Vos 2014-04-17 13:14:16 UTC
This bug is getting closed because a release has been made available that should address the reported issue. In case the problem is still not fixed with glusterfs-3.4.3, please reopen this bug report.

glusterfs-3.4.3 has been announced on the Gluster Developers mailinglist [1], packages for several distributions should already be or become available in the near future. Keep an eye on the Gluster Users mailinglist [2] and the update infrastructure for your distribution.

The fix for this bug likely to be included in all future GlusterFS releases i.e. release > 3.4.3. In the same line the recent release i.e. glusterfs-3.5.0 [3] likely to have the fix. You can verify this by reading the comments in this bug report and checking for comments mentioning "committed in release-3.5".

[1] http://thread.gmane.org/gmane.comp.file-systems.gluster.devel/5978
[2] http://news.gmane.org/gmane.comp.file-systems.gluster.user
[3] http://thread.gmane.org/gmane.comp.file-systems.gluster.devel/6137


Note You need to log in before you can comment on or make changes to this bug.