Bug 855787
Summary: | glusterfs: client crash while testing statedump | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | [Red Hat Storage] Red Hat Gluster Storage | Reporter: | Sachidananda Urs <sac> | ||||||
Component: | glusterfs | Assignee: | Raghavendra Bhat <rabhat> | ||||||
Status: | CLOSED ERRATA | QA Contact: | Sudhir D <sdharane> | ||||||
Severity: | unspecified | Docs Contact: | |||||||
Priority: | high | ||||||||
Version: | 2.0 | CC: | amarts, rhs-bugs, vbellur | ||||||
Target Milestone: | --- | ||||||||
Target Release: | --- | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Whiteboard: | |||||||||
Fixed In Version: | glusterfs-3.4.0qa8 | Doc Type: | Bug Fix | ||||||
Doc Text: | Story Points: | --- | |||||||
Clone Of: | Environment: | ||||||||
Last Closed: | 2013-09-23 22:33:22 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: | 885008 | ||||||||
Bug Blocks: | |||||||||
Attachments: |
|
Description
Sachidananda Urs
2012-09-10 09:43:07 UTC
Created attachment 611372 [details]
Client log file
Created attachment 611373 [details]
Core file
The below patch should fix it... amar@ganaka:~/work/glusterfs$ git diff diff --git a/xlators/performance/write-behind/src/write-behind.c b/xlators/performance/write-behind/src/write-behind.c index ad1e5f0..59cbd00 100644 --- a/xlators/performance/write-behind/src/write-behind.c +++ b/xlators/performance/write-behind/src/write-behind.c @@ -2604,7 +2604,8 @@ wb_flush_helper (call_frame_t *frame, xlator_t *this, fd_t *fd, dict_t *xdata) wb_request_unref (local->request); } - if (conf->flush_behind) { + int flag = conf->flush_behind; + if (flag) { flush_frame = copy_frame (frame); if (flush_frame == NULL) { op_errno = ENOMEM; @@ -2628,7 +2629,7 @@ wb_flush_helper (call_frame_t *frame, xlator_t *this, fd_t *fd, dict_t *xdata) STACK_DESTROY (process_frame->root); } - if (conf->flush_behind) { + if (flag) { STACK_UNWIND_STRICT (flush, frame, op_ret, op_errno, NULL); } -------------- also, if the proposed patch @ http://review.gluster.org/3947 goes in, then this race won't exist. patch accepted upstream Crash still happens and related to bug: https://bugzilla.redhat.com/show_bug.cgi?id=885008 Will continue testing once bug 885008 is fixed. marking as MODIFIED as there is no work for this particular bug, but will keep it in MODIFIED till the blocker bug for verification is fixed. Verified with load on multiple clients and series of graph changes in a loop on the servers. No crashes seen. Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory, and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. http://rhn.redhat.com/errata/RHBA-2013-1262.html |