Re: [PATCH] mm: Fix typo in VM_MPX definition

2017-09-26 Thread Kyle Huey
Could we get this patch into the next 4.14 rc too?  This "typo" causes a bunch
of sections in /proc/N/maps to be incorrectly labelled [mpx] which confuses rr.
We could probably work around if it we had to but doing this right is trivial.

Thanks,

- Kyle


Re: [PATCH] mm: Fix typo in VM_MPX definition

2017-09-18 Thread Rik van Riel
On Mon, 2017-09-18 at 17:02 +0300, Kirill A. Shutemov wrote:
> There's typo in recent change of VM_MPX definition. We want it to be
> VM_HIGH_ARCH_4, not VM_HIGH_ARCH_BIT_4.

Ugh, indeed! Good catch!

> Signed-off-by: Kirill A. Shutemov 
> Fixes: df3735c5b40f ("x86,mpx: make mpx depend on x86-64 to free up
> VMA flag")
> Cc: Rik van Riel 
> Cc: Dave Hansen 

Reviewed-by: Rik van Riel 


[PATCH] mm: Fix typo in VM_MPX definition

2017-09-18 Thread Kirill A. Shutemov
There's typo in recent change of VM_MPX definition. We want it to be
VM_HIGH_ARCH_4, not VM_HIGH_ARCH_BIT_4.

Signed-off-by: Kirill A. Shutemov 
Fixes: df3735c5b40f ("x86,mpx: make mpx depend on x86-64 to free up VMA flag")
Cc: Rik van Riel 
Cc: Dave Hansen 
---
 include/linux/mm.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/linux/mm.h b/include/linux/mm.h
index f8c10d336e42..065d99deb847 100644
--- a/include/linux/mm.h
+++ b/include/linux/mm.h
@@ -240,7 +240,7 @@ extern unsigned int kobjsize(const void *objp);
 
 #if defined(CONFIG_X86_INTEL_MPX)
 /* MPX specific bounds table or bounds directory */
-# define VM_MPXVM_HIGH_ARCH_BIT_4
+# define VM_MPXVM_HIGH_ARCH_4
 #else
 # define VM_MPXVM_NONE
 #endif
-- 
2.14.1