Re: [PATCH] ARM:mm: define arch-specific IOREMAP_MAX_ORDER only in !SMP && !LPAE case

2015-09-09 Thread Sergey Dyasly
On 09.09.2015 6:10, Nicolas Pitre wrote: On Tue, 8 Sep 2015, Sergey Dyasly wrote: On 08.09.2015 5:45, Zhang Zhen wrote: The arch-specific IOREMAP_MAX_ORDER is introduced in commit: ff0daca([ARM] Add section support to ioremap) and commit: a069c89 ([ARM] 3705/1: add supersection support

Re: [PATCH] ARM:mm: define arch-specific IOREMAP_MAX_ORDER only in !SMP && !LPAE case

2015-09-09 Thread Sergey Dyasly
On 09.09.2015 6:10, Nicolas Pitre wrote: On Tue, 8 Sep 2015, Sergey Dyasly wrote: On 08.09.2015 5:45, Zhang Zhen wrote: The arch-specific IOREMAP_MAX_ORDER is introduced in commit: ff0daca([ARM] Add section support to ioremap) and commit: a069c89 ([ARM] 3705/1: add supersection support

Re: [PATCH] ARM:mm: define arch-specific IOREMAP_MAX_ORDER only in !SMP && !LPAE case

2015-09-08 Thread Sergey Dyasly
On 08.09.2015 5:45, Zhang Zhen wrote: The arch-specific IOREMAP_MAX_ORDER is introduced in commit: ff0daca([ARM] Add section support to ioremap) and commit: a069c89 ([ARM] 3705/1: add supersection support to ioremap()). But supersections and sections mappings are only used in !SMP && !LPAE case.

Re: [PATCH] ARM:mm: define arch-specific IOREMAP_MAX_ORDER only in !SMP && !LPAE case

2015-09-08 Thread Sergey Dyasly
On 08.09.2015 5:45, Zhang Zhen wrote: The arch-specific IOREMAP_MAX_ORDER is introduced in commit: ff0daca([ARM] Add section support to ioremap) and commit: a069c89 ([ARM] 3705/1: add supersection support to ioremap()). But supersections and sections mappings are only used in !SMP && !LPAE case.

[PATCH] ARM: use default ioremap alignment for SMP or LPAE

2015-01-21 Thread Sergey Dyasly
itry Safonov Link: https://lkml.kernel.org/g/1419328813-2211-1-git-send-email-d.safo...@partner.samsung.com Signed-off-by: Sergey Dyasly --- arch/arm/include/asm/memory.h |2 ++ 1 file changed, 2 insertions(+) diff --git a/arch/arm/include/asm/memory.h b/arch/arm/include/asm/memory.h index 184de

[PATCH v2] checkpatch: add check for the buggy while_each_thread()

2015-01-21 Thread Sergey Dyasly
Now it's preferable to use for_each_thread() instead of while_each_thread(). Add a check to checkpatch.pl in order to prevent any new usages of the buggy while_each_thread() when possible. Cc: Oleg Nesterov Cc: Andy Whitcroft Cc: Joe Perches Signed-off-by: Sergey Dyasly --- Changes since v1

Re: [PATCH] checkpatch: add check for the buggy while_each_thread()

2015-01-21 Thread Sergey Dyasly
On Sun, 18 Jan 2015 17:27:33 -0800 Joe Perches wrote: > On Sun, 2015-01-18 at 17:24 +0300, Sergey Dyasly wrote: > > Now it's preferable to use for_each_thread() instead of while_each_thread(). > > Add a check to checkpatch.pl in order to prevent any new usages of the buggy > &

[PATCH v2] checkpatch: add check for the buggy while_each_thread()

2015-01-21 Thread Sergey Dyasly
Signed-off-by: Sergey Dyasly dse...@gmail.com --- Changes since v1: - Added \s* to the regular expression - Limited commit id to 12 digits - Prefer to use -- Consider using - Moved check to the end of the file scripts/checkpatch.pl | 6 ++ 1 file changed, 6 insertions(+) diff --git a/scripts

Re: [PATCH] checkpatch: add check for the buggy while_each_thread()

2015-01-21 Thread Sergey Dyasly
On Sun, 18 Jan 2015 17:27:33 -0800 Joe Perches j...@perches.com wrote: On Sun, 2015-01-18 at 17:24 +0300, Sergey Dyasly wrote: Now it's preferable to use for_each_thread() instead of while_each_thread(). Add a check to checkpatch.pl in order to prevent any new usages of the buggy

[PATCH] ARM: use default ioremap alignment for SMP or LPAE

2015-01-21 Thread Sergey Dyasly
-off-by: Sergey Dyasly s.dya...@samsung.com --- arch/arm/include/asm/memory.h |2 ++ 1 file changed, 2 insertions(+) diff --git a/arch/arm/include/asm/memory.h b/arch/arm/include/asm/memory.h index 184def0..c3ef139 100644 --- a/arch/arm/include/asm/memory.h +++ b/arch/arm/include/asm/memory.h

Re: [RFC][PATCH RESEND] mm: vmalloc: remove ioremap align constraint

2015-01-20 Thread Sergey Dyasly
On Sun, 04 Jan 2015 17:38:06 +0100 Arnd Bergmann wrote: > On Saturday 03 January 2015 18:59:46 Sergey Dyasly wrote: > > Hi Arnd, > > > > First, some background information. We originally encountered high > > fragmentation > > issue in vmalloc area: > >

Re: [RFC][PATCH RESEND] mm: vmalloc: remove ioremap align constraint

2015-01-20 Thread Sergey Dyasly
On Sun, 04 Jan 2015 17:38:06 +0100 Arnd Bergmann a...@arndb.de wrote: On Saturday 03 January 2015 18:59:46 Sergey Dyasly wrote: Hi Arnd, First, some background information. We originally encountered high fragmentation issue in vmalloc area: 1. Total size of vmalloc area

[PATCH] checkpatch: add check for the buggy while_each_thread()

2015-01-18 Thread Sergey Dyasly
Now it's preferable to use for_each_thread() instead of while_each_thread(). Add a check to checkpatch.pl in order to prevent any new usages of the buggy while_each_thread() when possible. Cc: Oleg Nesterov Cc: Andy Whitcroft Cc: Joe Perches Signed-off-by: Sergey Dyasly --- scripts

[PATCH] checkpatch: add check for the buggy while_each_thread()

2015-01-18 Thread Sergey Dyasly
Signed-off-by: Sergey Dyasly dse...@gmail.com --- scripts/checkpatch.pl | 6 ++ 1 file changed, 6 insertions(+) diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index f0bb6d6..0c5cc0b 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl @@ -3255,6 +3255,12 @@ sub process

Re: [RFC][PATCH RESEND] mm: vmalloc: remove ioremap align constraint

2015-01-03 Thread Sergey Dyasly
> Signed-off-by: Russell King > > and then later extended to 16MB supersection mappings, which indeed > is used to reduce TLB pressure. > > I don't see any downsides to it, why change it? > > Arnd -- Sergey Dyasly -- To unsubscribe from this list: send the

Re: [RFC][PATCH RESEND] mm: vmalloc: remove ioremap align constraint

2015-01-03 Thread Sergey Dyasly
mappings, which indeed is used to reduce TLB pressure. I don't see any downsides to it, why change it? Arnd -- Sergey Dyasly dse...@gmail.com -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info

Re: [PATCH 0/2] initial while_each_thread() fixes

2013-12-03 Thread Sergey Dyasly
read_group > can go away before we change all users of while_each_thread(). > > David, et al, I din't actually test 2/2, I do not know how. Please > review, although it looks simple. The patches look correct and my test case no longer hangs, so Reviewed-and-Tested-by: Sergey Dyasly >

Re: [PATCH 0/2] initial while_each_thread() fixes

2013-12-03 Thread Sergey Dyasly
, et al, I din't actually test 2/2, I do not know how. Please review, although it looks simple. The patches look correct and my test case no longer hangs, so Reviewed-and-Tested-by: Sergey Dyasly dse...@gmail.com Oleg. include/linux/init_task.h |2 ++ include/linux/sched.h | 12

Re: [PATCH] OOM killer: wait for tasks with pending SIGKILL to exit

2013-10-01 Thread Sergey Dyasly
It seems to me that we are going nowhere with this discussion... If you are ok with the first change in my patch regarding fatal_signal_pending, I can send new patch with just that change. On Mon, 30 Sep 2013 15:08:25 -0700 (PDT) David Rientjes wrote: > On Fri, 27 Sep 2013, Sergey Dya

Re: [PATCH] OOM killer: wait for tasks with pending SIGKILL to exit

2013-10-01 Thread Sergey Dyasly
, Sergey Dyasly wrote: What you are saying contradicts current OOMk code the way I read it. Comment in oom_kill_process() says: If the task is already exiting ... set TIF_MEMDIE so it can die quickly I just want to know the right solution. That's a comment, not code

Re: [PATCH] OOM killer: wait for tasks with pending SIGKILL to exit

2013-09-27 Thread Sergey Dyasly
On Wed, 25 Sep 2013 13:31:32 -0700 (PDT) David Rientjes wrote: > On Wed, 11 Sep 2013, Sergey Dyasly wrote: > > > > > /* > > > > * If this task is not being ptraced on exit, then wait > > > > for it > > > >

Re: [PATCH] OOM killer: wait for tasks with pending SIGKILL to exit

2013-09-27 Thread Sergey Dyasly
On Wed, 25 Sep 2013 13:31:32 -0700 (PDT) David Rientjes rient...@google.com wrote: On Wed, 11 Sep 2013, Sergey Dyasly wrote: /* * If this task is not being ptraced on exit, then wait for it * to finish before killing some other

Re: [PATCH] OOM killer: wait for tasks with pending SIGKILL to exit

2013-09-19 Thread Sergey Dyasly
Ping :) On Wed, 11 Sep 2013 19:06:05 +0400 Sergey Dyasly wrote: > On Mon, 9 Sep 2013 13:07:08 -0700 (PDT) > David Rientjes wrote: > > > > /* > > >* If this task is not being ptraced on exit, then wait for it > > >* to f

Re: [PATCH] OOM killer: wait for tasks with pending SIGKILL to exit

2013-09-19 Thread Sergey Dyasly
Ping :) On Wed, 11 Sep 2013 19:06:05 +0400 Sergey Dyasly dse...@gmail.com wrote: On Mon, 9 Sep 2013 13:07:08 -0700 (PDT) David Rientjes rient...@google.com wrote: /* * If this task is not being ptraced on exit, then wait for it * to finish before

Re: [PATCH] OOM killer: wait for tasks with pending SIGKILL to exit

2013-09-11 Thread Sergey Dyasly
nce that victim will become PF_EXITING between scan and kill. The only difference is in force_kill flag, and the only case where it's set is SysRq. And I think in this case OOM killer messages are a good thing to have even when victim is already exiting, instead of just silence. -- Sergey Dyasly

Re: [PATCH] OOM killer: wait for tasks with pending SIGKILL to exit

2013-09-11 Thread Sergey Dyasly
to have even when victim is already exiting, instead of just silence. -- Sergey Dyasly dse...@gmail.com -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please

[PATCH] OOM killer: wait for tasks with pending SIGKILL to exit

2013-09-09 Thread Sergey Dyasly
after victim has been selected. Signed-off-by: Sergey Dyasly --- mm/oom_kill.c | 17 + 1 file changed, 5 insertions(+), 12 deletions(-) diff --git a/mm/oom_kill.c b/mm/oom_kill.c index 98e75f2..ef83b81 100644 --- a/mm/oom_kill.c +++ b/mm/oom_kill.c @@ -275,13 +275,16 @@ enum

[PATCH] OOM killer: wait for tasks with pending SIGKILL to exit

2013-09-09 Thread Sergey Dyasly
after victim has been selected. Signed-off-by: Sergey Dyasly dse...@gmail.com --- mm/oom_kill.c | 17 + 1 file changed, 5 insertions(+), 12 deletions(-) diff --git a/mm/oom_kill.c b/mm/oom_kill.c index 98e75f2..ef83b81 100644 --- a/mm/oom_kill.c +++ b/mm/oom_kill.c @@ -275,13