Bug 476224 - convert NFS to new write_begin/write_end interfaces
Summary: convert NFS to new write_begin/write_end interfaces
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Enterprise Linux 5
Classification: Red Hat
Component: kernel
Version: 5.3
Hardware: All
OS: Linux
low
medium
Target Milestone: rc
: ---
Assignee: Jeff Layton
QA Contact: Red Hat Kernel QE team
URL:
Whiteboard:
Depends On: 445433
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-12-12 16:07 UTC by Jeff Layton
Modified: 2011-12-26 02:21 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2009-09-02 08:58:32 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
patch -- convert NFS to new aops (5.17 KB, patch)
2008-12-12 20:45 UTC, Jeff Layton
no flags Details | Diff
updated patch (5.32 KB, patch)
2009-01-09 20:22 UTC, Jeff Layton
no flags Details | Diff


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHSA-2009:1243 0 normal SHIPPED_LIVE Important: Red Hat Enterprise Linux 5.4 kernel security and bug fix update 2009-09-01 08:53:34 UTC

Description Jeff Layton 2008-12-12 16:07:10 UTC
Josef has backported write_begin and write_end to RHEL5.4. While the old prepare/commit_write interfaces are still there, he's adding an extra page copy into that codepath to fix a deadlock. This will incur a performance penalty. To avoid it, we'll want to convert NFS to use the new extended aops.

Comment 1 Jeff Layton 2008-12-12 20:45:24 UTC
Created attachment 326772 [details]
patch -- convert NFS to new aops

This patch builds cleanly, but needs testing.

Comment 2 Jeff Layton 2008-12-15 12:57:19 UTC
Patch seems to work. I've done a bit of basic testing with it and haven't noticed any regressions. We'll certainly want to test it more extensively though, particularly in the write codepaths.

Comment 3 Jeff Layton 2009-01-09 20:22:36 UTC
Created attachment 328585 [details]
updated patch

...to take into account the changes in Josef's patches

Comment 4 RHEL Program Management 2009-01-27 20:42:05 UTC
This request was evaluated by Red Hat Product Management for inclusion in a Red
Hat Enterprise Linux maintenance release.  Product Management has requested
further review of this request by Red Hat Engineering, for potential
inclusion in a Red Hat Enterprise Linux Update release for currently deployed
products.  This request is not yet committed for inclusion in an Update
release.

Comment 6 RHEL Program Management 2009-02-16 15:43:14 UTC
Updating PM score.

Comment 8 Don Zickus 2009-04-20 17:11:20 UTC
in kernel-2.6.18-140.el5
You can download this test kernel from http://people.redhat.com/dzickus/el5

Please do NOT transition this bugzilla state to VERIFIED until our QE team
has sent specific instructions indicating when to do so.  However feel free
to provide a comment indicating that this fix has been verified.

Comment 11 errata-xmlrpc 2009-09-02 08:58:32 UTC
An advisory has been issued which should help the problem
described in this bug report. This report is therefore being
closed with a resolution of ERRATA. For more information
on therefore solution and/or where to find the updated files,
please follow the link below. You may reopen this bug report
if the solution does not work for you.

http://rhn.redhat.com/errata/RHSA-2009-1243.html


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