Bug 1448364 - limited throughput with disperse volume over small number of bricks
Summary: limited throughput with disperse volume over small number of bricks
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: rpc
Version: mainline
Hardware: x86_64
OS: Linux
unspecified
low
Target Milestone: ---
Assignee: Raghavendra G
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks: 1358606 1420796 1450355 1450357 1450359 1456259
TreeView+ depends on / blocked
 
Reported: 2017-05-05 09:45 UTC by Raghavendra G
Modified: 2017-09-05 17:29 UTC (History)
10 users (show)

Fixed In Version: glusterfs-3.12.0
Clone Of: 1358606
: 1450355 1450357 1450359 1456259 (view as bug list)
Environment:
Last Closed: 2017-09-05 17:29:02 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

Comment 1 Worker Ant 2017-05-08 05:42:23 UTC
REVIEW: https://review.gluster.org/15036 (event/epoll: Add back socket for polling of events immediately after              reading the entire rpc message from the wire) posted (#3) for review on master by Raghavendra G (rgowdapp)

Comment 2 Worker Ant 2017-05-08 05:42:28 UTC
REVIEW: https://review.gluster.org/17200 (mount/fuse: Handle racing notify on more than one graph properly) posted (#1) for review on master by Raghavendra G (rgowdapp)

Comment 3 Worker Ant 2017-05-08 06:11:33 UTC
REVIEW: https://review.gluster.org/17200 (mount/fuse: Handle racing notify on more than one graph properly) posted (#2) for review on master by Raghavendra G (rgowdapp)

Comment 4 Worker Ant 2017-05-08 06:11:38 UTC
REVIEW: https://review.gluster.org/15036 (event/epoll: Add back socket for polling of events immediately after              reading the entire rpc message from the wire) posted (#4) for review on master by Raghavendra G (rgowdapp)

Comment 5 Worker Ant 2017-05-08 06:31:53 UTC
REVIEW: https://review.gluster.org/17200 (mount/fuse: Handle racing notify on more than one graph properly) posted (#3) for review on master by Raghavendra G (rgowdapp)

Comment 6 Worker Ant 2017-05-08 06:31:58 UTC
REVIEW: https://review.gluster.org/15036 (event/epoll: Add back socket for polling of events immediately after              reading the entire rpc message from the wire) posted (#5) for review on master by Raghavendra G (rgowdapp)

Comment 7 Worker Ant 2017-05-08 07:02:49 UTC
REVIEW: https://review.gluster.org/17200 (mount/fuse: Handle racing notify on more than one graph properly) posted (#4) for review on master by Raghavendra G (rgowdapp)

Comment 8 Worker Ant 2017-05-08 07:02:54 UTC
REVIEW: https://review.gluster.org/15036 (event/epoll: Add back socket for polling of events immediately after              reading the entire rpc message from the wire) posted (#6) for review on master by Raghavendra G (rgowdapp)

Comment 9 Worker Ant 2017-05-08 14:19:56 UTC
REVIEW: https://review.gluster.org/17200 (mount/fuse: Handle racing notify on more than one graph properly) posted (#5) for review on master by Raghavendra G (rgowdapp)

Comment 10 Worker Ant 2017-05-08 14:20:01 UTC
REVIEW: https://review.gluster.org/15036 (event/epoll: Add back socket for polling of events immediately after              reading the entire rpc message from the wire) posted (#7) for review on master by Raghavendra G (rgowdapp)

Comment 11 Worker Ant 2017-05-09 11:29:46 UTC
REVIEW: https://review.gluster.org/17200 (mount/fuse: Handle racing notify on more than one graph properly) posted (#6) for review on master by Raghavendra G (rgowdapp)

Comment 12 Worker Ant 2017-05-09 11:29:52 UTC
REVIEW: https://review.gluster.org/15036 (event/epoll: Add back socket for polling of events immediately after              reading the entire rpc message from the wire) posted (#8) for review on master by Raghavendra G (rgowdapp)

Comment 13 Worker Ant 2017-05-09 11:39:02 UTC
REVIEW: https://review.gluster.org/17200 (mount/fuse: Handle racing notify on more than one graph properly) posted (#7) for review on master by Raghavendra G (rgowdapp)

Comment 14 Worker Ant 2017-05-09 11:39:06 UTC
REVIEW: https://review.gluster.org/15036 (event/epoll: Add back socket for polling of events immediately after              reading the entire rpc message from the wire) posted (#9) for review on master by Raghavendra G (rgowdapp)

Comment 15 Worker Ant 2017-05-10 09:14:33 UTC
COMMIT: https://review.gluster.org/17200 committed in master by Raghavendra G (rgowdapp) 
------
commit e71119e942eb016ba5a11c3f986715f16da10b82
Author: Raghavendra G <rgowdapp>
Date:   Thu Nov 10 10:56:26 2016 +0530

    mount/fuse: Handle racing notify on more than one graph properly
    
    Make sure that we always use latest graph as a candidate for
    active-subvol.
    
    Change-Id: Ie37c818366f28ba6b1570d65a9eb17697d38a6c5
    BUG: 1448364
    Signed-off-by: Raghavendra G <rgowdapp>
    Reviewed-on: https://review.gluster.org/17200
    Smoke: Gluster Build System <jenkins.org>
    Reviewed-by: Amar Tumballi <amarts>
    NetBSD-regression: NetBSD Build System <jenkins.org>
    Reviewed-by: Jeff Darcy <jeff.us>
    CentOS-regression: Gluster Build System <jenkins.org>

Comment 16 Worker Ant 2017-05-10 10:16:57 UTC
REVIEW: https://review.gluster.org/15036 (event/epoll: Add back socket for polling of events immediately after              reading the entire rpc message from the wire) posted (#10) for review on master by Raghavendra G (rgowdapp)

Comment 17 Worker Ant 2017-05-10 10:17:02 UTC
REVIEW: https://review.gluster.org/17234 (tests/lock_revocation: mark as bad) posted (#1) for review on master by Raghavendra G (rgowdapp)

Comment 18 Worker Ant 2017-05-10 14:56:57 UTC
COMMIT: https://review.gluster.org/17234 committed in master by Jeff Darcy (jeff.us) 
------
commit d5865881de5653a0e810093a9867ab3962d00f67
Author: Raghavendra G <rgowdapp>
Date:   Wed May 10 15:44:33 2017 +0530

    tests/lock_revocation: mark as bad
    
    The test is failing in master. see gluster-devel for more details.
    
    Change-Id: I7a589ad2c54bd55d62f4e66fdf8037c19fc123ea
    BUG: 1448364
    Signed-off-by: Raghavendra G <rgowdapp>
    Reviewed-on: https://review.gluster.org/17234
    NetBSD-regression: NetBSD Build System <jenkins.org>
    Smoke: Gluster Build System <jenkins.org>
    CentOS-regression: Gluster Build System <jenkins.org>
    Reviewed-by: Jeff Darcy <jeff.us>

Comment 19 Worker Ant 2017-05-12 05:26:45 UTC
COMMIT: https://review.gluster.org/15036 committed in master by Raghavendra G (rgowdapp) 
------
commit cea8b702506ff914deadd056f4b7dd20a3ca7670
Author: Raghavendra G <rgowdapp>
Date:   Fri May 5 15:21:30 2017 +0530

    event/epoll: Add back socket for polling of events immediately after
                 reading the entire rpc message from the wire
    
    Currently socket is added back for future events after higher layers
    (rpc, xlators etc) have processed the message. If message processing
    involves signficant delay (as in writev replies processed by Erasure
    Coding), performance takes hit. Hence this patch modifies
    transport/socket to add back the socket for polling of events
    immediately after reading the entire rpc message, but before
    notification to higher layers.
    
    credits: Thanks to "Kotresh Hiremath Ravishankar"
             <khiremat> for assitance in fixing a regression in
             bitrot caused by this patch.
    
    Change-Id: I04b6b9d0b51a1cfb86ecac3c3d87a5f388cf5800
    BUG: 1448364
    Signed-off-by: Raghavendra G <rgowdapp>
    Reviewed-on: https://review.gluster.org/15036
    CentOS-regression: Gluster Build System <jenkins.org>
    NetBSD-regression: NetBSD Build System <jenkins.org>
    Smoke: Gluster Build System <jenkins.org>
    Reviewed-by: Amar Tumballi <amarts>

Comment 20 Worker Ant 2017-05-28 07:39:29 UTC
REVIEW: https://review.gluster.org/17391 (event/epoll: Add back socket for polling of events immediately after reading the entire rpc message from the wire) posted (#2) for review on release-3.11 by Raghavendra G (rgowdapp)

Comment 21 Worker Ant 2017-05-28 10:28:52 UTC
REVIEW: https://review.gluster.org/17391 (event/epoll: Add back socket for polling of events immediately after reading the entire rpc message from the wire) posted (#3) for review on release-3.11 by Raghavendra G (rgowdapp)

Comment 22 Milind Changire 2017-08-21 09:40:08 UTC
https://review.gluster.org/17391 has been merged

Comment 23 Shyamsundar 2017-09-05 17:29:02 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.12.0, please open a new bug report.

glusterfs-3.12.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/announce/2017-September/000082.html
[2] https://www.gluster.org/pipermail/gluster-users/


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