Bug 1397754

Summary: [SAMBA-CIFS] : IO hungs in cifs mount while graph switch on & off
Product: [Community] GlusterFS Reporter: rjoseph
Component: libgfapiAssignee: rjoseph
Status: CLOSED CURRENTRELEASE QA Contact: Sudhir D <sdharane>
Severity: high Docs Contact:
Priority: high    
Version: mainlineCC: bugs, rgowdapp, rhs-smb, rjoseph, sbhaloth, vdas
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: glusterfs-3.10.0 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: 1396449
: 1399914 1399915 1399916 (view as bug list) Environment:
Last Closed: 2017-03-06 17:36:17 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1396449, 1399914, 1399915, 1399916    

Comment 1 Worker Ant 2016-11-23 10:31:57 UTC
REVIEW: http://review.gluster.org/15913 (gfapi: glfs_subvol_done should not call glfs_lock) posted (#1) for review on master by Rajesh Joseph (rjoseph)

Comment 2 Worker Ant 2016-11-25 13:41:33 UTC
REVIEW: http://review.gluster.org/15913 (gfapi: glfs_subvol_done should not call glfs_lock) posted (#2) for review on master by Rajesh Joseph (rjoseph)

Comment 3 Worker Ant 2016-11-25 13:43:55 UTC
REVIEW: http://review.gluster.org/15913 (gfapi: glfs_subvol_done should not call glfs_lock) posted (#3) for review on master by Rajesh Joseph (rjoseph)

Comment 4 Worker Ant 2016-11-25 13:56:35 UTC
REVIEW: http://review.gluster.org/15913 (gfapi: glfs_subvol_done should not call glfs_lock) posted (#4) for review on master by Rajesh Joseph (rjoseph)

Comment 5 Worker Ant 2016-11-29 16:40:19 UTC
REVIEW: http://review.gluster.org/15913 (gfapi: glfs_subvol_done should NOT wait for graph migration.) posted (#5) for review on master by Rajesh Joseph (rjoseph)

Comment 6 Worker Ant 2016-11-30 07:52:46 UTC
COMMIT: http://review.gluster.org/15913 committed in master by Niels de Vos (ndevos) 
------
commit 17d10b42fc4041442e6cd0bfda45944edea498c6
Author: Rajesh Joseph <rjoseph>
Date:   Tue Nov 22 22:25:42 2016 +0530

    gfapi: glfs_subvol_done should NOT wait for graph migration.
    
    In graph_setup function glfs_subvol_done is called which
    is executed in an epoll thread. glfs_lock waits on other
    thread to finish graph migration. This can lead to dead lock
    if we consume all the epoll threads.
    
    In general any call-back function executed in epoll thread
    should not call any blocking call which waits on a network
    reply either directly or indirectly, e.g. syncop functions
    should not be called in these threads.
    
    As a fix we should not wait for migration in the call-back path.
    
    Change-Id: If96d0689fe1b4d74631e383048cdc30b01690dc2
    BUG: 1397754
    Signed-off-by: Rajesh Joseph <rjoseph>
    Reviewed-on: http://review.gluster.org/15913
    NetBSD-regression: NetBSD Build System <jenkins.org>
    Smoke: Gluster Build System <jenkins.org>
    Reviewed-by: Niels de Vos <ndevos>
    CentOS-regression: Gluster Build System <jenkins.org>

Comment 7 Shyamsundar 2017-03-06 17:36:17 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.10.0, please open a new bug report.

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

[1] http://lists.gluster.org/pipermail/gluster-users/2017-February/030119.html
[2] https://www.gluster.org/pipermail/gluster-users/