Bug 1303995 - SMB: SMB crashes with AIO enabled on reads + vers=3.0
Summary: SMB: SMB crashes with AIO enabled on reads + vers=3.0
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: gluster-smb
Version: mainline
Hardware: x86_64
OS: Linux
unspecified
urgent
Target Milestone: ---
Assignee: rjoseph
QA Contact:
URL:
Whiteboard:
Depends On: 1302901 1311146
Blocks: 1311578
TreeView+ depends on / blocked
 
Reported: 2016-02-02 16:21 UTC by rjoseph
Modified: 2016-06-16 13:56 UTC (History)
13 users (show)

Fixed In Version: glusterfs-3.8rc2
Clone Of: 1302901
: 1311578 (view as bug list)
Environment:
Last Closed: 2016-06-16 13:56:34 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

Comment 1 Vijay Bellur 2016-02-02 16:32:33 UTC
REVIEW: http://review.gluster.org/13340 (libgfapi: fd close is not correctly handled for async fop) posted (#1) for review on master by Rajesh Joseph (rjoseph)

Comment 2 Vijay Bellur 2016-02-03 12:07:58 UTC
REVIEW: http://review.gluster.org/13340 (libgfapi: fd close is not correctly handled for async fop) posted (#2) for review on master by Rajesh Joseph (rjoseph)

Comment 3 Vijay Bellur 2016-02-03 15:31:30 UTC
REVIEW: http://review.gluster.org/13340 (libgfapi: glfd close is not correctly handled for async fop) posted (#3) for review on master by Rajesh Joseph (rjoseph)

Comment 4 Vijay Bellur 2016-02-03 15:43:12 UTC
REVIEW: http://review.gluster.org/13340 (libgfapi: glfd close is not correctly handled for async fop) posted (#4) for review on master by Rajesh Joseph (rjoseph)

Comment 5 Vijay Bellur 2016-02-04 07:01:29 UTC
REVIEW: http://review.gluster.org/13340 (libgfapi: glfd close is not correctly handled for async fop) posted (#5) for review on master by Rajesh Joseph (rjoseph)

Comment 6 Vijay Bellur 2016-02-04 10:15:31 UTC
REVIEW: http://review.gluster.org/13340 (libgfapi: glfd close is not correctly handled for async fop) posted (#6) for review on master by Rajesh Joseph (rjoseph)

Comment 7 Vijay Bellur 2016-02-04 16:05:18 UTC
REVIEW: http://review.gluster.org/13340 (libgfapi: glfd close is not correctly handled for async fop) posted (#7) for review on master by Rajesh Joseph (rjoseph)

Comment 8 Vijay Bellur 2016-02-04 17:46:36 UTC
REVIEW: http://review.gluster.org/13340 (libgfapi: glfd close is not correctly handled for async fop) posted (#8) for review on master by Rajesh Joseph (rjoseph)

Comment 9 Vijay Bellur 2016-02-04 18:28:36 UTC
REVIEW: http://review.gluster.org/13340 (libgfapi: glfd close is not correctly handled for async fop) posted (#9) for review on master by Rajesh Joseph (rjoseph)

Comment 10 Vijay Bellur 2016-02-05 04:21:23 UTC
REVIEW: http://review.gluster.org/13340 (libgfapi: glfd close is not correctly handled for async fop) posted (#10) for review on master by Rajesh Joseph (rjoseph)

Comment 11 Vijay Bellur 2016-02-11 05:46:33 UTC
COMMIT: http://review.gluster.org/13340 committed in master by Shyamsundar Ranganathan (srangana) 
------
commit 88d772c05c45c467bfccebfc51f6a0e0ea9ca287
Author: Rajesh Joseph <rjoseph>
Date:   Tue Feb 2 04:22:04 2016 +0530

    libgfapi: glfd close is not correctly handled for async fop
    
    There is chance that before the async fop is complete client can send
    a close. libgfapi destroys glfd on close. Therefore it can lead to
    crash or unexpected behaviour when the pening fop reaches libgfapi
    layer. Currently we don't provide any api to cancel these outstanding
    fops neither we check if the glfd is already closed or not.
    
    Therefore as a fix provided refcount for glfd. Each fop (sync or async)
    will take a ref and once the fop is complete it will unref the refcount.
    We should not call the registered callback function if glfd is already
    closed. To achieve this we maintain state of glfd so that we can safely
    take a call if the fd is closed or not.
    
    Change-Id: Ibe71b2225312db3f1be66b244fcf8826c70c357d
    BUG: 1303995
    Signed-off-by: Rajesh Joseph <rjoseph>
    Reviewed-on: http://review.gluster.org/13340
    Smoke: Gluster Build System <jenkins.com>
    CentOS-regression: Gluster Build System <jenkins.com>
    Reviewed-by: Shyamsundar Ranganathan <srangana>
    NetBSD-regression: NetBSD Build System <jenkins.org>

Comment 12 Niels de Vos 2016-02-28 09:41:05 UTC
Bug 1311146 reports a regression that was introduced with this change.

Comment 13 rjoseph 2016-04-26 04:23:36 UTC
http://review.gluster.org/13666 fixes regression reported by Bug 1311146.

Comment 14 Niels de Vos 2016-06-16 13:56:34 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.8.0, please open a new bug report.

glusterfs-3.8.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://blog.gluster.org/2016/06/glusterfs-3-8-released/
[2] http://thread.gmane.org/gmane.comp.file-systems.gluster.user


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