Re: compiling master.

2016-11-03 Thread Naveen N. Rao
On 2016/11/03 03:55PM, Nicholas Piggin wrote: > On Wed, 2 Nov 2016 13:49:39 +0300 > Denis Kirjanov wrote: > > > Hi guys, > > > > compiling ppc head with bunch of asm errors on power8 box (gcc version > > 4.8.3 ) > > checked commit log but found nothing special. Looks

Re: [PATCH] powerpc/64s: relocation, register save fixes for system reset interrupt

2016-11-03 Thread Nicholas Piggin
On Thu, 3 Nov 2016 02:32:39 -0400 "Shreyas B. Prabhu" wrote: > On Thu, Nov 3, 2016 at 2:17 AM, Nicholas Piggin wrote: > > On Thu, 3 Nov 2016 01:56:46 -0400 > > "Shreyas B. Prabhu" wrote: > > > >> On Thu, Nov 3, 2016 at 1:21 AM,

Re: [PATCH 0/3] powerpc: Emulation support for load/store instructions on LE

2016-11-03 Thread Naveen N. Rao
On 2016/11/02 02:23PM, Ravi Bangoria wrote: > emulate_step is the basic infrastructure which is used by number of other > kernel infrastructures like kprobe, hw-breakpoint(data breakpoint) etc. > In case of kprobe, enabling emulation of load/store instructions will > speedup the execution of

Re: compiling master.

2016-11-03 Thread Nicholas Piggin
On Thu, 3 Nov 2016 12:23:17 +0530 "Naveen N. Rao" wrote: > On 2016/11/03 03:55PM, Nicholas Piggin wrote: > > On Wed, 2 Nov 2016 13:49:39 +0300 > > Denis Kirjanov wrote: > > > > > Hi guys, > > > > > > compiling ppc head with bunch of

[PATCH] powerpc/configs: Drop REISERFS from pseries & powernv

2016-11-03 Thread Michael Ellerman
No one uses reiserfs much these days, or is likely to in future. So drop it from pseries and powernv defconfigs to save time and space. It's still enabled in ppc64_defconfig so we get some build coverage. Signed-off-by: Michael Ellerman ---

Re: [PATCH 2/4] selftests/powerpc/64: Test all paths through copy routines

2016-11-03 Thread kbuild test robot
/powerpc-64-Make-exception-table-clearer-in-__copy_tofrom_user_base/20161103-133058 config: powerpc-maple_defconfig (attached as .config) compiler: powerpc64-linux-gnu-gcc (Debian 6.1.1-9) 6.1.1 20160705 reproduce: wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin

Re: compiling master.

2016-11-03 Thread Denis Kirjanov
On 11/3/16, Naveen N. Rao wrote: > On 2016/11/03 03:55PM, Nicholas Piggin wrote: >> On Wed, 2 Nov 2016 13:49:39 +0300 >> Denis Kirjanov wrote: >> >> > Hi guys, >> > >> > compiling ppc head with bunch of asm errors on power8 box (gcc

[PATCH] ext4: ext4_mb_mark_free_simple: Fix integer value truncation

2016-11-03 Thread Chandan Rajendra
'border' variable is set to a value of 2 times the block size of the underlying filesystem. With 64k block size, the resulting value won't fit into a 16-bit variable. Hence this commit changes the data type of 'border' to 'unsigned int'. Signed-off-by: Chandan Rajendra

Re: [PATCH 1/3] powerpc: Emulation support for load/store instructions on LE

2016-11-03 Thread Michael Ellerman
Ravi Bangoria writes: > On Thursday 03 November 2016 02:34 AM, Anton Blanchard wrote: >> Hi Ravi, >> >>> emulate_step() uses a number of underlying kernel functions that were >>> initially not enabled for LE. This has been rectified since. So, fix >>>

Re: compiling master.

2016-11-03 Thread Michael Ellerman
"Naveen N. Rao" writes: > diff --git a/arch/powerpc/include/asm/exception-64s.h > b/arch/powerpc/include/asm/exception-64s.h > index 2e4e7d8..9b7b302 100644 > --- a/arch/powerpc/include/asm/exception-64s.h > +++ b/arch/powerpc/include/asm/exception-64s.h > @@

Re: [PATCH] cxl: Fix error handling

2016-11-03 Thread Michael Ellerman
Jim Davis writes: > On Wed, Nov 2, 2016 at 4:12 AM, Michael Ellerman wrote: >> Jim Davis writes: >> >>> On Sun, Oct 30, 2016 at 10:37 PM, Michael Ellerman >>> wrote: More here:

Re: [PATCH 1/3] powerpc: Emulation support for load/store instructions on LE

2016-11-03 Thread Ravi Bangoria
On Thursday 03 November 2016 03:18 PM, Michael Ellerman wrote: > Ravi Bangoria writes: > >> On Thursday 03 November 2016 02:34 AM, Anton Blanchard wrote: >>> Hi Ravi, >>> emulate_step() uses a number of underlying kernel functions that were initially

Re: compiling master.

2016-11-03 Thread Naveen N. Rao
On 2016/11/03 08:51PM, Michael Ellerman wrote: > "Naveen N. Rao" writes: > > diff --git a/arch/powerpc/include/asm/exception-64s.h > > b/arch/powerpc/include/asm/exception-64s.h > > index 2e4e7d8..9b7b302 100644 > > --- a/arch/powerpc/include/asm/exception-64s.h

Re: compiling master.

2016-11-03 Thread Naveen N. Rao
On 2016/11/03 06:09PM, Nicholas Piggin wrote: > On Thu, 3 Nov 2016 12:23:17 +0530 > "Naveen N. Rao" wrote: > > diff --git a/arch/powerpc/include/asm/exception-64s.h > > b/arch/powerpc/include/asm/exception-64s.h > > index 2e4e7d8..9b7b302 100644 > > ---

Re: compiling master.

2016-11-03 Thread Denis Kirjanov
On 11/3/16, Naveen N. Rao wrote: > On 2016/11/03 08:51PM, Michael Ellerman wrote: >> "Naveen N. Rao" writes: >> > diff --git a/arch/powerpc/include/asm/exception-64s.h >> > b/arch/powerpc/include/asm/exception-64s.h >> > index

[PATCH] ppc: cpufreq: Use get_cpu()/put_cpu() instead of smp_processor_id() to avoid the oops

2016-11-03 Thread Denis Kirjanov
[ 67.700897] BUG: using smp_processor_id() in preemptible [] code: cat/7343 [ 67.700988] caller is .powernv_cpufreq_throttle_check+0x2c/0x710 [ 67.700998] CPU: 13 PID: 7343 Comm: cat Not tainted 4.8.0-rc5-dirty #1 [ 67.701038] Call Trace: [ 67.701066] [c007d25b75b0]

[PATCH v3] console: use first console if stdout-path device doesn't appear

2016-11-03 Thread Paul Burton
If a device tree specified a preferred device for kernel console output via the stdout-path or linux,stdout-path chosen node properties there's no guarantee that it will have specified a device for which we have a driver. It may also be the case that we do have a driver but it doesn't call

Re: [PATCH v2] console: use first console if stdout-path device doesn't appear

2016-11-03 Thread Paul Burton
On Monday, 31 October 2016 18:31:43 GMT Larry Finger wrote: > On 10/31/2016 06:09 PM, Sergey Senozhatsky wrote: > > Hi, > > > > On (10/31/16 15:50), Paul Burton wrote: > > [..] > > > >> Actually whilst this fixes the output in QEMU it has other problems. I'm > >> still digging... > > > > I

Re: [PATCH] ppc: cpufreq: Use get_cpu()/put_cpu() instead of smp_processor_id() to avoid the oops

2016-11-03 Thread Gautham R Shenoy
Hi Denis, On Thu, Nov 03, 2016 at 07:20:41AM -0400, Denis Kirjanov wrote: > [ 67.700897] BUG: using smp_processor_id() in preemptible [] code: > cat/7343 > [ 67.700988] caller is .powernv_cpufreq_throttle_check+0x2c/0x710 > [ 67.700998] CPU: 13 PID: 7343 Comm: cat Not tainted

Re: [PATCH v3 03/16] scsi: fc: Export fc_bsg_jobdone and use it in FC drivers

2016-11-03 Thread Christoph Hellwig
On Thu, Oct 13, 2016 at 05:00:19PM +0200, Johannes Thumshirn wrote: > Export fc_bsg_jobdone so drivers can use it directly instead of doing > the round-trip via struct fc_bsg_job::job_done() and use it in the LLDDs. > > As we've converted all LLDDs over to use fc_bsg_jobdone() directly, > we can

Re: [PATCH v3 04/16] scsi: Unify interfaces of fc_bsg_jobdone and bsg_job_done

2016-11-03 Thread Christoph Hellwig
On Thu, Oct 13, 2016 at 05:00:20PM +0200, Johannes Thumshirn wrote: > Unify the interfaces of fc_bsg_jobdone and bsg_job_done. This will reduce the > diff when moving from 'struct fc_bsg_job' to a plain 'struct bsg_job' later > on. > > Signed-off-by: Johannes Thumshirn >

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

2016-11-03 Thread Christoph Hellwig
On Fri, Oct 14, 2016 at 09:38:21AM +0200, Johannes Thumshirn wrote: > 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

Re: [PATCH v3] console: use first console if stdout-path device doesn't appear

2016-11-03 Thread Sergey Senozhatsky
On (11/03/16 12:57), Paul Burton wrote: > If a device tree specified a preferred device for kernel console output > via the stdout-path or linux,stdout-path chosen node properties there's > no guarantee that it will have specified a device for which we have a > driver. It may also be the case that

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

2016-11-03 Thread Johannes Thumshirn
On Thu, Nov 03, 2016 at 08:17:51AM -0700, Christoph Hellwig wrote: > On Fri, Oct 14, 2016 at 09:38:21AM +0200, Johannes Thumshirn wrote: > > On Thu, Oct 13, 2016 at 05:55:11PM +0200, Steffen Maier wrote: > > > Hm, still behaves for me like I reported for v2: > > >

Re: [PATCH net-next v6 02/10] dpaa_eth: add support for DPAA Ethernet

2016-11-03 Thread David Miller
From: Madalin Bucur Date: Wed, 2 Nov 2016 22:17:26 +0200 > This introduces the Freescale Data Path Acceleration Architecture > +static inline size_t bpool_buffer_raw_size(u8 index, u8 cnt) > +{ > + u8 i; > + size_t res = DPAA_BP_RAW_SIZE / 2; Always order local

Re: [PATCH v3] console: use first console if stdout-path device doesn't appear

2016-11-03 Thread Paul Burton
Hi Sergey, On Friday, 4 November 2016 02:40:40 GMT Sergey Senozhatsky wrote: > On (11/03/16 12:57), Paul Burton wrote: > > If a device tree specified a preferred device for kernel console output > > via the stdout-path or linux,stdout-path chosen node properties there's > > no guarantee that it

Re: [PATCH v3 03/16] scsi: fc: Export fc_bsg_jobdone and use it in FC drivers

2016-11-03 Thread Tyrel Datwyler
On 10/13/2016 08:00 AM, Johannes Thumshirn wrote: > Export fc_bsg_jobdone so drivers can use it directly instead of doing > the round-trip via struct fc_bsg_job::job_done() and use it in the LLDDs. > > As we've converted all LLDDs over to use fc_bsg_jobdone() directly, > we can remove the

Re: [PATCH v3 04/16] scsi: Unify interfaces of fc_bsg_jobdone and bsg_job_done

2016-11-03 Thread Tyrel Datwyler
On 10/13/2016 08:00 AM, Johannes Thumshirn wrote: > Unify the interfaces of fc_bsg_jobdone and bsg_job_done. This will reduce the > diff when moving from 'struct fc_bsg_job' to a plain 'struct bsg_job' later > on. > > Signed-off-by: Johannes Thumshirn > Reviewed-by: Hannes

Re: [PATCH v3 05/16] scsi: fc: provide fc_bsg_to_shost() helper

2016-11-03 Thread Tyrel Datwyler
On 10/13/2016 08:00 AM, Johannes Thumshirn wrote: > Provide fc_bsg_to_shost() helper that will become handy when we're moving from > struct fc_bsg_job to a plain struct bsg_job. Also use this little helper in > the LLDDs. > > Signed-off-by: Johannes Thumshirn > Reviewed-by:

Re: [PATCH v3 06/16] scsi: fc: provide fc_bsg_to_rport() helper

2016-11-03 Thread Tyrel Datwyler
On 10/13/2016 08:00 AM, Johannes Thumshirn wrote: > Provide fc_bsg_to_rport() helper that will become handy when we're moving > from struct fc_bsg_job to a plain struct bsg_job. Also move all LLDDs to use > the new helper. > > Signed-off-by: Johannes Thumshirn > Reviewed-by:

Re: [PATCH v3 10/16] scsi: change FC drivers to use 'struct bsg_job'

2016-11-03 Thread Tyrel Datwyler
On 10/13/2016 08:00 AM, Johannes Thumshirn wrote: > Change FC drivers to use 'struct bsg_job' from bsg-lib.h instead of 'struct > fc_bsg_job' from scsi_transport_fc.h and remove 'struct fc_bsg_job'. > > Signed-off-by: Johannes Thumshirn > Reviewed-by: Hannes Reinecke

Re: [PATCH v3 13/16] scsi: fc: use bsg_job_done

2016-11-03 Thread Tyrel Datwyler
On 10/13/2016 08:00 AM, Johannes Thumshirn wrote: > fc_bsg_jobdone() and bsg_job_done() are 1:1 copies now so use the bsg-lib one > instead of the FC private implementation. > > Signed-off-by: Johannes Thumshirn > Reviewed-by: Hannes Reinecke For ibmvfc

[PATCH 3/4 v2] selftests/powerpc/64: Test exception cases in copy_tofrom_user

2016-11-03 Thread Paul Mackerras
This adds a set of test cases to test the behaviour of copy_tofrom_user when exceptions are encountered accessing the source or destination. Currently, copy_tofrom_user does not always copy as many bytes as possible when an exception occurs on a store to the destination, and that is reflected in

Re: [PATCH] ppc: cpufreq: Use get_cpu()/put_cpu() instead of smp_processor_id() to avoid the oops

2016-11-03 Thread Denis Kirjanov
On Thursday, November 3, 2016, Gautham R Shenoy wrote: > Hi Denis, > > On Thu, Nov 03, 2016 at 07:20:41AM -0400, Denis Kirjanov wrote: > > [ 67.700897] BUG: using smp_processor_id() in preemptible [] > code: cat/7343 > > [ 67.700988] caller is

Re: [PATCH] ext4: ext4_mb_mark_free_simple: Fix integer value truncation

2016-11-03 Thread Andreas Dilger
On Nov 3, 2016, at 3:14 AM, Chandan Rajendra wrote: > > 'border' variable is set to a value of 2 times the block size of the > underlying filesystem. With 64k block size, the resulting value won't > fit into a 16-bit variable. Hence this commit changes the data type

[PATCH 2/4 v2] selftests/powerpc/64: Test all paths through copy routines

2016-11-03 Thread Paul Mackerras
The hand-coded assembler 64-bit copy routines include feature sections that select one code path or another depending on which CPU we are executing on. The self-tests for these copy routines end up testing just one path. This adds a mechanism for selecting any desired code path at compile time,

Re: [PATCH] powerpc/64s: relocation, register save fixes for system reset interrupt

2016-11-03 Thread Nicholas Piggin
On Thu, 3 Nov 2016 01:56:46 -0400 "Shreyas B. Prabhu" wrote: > On Thu, Nov 3, 2016 at 1:21 AM, Nicholas Piggin wrote: > > On Wed, 2 Nov 2016 14:15:48 +0530 > > Gautham R Shenoy wrote: > > > >> Hi Nick, > >> > >> On Wed, Nov 02,

Re: [PATCH] powerpc/64s: relocation, register save fixes for system reset interrupt

2016-11-03 Thread Shreyas B. Prabhu
On Thu, Nov 3, 2016 at 2:17 AM, Nicholas Piggin wrote: > On Thu, 3 Nov 2016 01:56:46 -0400 > "Shreyas B. Prabhu" wrote: > >> On Thu, Nov 3, 2016 at 1:21 AM, Nicholas Piggin wrote: >> > On Wed, 2 Nov 2016 14:15:48 +0530 >> > Gautham R

Re: [RFC v2 5/7] powerpc: Rename context.vdso_base to context.vdso

2016-11-03 Thread Michael Ellerman
Christopher Covington writes: > Checkpoint/Restore In Userspace (CRIU) needs to be able to unmap and remap > the VDSO to successfully checkpoint and restore applications in the face of > changing VDSO addresses due to Address Space Layout Randomization (ASLR, > randmaps).

[PATCH 2/2] cpufreq: powernv: Use PMSR to verify global and local pstate

2016-11-03 Thread Akshay Adiga
As fast_switch may get called in interrupt disable mode, it does not update the global_pstate_info data structure. Hence the global_pstate_info has stale data whenever pstate is updated through fast_swtich(). So the gpstate_timer can fire after a fast_switch() call has update the pstates to a

[PATCH 1/2] cpufreq: powernv: Adding fast_switch for schedutil

2016-11-03 Thread Akshay Adiga
Adding fast_switch which does light weight operation to set the desired pstate. Signed-off-by: Akshay Adiga --- drivers/cpufreq/powernv-cpufreq.c | 22 +- 1 file changed, 21 insertions(+), 1 deletion(-) diff --git

Re: [PATCH 1/3] powerpc: Emulation support for load/store instructions on LE

2016-11-03 Thread Andrew Donnellan
On 03/11/16 21:27, Ravi Bangoria wrote: Yes, kernel-space hw-breakpoint feature is broken on LE without this. Is there any actual user-visible feature that depends on this, or is this solely for debugging and development purposes? It would of course be *nice* to have it in stable trees

[PATCH] powerpc/asm: Allow including ppc_asm.h in asm files

2016-11-03 Thread Michael Ellerman
There's no reason to #error if we include ppc_asm.h in asm files, the ifdef already prevents any problems. Signed-off-by: Michael Ellerman --- arch/powerpc/include/asm/ppc_asm.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git

Re: [mm PATCH v2 18/26] arch/powerpc: Add option to skip DMA sync as a part of mapping

2016-11-03 Thread Michael Ellerman
Alexander Duyck writes: > This change allows us to pass DMA_ATTR_SKIP_CPU_SYNC which allows us to > avoid invoking cache line invalidation if the driver will just handle it > via a sync_for_cpu or sync_for_device call. > > Cc: Benjamin Herrenschmidt

Re: [PATCH 1/3] powerpc: Emulation support for load/store instructions on LE

2016-11-03 Thread Ravi Bangoria
On Friday 04 November 2016 07:37 AM, Andrew Donnellan wrote: > On 03/11/16 21:27, Ravi Bangoria wrote: >> Yes, kernel-space hw-breakpoint feature is broken on LE without this. > > Is there any actual user-visible feature that depends on this, or is this > solely for debugging and development

Re: [RFC v2 6/7] mm/powerpc: Use generic VDSO remap and unmap functions

2016-11-03 Thread Michael Ellerman
Christopher Covington writes: > The PowerPC VDSO remap and unmap code was copied to a generic location, > only modifying the variable name expected in mm->context (vdso instead of > vdso_base) to match most other architectures. Having adopted this generic > naming, drop the

[PATCH] vfio: Fix build break when SPAPR_TCE_IOMMU=n

2016-11-03 Thread Michael Ellerman
Currently the kconfig logic for VFIO_IOMMU_SPAPR_TCE and VFIO_SPAPR_EEH is broken when SPAPR_TCE_IOMMU=n. Leading to: warning: (VFIO) selects VFIO_IOMMU_SPAPR_TCE which has unmet direct dependencies (VFIO && SPAPR_TCE_IOMMU) warning: (VFIO) selects VFIO_IOMMU_SPAPR_TCE which has unmet