Bug 1445189 - Bug in stripe xlator makes write op never returned
Summary: Bug in stripe xlator makes write op never returned
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: GlusterFS
Classification: Community
Component: stripe
Version: mainline
Hardware: Unspecified
OS: Linux
unspecified
high
Target Milestone: ---
Assignee: bugs@gluster.org
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2017-04-25 08:35 UTC by Zhang Huan
Modified: 2017-04-25 11:16 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2017-04-25 11:16:12 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)
patch file to fix this issue (3.41 KB, application/mbox)
2017-04-25 08:42 UTC, Zhang Huan
no flags Details

Description Zhang Huan 2017-04-25 08:35:39 UTC
Description of problem:

There is a lock issue in the stripe xlator. It took the wrong lock for a frame so that the call counter might not be updated correctly, making an fop never returned.

This issue is easy to reproduce when gluster client is configured with more than one event threads and request size is larger than stripe size. 

Version-Release number of selected component (if applicable):


How reproducible:
Very often

Steps to Reproduce:
1. create a stripe volume
2. use fio to write to this volume, make sure block size larger than stripe size

Actual results:
fio process hangs

Expected results:
fio finish test successfully

Additional info:

Comment 1 Zhang Huan 2017-04-25 08:42:47 UTC
Created attachment 1273851 [details]
patch file to fix this issue

I don't know why I got notified that I don't have permission to submit this patch. I also notice that I don't permission to sign into review.gluster.org with my github account. Therefore I attach the patch of this fix here. Hope it would help.

Comment 2 Kaleb KEITHLEY 2017-04-25 11:16:12 UTC
Stripe xlator is deprecated. Use the shard option instead.


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