Re: Oops on Power8 (was Re: [PATCH v2 1/7] workqueue: make workqueue available early during boot)

2016-10-14 Thread Balbir Singh
On 15/10/16 02:08, Tejun Heo wrote: > Hello, Balbir. > > On Tue, Oct 11, 2016 at 11:21:09PM +1100, Balbir Singh wrote: >> FYI: I see the samething on my cpu as well, its just that I get lucky >> and cpu_online(cpu) returns false. > > Are you seeing this on x86 or is your test setup also a

Re: Oops on Power8 (was Re: [PATCH v2 1/7] workqueue: make workqueue available early during boot)

2016-10-14 Thread Balbir Singh
On 15/10/16 02:07, Tejun Heo wrote: > Hello, Michael. > > On Tue, Oct 11, 2016 at 10:22:13PM +1100, Michael Ellerman wrote: >> The oops happens because we're in enqueue_task_fair() and p->se->cfs_rq >> is NULL. >> >> The cfs_rq is NULL because we did set_task_rq(p, 2048), where 2048 is >>

Re: [PATCH] powerpc/mm: simplify loop control in parse_numa_properties()

2016-10-14 Thread Reza Arbab
On Fri, Oct 14, 2016 at 11:33:12AM +1100, Michael Ellerman wrote: Reza Arbab writes: The flow of the main loop in parse_numa_properties() is overly complicated. Simplify it to be less confusing and easier to read. No functional change. I think you're right, but it's

Re: Oops on Power8 (was Re: [PATCH v2 1/7] workqueue: make workqueue available early during boot)

2016-10-14 Thread Tejun Heo
Hello, Balbir. On Tue, Oct 11, 2016 at 11:21:09PM +1100, Balbir Singh wrote: > FYI: I see the samething on my cpu as well, its just that I get lucky > and cpu_online(cpu) returns false. Are you seeing this on x86 or is your test setup also a power machine? Thanks. -- tejun

Re: Oops on Power8 (was Re: [PATCH v2 1/7] workqueue: make workqueue available early during boot)

2016-10-14 Thread Tejun Heo
Hello, Michael. On Tue, Oct 11, 2016 at 10:22:13PM +1100, Michael Ellerman wrote: > The oops happens because we're in enqueue_task_fair() and p->se->cfs_rq > is NULL. > > The cfs_rq is NULL because we did set_task_rq(p, 2048), where 2048 is > NR_CPUS. That causes us to index past the end of the

Re: [PowerPC] Kernel panic while running CPU toggle test on 4.8.0 kernel

2016-10-14 Thread Balbir Singh
On 14/10/16 17:24, Abdul Haleem wrote: > Hi, > > Kernel Oops, followed by kernel panic seen while running rcutorture test on > 4.8.0 kernel > Machine Type : PowerPC Bare Metal > > RCU torture test steps: > 1. modprobe rcutorture > 2. start CPU offline and online while torture test is running

RE: [PATCH] powerpc: link error on orphan sections

2016-10-14 Thread David Laight
From: Michael Ellerman > Sent: 14 October 2016 01:46 ... > > +LDFLAGS_vmlinux:= $(LDFLAGS_vmlinux-y) --orphan-handling=error > > At least some old(er) toolchains don't support that: > > /opt/cross/kisskb/gcc-4.6.3-nolibc/powerpc-linux/bin/powerpc-linux-ld: > unrecognized option '-- >

Re: [PATCH][RFC] powerpc/64s: Use emergency stack for 0x100 system reset interrupt

2016-10-14 Thread Balbir Singh
On 13/10/16 17:51, Nicholas Piggin wrote: > Also use EX_NMI save space. This should give cleaner debug data in > case kernel stacks have become corrupted. > > MSR_RI is kept clear for the 0x100 handler, because we can't mask it, > and can't tolerate a reentrant NMI. Technically the 0x100 is not

Re: [PATCH v2] powerpc/64: option to force run-at-load to test relocation

2016-10-14 Thread Balbir Singh
On 14/10/16 18:31, Nicholas Piggin wrote: > This adds a config option that can help exercise the case when > the kernel is not running at PAGE_OFFSET. > > Signed-off-by: Nicholas Piggin > --- Reviewed-by: Balbir Singh

[RESEND] [PATCH v3] cxl: Prevent adapter reset if an active context exists

2016-10-14 Thread Vaibhav Jain
This patch prevents resetting the cxl adapter via sysfs in presence of one or more active cxl_context on it. This protects against an unrecoverable error caused by PSL owning a dirty cache line even after reset and host tries to touch the same cache line. In case a force reset of the card is

Re: [PATCH v3] cxl: Prevent adapter reset if an active context exists

2016-10-14 Thread Vaibhav Jain
Hi Mpe, Michael Ellerman writes: > This is arguably a change to the ABI, but I assume we're OK with that as > it's arguably a bug in the existing implementation? Yes cross checked with Fred and he is ok with this. > > If so do we have a Fixes: ? and/or should this go to

[PATCH v4 09/12] powerpc/tracing: fix compat syscall handling

2016-10-14 Thread Marcin Nowakowski
Adapt the code to make use of new syscall handling interface Signed-off-by: Marcin Nowakowski Cc: Steven Rostedt Cc: Ingo Molnar Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc:

[PATCH v4 04/12] tracing/syscalls: add compat syscall metadata

2016-10-14 Thread Marcin Nowakowski
Now that compat syscalls are properly distinguished from native calls, we can add metadata for compat syscalls as well. All the macros used to generate the metadata are the same as for standard syscalls, but with a compat_ prefix to distinguish them easily. Signed-off-by: Marcin Nowakowski

Re: [PATCH v3 02/16] scsi: don't use fc_bsg_job::request and fc_bsg_job::reply directly

2016-10-14 Thread Johannes Thumshirn
On Thu, Oct 13, 2016 at 05:55:11PM +0200, Steffen Maier wrote: > Hm, still behaves for me like I reported for v2: > http://marc.info/?l=linux-scsi=147637177902937=2 Hi Steffen, Can you please try the following on top of 2/16? diff --git a/drivers/scsi/scsi_transport_fc.c

[PATCH v2] powerpc: link warning for orphan sections

2016-10-14 Thread Nicholas Piggin
Add --orphan-handling=warn to final link flags. This ensures we can handle all sections explicitly. This would have caught subtle breakage such as 7de3b27bac47da9de08409df1d69664acbb72197 at build-time. Also bring some wayward sections into the fold: - .text.hot and .text.unlikely are compiler

[PATCH v2] powerpc/64: option to force run-at-load to test relocation

2016-10-14 Thread Nicholas Piggin
This adds a config option that can help exercise the case when the kernel is not running at PAGE_OFFSET. Signed-off-by: Nicholas Piggin --- Changes since v1: - Improved style as suggested by Balbir and Michael arch/powerpc/Kconfig | 9 +

Re: [PATCH 0/6] kvm: powerpc halt polling updates

2016-10-14 Thread Nicholas Piggin
On Fri, 14 Oct 2016 11:53:18 +1100 Suraj Jitindar Singh wrote: > This patch series makes some updates and bug fixes to the powerpc kvm-hv > halt polling code. > > The first two patches are concerned with exporting the generic kvm module > parameter variables and

Re: [PATCH] powerpc/64: option to force run-at-load to test relocation

2016-10-14 Thread Michael Ellerman
Balbir Singh writes: > On 12/10/16 17:57, Nicholas Piggin wrote: >> diff --git a/arch/powerpc/kernel/head_64.S b/arch/powerpc/kernel/head_64.S >> index 79da0641..bc9ceac 100644 >> --- a/arch/powerpc/kernel/head_64.S >> +++ b/arch/powerpc/kernel/head_64.S >> @@ -111,8

Re: [PATCH v3] cxl: Prevent adapter reset if an active context exists

2016-10-14 Thread Andrew Donnellan
On 13/10/16 22:08, Vaibhav Jain wrote: This patch prevents resetting the cxl adapter via sysfs in presence of one or more active cxl_context on it. This protects against an unrecoverable error caused by PSL owning a dirty cache line even after reset and host tries to touch the same cache line.