Re: [PATCH] dm ioctl: Restore __GFP_HIGH in copy_params()

2017-05-22 Thread Michal Hocko
On Mon 22-05-17 10:52:44, Mikulas Patocka wrote: > > > On Mon, 22 May 2017, Michal Hocko wrote: [...] > > I am not sure I understand. OOM killer is invoked for _all_ allocations > > <= PAGE_ALLOC_COSTLY_ORDER that do not have __GFP_NORETRY as long as the > > OOM killer is not disabled

Re: [PATCH] dm ioctl: Restore __GFP_HIGH in copy_params()

2017-05-22 Thread Michal Hocko
On Mon 22-05-17 10:52:44, Mikulas Patocka wrote: > > > On Mon, 22 May 2017, Michal Hocko wrote: [...] > > I am not sure I understand. OOM killer is invoked for _all_ allocations > > <= PAGE_ALLOC_COSTLY_ORDER that do not have __GFP_NORETRY as long as the > > OOM killer is not disabled

Re: [PATCH] dm ioctl: Restore __GFP_HIGH in copy_params()

2017-05-22 Thread Mikulas Patocka
On Mon, 22 May 2017, Michal Hocko wrote: > On Mon 22-05-17 08:00:11, Mikulas Patocka wrote: > > > > On Mon, 22 May 2017, Michal Hocko wrote: > > > > > > Sometimes, I/O to a device mapper device is blocked until the userspace > > > > daemon dmeventd does some action (for example, when

Re: [PATCH] dm ioctl: Restore __GFP_HIGH in copy_params()

2017-05-22 Thread Mikulas Patocka
On Mon, 22 May 2017, Michal Hocko wrote: > On Mon 22-05-17 08:00:11, Mikulas Patocka wrote: > > > > On Mon, 22 May 2017, Michal Hocko wrote: > > > > > > Sometimes, I/O to a device mapper device is blocked until the userspace > > > > daemon dmeventd does some action (for example, when

Re: [PATCH] dm ioctl: Restore __GFP_HIGH in copy_params()

2017-05-22 Thread Michal Hocko
On Mon 22-05-17 08:00:11, Mikulas Patocka wrote: > > > On Mon, 22 May 2017, Michal Hocko wrote: > > > On Fri 19-05-17 19:43:23, Mikulas Patocka wrote: > > > > > > > > > On Fri, 19 May 2017, Michal Hocko wrote: > > > > > > > On Thu 18-05-17 19:50:46, Junaid Shahid wrote: > > > > > (Adding

Re: [PATCH] dm ioctl: Restore __GFP_HIGH in copy_params()

2017-05-22 Thread Michal Hocko
On Mon 22-05-17 08:00:11, Mikulas Patocka wrote: > > > On Mon, 22 May 2017, Michal Hocko wrote: > > > On Fri 19-05-17 19:43:23, Mikulas Patocka wrote: > > > > > > > > > On Fri, 19 May 2017, Michal Hocko wrote: > > > > > > > On Thu 18-05-17 19:50:46, Junaid Shahid wrote: > > > > > (Adding

Re: [PATCH] dm ioctl: Restore __GFP_HIGH in copy_params()

2017-05-22 Thread Mikulas Patocka
On Mon, 22 May 2017, Michal Hocko wrote: > On Fri 19-05-17 19:43:23, Mikulas Patocka wrote: > > > > > > On Fri, 19 May 2017, Michal Hocko wrote: > > > > > On Thu 18-05-17 19:50:46, Junaid Shahid wrote: > > > > (Adding back the correct linux-mm email address and also adding > > > >

Re: [PATCH] dm ioctl: Restore __GFP_HIGH in copy_params()

2017-05-22 Thread Mikulas Patocka
On Mon, 22 May 2017, Michal Hocko wrote: > On Fri 19-05-17 19:43:23, Mikulas Patocka wrote: > > > > > > On Fri, 19 May 2017, Michal Hocko wrote: > > > > > On Thu 18-05-17 19:50:46, Junaid Shahid wrote: > > > > (Adding back the correct linux-mm email address and also adding > > > >

Re: [PATCH] dm ioctl: Restore __GFP_HIGH in copy_params()

2017-05-22 Thread Michal Hocko
On Fri 19-05-17 19:43:23, Mikulas Patocka wrote: > > > On Fri, 19 May 2017, Michal Hocko wrote: > > > On Thu 18-05-17 19:50:46, Junaid Shahid wrote: > > > (Adding back the correct linux-mm email address and also adding > > > linux-kernel.) > > > > > > On Thursday, May 18, 2017 01:41:33 PM

Re: [PATCH] dm ioctl: Restore __GFP_HIGH in copy_params()

2017-05-22 Thread Michal Hocko
On Fri 19-05-17 19:43:23, Mikulas Patocka wrote: > > > On Fri, 19 May 2017, Michal Hocko wrote: > > > On Thu 18-05-17 19:50:46, Junaid Shahid wrote: > > > (Adding back the correct linux-mm email address and also adding > > > linux-kernel.) > > > > > > On Thursday, May 18, 2017 01:41:33 PM

Re: [PATCH] dm ioctl: Restore __GFP_HIGH in copy_params()

2017-05-20 Thread Mikulas Patocka
On Sat, 20 May 2017, Michal Hocko wrote: > On Fri 19-05-17 19:50:24, Mikulas Patocka wrote: > > > > > > On Thu, 18 May 2017, Junaid Shahid wrote: > > > > > d224e9381897 (drivers/md/dm-ioctl.c: use kvmalloc rather than opencoded > > > variant) left out the __GFP_HIGH flag when converting from

Re: [PATCH] dm ioctl: Restore __GFP_HIGH in copy_params()

2017-05-20 Thread Mikulas Patocka
On Sat, 20 May 2017, Michal Hocko wrote: > On Fri 19-05-17 19:50:24, Mikulas Patocka wrote: > > > > > > On Thu, 18 May 2017, Junaid Shahid wrote: > > > > > d224e9381897 (drivers/md/dm-ioctl.c: use kvmalloc rather than opencoded > > > variant) left out the __GFP_HIGH flag when converting from

Re: [PATCH] dm ioctl: Restore __GFP_HIGH in copy_params()

2017-05-19 Thread Mikulas Patocka
On Fri, 19 May 2017, Michal Hocko wrote: > On Thu 18-05-17 19:50:46, Junaid Shahid wrote: > > (Adding back the correct linux-mm email address and also adding > > linux-kernel.) > > > > On Thursday, May 18, 2017 01:41:33 PM David Rientjes wrote: > [...] > > > Let's ask Mikulas, who changed

Re: [PATCH] dm ioctl: Restore __GFP_HIGH in copy_params()

2017-05-19 Thread Mikulas Patocka
On Fri, 19 May 2017, Michal Hocko wrote: > On Thu 18-05-17 19:50:46, Junaid Shahid wrote: > > (Adding back the correct linux-mm email address and also adding > > linux-kernel.) > > > > On Thursday, May 18, 2017 01:41:33 PM David Rientjes wrote: > [...] > > > Let's ask Mikulas, who changed

Re: [PATCH] dm ioctl: Restore __GFP_HIGH in copy_params()

2017-05-19 Thread Michal Hocko
On Thu 18-05-17 19:50:46, Junaid Shahid wrote: > (Adding back the correct linux-mm email address and also adding linux-kernel.) > > On Thursday, May 18, 2017 01:41:33 PM David Rientjes wrote: [...] > > Let's ask Mikulas, who changed this from PF_MEMALLOC to __GFP_HIGH, > > assuming there was a

Re: [PATCH] dm ioctl: Restore __GFP_HIGH in copy_params()

2017-05-19 Thread Michal Hocko
On Thu 18-05-17 19:50:46, Junaid Shahid wrote: > (Adding back the correct linux-mm email address and also adding linux-kernel.) > > On Thursday, May 18, 2017 01:41:33 PM David Rientjes wrote: [...] > > Let's ask Mikulas, who changed this from PF_MEMALLOC to __GFP_HIGH, > > assuming there was a

Re: [PATCH] dm ioctl: Restore __GFP_HIGH in copy_params()

2017-05-18 Thread Junaid Shahid
(Adding back the correct linux-mm email address and also adding linux-kernel.) On Thursday, May 18, 2017 01:41:33 PM David Rientjes wrote: > On Thu, 18 May 2017, Michal Hocko wrote: > > > On Thu 18-05-17 11:50:40, Junaid Shahid wrote: > > > d224e9381897 (drivers/md/dm-ioctl.c: use kvmalloc

Re: [PATCH] dm ioctl: Restore __GFP_HIGH in copy_params()

2017-05-18 Thread Junaid Shahid
(Adding back the correct linux-mm email address and also adding linux-kernel.) On Thursday, May 18, 2017 01:41:33 PM David Rientjes wrote: > On Thu, 18 May 2017, Michal Hocko wrote: > > > On Thu 18-05-17 11:50:40, Junaid Shahid wrote: > > > d224e9381897 (drivers/md/dm-ioctl.c: use kvmalloc