Bug 682423

Summary:	better handling for temporary CIFS writeback errors
Product:	Red Hat Enterprise Linux 5	Reporter:	Jeff Layton <jlayton>
Component:	kernel	Assignee:	Jeff Layton <jlayton>
Status:	CLOSED WONTFIX	QA Contact:	Red Hat Kernel QE team <kernel-qe>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	5.6	CC:	bfields, rwheeler, sprabhu, steved
Target Milestone:	rc
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2012-11-15 19:57:36 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:

Description Jeff Layton 2011-03-05 12:03:01 UTC

Currently, when the CIFS code receives any error during writeback it will mark the page and mapping with an error and quit trying to write back. This is true even for temporary errors like EAGAIN during timeouts. I introduced a patch upstream to fix this a few months ago.

Opening this bug to consider backporting that for RHEL5.

Comment 1 Jeff Layton 2011-03-05 12:05:06 UTC

The patch I'm considering is this one. I don't have a great reproducer for this, but I think it's something we should consider even for RHEL5:

commit 941b853d779de3298e39f1eb4e252984464eaea8
Author: Jeff Layton <jlayton>
Date:   Tue Jan 11 07:24:01 2011 -0500

    cifs: don't fail writepages on -EAGAIN errors

Comment 2 Jeff Layton 2012-11-15 19:54:45 UTC

So far, I've not heard great hue and cry for this patch. At this point I think we're best off just closing this WONTFIX. We can revisit it later if there is need...

Comment 3 RHEL Program Management 2012-11-15 19:57:36 UTC

Development Management has reviewed and declined this request.
You may appeal this decision by reopening this request.