All of lore.kernel.org
 help / color / mirror / Atom feed
* 4.5-rc1: mm/gup.c warning when writing to /proc/self/mem
@ 2016-01-30 17:58 ` Dave Jones
  0 siblings, 0 replies; 6+ messages in thread
From: Dave Jones @ 2016-01-30 17:58 UTC (permalink / raw)
  To: linux-mm; +Cc: Linux Kernel

Hit this overnight. Just started seeing this after I added "create mmap's
of fd's we open()'d" to trinity.

	Dave

WARNING: CPU: 1 PID: 16733 at mm/gup.c:434 __get_user_pages+0x5f9/0x990()
CPU: 1 PID: 16733 Comm: trinity-c30 Tainted: G        W       4.5.0-rc1-think+ #12
 0000000000000009 000000006648ff5c ffff88000f0779a0 ffffffff99565971
------------[ cut here ]------------
WARNING: CPU: 0 PID: 16731 at mm/gup.c:434 __get_user_pages+0x5f9/0x990()
 0000000000000000 ffff88000f0779e0 ffffffff990b168f ffffffff992aba69
 ffff880450cf1000 0000000000000000 ffff88023780e600 0000000000000017
Call Trace:
 [<ffffffff99565971>] dump_stack+0x4e/0x7d
 [<ffffffff990b168f>] warn_slowpath_common+0x9f/0xe0
 [<ffffffff992aba69>] ? __get_user_pages+0x5f9/0x990
 [<ffffffff990b18aa>] warn_slowpath_null+0x1a/0x20
 [<ffffffff992aba69>] __get_user_pages+0x5f9/0x990
 [<ffffffff992ab470>] ? follow_page_mask+0x530/0x530
 [<ffffffff992ad54a>] ? __access_remote_vm+0xca/0x340
 [<ffffffff992ac2e2>] get_user_pages+0x52/0x60
 [<ffffffff992ad610>] __access_remote_vm+0x190/0x340
 [<ffffffff990f2531>] ? preempt_count_sub+0xc1/0x120
 [<ffffffff992ad480>] ? __might_fault+0xf0/0xf0
 [<ffffffff992ad417>] ? __might_fault+0x87/0xf0
 [<ffffffff992b607f>] access_remote_vm+0x1f/0x30
 [<ffffffff993c5703>] mem_rw.isra.15+0xe3/0x1d0
 [<ffffffff993c5833>] mem_write+0x43/0x50
 [<ffffffff9930a6ed>] __vfs_write+0xdd/0x260
 [<ffffffff9930a610>] ? __vfs_read+0x260/0x260
 [<ffffffff99d136cb>] ? mutex_lock_nested+0x38b/0x590
 [<ffffffff99133152>] ? __lock_is_held+0x92/0xd0
 [<ffffffff990f2531>] ? preempt_count_sub+0xc1/0x120
 [<ffffffff99131035>] ? update_fast_ctr+0x65/0x90
 [<ffffffff991310e7>] ? percpu_down_read+0x57/0xa0
 [<ffffffff99310bc4>] ? __sb_start_write+0xb4/0xf0
 [<ffffffff9930bec6>] vfs_write+0xf6/0x260
 [<ffffffff9930d84f>] SyS_write+0xbf/0x160
 [<ffffffff9930d790>] ? SyS_read+0x160/0x160
 [<ffffffff99002017>] ? trace_hardirqs_on_thunk+0x17/0x19
 [<ffffffff99d19557>] entry_SYSCALL_64_fastpath+0x12/0x6b
CPU: 0 PID: 16731 Comm: trinity-c28 Tainted: G        W       4.5.0-rc1-think+ #12
 0000000000000009 000000002962eec9 ffff8802e7b7f8d8 ffffffff99565971
 0000000000000000 ffff8802e7b7f918 ffffffff990b168f ffffffff992aba69
 ffff8803ed6f1000 00000000000000a0 ffff88023780e600 0000000000000017
Call Trace:
 [<ffffffff99565971>] dump_stack+0x4e/0x7d
 [<ffffffff990b168f>] warn_slowpath_common+0x9f/0xe0
 [<ffffffff992aba69>] ? __get_user_pages+0x5f9/0x990
 [<ffffffff990b18aa>] warn_slowpath_null+0x1a/0x20
 [<ffffffff992aba69>] __get_user_pages+0x5f9/0x990
 [<ffffffff99015979>] ? native_sched_clock+0x69/0x160
 [<ffffffff992ab470>] ? follow_page_mask+0x530/0x530
 [<ffffffff992ad54a>] ? __access_remote_vm+0xca/0x340
 [<ffffffff992ac2e2>] get_user_pages+0x52/0x60
 [<ffffffff992ad610>] __access_remote_vm+0x190/0x340
 [<ffffffff990f2531>] ? preempt_count_sub+0xc1/0x120
 [<ffffffff992ad480>] ? __might_fault+0xf0/0xf0
 [<ffffffff992ad417>] ? __might_fault+0x87/0xf0
 [<ffffffff992b607f>] access_remote_vm+0x1f/0x30
 [<ffffffff993c5703>] mem_rw.isra.15+0xe3/0x1d0
 [<ffffffff993c5833>] mem_write+0x43/0x50
 [<ffffffff9930a950>] do_loop_readv_writev+0xe0/0x110
 [<ffffffff993c57f0>] ? mem_rw.isra.15+0x1d0/0x1d0
 [<ffffffff9930c3bb>] do_readv_writev+0x38b/0x3c0
 [<ffffffff99132960>] ? trace_hardirqs_off_caller+0x70/0x110
 [<ffffffff993c57f0>] ? mem_rw.isra.15+0x1d0/0x1d0
 [<ffffffff9930c030>] ? vfs_write+0x260/0x260
 [<ffffffff99595e17>] ? debug_smp_processor_id+0x17/0x20
 [<ffffffff990f2531>] ? preempt_count_sub+0xc1/0x120
 [<ffffffff991330e5>] ? __lock_is_held+0x25/0xd0
 [<ffffffff991381e3>] ? mark_held_locks+0x23/0xc0
 [<ffffffff9926080a>] ? context_tracking_exit.part.5+0x2a/0x50
 [<ffffffff99138406>] ? trace_hardirqs_on_caller+0x186/0x280
 [<ffffffff9913850d>] ? trace_hardirqs_on+0xd/0x10
 [<ffffffff9930c4b9>] vfs_writev+0x59/0x70
 [<ffffffff9930e2fd>] SyS_pwritev+0x15d/0x180
 [<ffffffff9930e1a0>] ? SyS_preadv+0x180/0x180
 [<ffffffff99002017>] ? trace_hardirqs_on_thunk+0x17/0x19
 [<ffffffff99d19557>] entry_SYSCALL_64_fastpath+0x12/0x6b
---[ end trace 96115a52264cceaf ]---
---[ end trace 96115a52264cceb0 ]---

^ permalink raw reply	[flat|nested] 6+ messages in thread
* 4.5-rc1: mm/gup.c warning when writing to /proc/self/mem
@ 2016-01-30 17:58 ` Dave Jones
  0 siblings, 0 replies; 6+ messages in thread
From: Dave Jones @ 2016-01-30 17:58 UTC (permalink / raw)
  To: linux-mm; +Cc: Linux Kernel

Hit this overnight. Just started seeing this after I added "create mmap's
of fd's we open()'d" to trinity.

	Dave

WARNING: CPU: 1 PID: 16733 at mm/gup.c:434 __get_user_pages+0x5f9/0x990()
CPU: 1 PID: 16733 Comm: trinity-c30 Tainted: G        W       4.5.0-rc1-think+ #12
 0000000000000009 000000006648ff5c ffff88000f0779a0 ffffffff99565971
------------[ cut here ]------------
WARNING: CPU: 0 PID: 16731 at mm/gup.c:434 __get_user_pages+0x5f9/0x990()
 0000000000000000 ffff88000f0779e0 ffffffff990b168f ffffffff992aba69
 ffff880450cf1000 0000000000000000 ffff88023780e600 0000000000000017
Call Trace:
 [<ffffffff99565971>] dump_stack+0x4e/0x7d
 [<ffffffff990b168f>] warn_slowpath_common+0x9f/0xe0
 [<ffffffff992aba69>] ? __get_user_pages+0x5f9/0x990
 [<ffffffff990b18aa>] warn_slowpath_null+0x1a/0x20
 [<ffffffff992aba69>] __get_user_pages+0x5f9/0x990
 [<ffffffff992ab470>] ? follow_page_mask+0x530/0x530
 [<ffffffff992ad54a>] ? __access_remote_vm+0xca/0x340
 [<ffffffff992ac2e2>] get_user_pages+0x52/0x60
 [<ffffffff992ad610>] __access_remote_vm+0x190/0x340
 [<ffffffff990f2531>] ? preempt_count_sub+0xc1/0x120
 [<ffffffff992ad480>] ? __might_fault+0xf0/0xf0
 [<ffffffff992ad417>] ? __might_fault+0x87/0xf0
 [<ffffffff992b607f>] access_remote_vm+0x1f/0x30
 [<ffffffff993c5703>] mem_rw.isra.15+0xe3/0x1d0
 [<ffffffff993c5833>] mem_write+0x43/0x50
 [<ffffffff9930a6ed>] __vfs_write+0xdd/0x260
 [<ffffffff9930a610>] ? __vfs_read+0x260/0x260
 [<ffffffff99d136cb>] ? mutex_lock_nested+0x38b/0x590
 [<ffffffff99133152>] ? __lock_is_held+0x92/0xd0
 [<ffffffff990f2531>] ? preempt_count_sub+0xc1/0x120
 [<ffffffff99131035>] ? update_fast_ctr+0x65/0x90
 [<ffffffff991310e7>] ? percpu_down_read+0x57/0xa0
 [<ffffffff99310bc4>] ? __sb_start_write+0xb4/0xf0
 [<ffffffff9930bec6>] vfs_write+0xf6/0x260
 [<ffffffff9930d84f>] SyS_write+0xbf/0x160
 [<ffffffff9930d790>] ? SyS_read+0x160/0x160
 [<ffffffff99002017>] ? trace_hardirqs_on_thunk+0x17/0x19
 [<ffffffff99d19557>] entry_SYSCALL_64_fastpath+0x12/0x6b
CPU: 0 PID: 16731 Comm: trinity-c28 Tainted: G        W       4.5.0-rc1-think+ #12
 0000000000000009 000000002962eec9 ffff8802e7b7f8d8 ffffffff99565971
 0000000000000000 ffff8802e7b7f918 ffffffff990b168f ffffffff992aba69
 ffff8803ed6f1000 00000000000000a0 ffff88023780e600 0000000000000017
Call Trace:
 [<ffffffff99565971>] dump_stack+0x4e/0x7d
 [<ffffffff990b168f>] warn_slowpath_common+0x9f/0xe0
 [<ffffffff992aba69>] ? __get_user_pages+0x5f9/0x990
 [<ffffffff990b18aa>] warn_slowpath_null+0x1a/0x20
 [<ffffffff992aba69>] __get_user_pages+0x5f9/0x990
 [<ffffffff99015979>] ? native_sched_clock+0x69/0x160
 [<ffffffff992ab470>] ? follow_page_mask+0x530/0x530
 [<ffffffff992ad54a>] ? __access_remote_vm+0xca/0x340
 [<ffffffff992ac2e2>] get_user_pages+0x52/0x60
 [<ffffffff992ad610>] __access_remote_vm+0x190/0x340
 [<ffffffff990f2531>] ? preempt_count_sub+0xc1/0x120
 [<ffffffff992ad480>] ? __might_fault+0xf0/0xf0
 [<ffffffff992ad417>] ? __might_fault+0x87/0xf0
 [<ffffffff992b607f>] access_remote_vm+0x1f/0x30
 [<ffffffff993c5703>] mem_rw.isra.15+0xe3/0x1d0
 [<ffffffff993c5833>] mem_write+0x43/0x50
 [<ffffffff9930a950>] do_loop_readv_writev+0xe0/0x110
 [<ffffffff993c57f0>] ? mem_rw.isra.15+0x1d0/0x1d0
 [<ffffffff9930c3bb>] do_readv_writev+0x38b/0x3c0
 [<ffffffff99132960>] ? trace_hardirqs_off_caller+0x70/0x110
 [<ffffffff993c57f0>] ? mem_rw.isra.15+0x1d0/0x1d0
 [<ffffffff9930c030>] ? vfs_write+0x260/0x260
 [<ffffffff99595e17>] ? debug_smp_processor_id+0x17/0x20
 [<ffffffff990f2531>] ? preempt_count_sub+0xc1/0x120
 [<ffffffff991330e5>] ? __lock_is_held+0x25/0xd0
 [<ffffffff991381e3>] ? mark_held_locks+0x23/0xc0
 [<ffffffff9926080a>] ? context_tracking_exit.part.5+0x2a/0x50
 [<ffffffff99138406>] ? trace_hardirqs_on_caller+0x186/0x280
 [<ffffffff9913850d>] ? trace_hardirqs_on+0xd/0x10
 [<ffffffff9930c4b9>] vfs_writev+0x59/0x70
 [<ffffffff9930e2fd>] SyS_pwritev+0x15d/0x180
 [<ffffffff9930e1a0>] ? SyS_preadv+0x180/0x180
 [<ffffffff99002017>] ? trace_hardirqs_on_thunk+0x17/0x19
 [<ffffffff99d19557>] entry_SYSCALL_64_fastpath+0x12/0x6b
---[ end trace 96115a52264cceaf ]---
---[ end trace 96115a52264cceb0 ]---

--
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] 6+ messages in thread
* Re: 4.5-rc1: mm/gup.c warning when writing to /proc/self/mem
  2016-01-30 17:58 ` Dave Jones
@ 2016-01-30 19:53   ` Kirill A. Shutemov
  -1 siblings, 0 replies; 6+ messages in thread
From: Kirill A. Shutemov @ 2016-01-30 19:53 UTC (permalink / raw)
  To: Dave Jones, linux-mm, Linux Kernel, Hugh Dickins, Linus Torvalds,
	Andrew Morton

On Sat, Jan 30, 2016 at 12:58:31PM -0500, Dave Jones wrote:
> Hit this overnight. Just started seeing this after I added "create mmap's
> of fd's we open()'d" to trinity.

The WARN_ON_ONCE() came form Hugh's patch:
 cda540ace6a1 ("mm: get_user_pages(write,force) refuse to COW in shared areas")

This warning is expected if you try to write via /proc/<pid>/mem into
write-protected shared mapping without FMODE_WRITE on the underlying file.
You're not supposed to do that and -EFAULT is right answer for an attempt.

The WARN_ON_ONCE() was added almost two years ago to catch other not
expected users of get_user_pages(write=1,force=1). IIUC, none were found.

Probably we should consider removing the warning.

> 
> 	Dave
> 
> WARNING: CPU: 1 PID: 16733 at mm/gup.c:434 __get_user_pages+0x5f9/0x990()
> CPU: 1 PID: 16733 Comm: trinity-c30 Tainted: G        W       4.5.0-rc1-think+ #12
>  0000000000000009 000000006648ff5c ffff88000f0779a0 ffffffff99565971
> ------------[ cut here ]------------
> WARNING: CPU: 0 PID: 16731 at mm/gup.c:434 __get_user_pages+0x5f9/0x990()
>  0000000000000000 ffff88000f0779e0 ffffffff990b168f ffffffff992aba69
>  ffff880450cf1000 0000000000000000 ffff88023780e600 0000000000000017
> Call Trace:
>  [<ffffffff99565971>] dump_stack+0x4e/0x7d
>  [<ffffffff990b168f>] warn_slowpath_common+0x9f/0xe0
>  [<ffffffff992aba69>] ? __get_user_pages+0x5f9/0x990
>  [<ffffffff990b18aa>] warn_slowpath_null+0x1a/0x20
>  [<ffffffff992aba69>] __get_user_pages+0x5f9/0x990
>  [<ffffffff992ab470>] ? follow_page_mask+0x530/0x530
>  [<ffffffff992ad54a>] ? __access_remote_vm+0xca/0x340
>  [<ffffffff992ac2e2>] get_user_pages+0x52/0x60
>  [<ffffffff992ad610>] __access_remote_vm+0x190/0x340
>  [<ffffffff990f2531>] ? preempt_count_sub+0xc1/0x120
>  [<ffffffff992ad480>] ? __might_fault+0xf0/0xf0
>  [<ffffffff992ad417>] ? __might_fault+0x87/0xf0
>  [<ffffffff992b607f>] access_remote_vm+0x1f/0x30
>  [<ffffffff993c5703>] mem_rw.isra.15+0xe3/0x1d0
>  [<ffffffff993c5833>] mem_write+0x43/0x50
>  [<ffffffff9930a6ed>] __vfs_write+0xdd/0x260
>  [<ffffffff9930a610>] ? __vfs_read+0x260/0x260
>  [<ffffffff99d136cb>] ? mutex_lock_nested+0x38b/0x590
>  [<ffffffff99133152>] ? __lock_is_held+0x92/0xd0
>  [<ffffffff990f2531>] ? preempt_count_sub+0xc1/0x120
>  [<ffffffff99131035>] ? update_fast_ctr+0x65/0x90
>  [<ffffffff991310e7>] ? percpu_down_read+0x57/0xa0
>  [<ffffffff99310bc4>] ? __sb_start_write+0xb4/0xf0
>  [<ffffffff9930bec6>] vfs_write+0xf6/0x260
>  [<ffffffff9930d84f>] SyS_write+0xbf/0x160
>  [<ffffffff9930d790>] ? SyS_read+0x160/0x160
>  [<ffffffff99002017>] ? trace_hardirqs_on_thunk+0x17/0x19
>  [<ffffffff99d19557>] entry_SYSCALL_64_fastpath+0x12/0x6b
> CPU: 0 PID: 16731 Comm: trinity-c28 Tainted: G        W       4.5.0-rc1-think+ #12
>  0000000000000009 000000002962eec9 ffff8802e7b7f8d8 ffffffff99565971
>  0000000000000000 ffff8802e7b7f918 ffffffff990b168f ffffffff992aba69
>  ffff8803ed6f1000 00000000000000a0 ffff88023780e600 0000000000000017
> Call Trace:
>  [<ffffffff99565971>] dump_stack+0x4e/0x7d
>  [<ffffffff990b168f>] warn_slowpath_common+0x9f/0xe0
>  [<ffffffff992aba69>] ? __get_user_pages+0x5f9/0x990
>  [<ffffffff990b18aa>] warn_slowpath_null+0x1a/0x20
>  [<ffffffff992aba69>] __get_user_pages+0x5f9/0x990
>  [<ffffffff99015979>] ? native_sched_clock+0x69/0x160
>  [<ffffffff992ab470>] ? follow_page_mask+0x530/0x530
>  [<ffffffff992ad54a>] ? __access_remote_vm+0xca/0x340
>  [<ffffffff992ac2e2>] get_user_pages+0x52/0x60
>  [<ffffffff992ad610>] __access_remote_vm+0x190/0x340
>  [<ffffffff990f2531>] ? preempt_count_sub+0xc1/0x120
>  [<ffffffff992ad480>] ? __might_fault+0xf0/0xf0
>  [<ffffffff992ad417>] ? __might_fault+0x87/0xf0
>  [<ffffffff992b607f>] access_remote_vm+0x1f/0x30
>  [<ffffffff993c5703>] mem_rw.isra.15+0xe3/0x1d0
>  [<ffffffff993c5833>] mem_write+0x43/0x50
>  [<ffffffff9930a950>] do_loop_readv_writev+0xe0/0x110
>  [<ffffffff993c57f0>] ? mem_rw.isra.15+0x1d0/0x1d0
>  [<ffffffff9930c3bb>] do_readv_writev+0x38b/0x3c0
>  [<ffffffff99132960>] ? trace_hardirqs_off_caller+0x70/0x110
>  [<ffffffff993c57f0>] ? mem_rw.isra.15+0x1d0/0x1d0
>  [<ffffffff9930c030>] ? vfs_write+0x260/0x260
>  [<ffffffff99595e17>] ? debug_smp_processor_id+0x17/0x20
>  [<ffffffff990f2531>] ? preempt_count_sub+0xc1/0x120
>  [<ffffffff991330e5>] ? __lock_is_held+0x25/0xd0
>  [<ffffffff991381e3>] ? mark_held_locks+0x23/0xc0
>  [<ffffffff9926080a>] ? context_tracking_exit.part.5+0x2a/0x50
>  [<ffffffff99138406>] ? trace_hardirqs_on_caller+0x186/0x280
>  [<ffffffff9913850d>] ? trace_hardirqs_on+0xd/0x10
>  [<ffffffff9930c4b9>] vfs_writev+0x59/0x70
>  [<ffffffff9930e2fd>] SyS_pwritev+0x15d/0x180
>  [<ffffffff9930e1a0>] ? SyS_preadv+0x180/0x180
>  [<ffffffff99002017>] ? trace_hardirqs_on_thunk+0x17/0x19
>  [<ffffffff99d19557>] entry_SYSCALL_64_fastpath+0x12/0x6b
> ---[ end trace 96115a52264cceaf ]---
> ---[ end trace 96115a52264cceb0 ]---
> 
> --
> 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>

-- 
 Kirill A. Shutemov

^ permalink raw reply	[flat|nested] 6+ messages in thread
* Re: 4.5-rc1: mm/gup.c warning when writing to /proc/self/mem
@ 2016-01-30 19:53   ` Kirill A. Shutemov
  0 siblings, 0 replies; 6+ messages in thread
From: Kirill A. Shutemov @ 2016-01-30 19:53 UTC (permalink / raw)
  To: Dave Jones, linux-mm, Linux Kernel, Hugh Dickins, Linus Torvalds,
	Andrew Morton

On Sat, Jan 30, 2016 at 12:58:31PM -0500, Dave Jones wrote:
> Hit this overnight. Just started seeing this after I added "create mmap's
> of fd's we open()'d" to trinity.

The WARN_ON_ONCE() came form Hugh's patch:
 cda540ace6a1 ("mm: get_user_pages(write,force) refuse to COW in shared areas")

This warning is expected if you try to write via /proc/<pid>/mem into
write-protected shared mapping without FMODE_WRITE on the underlying file.
You're not supposed to do that and -EFAULT is right answer for an attempt.

The WARN_ON_ONCE() was added almost two years ago to catch other not
expected users of get_user_pages(write=1,force=1). IIUC, none were found.

Probably we should consider removing the warning.

> 
> 	Dave
> 
> WARNING: CPU: 1 PID: 16733 at mm/gup.c:434 __get_user_pages+0x5f9/0x990()
> CPU: 1 PID: 16733 Comm: trinity-c30 Tainted: G        W       4.5.0-rc1-think+ #12
>  0000000000000009 000000006648ff5c ffff88000f0779a0 ffffffff99565971
> ------------[ cut here ]------------
> WARNING: CPU: 0 PID: 16731 at mm/gup.c:434 __get_user_pages+0x5f9/0x990()
>  0000000000000000 ffff88000f0779e0 ffffffff990b168f ffffffff992aba69
>  ffff880450cf1000 0000000000000000 ffff88023780e600 0000000000000017
> Call Trace:
>  [<ffffffff99565971>] dump_stack+0x4e/0x7d
>  [<ffffffff990b168f>] warn_slowpath_common+0x9f/0xe0
>  [<ffffffff992aba69>] ? __get_user_pages+0x5f9/0x990
>  [<ffffffff990b18aa>] warn_slowpath_null+0x1a/0x20
>  [<ffffffff992aba69>] __get_user_pages+0x5f9/0x990
>  [<ffffffff992ab470>] ? follow_page_mask+0x530/0x530
>  [<ffffffff992ad54a>] ? __access_remote_vm+0xca/0x340
>  [<ffffffff992ac2e2>] get_user_pages+0x52/0x60
>  [<ffffffff992ad610>] __access_remote_vm+0x190/0x340
>  [<ffffffff990f2531>] ? preempt_count_sub+0xc1/0x120
>  [<ffffffff992ad480>] ? __might_fault+0xf0/0xf0
>  [<ffffffff992ad417>] ? __might_fault+0x87/0xf0
>  [<ffffffff992b607f>] access_remote_vm+0x1f/0x30
>  [<ffffffff993c5703>] mem_rw.isra.15+0xe3/0x1d0
>  [<ffffffff993c5833>] mem_write+0x43/0x50
>  [<ffffffff9930a6ed>] __vfs_write+0xdd/0x260
>  [<ffffffff9930a610>] ? __vfs_read+0x260/0x260
>  [<ffffffff99d136cb>] ? mutex_lock_nested+0x38b/0x590
>  [<ffffffff99133152>] ? __lock_is_held+0x92/0xd0
>  [<ffffffff990f2531>] ? preempt_count_sub+0xc1/0x120
>  [<ffffffff99131035>] ? update_fast_ctr+0x65/0x90
>  [<ffffffff991310e7>] ? percpu_down_read+0x57/0xa0
>  [<ffffffff99310bc4>] ? __sb_start_write+0xb4/0xf0
>  [<ffffffff9930bec6>] vfs_write+0xf6/0x260
>  [<ffffffff9930d84f>] SyS_write+0xbf/0x160
>  [<ffffffff9930d790>] ? SyS_read+0x160/0x160
>  [<ffffffff99002017>] ? trace_hardirqs_on_thunk+0x17/0x19
>  [<ffffffff99d19557>] entry_SYSCALL_64_fastpath+0x12/0x6b
> CPU: 0 PID: 16731 Comm: trinity-c28 Tainted: G        W       4.5.0-rc1-think+ #12
>  0000000000000009 000000002962eec9 ffff8802e7b7f8d8 ffffffff99565971
>  0000000000000000 ffff8802e7b7f918 ffffffff990b168f ffffffff992aba69
>  ffff8803ed6f1000 00000000000000a0 ffff88023780e600 0000000000000017
> Call Trace:
>  [<ffffffff99565971>] dump_stack+0x4e/0x7d
>  [<ffffffff990b168f>] warn_slowpath_common+0x9f/0xe0
>  [<ffffffff992aba69>] ? __get_user_pages+0x5f9/0x990
>  [<ffffffff990b18aa>] warn_slowpath_null+0x1a/0x20
>  [<ffffffff992aba69>] __get_user_pages+0x5f9/0x990
>  [<ffffffff99015979>] ? native_sched_clock+0x69/0x160
>  [<ffffffff992ab470>] ? follow_page_mask+0x530/0x530
>  [<ffffffff992ad54a>] ? __access_remote_vm+0xca/0x340
>  [<ffffffff992ac2e2>] get_user_pages+0x52/0x60
>  [<ffffffff992ad610>] __access_remote_vm+0x190/0x340
>  [<ffffffff990f2531>] ? preempt_count_sub+0xc1/0x120
>  [<ffffffff992ad480>] ? __might_fault+0xf0/0xf0
>  [<ffffffff992ad417>] ? __might_fault+0x87/0xf0
>  [<ffffffff992b607f>] access_remote_vm+0x1f/0x30
>  [<ffffffff993c5703>] mem_rw.isra.15+0xe3/0x1d0
>  [<ffffffff993c5833>] mem_write+0x43/0x50
>  [<ffffffff9930a950>] do_loop_readv_writev+0xe0/0x110
>  [<ffffffff993c57f0>] ? mem_rw.isra.15+0x1d0/0x1d0
>  [<ffffffff9930c3bb>] do_readv_writev+0x38b/0x3c0
>  [<ffffffff99132960>] ? trace_hardirqs_off_caller+0x70/0x110
>  [<ffffffff993c57f0>] ? mem_rw.isra.15+0x1d0/0x1d0
>  [<ffffffff9930c030>] ? vfs_write+0x260/0x260
>  [<ffffffff99595e17>] ? debug_smp_processor_id+0x17/0x20
>  [<ffffffff990f2531>] ? preempt_count_sub+0xc1/0x120
>  [<ffffffff991330e5>] ? __lock_is_held+0x25/0xd0
>  [<ffffffff991381e3>] ? mark_held_locks+0x23/0xc0
>  [<ffffffff9926080a>] ? context_tracking_exit.part.5+0x2a/0x50
>  [<ffffffff99138406>] ? trace_hardirqs_on_caller+0x186/0x280
>  [<ffffffff9913850d>] ? trace_hardirqs_on+0xd/0x10
>  [<ffffffff9930c4b9>] vfs_writev+0x59/0x70
>  [<ffffffff9930e2fd>] SyS_pwritev+0x15d/0x180
>  [<ffffffff9930e1a0>] ? SyS_preadv+0x180/0x180
>  [<ffffffff99002017>] ? trace_hardirqs_on_thunk+0x17/0x19
>  [<ffffffff99d19557>] entry_SYSCALL_64_fastpath+0x12/0x6b
> ---[ end trace 96115a52264cceaf ]---
> ---[ end trace 96115a52264cceb0 ]---
> 
> --
> 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>

-- 
 Kirill A. Shutemov

--
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] 6+ messages in thread
* Re: 4.5-rc1: mm/gup.c warning when writing to /proc/self/mem
  2016-01-30 19:53   ` Kirill A. Shutemov
@ 2016-01-31  2:03     ` Hugh Dickins
  -1 siblings, 0 replies; 6+ messages in thread
From: Hugh Dickins @ 2016-01-31  2:03 UTC (permalink / raw)
  To: Kirill A. Shutemov
  Cc: Dave Jones, linux-mm, Linux Kernel, Hugh Dickins, Linus Torvalds,
	Andrew Morton

On Sat, 30 Jan 2016, Kirill A. Shutemov wrote:
> On Sat, Jan 30, 2016 at 12:58:31PM -0500, Dave Jones wrote:
> > Hit this overnight. Just started seeing this after I added "create mmap's
> > of fd's we open()'d" to trinity.
> 
> The WARN_ON_ONCE() came form Hugh's patch:
>  cda540ace6a1 ("mm: get_user_pages(write,force) refuse to COW in shared areas")
> 
> This warning is expected if you try to write via /proc/<pid>/mem into
> write-protected shared mapping without FMODE_WRITE on the underlying file.

Other way round: it happens only when you do have FMODE_WRITE on the file.
It was always a strange case.

> You're not supposed to do that and -EFAULT is right answer for an attempt.
> 
> The WARN_ON_ONCE() was added almost two years ago to catch other not
> expected users of get_user_pages(write=1,force=1). IIUC, none were found.
> 
> Probably we should consider removing the warning.

Yes, I agree: as a _ONCE, it doesn't do a whole lot of harm,
but we just don't need it any longer.  And it reminds me of
something else you pointed out to me back then...

> 
> > 
> > 	Dave
> > 
> > WARNING: CPU: 1 PID: 16733 at mm/gup.c:434 __get_user_pages+0x5f9/0x990()


[PATCH] mm: retire GUP WARN_ON_ONCE that outlived its usefulness

Trinity is now hitting the WARN_ON_ONCE we added in v3.15 commit
cda540ace6a1 ("mm: get_user_pages(write,force) refuse to COW in shared
areas").  The warning has served its purpose, nobody was harmed by that
change, so just remove the warning to generate less noise from Trinity.

Which reminds me of the comment I wrongly left behind with that commit
(but was spotted at the time by Kirill), which has since moved into a
separate function, and become even more obscure: delete it.

Reported-by: Dave Jones <davej@codemonkey.org.uk>
Suggested-by: Kirill A. Shutemov <kirill@shutemov.name>
Signed-off-by: Hugh Dickins <hughd@google.com>
---

 mm/gup.c    |    4 +---
 mm/memory.c |    5 -----
 2 files changed, 1 insertion(+), 8 deletions(-)

--- 4.5-rc1/mm/gup.c	2016-01-24 14:54:58.031544001 -0800
+++ linux/mm/gup.c	2016-01-30 17:14:21.443281994 -0800
@@ -430,10 +430,8 @@ static int check_vma_flags(struct vm_are
 			 * Anon pages in shared mappings are surprising: now
 			 * just reject it.
 			 */
-			if (!is_cow_mapping(vm_flags)) {
-				WARN_ON_ONCE(vm_flags & VM_MAYWRITE);
+			if (!is_cow_mapping(vm_flags))
 				return -EFAULT;
-			}
 		}
 	} else if (!(vm_flags & VM_READ)) {
 		if (!(gup_flags & FOLL_FORCE))
--- 4.5-rc1/mm/memory.c	2016-01-24 14:54:58.051544131 -0800
+++ linux/mm/memory.c	2016-01-30 17:14:21.443281994 -0800
@@ -2232,11 +2232,6 @@ static int wp_page_shared(struct mm_stru
 
 	page_cache_get(old_page);
 
-	/*
-	 * Only catch write-faults on shared writable pages,
-	 * read-only shared pages can get COWed by
-	 * get_user_pages(.write=1, .force=1).
-	 */
 	if (vma->vm_ops && vma->vm_ops->page_mkwrite) {
 		int tmp;
 

^ permalink raw reply	[flat|nested] 6+ messages in thread
* Re: 4.5-rc1: mm/gup.c warning when writing to /proc/self/mem
@ 2016-01-31  2:03     ` Hugh Dickins
  0 siblings, 0 replies; 6+ messages in thread
From: Hugh Dickins @ 2016-01-31  2:03 UTC (permalink / raw)
  To: Kirill A. Shutemov
  Cc: Dave Jones, linux-mm, Linux Kernel, Hugh Dickins, Linus Torvalds,
	Andrew Morton

On Sat, 30 Jan 2016, Kirill A. Shutemov wrote:
> On Sat, Jan 30, 2016 at 12:58:31PM -0500, Dave Jones wrote:
> > Hit this overnight. Just started seeing this after I added "create mmap's
> > of fd's we open()'d" to trinity.
> 
> The WARN_ON_ONCE() came form Hugh's patch:
>  cda540ace6a1 ("mm: get_user_pages(write,force) refuse to COW in shared areas")
> 
> This warning is expected if you try to write via /proc/<pid>/mem into
> write-protected shared mapping without FMODE_WRITE on the underlying file.

Other way round: it happens only when you do have FMODE_WRITE on the file.
It was always a strange case.

> You're not supposed to do that and -EFAULT is right answer for an attempt.
> 
> The WARN_ON_ONCE() was added almost two years ago to catch other not
> expected users of get_user_pages(write=1,force=1). IIUC, none were found.
> 
> Probably we should consider removing the warning.

Yes, I agree: as a _ONCE, it doesn't do a whole lot of harm,
but we just don't need it any longer.  And it reminds me of
something else you pointed out to me back then...

> 
> > 
> > 	Dave
> > 
> > WARNING: CPU: 1 PID: 16733 at mm/gup.c:434 __get_user_pages+0x5f9/0x990()


[PATCH] mm: retire GUP WARN_ON_ONCE that outlived its usefulness

Trinity is now hitting the WARN_ON_ONCE we added in v3.15 commit
cda540ace6a1 ("mm: get_user_pages(write,force) refuse to COW in shared
areas").  The warning has served its purpose, nobody was harmed by that
change, so just remove the warning to generate less noise from Trinity.

Which reminds me of the comment I wrongly left behind with that commit
(but was spotted at the time by Kirill), which has since moved into a
separate function, and become even more obscure: delete it.

Reported-by: Dave Jones <davej@codemonkey.org.uk>
Suggested-by: Kirill A. Shutemov <kirill@shutemov.name>
Signed-off-by: Hugh Dickins <hughd@google.com>
---

 mm/gup.c    |    4 +---
 mm/memory.c |    5 -----
 2 files changed, 1 insertion(+), 8 deletions(-)

--- 4.5-rc1/mm/gup.c	2016-01-24 14:54:58.031544001 -0800
+++ linux/mm/gup.c	2016-01-30 17:14:21.443281994 -0800
@@ -430,10 +430,8 @@ static int check_vma_flags(struct vm_are
 			 * Anon pages in shared mappings are surprising: now
 			 * just reject it.
 			 */
-			if (!is_cow_mapping(vm_flags)) {
-				WARN_ON_ONCE(vm_flags & VM_MAYWRITE);
+			if (!is_cow_mapping(vm_flags))
 				return -EFAULT;
-			}
 		}
 	} else if (!(vm_flags & VM_READ)) {
 		if (!(gup_flags & FOLL_FORCE))
--- 4.5-rc1/mm/memory.c	2016-01-24 14:54:58.051544131 -0800
+++ linux/mm/memory.c	2016-01-30 17:14:21.443281994 -0800
@@ -2232,11 +2232,6 @@ static int wp_page_shared(struct mm_stru
 
 	page_cache_get(old_page);
 
-	/*
-	 * Only catch write-faults on shared writable pages,
-	 * read-only shared pages can get COWed by
-	 * get_user_pages(.write=1, .force=1).
-	 */
 	if (vma->vm_ops && vma->vm_ops->page_mkwrite) {
 		int tmp;
 

--
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] 6+ messages in thread
end of thread, other threads:[~2016-01-31  2:03 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-01-30 17:58 4.5-rc1: mm/gup.c warning when writing to /proc/self/mem Dave Jones
2016-01-30 17:58 ` Dave Jones
2016-01-30 19:53 ` Kirill A. Shutemov
2016-01-30 19:53   ` Kirill A. Shutemov
2016-01-31  2:03   ` Hugh Dickins
2016-01-31  2:03     ` Hugh Dickins

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.