diff options
| author | Linus Torvalds <torvalds@linux-foundation.org> | 2014-12-17 19:52:37 (GMT) |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2014-12-17 19:52:37 (GMT) |
| commit | cf3c0a1579eff90195a791c5f464463c1011ef4a (patch) | |
| tree | e66fcbf1d65d1a43b9efde092045890f6aa52487 /arch/x86/mm/fault.c | |
| parent | cc669743a39e3f61c9ca5e786e959bf478ccd197 (diff) | |
| download | linux-cf3c0a1579eff90195a791c5f464463c1011ef4a.tar.xz | |
x86: mm: fix VM_FAULT_RETRY handling
My commit 26178ec11ef3 ("x86: mm: consolidate VM_FAULT_RETRY handling")
had a really stupid typo: the FAULT_FLAG_USER bit is in the 'flags'
variable, not the 'fault' variable. Duh,
The one silver lining in this is that Dave finding this at least
confirms that trinity actually triggers this special path easily, in a
way normal use does not.
Reported-by: Dave Jones <davej@redhat.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'arch/x86/mm/fault.c')
| -rw-r--r-- | arch/x86/mm/fault.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/x86/mm/fault.c b/arch/x86/mm/fault.c index b74a7e1..38dcec4 100644 --- a/arch/x86/mm/fault.c +++ b/arch/x86/mm/fault.c @@ -1247,7 +1247,7 @@ good_area: } /* User mode? Just return to handle the fatal exception */ - if (fault & FAULT_FLAG_USER) + if (flags & FAULT_FLAG_USER) return; /* Not returning to user mode? Handle exceptions or die: */ |