Bug 626983 - poor NFS client performance on SMP and EL5
Summary: poor NFS client performance on SMP and EL5
Keywords:
Status: CLOSED DUPLICATE of bug 519126
Alias: None
Product: Red Hat Enterprise Linux 5
Classification: Red Hat
Component: kernel
Version: 5.5
Hardware: All
OS: Linux
low
high
Target Milestone: rc
: ---
Assignee: Jeff Layton
QA Contact: Red Hat Kernel QE team
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2010-08-24 19:20 UTC by Herbert van den Bergh
Modified: 2018-11-29 21:11 UTC (History)
9 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2010-12-04 12:15:09 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
NFS client BKL removal (86.41 KB, patch)
2010-08-24 19:21 UTC, Herbert van den Bergh
no flags Details | Diff
tarball with patches split into individual files (18.07 KB, application/octet-stream)
2010-10-05 12:17 UTC, Jeff Layton
no flags Details
patch -- forward ported patch (56.28 KB, patch)
2011-03-28 18:35 UTC, Jeff Layton
no flags Details | Diff

Description Herbert van den Bergh 2010-08-24 19:20:24 UTC
Contention for the BKL in NFS client code, combined with
CONFIG_PREEMPT_BKL being enabled in EL5, causes severe overhead on SMP
systems in scheduling and locking code.  The patch stream backported from
upstream modifies the NFS client code to remove the BKL contention.

Comment 1 Herbert van den Bergh 2010-08-24 19:21:38 UTC
Created attachment 440741 [details]
NFS client BKL removal

Comment 2 Jeff Layton 2010-10-05 12:17:47 UTC
Created attachment 451651 [details]
tarball with patches split into individual files

Tarball with patches split out as individual files. There are some definite merge conflicts with 5.6-ish kernels, so this will take some work if we want to go with this.

Comment 3 Jeff Layton 2010-10-07 13:30:23 UTC
I've forward ported the patches to more recent kernels and have test kernels on my people.redhat.com pages. Any testing of them would be appreciated:

http://people.redhat.com/jlayton/

Comment 4 Jeff Layton 2010-12-04 11:59:21 UTC
I've experimented with these patches over the last several months and haven't seen any problems. That said, they make me really nervous. This is just the sort of thing that can cause subtle, difficult to detect regressions.

At this point, I think this is too risky, especially given that RHEL6 has most of this already. For now I'm going to NAK this bug, and suggest that anyone that's hitting this problem consider a move to RHEL6.

Comment 5 RHEL Program Management 2010-12-04 12:15:09 UTC
Development Management has reviewed and declined this request.  You may appeal
this decision by reopening this request.

Comment 6 Jeff Layton 2011-03-28 18:35:20 UTC
Created attachment 488222 [details]
patch -- forward ported patch

For the record, here's the patchset I was playing with when I was considering this, if anyone is interested...

Comment 9 Harshula Jayasuriya 2014-12-18 09:25:50 UTC

*** This bug has been marked as a duplicate of bug 519126 ***


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