Bug 1361678
| Summary: | thread CPU saturation limiting throughput on write workloads | ||
|---|---|---|---|
| Product: | [Community] GlusterFS | Reporter: | Pranith Kumar K <pkarampu> |
| Component: | fuse | Assignee: | Pranith Kumar K <pkarampu> |
| Status: | CLOSED CURRENTRELEASE | QA Contact: | |
| Severity: | unspecified | Docs Contact: | |
| Priority: | unspecified | ||
| Version: | mainline | CC: | bugs, dblack, jahernan, mpillai, pkarampu, rtalur, sarumuga |
| Target Milestone: | --- | Keywords: | Performance, Triaged |
| Target Release: | --- | ||
| Hardware: | x86_64 | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Fixed In Version: | glusterfs-3.9.0 | Doc Type: | If docs needed, set a value |
| Doc Text: | Story Points: | --- | |
| Clone Of: | 1349953 | Environment: | |
| Last Closed: | 2017-03-27 18:19:04 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: | 1349953 | ||
| Bug Blocks: | 1359180 | ||
|
Description
Pranith Kumar K
2016-07-29 17:40:42 UTC
REVIEW: http://review.gluster.org/15051 (mgmt/glusterd: Enable client-io-threads by default) posted (#1) for review on master by Pranith Kumar Karampuri (pkarampu) REVIEW: http://review.gluster.org/15051 (mgmt/glusterd: Enable client-io-threads by default) posted (#2) for review on master by Pranith Kumar Karampuri (pkarampu) REVIEW: http://review.gluster.org/15070 (cluster/afr: copy loc before passing to syncop) posted (#1) for review on master by Pranith Kumar Karampuri (pkarampu) REVIEW: http://review.gluster.org/15070 (cluster/afr: copy loc before passing to syncop) posted (#2) for review on master by Pranith Kumar Karampuri (pkarampu) COMMIT: http://review.gluster.org/15070 committed in master by Pranith Kumar Karampuri (pkarampu) ------ commit 67ebde53ea26065d41cacf054fa4337c22de5903 Author: Pranith Kumar K <pkarampu> Date: Tue Aug 2 15:19:00 2016 +0530 cluster/afr: copy loc before passing to syncop Problem: When io-threads is enabled on the client side, io-threads destroys the call-stub in which the loc is stored as soon as the c-stack unwinds. Because afr is creating a syncop with the address of loc passed in setxattr by the time syncop tries to access it, io-threads would have already freed the call-stub. This will lead to crash. Fix: Copy loc to frame->local and use it's address. BUG: 1361678 Change-Id: I16987e491e24b0b4e3d868a6968e802e47c77f7a Signed-off-by: Pranith Kumar K <pkarampu> Reviewed-on: http://review.gluster.org/15070 Reviewed-by: Ravishankar N <ravishankar> Smoke: Gluster Build System <jenkins.org> NetBSD-regression: NetBSD Build System <jenkins.org> CentOS-regression: Gluster Build System <jenkins.org> COMMIT: http://review.gluster.org/15051 committed in master by Atin Mukherjee (amukherj) ------ commit e068c1997314046658dd502e9118dab32decf879 Author: Pranith Kumar K <pkarampu> Date: Fri Jul 29 21:55:58 2016 +0530 mgmt/glusterd: Enable client-io-threads by default BUG: 1361678 Change-Id: Ic013a772ffc3f8c93673bbee064ff4cc372fe128 Signed-off-by: Pranith Kumar K <pkarampu> Reviewed-on: http://review.gluster.org/15051 NetBSD-regression: NetBSD Build System <jenkins.org> CentOS-regression: Gluster Build System <jenkins.org> Smoke: Gluster Build System <jenkins.org> Reviewed-by: Prashanth Pai <ppai> Reviewed-by: Atin Mukherjee <amukherj> 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.9.0, please open a new bug report. glusterfs-3.9.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/2016-November/029281.html [2] https://www.gluster.org/pipermail/gluster-users/ |