[PATCH v9 10/13] arch/tile: adopt prepare_exit_to_usermode() model from x86

2016-01-04 Thread Chris Metcalf
all out the bug explicitly now, but don't yet fix it. Signed-off-by: Chris Metcalf <cmetc...@ezchip.com> --- arch/tile/include/asm/processor.h | 2 +- arch/tile/include/asm/thread_info.h | 8 +++- arch/tile/kernel/intvec_32.S| 46 +++-- arch

[PATCH v9 13/13] arm, tile: turn off timer tick for oneshot_stopped state

2016-01-04 Thread Chris Metcalf
l properly re-enable it. This fix avoids a small performance hiccup for regular applications, but for TASK_ISOLATION code, it fixes a potentially serious kernel timer interruption to the time-sensitive application. Signed-off-by: Chris Metcalf <cmetc...@ezchip.com> --- arch/tile/kernel/time.c

[PATCH v9 06/13] task_isolation: add debug boot flag

2016-01-04 Thread Chris Metcalf
to deliver an interrupt to a task_isolation core. It may be worth considering other ways to generate useful debugging output rather than console spew, but for now that is simple and direct. Signed-off-by: Chris Metcalf <cmetc...@ezchip.com> --- Documentation/kernel-parameters.txt | 8

[PATCH v9 03/13] lru_add_drain_all: factor out lru_add_drain_needed

2016-01-04 Thread Chris Metcalf
This per-cpu check was being done in the loop in lru_add_drain_all(), but having it be callable for a particular cpu is helpful for the task-isolation patches. Signed-off-by: Chris Metcalf <cmetc...@ezchip.com> --- include/linux/swap.h | 1 + mm/swap.c| 13 + 2

[PATCH v9 12/13] arch/tile: enable task isolation functionality

2016-01-04 Thread Chris Metcalf
not generate signals to the application. Signed-off-by: Chris Metcalf <cmetc...@ezchip.com> --- arch/tile/kernel/process.c | 6 +- arch/tile/kernel/ptrace.c | 6 ++ arch/tile/kernel/single_step.c | 5 + arch/tile/kernel/smp.c | 26 ++ arc

[PATCH v9 07/13] arch/x86: enable task isolation functionality

2016-01-04 Thread Chris Metcalf
strict reporting for the kernel exception types that do not result in signals, namely non-signalling page faults and non-signalling MPX fixups. Signed-off-by: Chris Metcalf <cmetc...@ezchip.com> --- arch/x86/entry/common.c | 10 +- arch/x86/kernel/traps.c | 2 ++ arch/x86/mm/f

[PATCH v9 00/13] support "task_isolation" mode for nohz_full

2016-01-04 Thread Chris Metcalf
ESHOT_STOPPED state"). - For tile, I adopted the arm model of doing user_exit() calls in the early assembly code (a new patch in this series). I also added a missing task_isolation_debug hook for tile's IPI and remote cache flush code. Chris Metcalf (12): vmstat: add vm

Re: [PATCH v9 06/13] task_isolation: add debug boot flag

2016-01-04 Thread Chris Metcalf
On 1/4/2016 5:52 PM, Steven Rostedt wrote: On Mon, 4 Jan 2016 14:34:44 -0500 Chris Metcalf<cmetc...@ezchip.com> wrote: >+#ifdef CONFIG_TASK_ISOLATION >+void task_isolation_debug(int cpu) >+{ >+ struct task_struct *p; >+ >+ if (!task_isolation_possible(c

Re: [PATCH 1/2] compat ABI: use non-compat openat and open_by_handle_at variants

2015-12-28 Thread Chris Metcalf
is tile32, that >continues with compat version of syscalls. > >Signed-off-by: Yury Norov Looks good, Acked-by: Arnd Bergmann (adding Chris to Cc a well, in case he has further comments) Seems plausible. Acked-by: Chris Metcalf [for tile] -- Chris Metcalf, EZChip Semiconductor htt

Re: [PATCH 1/2] compat ABI: use non-compat openat and open_by_handle_at variants

2015-12-28 Thread Chris Metcalf
is tile32, that >continues with compat version of syscalls. > >Signed-off-by: Yury Norov<yno...@caviumnetworks.com> Looks good, Acked-by: Arnd Bergmann<a...@arndb.de> (adding Chris to Cc a well, in case he has further comments) Seems plausible. Acked-by: Chris Metcalf &

[PATCH] tile: fix bug in setting PT_FLAGS_DISABLE_IRQ on kernel entry

2015-12-23 Thread Chris Metcalf
Signed-off-by: Chris Metcalf --- arch/tile/kernel/intvec_64.S | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/tile/kernel/intvec_64.S b/arch/tile/kernel/intvec_64.S index 58964d209d4d..87cf7f70fd02 100644 --- a/arch/tile/kernel/intvec_64.S +++ b/arch/tile/kernel/intvec_64.S

[PATCH] tile: fix bug in setting PT_FLAGS_DISABLE_IRQ on kernel entry

2015-12-23 Thread Chris Metcalf
Signed-off-by: Chris Metcalf <cmetc...@ezchip.com> --- arch/tile/kernel/intvec_64.S | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/tile/kernel/intvec_64.S b/arch/tile/kernel/intvec_64.S index 58964d209d4d..87cf7f70fd02 100644 --- a/arch/tile/kernel/intvec_64.S +++ b/

[PATCH] tile: provide CONFIG_PAGE_SIZE_64KB etc for tilepro

2015-12-22 Thread Chris Metcalf
. Signed-off-by: Chris Metcalf --- arch/tile/Kconfig| 11 +-- arch/tile/include/asm/page.h | 8 +--- 2 files changed, 10 insertions(+), 9 deletions(-) diff --git a/arch/tile/Kconfig b/arch/tile/Kconfig index 49be476f9feb..5aa0c97c8a5e 100644 --- a/arch/tile/Kconfig +++ b/arch

[PATCH] lib/clz_tab.c: put in lib-y rather than obj-y

2015-12-22 Thread Chris Metcalf
-by: Chris Metcalf --- I have this in the tile tree for linux-next since it broke tilepro, but perhaps it makes more sense for it to get pushed through Andrew's tree or somewhere else? lib/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/Makefile b/lib/Makefile

[PATCH] tile: fix a -Wframe-larger-than warning

2015-12-22 Thread Chris Metcalf
The warning occurs in setup.c, where it is known that it can't be a problem, but it's still a good idea to silence the warning. The onstack array is converted from an s32 to a u8, which still is plenty of range for the values being managed there. Signed-off-by: Chris Metcalf --- arch/tile

[PATCH] tile: fix tilepro casts for readl, writel, etc

2015-12-22 Thread Chris Metcalf
Missing parentheses could cause an argument of the form "integer + pointer" to get cast to "(long)integer + pointer" and remain a pointer type, causing compiler warnings. Signed-off-by: Chris Metcalf --- arch/tile/include/asm/io.h | 16 1 file changed

[PATCH] tile: provide CONFIG_PAGE_SIZE_64KB etc for tilepro

2015-12-22 Thread Chris Metcalf
. Signed-off-by: Chris Metcalf <cmetc...@ezchip.com> --- arch/tile/Kconfig| 11 +-- arch/tile/include/asm/page.h | 8 +--- 2 files changed, 10 insertions(+), 9 deletions(-) diff --git a/arch/tile/Kconfig b/arch/tile/Kconfig index 49be476f9feb..5aa0c97c8a5e 100644 --- a/arc

[PATCH] tile: fix a -Wframe-larger-than warning

2015-12-22 Thread Chris Metcalf
The warning occurs in setup.c, where it is known that it can't be a problem, but it's still a good idea to silence the warning. The onstack array is converted from an s32 to a u8, which still is plenty of range for the values being managed there. Signed-off-by: Chris Metcalf <cmetc...@ezchip.

[PATCH] tile: fix tilepro casts for readl, writel, etc

2015-12-22 Thread Chris Metcalf
Missing parentheses could cause an argument of the form "integer + pointer" to get cast to "(long)integer + pointer" and remain a pointer type, causing compiler warnings. Signed-off-by: Chris Metcalf <cmetc...@ezchip.com> --- arch/tile/include/asm/io.h | 16 -

[PATCH] lib/clz_tab.c: put in lib-y rather than obj-y

2015-12-22 Thread Chris Metcalf
-by: Chris Metcalf <cmetc...@ezchip.com> --- I have this in the tile tree for linux-next since it broke tilepro, but perhaps it makes more sense for it to get pushed through Andrew's tree or somewhere else? lib/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [PATCH 38/43] MAINTAINERS: add git URL for tile

2015-12-18 Thread Chris Metcalf
On 12/18/2015 02:52 AM, Fengguang Wu wrote: CC: Chris Metcalf Signed-off-by: Fengguang Wu --- MAINTAINERS |1 + 1 file changed, 1 insertion(+) Thanks! Taken into the tile tree. -- Chris Metcalf, EZChip Semiconductor http://www.ezchip.com -- To unsubscribe from this list: send

Re: [PATCH] Revert "kernel/stop_machine.c: remove CONFIG_SMP dependencies"

2015-12-18 Thread Chris Metcalf
org/sudipm-mukherjee/parport/jobs/97618542 kernel/stop_machine.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) Acked-by: Chris Metcalf [for tile] -- Chris Metcalf, EZChip Semiconductor http://www.ezchip.com -- To unsubscribe from this list: send the line "unsubscribe linux-kern

Re: [PATCH 38/43] MAINTAINERS: add git URL for tile

2015-12-18 Thread Chris Metcalf
On 12/18/2015 02:52 AM, Fengguang Wu wrote: CC: Chris Metcalf<cmetc...@ezchip.com> Signed-off-by: Fengguang Wu<fengguang...@intel.com> --- MAINTAINERS |1 + 1 file changed, 1 insertion(+) Thanks! Taken into the tile tree. -- Chris Metcalf, EZChip Semiconductor http://ww

Re: [PATCH] Revert "kernel/stop_machine.c: remove CONFIG_SMP dependencies"

2015-12-18 Thread Chris Metcalf
avis-ci.org/sudipm-mukherjee/parport/jobs/97618550 m32r build at: https://travis-ci.org/sudipm-mukherjee/parport/jobs/97618542 kernel/stop_machine.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) Acked-by: Chris Metcalf <cmetc...@ezchip.com> [for tile] -- Chris

[PATCH] tile: include the syscall number in the backtrace

2015-12-17 Thread Chris Metcalf
This information is easily available in the backtrace data and can be helpful when trying to figure out the backtrace, particularly if we're early in kernel entry or late in kernel exit. Signed-off-by: Chris Metcalf --- arch/tile/kernel/stack.c | 17 - 1 file changed, 8

[PATCH] tile: include the syscall number in the backtrace

2015-12-17 Thread Chris Metcalf
This information is easily available in the backtrace data and can be helpful when trying to figure out the backtrace, particularly if we're early in kernel entry or late in kernel exit. Signed-off-by: Chris Metcalf <cmetc...@ezchip.com> --- arch/tile/kernel/stack.c | 17 --

Re: [PATCH 1/9] atomic: Export fetch_or()

2015-12-16 Thread Chris Metcalf
, but if no one else is objecting I'll shut up now. Otherwise, for the series: Reviewed-by: Chris Metcalf -- Chris Metcalf, EZChip Semiconductor http://www.ezchip.com -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kerne

Re: [PATCH 1/9] atomic: Export fetch_or()

2015-12-16 Thread Chris Metcalf
, but if no one else is objecting I'll shut up now. Otherwise, for the series: Reviewed-by: Chris Metcalf <cmetc...@ezchip.com> -- Chris Metcalf, EZChip Semiconductor http://www.ezchip.com -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a m

Re: [PATCH 1/7] atomic: Export fetch_or()

2015-11-30 Thread Chris Metcalf
(Resending as plain text. Not sure what Thunderbird was smoking to make this message multipart/alternative originally...) On 11/30/2015 12:36 PM, Frederic Weisbecker wrote: On Tue, Nov 24, 2015 at 04:48:35PM -0500, Chris Metcalf wrote: >Unless you want to take a big diversion into atom

Re: [PATCH 1/7] atomic: Export fetch_or()

2015-11-30 Thread Chris Metcalf
(Resending as plain text. Not sure what Thunderbird was smoking to make this message multipart/alternative originally...) On 11/30/2015 12:36 PM, Frederic Weisbecker wrote: On Tue, Nov 24, 2015 at 04:48:35PM -0500, Chris Metcalf wrote: >Unless you want to take a big diversion into atom

Re: [PATCH 1/7] atomic: Export fetch_or()

2015-11-24 Thread Chris Metcalf
rly a win to mark the data types as being atomic to the extent we can do so, I think. -- Chris Metcalf, EZChip Semiconductor http://www.ezchip.com -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More maj

Re: [PATCH 3/7] perf: Migrate perf to use new tick dependency mask model

2015-11-24 Thread Chris Metcalf
_inc_return [cpu 2] tick_nohz_set_dep() [cpu 1] tick_nohz_clear_dep() Or perhaps this is a true race condition possibility? I think we're OK for the sched cases since they're protected under the rq lock, I think. I'm not sure about the POSIX cpu timers. -- Chris Metcalf, EZChip Semiconductor htt

Re: [PATCH 2/7] nohz: New tick dependency mask

2015-11-24 Thread Chris Metcalf
actually try to kick if we are in an NMI? -- Chris Metcalf, EZChip Semiconductor http://www.ezchip.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/majordom

Re: [PATCH 1/7] atomic: Export fetch_or()

2015-11-24 Thread Chris Metcalf
On 11/13/2015 09:22 AM, Frederic Weisbecker wrote: Export fetch_or() that's implemented and used internally by the scheduler. We are going to use it for NO_HZ so make it generally available. Cc: Christoph Lameter Cc: Chris Metcalf Cc: Ingo Molnar Cc: Luiz Capitulino Cc: Peter Zijlstra Cc: Rik

Re: [PATCH 1/7] atomic: Export fetch_or()

2015-11-24 Thread Chris Metcalf
rly a win to mark the data types as being atomic to the extent we can do so, I think. -- Chris Metcalf, EZChip Semiconductor http://www.ezchip.com -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More maj

Re: [PATCH 2/7] nohz: New tick dependency mask

2015-11-24 Thread Chris Metcalf
actually try to kick if we are in an NMI? -- Chris Metcalf, EZChip Semiconductor http://www.ezchip.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/majordom

Re: [PATCH 3/7] perf: Migrate perf to use new tick dependency mask model

2015-11-24 Thread Chris Metcalf
_inc_return [cpu 2] tick_nohz_set_dep() [cpu 1] tick_nohz_clear_dep() Or perhaps this is a true race condition possibility? I think we're OK for the sched cases since they're protected under the rq lock, I think. I'm not sure about the POSIX cpu timers. -- Chris Metcalf, EZChip Semiconductor htt

Re: [PATCH 1/7] atomic: Export fetch_or()

2015-11-24 Thread Chris Metcalf
On 11/13/2015 09:22 AM, Frederic Weisbecker wrote: Export fetch_or() that's implemented and used internally by the scheduler. We are going to use it for NO_HZ so make it generally available. Cc: Christoph Lameter<c...@linux.com> Cc: Chris Metcalf<cmetc...@ezchip.com> Cc: In

[PATCH] Documentation/memory-barriers.txt: fix ACCESS_ONCE thinko

2015-11-23 Thread Chris Metcalf
Q, P)". This doesn't match the following text, which is "Q = LOAD P". Change the statement to be "Q = READ_ONCE(P)". Signed-off-by: Chris Metcalf --- Documentation/memory-barriers.txt | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/Docume

[PATCH] Documentation/memory-barriers.txt: fix ACCESS_ONCE thinko

2015-11-23 Thread Chris Metcalf
Q, P)". This doesn't match the following text, which is "Q = LOAD P". Change the statement to be "Q = READ_ONCE(P)". Signed-off-by: Chris Metcalf <cmetc...@ezchip.com> --- Documentation/memory-barriers.txt | 8 1 file changed, 4 insertions(+), 4 deletion

Re: [PATCH] mm: fix incorrect behavior when process virtual address space limit is exceeded

2015-11-17 Thread Chris Metcalf
separate patch. In this case we can unexport mmap_region(). The problem is that we are mapping a region of virtual address space that the chip provides for setting up interrupt handlers (at 0xfc00) but that is above the TASK_SIZE cutoff, so do_mmap() would fail the call in get_unmapped_area().

Re: [PATCH] mm: fix incorrect behavior when process virtual address space limit is exceeded

2015-11-17 Thread Chris Metcalf
separate patch. In this case we can unexport mmap_region(). The problem is that we are mapping a region of virtual address space that the chip provides for setting up interrupt handlers (at 0xfc00) but that is above the TASK_SIZE cutoff, so do_mmap() would fail the call in get_unmapped_area().

Re: [PATCH v6 13/17] arm64:ilp32: add sys_ilp32.c and a separate table (in entry.S) to use it

2015-11-15 Thread Chris Metcalf
and _FILE_OFFSET_BITS == 64 on top of the 64-bit __kernel_off_t? I just did a little bit of digging through glibc history and found that Chris Metcalf added the files that are now in sysdeps/unix/sysv/linux/generic/wordsize-32/ and that provide the implementation for 32-bit off_t in glibc on top of the 64-bit

Re: [PATCH v6 13/17] arm64:ilp32: add sys_ilp32.c and a separate table (in entry.S) to use it

2015-11-15 Thread Chris Metcalf
hey also implement both _FILE_OFFSET_BITS == 32 and _FILE_OFFSET_BITS == 64 on top of the 64-bit __kernel_off_t? I just did a little bit of digging through glibc history and found that Chris Metcalf added the files that are now in sysdeps/unix/sysv/linux/generic/wordsize-32/ and that provide the imp

Re: [PATCH v2] Documentation/SubmittingPatches: discuss In-Reply-To

2015-11-11 Thread Chris Metcalf
On 11/11/2015 12:07 PM, Jonathan Corbet wrote: On Thu, 5 Nov 2015 15:21:47 -0500 Chris Metcalf wrote: Add a paragraph suggesting best practices for when to link patches to previous LKML messages via In-Reply-To. We're getting there, but in the middle of the discussion on the format

Re: [PATCH v2] Documentation/SubmittingPatches: discuss In-Reply-To

2015-11-11 Thread Chris Metcalf
On 11/11/2015 12:07 PM, Jonathan Corbet wrote: On Thu, 5 Nov 2015 15:21:47 -0500 Chris Metcalf <cmetc...@ezchip.com> wrote: Add a paragraph suggesting best practices for when to link patches to previous LKML messages via In-Reply-To. We're getting there, but in the middle of the disc

[PATCH v2] Documentation/SubmittingPatches: discuss In-Reply-To

2015-11-05 Thread Chris Metcalf
Add a paragraph suggesting best practices for when to link patches to previous LKML messages via In-Reply-To. Signed-off-by: Chris Metcalf --- Documentation/SubmittingPatches | 10 ++ 1 file changed, 10 insertions(+) diff --git a/Documentation/SubmittingPatches b/Documentation

Re: [PATCH] Documentation/email-clients.txt: discuss In-Reply-To

2015-11-05 Thread Chris Metcalf
On 11/05/2015 01:31 PM, Jonathan Corbet wrote: On Fri, 23 Oct 2015 12:13:01 -0400 Chris Metcalf wrote: +When manually adding In-Reply-To: headers to a patch (e.g., using `git +send email`), use common sense to associate the patch with previous +relevant discussion, e.g. link a bug fix

[PATCH v2] Documentation/SubmittingPatches: discuss In-Reply-To

2015-11-05 Thread Chris Metcalf
Add a paragraph suggesting best practices for when to link patches to previous LKML messages via In-Reply-To. Signed-off-by: Chris Metcalf <cmetc...@ezchip.com> --- Documentation/SubmittingPatches | 10 ++ 1 file changed, 10 insertions(+) diff --git a/Documentation/SubmittingPat

Re: [PATCH] Documentation/email-clients.txt: discuss In-Reply-To

2015-11-05 Thread Chris Metcalf
On 11/05/2015 01:31 PM, Jonathan Corbet wrote: On Fri, 23 Oct 2015 12:13:01 -0400 Chris Metcalf <cmetc...@ezchip.com> wrote: +When manually adding In-Reply-To: headers to a patch (e.g., using `git +send email`), use common sense to associate the patch with previous +relevant discussio

Re: crazy idea: big percpu lock (Re: task isolation)

2015-10-28 Thread Chris Metcalf
d a lot of these TLB flushes since we needed to adjust the kernel linear mapping as well as the userspace mappings. It's a lot less common otherwise, just vunmap and the like, but still a good thing for a follow-up patch. -- Chris Metcalf, EZChip Semiconductor http://www.ezchip.com -- To

Re: crazy idea: big percpu lock (Re: task isolation)

2015-10-28 Thread Chris Metcalf
d a lot of these TLB flushes since we needed to adjust the kernel linear mapping as well as the userspace mappings. It's a lot less common otherwise, just vunmap and the like, but still a good thing for a follow-up patch. -- Chris Metcalf, EZChip Semiconductor http://www.ezchip.com -- To

Re: [PATCH v8 06/14] task_isolation: provide strict mode configurable signal

2015-10-27 Thread Chris Metcalf
On 10/22/2015 05:00 PM, Andy Lutomirski wrote: On Thu, Oct 22, 2015 at 1:44 PM, Chris Metcalf wrote: On 10/21/2015 02:53 PM, Andy Lutomirski wrote: On Tue, Oct 20, 2015 at 11:41 PM, Gilad Ben Yossef wrote: From: Andy Lutomirski [mailto:l...@amacapital.net] Sent: Wednesday, October 21

Re: [PATCH v8 04/14] task_isolation: add initial support

2015-10-27 Thread Chris Metcalf
On 10/21/2015 12:12 PM, Frederic Weisbecker wrote: On Tue, Oct 20, 2015 at 04:36:02PM -0400, Chris Metcalf wrote: +/* + * This routine controls whether we can enable task-isolation mode. + * The task must be affinitized to a single nohz_full core or we will + * return EINVAL. Although

Re: [PATCH v8 04/14] task_isolation: add initial support

2015-10-27 Thread Chris Metcalf
On 10/21/2015 12:12 PM, Frederic Weisbecker wrote: On Tue, Oct 20, 2015 at 04:36:02PM -0400, Chris Metcalf wrote: +/* + * This routine controls whether we can enable task-isolation mode. + * The task must be affinitized to a single nohz_full core or we will + * return EINVAL. Although

Re: [PATCH v8 06/14] task_isolation: provide strict mode configurable signal

2015-10-27 Thread Chris Metcalf
On 10/22/2015 05:00 PM, Andy Lutomirski wrote: On Thu, Oct 22, 2015 at 1:44 PM, Chris Metcalf <cmetc...@ezchip.com> wrote: On 10/21/2015 02:53 PM, Andy Lutomirski wrote: On Tue, Oct 20, 2015 at 11:41 PM, Gilad Ben Yossef <gil...@ezchip.com> wrote: From: Andy Lutomirs

Re: [PATCH v8 04/14] task_isolation: add initial support

2015-10-26 Thread Chris Metcalf
after tick_nohz_stop_sched_tick() manages to stop the tick, and it won't if rcu_needs_cpu() is true. -- Chris Metcalf, EZChip Semiconductor http://www.ezchip.com -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kerne

Re: [PATCH v8 04/14] task_isolation: add initial support

2015-10-26 Thread Chris Metcalf
e series. But the rest of it seems like things you just control for when you are running the application, and if you do it right, the application runs. If you don't (e.g. you intentionally schedule multiple processes on the same core), the app doesn't run, and you fix it in development. --

Re: [PATCH v8 04/14] task_isolation: add initial support

2015-10-26 Thread Chris Metcalf
f complexity both in the API (documenting a new scheduler class) and in the implementation (putting new requirements into the scheduler implementations). So I'm somewhat dubious, although willing to be pushed in that direction if that's the consensus. On balance I think it still feels to me li

Re: [PATCH v8 04/14] task_isolation: add initial support

2015-10-26 Thread Chris Metcalf
e series. But the rest of it seems like things you just control for when you are running the application, and if you do it right, the application runs. If you don't (e.g. you intentionally schedule multiple processes on the same core), the app doesn't run, and you fix it in development. --

Re: [PATCH v8 04/14] task_isolation: add initial support

2015-10-26 Thread Chris Metcalf
after tick_nohz_stop_sched_tick() manages to stop the tick, and it won't if rcu_needs_cpu() is true. -- Chris Metcalf, EZChip Semiconductor http://www.ezchip.com -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kerne

Re: [PATCH v8 04/14] task_isolation: add initial support

2015-10-26 Thread Chris Metcalf
f complexity both in the API (documenting a new scheduler class) and in the implementation (putting new requirements into the scheduler implementations). So I'm somewhat dubious, although willing to be pushed in that direction if that's the consensus. On balance I think it still feels to me li

[PATCH] Documentation/email-clients.txt: discuss In-Reply-To

2015-10-23 Thread Chris Metcalf
In-Reply-To: <20151023090459.gw17...@twins.programming.kicks-ass.net> This commit adds a paragraph suggesting best practices for when to link patches to previous LKML messages via In-Reply-To. Signed-off-by: Chris Metcalf --- Documentation/email-clients.txt | 10 ++ 1 file chang

[PATCH] Documentation/email-clients.txt: discuss In-Reply-To

2015-10-23 Thread Chris Metcalf
In-Reply-To: <20151023090459.gw17...@twins.programming.kicks-ass.net> This commit adds a paragraph suggesting best practices for when to link patches to previous LKML messages via In-Reply-To. Signed-off-by: Chris Metcalf <cmetc...@ezchip.com> --- Documentation/email-clie

Re: [PATCH v8 06/14] task_isolation: provide strict mode configurable signal

2015-10-22 Thread Chris Metcalf
On 10/21/2015 02:53 PM, Andy Lutomirski wrote: On Tue, Oct 20, 2015 at 11:41 PM, Gilad Ben Yossef wrote: From: Andy Lutomirski [mailto:l...@amacapital.net] Sent: Wednesday, October 21, 2015 4:43 AM To: Chris Metcalf Subject: Re: [PATCH v8 06/14] task_isolation: provide strict mode

Re: [PATCH v8 00/14] support "task_isolation" mode for nohz_full

2015-10-22 Thread Chris Metcalf
his option? Or is it more a question of scale, and the 80-odd patches that I've posted so far just pushed an otherwise good system into a more dysfunctional mode? If so, perhaps some text in Documentation/SubmittingPatches would be helpful here. -- Chris Metcalf, EZChip Semiconductor http://www.

Re: [PATCH v8 06/14] task_isolation: provide strict mode configurable signal

2015-10-22 Thread Chris Metcalf
On 10/21/2015 02:53 PM, Andy Lutomirski wrote: On Tue, Oct 20, 2015 at 11:41 PM, Gilad Ben Yossef <gil...@ezchip.com> wrote: From: Andy Lutomirski [mailto:l...@amacapital.net] Sent: Wednesday, October 21, 2015 4:43 AM To: Chris Metcalf Subject: Re: [PATCH v8 06/14] task_isolation: p

Re: [PATCH v8 00/14] support "task_isolation" mode for nohz_full

2015-10-22 Thread Chris Metcalf
his option? Or is it more a question of scale, and the 80-odd patches that I've posted so far just pushed an otherwise good system into a more dysfunctional mode? If so, perhaps some text in Documentation/SubmittingPatches would be helpful here. -- Chris Metcalf, EZChip Semiconductor http://www.

Re: [PATCH v8 06/14] task_isolation: provide strict mode configurable signal

2015-10-20 Thread Chris Metcalf
On 10/20/2015 8:56 PM, Steven Rostedt wrote: On Tue, 20 Oct 2015 16:36:04 -0400 Chris Metcalf wrote: Allow userspace to override the default SIGKILL delivered when a task_isolation process in STRICT mode does a syscall or otherwise synchronously enters the kernel. Is this really a good idea

Re: [PATCH v8 04/14] task_isolation: add initial support

2015-10-20 Thread Chris Metcalf
On 10/20/2015 04:56 PM, Andy Lutomirski wrote: On Tue, Oct 20, 2015 at 1:36 PM, Chris Metcalf wrote: +/* + * In task isolation mode we try to return to userspace only after + * attempting to make sure we won't be interrupted again. To handle + * the periodic scheduler tick, we test to make

Re: [PATCH v8 08/14] nohz_full: allow disabling the 1Hz minimum tick at boot

2015-10-20 Thread Chris Metcalf
On 10/20/2015 05:03 PM, Frederic Weisbecker wrote: On Tue, Oct 20, 2015 at 04:36:06PM -0400, Chris Metcalf wrote: While the current fallback to 1-second tick is still required for a number of kernel accounting tasks (e.g. vruntime, load balancing data, and load accounting), it's useful

[PATCH v8 04/14] task_isolation: add initial support

2015-10-20 Thread Chris Metcalf
ion intentionally calls into the kernel. Separate patches that follow provide these changes for x86, arm64, and tile. Signed-off-by: Chris Metcalf --- include/linux/isolation.h | 38 ++ include/linux/sched.h | 3 ++ include/uapi/linux/prctl.h | 5 +++ init/Kconfi

[PATCH v8 08/14] nohz_full: allow disabling the 1Hz minimum tick at boot

2015-10-20 Thread Chris Metcalf
possibly-spurious timer interrupts are scheduled when the basic 1Hz tick has been turned off. Signed-off-by: Chris Metcalf --- kernel/sched/core.c | 16 1 file changed, 16 insertions(+) diff --git a/kernel/sched/core.c b/kernel/sched/core.c index b79f8e0aeffb..634d5c2ab08a 100644

[PATCH v8 14/14] arch/tile: enable task isolation functionality

2015-10-20 Thread Chris Metcalf
not generate signals to the application. In addition, we add an overriding task_isolation_wait() call that runs a nap instruction while waiting for an interrupt, to make the task_isolation_enter() loop run in a lower-power state. Signed-off-by: Chris Metcalf --- arch/tile/kernel/process.c | 6

[PATCH v8 02/14] vmstat: add vmstat_idle function

2015-10-20 Thread Chris Metcalf
This function checks to see if a vmstat worker is not running, and the vmstat diffs don't require an update. The function is called from the task-isolation code to see if we need to actually do some work to quiet vmstat. Signed-off-by: Chris Metcalf --- include/linux/vmstat.h | 2 ++ mm

[PATCH v8 05/14] task_isolation: support PR_TASK_ISOLATION_STRICT mode

2015-10-20 Thread Chris Metcalf
to allow exiting the task without a pointless signal killing you as you try to do so. Signed-off-by: Chris Metcalf --- include/linux/isolation.h | 21 + include/uapi/linux/prctl.h | 1 + kernel/isolation.c | 42 ++ 3 files changed

[PATCH v8 13/14] arch/tile: turn off timer tick for oneshot_stopped state

2015-10-20 Thread Chris Metcalf
disastrous kernel timer interruption that could cause packets to be dropped. Signed-off-by: Chris Metcalf --- arch/tile/kernel/time.c | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/tile/kernel/time.c b/arch/tile/kernel/time.c index 178989e6d3e3..fbedf380d9d4 100644 --- a/arch/tile/ker

[PATCH v8 07/14] task_isolation: add debug boot flag

2015-10-20 Thread Chris Metcalf
to deliver an interrupt to a task_isolation core. It may be worth considering other ways to generate useful debugging output rather than console spew, but for now that is simple and direct. Signed-off-by: Chris Metcalf --- Documentation/kernel-parameters.txt | 7 +++ include/linux/isolation.h

[PATCH v8 10/14] arch/arm64: adopt prepare_exit_to_usermode() model from x86

2015-10-20 Thread Chris Metcalf
c5c46f59e4e7 ("x86/entry: Add new, comprehensible entry and exit handlers written in C"). Signed-off-by: Chris Metcalf --- arch/arm64/kernel/entry.S | 6 +++--- arch/arm64/kernel/signal.c | 32 ++-- 2 files changed, 25 insertions(+), 13 deletions(-) diff --git a/

[PATCH v8 12/14] arch/tile: adopt prepare_exit_to_usermode() model from x86

2015-10-20 Thread Chris Metcalf
all out the bug explicitly now, but don't yet fix it. Signed-off-by: Chris Metcalf --- arch/tile/include/asm/processor.h | 2 +- arch/tile/include/asm/thread_info.h | 8 +++- arch/tile/kernel/intvec_32.S| 46 +++-- arch/tile/kernel/intvec_64.S| 49 +++

[PATCH v8 06/14] task_isolation: provide strict mode configurable signal

2015-10-20 Thread Chris Metcalf
Allow userspace to override the default SIGKILL delivered when a task_isolation process in STRICT mode does a syscall or otherwise synchronously enters the kernel. Signed-off-by: Chris Metcalf --- include/uapi/linux/prctl.h | 2 ++ kernel/isolation.c | 9 - 2 files changed, 10

[PATCH v8 09/14] arch/x86: enable task isolation functionality

2015-10-20 Thread Chris Metcalf
strict reporting for the kernel exception types that do not result in signals, namely non-signalling page faults and non-signalling MPX fixups. Signed-off-by: Chris Metcalf --- arch/x86/entry/common.c | 10 +- arch/x86/kernel/traps.c | 2 ++ arch/x86/mm/fault.c | 2 ++ 3 files

[PATCH v8 11/14] arch/arm64: enable task isolation functionality

2015-10-20 Thread Chris Metcalf
CT mode in do_mem_abort() to handle the case of page faults. Signed-off-by: Chris Metcalf --- arch/arm64/include/asm/thread_info.h | 18 -- arch/arm64/kernel/ptrace.c | 12 +--- arch/arm64/kernel/signal.c | 7 +-- arch/arm64/mm/fault.c

[PATCH v8 03/14] lru_add_drain_all: factor out lru_add_drain_needed

2015-10-20 Thread Chris Metcalf
This per-cpu check was being done in the loop in lru_add_drain_all(), but having it be callable for a particular cpu is helpful for the task-isolation patches. Signed-off-by: Chris Metcalf --- include/linux/swap.h | 1 + mm/swap.c| 13 + 2 files changed, 10 insertions

[PATCH v8 00/14] support "task_isolation" mode for nohz_full

2015-10-20 Thread Chris Metcalf
he one hand, and making voluntary calls in and out of the kernel more expensive, for tasks that want it. The series is available at: git://git.kernel.org/pub/scm/linux/kernel/git/cmetcalf/linux-tile.git dataplane Chris Metcalf (13): vmstat: add vmstat_idle function lru_add_drain_all: facto

[PATCH v8 01/14] vmstat: provide a function to quiet down the diff processing

2015-10-20 Thread Chris Metcalf
erd thread will continue scanning the differentials from another processor and will reenable the vmstat workers if it detects any changes. Signed-off-by: Christoph Lameter Signed-off-by: Chris Metcalf --- include/linux/vmstat.h | 2 ++ mm/vmstat.c| 14 ++ 2 files changed, 16

[PATCH v8 01/14] vmstat: provide a function to quiet down the diff processing

2015-10-20 Thread Chris Metcalf
hat the shepherd thread will continue scanning the differentials from another processor and will reenable the vmstat workers if it detects any changes. Signed-off-by: Christoph Lameter <c...@linux.com> Signed-off-by: Chris Metcalf <cmetc...@ezchip.com> --- include/linux/v

[PATCH v8 00/14] support "task_isolation" mode for nohz_full

2015-10-20 Thread Chris Metcalf
he one hand, and making voluntary calls in and out of the kernel more expensive, for tasks that want it. The series is available at: git://git.kernel.org/pub/scm/linux/kernel/git/cmetcalf/linux-tile.git dataplane Chris Metcalf (13): vmstat: add vmstat_idle function lru_add_drain_all: facto

[PATCH v8 03/14] lru_add_drain_all: factor out lru_add_drain_needed

2015-10-20 Thread Chris Metcalf
This per-cpu check was being done in the loop in lru_add_drain_all(), but having it be callable for a particular cpu is helpful for the task-isolation patches. Signed-off-by: Chris Metcalf <cmetc...@ezchip.com> --- include/linux/swap.h | 1 + mm/swap.c| 13 + 2

[PATCH v8 11/14] arch/arm64: enable task isolation functionality

2015-10-20 Thread Chris Metcalf
CT mode in do_mem_abort() to handle the case of page faults. Signed-off-by: Chris Metcalf <cmetc...@ezchip.com> --- arch/arm64/include/asm/thread_info.h | 18 -- arch/arm64/kernel/ptrace.c | 12 +--- arch/arm64/kernel/signal.c | 7 +

[PATCH v8 07/14] task_isolation: add debug boot flag

2015-10-20 Thread Chris Metcalf
to deliver an interrupt to a task_isolation core. It may be worth considering other ways to generate useful debugging output rather than console spew, but for now that is simple and direct. Signed-off-by: Chris Metcalf <cmetc...@ezchip.com> --- Documentation/kernel-parameters.txt | 7 ++

[PATCH v8 06/14] task_isolation: provide strict mode configurable signal

2015-10-20 Thread Chris Metcalf
Allow userspace to override the default SIGKILL delivered when a task_isolation process in STRICT mode does a syscall or otherwise synchronously enters the kernel. Signed-off-by: Chris Metcalf <cmetc...@ezchip.com> --- include/uapi/linux/prctl.h | 2 ++ kernel/isolation.c

[PATCH v8 12/14] arch/tile: adopt prepare_exit_to_usermode() model from x86

2015-10-20 Thread Chris Metcalf
all out the bug explicitly now, but don't yet fix it. Signed-off-by: Chris Metcalf <cmetc...@ezchip.com> --- arch/tile/include/asm/processor.h | 2 +- arch/tile/include/asm/thread_info.h | 8 +++- arch/tile/kernel/intvec_32.S| 46 +++-- arch

[PATCH v8 10/14] arch/arm64: adopt prepare_exit_to_usermode() model from x86

2015-10-20 Thread Chris Metcalf
c5c46f59e4e7 ("x86/entry: Add new, comprehensible entry and exit handlers written in C"). Signed-off-by: Chris Metcalf <cmetc...@ezchip.com> --- arch/arm64/kernel/entry.S | 6 +++--- arch/arm64/kernel/signal.c | 32 ++-- 2 files changed, 25 insertions(

[PATCH v8 09/14] arch/x86: enable task isolation functionality

2015-10-20 Thread Chris Metcalf
strict reporting for the kernel exception types that do not result in signals, namely non-signalling page faults and non-signalling MPX fixups. Signed-off-by: Chris Metcalf <cmetc...@ezchip.com> --- arch/x86/entry/common.c | 10 +- arch/x86/kernel/traps.c | 2 ++ arch/x86/mm/f

Re: [PATCH v8 04/14] task_isolation: add initial support

2015-10-20 Thread Chris Metcalf
On 10/20/2015 04:56 PM, Andy Lutomirski wrote: On Tue, Oct 20, 2015 at 1:36 PM, Chris Metcalf <cmetc...@ezchip.com> wrote: +/* + * In task isolation mode we try to return to userspace only after + * attempting to make sure we won't be interrupted again. To handle + * the periodic sch

[PATCH v8 04/14] task_isolation: add initial support

2015-10-20 Thread Chris Metcalf
ion intentionally calls into the kernel. Separate patches that follow provide these changes for x86, arm64, and tile. Signed-off-by: Chris Metcalf <cmetc...@ezchip.com> --- include/linux/isolation.h | 38 ++ include/linux/sched.h | 3 ++ include/uapi

Re: [PATCH v8 08/14] nohz_full: allow disabling the 1Hz minimum tick at boot

2015-10-20 Thread Chris Metcalf
On 10/20/2015 05:03 PM, Frederic Weisbecker wrote: On Tue, Oct 20, 2015 at 04:36:06PM -0400, Chris Metcalf wrote: While the current fallback to 1-second tick is still required for a number of kernel accounting tasks (e.g. vruntime, load balancing data, and load accounting), it's useful

[PATCH v8 14/14] arch/tile: enable task isolation functionality

2015-10-20 Thread Chris Metcalf
not generate signals to the application. In addition, we add an overriding task_isolation_wait() call that runs a nap instruction while waiting for an interrupt, to make the task_isolation_enter() loop run in a lower-power state. Signed-off-by: Chris Metcalf <cmetc...@ezchip.com> --- arc

[PATCH v8 02/14] vmstat: add vmstat_idle function

2015-10-20 Thread Chris Metcalf
This function checks to see if a vmstat worker is not running, and the vmstat diffs don't require an update. The function is called from the task-isolation code to see if we need to actually do some work to quiet vmstat. Signed-off-by: Chris Metcalf <cmetc...@ezchip.com> --- include

<    3   4   5   6   7   8   9   10   11   12   >