x86: mm: fix VM_FAULT_RETRY handling
authorLinus Torvalds <torvalds@linux-foundation.org>
Wed, 17 Dec 2014 19:52:37 +0000 (11:52 -0800)
committerLinus Torvalds <torvalds@linux-foundation.org>
Wed, 17 Dec 2014 19:52:37 +0000 (11:52 -0800)
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>
arch/x86/mm/fault.c

index b74a7e1..38dcec4 100644 (file)
@@ -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: */