* [PATCH] mm: Fix typo in VM_MPX definition
@ 2017-09-18 14:02 Kirill A. Shutemov
2017-09-18 14:36 ` Rik van Riel
2017-09-26 19:19 ` Kyle Huey
0 siblings, 2 replies; 3+ messages in thread
From: Kirill A. Shutemov @ 2017-09-18 14:02 UTC (permalink / raw)
To: Andrew Morton
Cc: linux-mm, linux-kernel, Kirill A. Shutemov, Rik van Riel,
Dave Hansen
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 <kirill.shutemov@linux.intel.com>
Fixes: df3735c5b40f ("x86,mpx: make mpx depend on x86-64 to free up VMA flag")
Cc: Rik van Riel <riel@redhat.com>
Cc: Dave Hansen <dave.hansen@intel.com>
---
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_MPX VM_HIGH_ARCH_BIT_4
+# define VM_MPX VM_HIGH_ARCH_4
#else
# define VM_MPX VM_NONE
#endif
--
2.14.1
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] mm: Fix typo in VM_MPX definition
2017-09-18 14:02 [PATCH] mm: Fix typo in VM_MPX definition Kirill A. Shutemov
@ 2017-09-18 14:36 ` Rik van Riel
2017-09-26 19:19 ` Kyle Huey
1 sibling, 0 replies; 3+ messages in thread
From: Rik van Riel @ 2017-09-18 14:36 UTC (permalink / raw)
To: Kirill A. Shutemov, Andrew Morton; +Cc: linux-mm, linux-kernel, Dave Hansen
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 <kirill.shutemov@linux.intel.com>
> Fixes: df3735c5b40f ("x86,mpx: make mpx depend on x86-64 to free up
> VMA flag")
> Cc: Rik van Riel <riel@redhat.com>
> Cc: Dave Hansen <dave.hansen@intel.com>
Reviewed-by: Rik van Riel <riel@redhat.com>
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] mm: Fix typo in VM_MPX definition
2017-09-18 14:02 [PATCH] mm: Fix typo in VM_MPX definition Kirill A. Shutemov
2017-09-18 14:36 ` Rik van Riel
@ 2017-09-26 19:19 ` Kyle Huey
1 sibling, 0 replies; 3+ messages in thread
From: Kyle Huey @ 2017-09-26 19:19 UTC (permalink / raw)
To: Linus Torvalds
Cc: Robert O'Callahan, Kirill A. Shutemov, Andrew Morton,
linux-mm, linux-kernel, Rik van Riel, Dave Hansen
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
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-09-26 19:19 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-09-18 14:02 [PATCH] mm: Fix typo in VM_MPX definition Kirill A. Shutemov
2017-09-18 14:36 ` Rik van Riel
2017-09-26 19:19 ` Kyle Huey
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).