build regression from c153693: Simplify module TOC handling

2016-02-09 Thread Peter Robinson
Hi Alan, Your patch for "powerpc: Simplify module TOC handling" is causing the Fedora ppc64le to fail to build with depmod failures. Reverting the commit fixes it for us on rawhide. We're getting the out put below, full logs at [1]. Let me know if you have any other queries. Regards, Peter [1]

[PATCH v7 02/23] powerpc/8xx: Map linear kernel RAM with 8M pages

2016-02-09 Thread Christophe Leroy
On a live running system (VoIP gateway for Air Trafic Control), over a 10 minutes period (with 277s idle), we get 87 millions DTLB misses and approximatly 35 secondes are spent in DTLB handler. This represents 5.8% of the overall time and even 10.8% of the non-idle time. Among those 87 millions

[PATCH v7 03/23] powerpc: Update documentation for noltlbs kernel parameter

2016-02-09 Thread Christophe Leroy
Now the noltlbs kernel parameter is also applicable to PPC8xx Signed-off-by: Christophe Leroy --- v2: no change v3: no change v4: no change v5: no change v6: no change v7: no change Documentation/kernel-parameters.txt | 2 +- 1 file changed, 1 insertion(+), 1

[PATCH v7 10/23] powerpc/8xx: map more RAM at startup when needed

2016-02-09 Thread Christophe Leroy
On recent kernels, with some debug options like for instance CONFIG_LOCKDEP, the BSS requires more than 8M memory, allthough the kernel code fits in the first 8M. Today, it is necessary to activate CONFIG_PIN_TLB to get more than 8M at startup, allthough pinning TLB is not necessary for that.

[PATCH v7 17/23] powerpc/8xx: rewrite flush_instruction_cache() in C

2016-02-09 Thread Christophe Leroy
On PPC8xx, flushing instruction cache is performed by writing in register SPRN_IC_CST. This registers suffers CPU6 ERRATA. The patch rewrites the fonction in C so that CPU6 ERRATA will be handled transparently Signed-off-by: Christophe Leroy --- v2: no change v3: no

[PATCH][v3] mpc85xx/lbc: modify suspend/resume entry sequence

2016-02-09 Thread Raghav Dogra
Modify platform driver suspend/resume to syscore suspend/resume. This is because p1022ds needs to use localbus when entering the PCIE resume. Signed-off-by: Raghav Dogra --- Changes for v3: rebased to linux.git main branch arch/powerpc/sysdev/Makefile | 2 +-

[PATCH v7 08/23] powerpc/8xx: Map IMMR area with 512k page at a fixed address

2016-02-09 Thread Christophe Leroy
Once the linear memory space has been mapped with 8Mb pages, as seen in the related commit, we get 11 millions DTLB missed during the reference 600s period. 77% of the misses are on user addresses and 23% are on kernel addresses (1 fourth for linear address space and 3 fourth for virtual address

[PATCH v7 21/23] powerpc: Simplify test in __dma_sync()

2016-02-09 Thread Christophe Leroy
This simplification helps the compiler. We now have only one test instead of two, so it reduces the number of branches. Signed-off-by: Christophe Leroy --- v2: new v3: no change v4: no change v5: no change v6: no change v7: no change arch/powerpc/mm/dma-noncoherent.c |

Re: [PATCH v3 0/2] Consolidate redundant register/stack access code

2016-02-09 Thread Michael Ellerman
On Tue, 2016-02-09 at 00:38 -0500, David Long wrote: > From: "David A. Long" > > Move duplicate and functionally equivalent code for accessing registers > and stack (CONFIG_HAVE_REGS_AND_STACK_ACCESS_API) from arch subdirs into > common kernel files. > > I'm sending this

[PATCH v7 05/23] powerpc32: Fix pte_offset_kernel() to return NULL for bad pages

2016-02-09 Thread Christophe Leroy
The fixmap related functions try to map kernel pages that are already mapped through Large TLBs. pte_offset_kernel() has to return NULL for LTLBs, otherwise the caller will try to access level 2 table which doesn't exist Signed-off-by: Christophe Leroy --- v3: new v4: no

[PATCH v7 13/23] powerpc/8xx: Add missing SPRN defines into reg_8xx.h

2016-02-09 Thread Christophe Leroy
Add missing SPRN defines into reg_8xx.h Some of them are defined in mmu-8xx.h, so we include mmu-8xx.h in reg_8xx.h, for that we remove references to PAGE_SHIFT in mmu-8xx.h to have it self sufficient, as includers of reg_8xx.h don't all include asm/page.h Signed-off-by: Christophe Leroy

[PATCH v7 19/23] powerpc32: Remove clear_pages() and define clear_page() inline

2016-02-09 Thread Christophe Leroy
clear_pages() is never used expect by clear_page, and PPC32 is the only architecture (still) having this function. Neither PPC64 nor any other architecture has it. This patch removes clear_pages() and moves clear_page() function inline (same as PPC64) as it only is a few isns Signed-off-by:

Re: [PATCH v3 0/2] Consolidate redundant register/stack access code

2016-02-09 Thread Ingo Molnar
* Michael Ellerman wrote: > On Tue, 2016-02-09 at 00:38 -0500, David Long wrote: > > > From: "David A. Long" > > > > Move duplicate and functionally equivalent code for accessing registers > > and stack (CONFIG_HAVE_REGS_AND_STACK_ACCESS_API) from

[PATCH v7 00/23] powerpc/8xx: Use large pages for RAM and IMMR and other improvments

2016-02-09 Thread Christophe Leroy
The main purpose of this patchset is to dramatically reduce the time spent in DTLB miss handler. This is achieved by: 1/ Mapping RAM with 8M pages 2/ Mapping IMMR with a fixed 512K page On a live running system (VoIP gateway for Air Trafic Control), over a 10 minutes period (with 277s idle), we

[PATCH v7 07/23] powerpc/8xx: Fix vaddr for IMMR early remap

2016-02-09 Thread Christophe Leroy
Memory: 124428K/131072K available (3748K kernel code, 188K rwdata, 648K rodata, 508K init, 290K bss, 6644K reserved) Kernel virtual memory layout: * 0xfffdf000..0xf000 : fixmap * 0xfde0..0xfe00 : consistent mem * 0xfddf6000..0xfde0 : early ioremap *

[PATCH v7 06/23] powerpc32: refactor x_mapped_by_bats() and x_mapped_by_tlbcam() together

2016-02-09 Thread Christophe Leroy
x_mapped_by_bats() and x_mapped_by_tlbcam() serve the same kind of purpose, and are never defined at the same time. So rename them x_block_mapped() and define them in the relevant places Signed-off-by: Christophe Leroy --- v2: no change v3: Functions are mutually

[PATCH v7 15/23] powerpc/8xx: remove special handling of CPU6 errata in set_dec()

2016-02-09 Thread Christophe Leroy
CPU6 ERRATA is now handled directly in mtspr(), so we can use the standard set_dec() fonction in all cases. Signed-off-by: Christophe Leroy --- v2: no change v3: no change v4: no change v5: no change v6: no change v7: no change arch/powerpc/include/asm/time.h | 6

[PATCH v7 16/23] powerpc/8xx: rewrite set_context() in C

2016-02-09 Thread Christophe Leroy
There is no real need to have set_context() in assembly. Now that we have mtspr() handling CPU6 ERRATA directly, we can rewrite set_context() in C language for easier maintenance. Signed-off-by: Christophe Leroy --- v2: no change v3: no change v4: no change v5: no change

Re: [PATCH v3 2/2] powerpc: tracing: don't trace hcalls on offline CPUs

2016-02-09 Thread Michael Ellerman
On Fri, 2016-02-05 at 09:36 -0500, Steven Rostedt wrote: > On Fri, 5 Feb 2016 14:20:17 +0300 > Denis Kirjanov wrote: > > > > > Signed-off-by: Denis Kirjanov > > > > > > Hi Steven, > > > > > > please apply with Michael's acked-by tag. > > > > ping >

Re: powerpc/perf/hv-gpci: Increase request buffer size

2016-02-09 Thread Michael Ellerman
On Tue, 2016-09-02 at 03:08:30 UTC, Sukadev Bhattiprolu wrote: > >From 31edd352fb7c2a72913f1977fa1bf168109089ad Mon Sep 17 00:00:00 2001 > From: Sukadev Bhattiprolu > Date: Tue, 9 Feb 2016 02:47:45 -0500 > Subject: [PATCH] powerpc/perf/hv-gpci: Increase request buffer

[PATCH v7 01/23] powerpc/8xx: Save r3 all the time in DTLB miss handler

2016-02-09 Thread Christophe Leroy
We are spending between 40 and 160 cycles with a mean of 65 cycles in the DTLB handling routine (measured with mftbl) so make it more simple althought it adds one instruction. With this modification, we get three registers available at all time, which will help with following patch.

[PATCH v7 09/23] powerpc/8xx: CONFIG_PIN_TLB unneeded for CONFIG_PPC_EARLY_DEBUG_CPM

2016-02-09 Thread Christophe Leroy
IMMR is now mapped by page tables so it is not anymore necessary to PIN TLBs Signed-off-by: Christophe Leroy --- v2: no change v3: no change v4: no change v5: no change v6: no change v7: no change arch/powerpc/Kconfig.debug | 1 - 1 file changed, 1 deletion(-) diff

Re: powerpc/xmon: Add xmon command to dump process/task similar to ps(1)

2016-02-09 Thread Michael Ellerman
On Mon, 2015-23-11 at 15:01:15 UTC, Douglas Miller wrote: > Add 'P' command with optional task_struct address to dump all/one task's > information: task pointer, kernel stack pointer, PID, PPID, state > (interpreted), CPU where (last) running, and command. > > Introduce XMON_PROTECT macro to

Re: [PATCH v7 04/10] ppc64 ftrace_with_regs configuration variables

2016-02-09 Thread Torsten Duwe
On Mon, Feb 08, 2016 at 10:49:28AM -0500, Steven Rostedt wrote: > On Mon, 8 Feb 2016 16:23:06 +0100 > Petr Mladek wrote: > > > >From 2b0fcb678d7720d03f9c9f233b61ed9ed4d420b3 Mon Sep 17 00:00:00 2001 > > From: Petr Mladek > > Date: Mon, 8 Feb 2016 16:03:03

[PATCH v7 04/23] powerpc/8xx: move setup_initial_memory_limit() into 8xx_mmu.c

2016-02-09 Thread Christophe Leroy
Now we have a 8xx specific .c file for that so put it in there as other powerpc variants do Signed-off-by: Christophe Leroy --- v2: no change v3: no change v4: no change v5: no change v6: no change v7: no change arch/powerpc/mm/8xx_mmu.c | 17 +

[PATCH v7 14/23] powerpc/8xx: Handle CPU6 ERRATA directly in mtspr() macro

2016-02-09 Thread Christophe Leroy
MPC8xx has an ERRATA on the use of mtspr() for some registers This patch includes the ERRATA handling directly into mtspr() macro so that mtspr() users don't need to bother about that errata Signed-off-by: Christophe Leroy --- v2: no change v3: no change v4: no change

[PATCH v7 22/23] powerpc32: small optimisation in flush_icache_range()

2016-02-09 Thread Christophe Leroy
Inlining of _dcache_range() functions has shown that the compiler does the same thing a bit better with one insn less Signed-off-by: Christophe Leroy --- v2: new v3: no change v4: no change v5: no change v6: no change v7: no change arch/powerpc/kernel/misc_32.S | 5

Re: Fix kgdb on little endian ppc64le

2016-02-09 Thread Michael Ellerman
On Mon, 2016-01-02 at 06:03:25 UTC, Balbir Singh wrote: > From: Balbir Singh > > I spent some time trying to use kgdb and debugged my inability to > resume from kgdb_handle_breakpoint(). NIP is not incremented > and that leads to a loop in the debugger. > > I've tested this

Re: [1/2] powerpc/powernv: new function to access OPAL msglog

2016-02-09 Thread Michael Ellerman
On Tue, 2016-02-09 at 16:29 +1100, Andrew Donnellan wrote: > On 08/02/16 22:31, Michael Ellerman wrote: > > Pulling the memcons out of the bin_attr here is not all that nice. This > > routine > > should really stand on its own without reference to the bin_attr. In theory > > I > > might want to

[PATCH v7 12/23] powerpc32: remove ioremap_base

2016-02-09 Thread Christophe Leroy
ioremap_base is not initialised and is nowhere used so remove it Signed-off-by: Christophe Leroy --- v2: no change v3: fix comment as well v4: no change v5: no change v6: no change v7: no change arch/powerpc/include/asm/nohash/32/pgtable.h | 2 +-

[PATCH v7 11/23] powerpc32: Remove useless/wrong MMU:setio progress message

2016-02-09 Thread Christophe Leroy
Commit 771168494719 ("[POWERPC] Remove unused machine call outs") removed the call to setup_io_mappings(), so remove the associated progress line message Signed-off-by: Christophe Leroy --- v2: no change v3: no change v4: no change v5: no change v6: no change v7: no

[PATCH v7 18/23] powerpc: add inline functions for cache related instructions

2016-02-09 Thread Christophe Leroy
This patch adds inline functions to use dcbz, dcbi, dcbf, dcbst from C functions Signed-off-by: Christophe Leroy --- v2: new v3: no change v4: no change v5: no change v6: no change v7: no change arch/powerpc/include/asm/cache.h | 19 +++ 1 file changed,

[PATCH v7 20/23] powerpc32: move xxxxx_dcache_range() functions inline

2016-02-09 Thread Christophe Leroy
flush/clean/invalidate _dcache_range() functions are all very similar and are quite short. They are mainly used in __dma_sync() perf_event locate them in the top 3 consumming functions during heavy ethernet activity They are good candidate for inlining, as __dma_sync() does almost nothing but

[PATCH v7 23/23] powerpc32: Remove one insn in mulhdu

2016-02-09 Thread Christophe Leroy
Remove one instruction in mulhdu Signed-off-by: Christophe Leroy --- v2: new v3: no change v4: no change v5: no change v6: no change v7: no change arch/powerpc/kernel/misc_32.S | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff --git

Re: [RFC PATCH kernel] powerpc/ioda: Set "read" permission when "write" is set

2016-02-09 Thread Douglas Miller
We finally got the chance to test it end of last week. I forgot to update everyone Monday. B all appearances, the patch fixes the problem. We did not see any new issues with the patch (vs. same test scenarios without). I'll also update the bugzilla. Thanks, Doug On 02/08/2016 07:37 PM,

RE: PowerPC agpmode issues

2016-02-09 Thread luigi burdo
Mike and Gerhard, dont think the situation of the pcie powerpc is bettrer.but compared with last years with the new kernels and last xorg on a radeonhd 4650 i have an increase of performance about 250x ... example QuakeSpasm was gaving 640x480157fps on Radeon 4650... Now is 380 fps yes compared

[powerpc:topic/math-emu 6/9] arch/sh/math-emu/math.c:129:1: warning: ISO C90 forbids mixed declarations and code

2016-02-09 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git topic/math-emu head: 0d351023a638b7c82abdd8d66ebf5b5b3d6cb169 commit: 3b76bfd2f01f3b9101f040878e2636cdb7f3bbdd [6/9] sh/math-emu: Move sh from math-emu-old to math-emu config: sh-allyesconfig (attached as .config)

enable kdump capture kernel functionality

2016-02-09 Thread Cosmin Banu
Hi, I am trying to rebuild the kernel for one of our powerpc devices to include kdump capture kernel functionality, but I'm having trouble getting it to load for use on panic. I'm running Linux 3.14.60 on e500v2 (COMX-P2020 module). I've tried following the documentation at

Re: [PATCH v3 13/18] cxl: sysfs support for guests

2016-02-09 Thread Frederic Barrat
Le 08/02/2016 04:02, Stewart Smith a écrit : Frederic Barrat writes: --- a/Documentation/ABI/testing/sysfs-class-cxl +++ b/Documentation/ABI/testing/sysfs-class-cxl @@ -183,7 +183,7 @@ Description:read only Identifies the revision level of

Re: powerpc/xmon: Add xmon command to dump process/task similar to ps(1)

2016-02-09 Thread Douglas Miller
That looks fine to me. Thanks! On 02/09/2016 04:58 AM, Michael Ellerman wrote: On Mon, 2015-23-11 at 15:01:15 UTC, Douglas Miller wrote: Add 'P' command with optional task_struct address to dump all/one task's information: task pointer, kernel stack pointer, PID, PPID, state (interpreted),

Re: [PATCH V2 00/29] Book3s abstraction in preparation for new MMU model

2016-02-09 Thread Aneesh Kumar K.V
Hi Scott, I missed adding you on CC:, Can you take a look at this and make sure we are not breaking anything on freescale. "Aneesh Kumar K.V" writes: > Hello, > > This is a large series, mostly consisting of code movement. No new features > are done in this

Re: PowerPC agpmode issues

2016-02-09 Thread Gerhard Pircher
> On 9 Feb 2016 03:27, "Mike" wrote: > Ok, so its quirks to be added then? Something not implemented in KMS > that was in UMS? > Reports are that the same issue exsist on PPC Amiga Ones with a VIA > chipset, and the Pegasos 2 with the Artica s chipset, i posted a > mail

Re: [V3] powerpc/mm: Fix Multi hit ERAT cause by recent THP update

2016-02-09 Thread Michael Ellerman
On Tue, 2016-09-02 at 01:20:31 UTC, "Aneesh Kumar K.V" wrote: > With ppc64 we use the deposited pgtable_t to store the hash pte slot > information. We should not withdraw the deposited pgtable_t without > marking the pmd none. This ensure that low level hash fault handling > will skip this huge

Re: [v3,2/2] powerpc: tracing: don't trace hcalls on offline CPUs

2016-02-09 Thread Michael Ellerman
On Mon, 2015-14-12 at 20:18:06 UTC, Denis Kirjanov wrote: > ./drmgr -c cpu -a -r gives the following warning: > > ... Applied to powerpc fixes, thanks. https://git.kernel.org/powerpc/c/168a20bb35122539682671d15c cheers ___ Linuxppc-dev mailing list

Re: powerpc: fix dedotify for binutils >= 2.26

2016-02-09 Thread Michael Ellerman
On Fri, 2016-05-02 at 18:50:03 UTC, Andreas Schwab wrote: > Since binutils 2.26 BFD is doing suffix merging on STRTAB sections. But > dedotify modifies the symbol names in place, which can also modify > unrelated symbols with a name that matches a suffix of a dotted name. To > remove the leading

Re: [PATCH v7 00/23] powerpc/8xx: Use large pages for RAM and IMMR and other improvments

2016-02-09 Thread Christophe Leroy
Le 09/02/2016 11:23, Christophe Leroy a écrit : The main purpose of this patchset is to dramatically reduce the time spent in DTLB miss handler. This is achieved by: 1/ Mapping RAM with 8M pages 2/ Mapping IMMR with a fixed 512K page Change in v7: * Don't include x_block_mapped() from

[PATCH v8 02/23] powerpc/8xx: Map linear kernel RAM with 8M pages

2016-02-09 Thread Christophe Leroy
On a live running system (VoIP gateway for Air Trafic Control), over a 10 minutes period (with 277s idle), we get 87 millions DTLB misses and approximatly 35 secondes are spent in DTLB handler. This represents 5.8% of the overall time and even 10.8% of the non-idle time. Among those 87 millions

[PATCH v8 08/23] powerpc/8xx: Map IMMR area with 512k page at a fixed address

2016-02-09 Thread Christophe Leroy
Once the linear memory space has been mapped with 8Mb pages, as seen in the related commit, we get 11 millions DTLB missed during the reference 600s period. 77% of the misses are on user addresses and 23% are on kernel addresses (1 fourth for linear address space and 3 fourth for virtual address

[PATCH v8 11/23] powerpc32: Remove useless/wrong MMU:setio progress message

2016-02-09 Thread Christophe Leroy
Commit 771168494719 ("[POWERPC] Remove unused machine call outs") removed the call to setup_io_mappings(), so remove the associated progress line message Signed-off-by: Christophe Leroy --- v2: no change v3: no change v4: no change v5: no change v6: no change v8: no

[PATCH v8 00/23] powerpc/8xx: Use large pages for RAM and IMMR and other improvments

2016-02-09 Thread Christophe Leroy
The main purpose of this patchset is to dramatically reduce the time spent in DTLB miss handler. This is achieved by: 1/ Mapping RAM with 8M pages 2/ Mapping IMMR with a fixed 512K page On a live running system (VoIP gateway for Air Trafic Control), over a 10 minutes period (with 277s idle), we

[PATCH v8 01/23] powerpc/8xx: Save r3 all the time in DTLB miss handler

2016-02-09 Thread Christophe Leroy
We are spending between 40 and 160 cycles with a mean of 65 cycles in the DTLB handling routine (measured with mftbl) so make it more simple althought it adds one instruction. With this modification, we get three registers available at all time, which will help with following patch.

[PATCH v8 03/23] powerpc: Update documentation for noltlbs kernel parameter

2016-02-09 Thread Christophe Leroy
Now the noltlbs kernel parameter is also applicable to PPC8xx Signed-off-by: Christophe Leroy --- v2: no change v3: no change v4: no change v5: no change v6: no change v8: no change Documentation/kernel-parameters.txt | 2 +- 1 file changed, 1 insertion(+), 1

[PATCH v8 04/23] powerpc/8xx: move setup_initial_memory_limit() into 8xx_mmu.c

2016-02-09 Thread Christophe Leroy
Now we have a 8xx specific .c file for that so put it in there as other powerpc variants do Signed-off-by: Christophe Leroy --- v2: no change v3: no change v4: no change v5: no change v6: no change v8: no change arch/powerpc/mm/8xx_mmu.c | 17 +

[PATCH v8 05/23] powerpc32: Fix pte_offset_kernel() to return NULL for bad pages

2016-02-09 Thread Christophe Leroy
The fixmap related functions try to map kernel pages that are already mapped through Large TLBs. pte_offset_kernel() has to return NULL for LTLBs, otherwise the caller will try to access level 2 table which doesn't exist Signed-off-by: Christophe Leroy --- v3: new v4: no

[PATCH v8 06/23] powerpc32: refactor x_mapped_by_bats() and x_mapped_by_tlbcam() together

2016-02-09 Thread Christophe Leroy
x_mapped_by_bats() and x_mapped_by_tlbcam() serve the same kind of purpose, and are never defined at the same time. So rename them x_block_mapped() and define them in the relevant places Signed-off-by: Christophe Leroy --- v2: no change v3: Functions are mutually

[PATCH v8 10/23] powerpc/8xx: map more RAM at startup when needed

2016-02-09 Thread Christophe Leroy
On recent kernels, with some debug options like for instance CONFIG_LOCKDEP, the BSS requires more than 8M memory, allthough the kernel code fits in the first 8M. Today, it is necessary to activate CONFIG_PIN_TLB to get more than 8M at startup, allthough pinning TLB is not necessary for that.

[PATCH v8 12/23] powerpc32: remove ioremap_base

2016-02-09 Thread Christophe Leroy
ioremap_base is not initialised and is nowhere used so remove it Signed-off-by: Christophe Leroy --- v2: no change v3: fix comment as well v4: no change v5: no change v6: no change v8: no change arch/powerpc/include/asm/nohash/32/pgtable.h | 2 +-

[PATCH v8 07/23] powerpc/8xx: Fix vaddr for IMMR early remap

2016-02-09 Thread Christophe Leroy
Memory: 124428K/131072K available (3748K kernel code, 188K rwdata, 648K rodata, 508K init, 290K bss, 6644K reserved) Kernel virtual memory layout: * 0xfffdf000..0xf000 : fixmap * 0xfde0..0xfe00 : consistent mem * 0xfddf6000..0xfde0 : early ioremap *

[PATCH v8 14/23] powerpc/8xx: Handle CPU6 ERRATA directly in mtspr() macro

2016-02-09 Thread Christophe Leroy
MPC8xx has an ERRATA on the use of mtspr() for some registers This patch includes the ERRATA handling directly into mtspr() macro so that mtspr() users don't need to bother about that errata Signed-off-by: Christophe Leroy --- v2: no change v3: no change v4: no change

[PATCH v8 09/23] powerpc/8xx: CONFIG_PIN_TLB unneeded for CONFIG_PPC_EARLY_DEBUG_CPM

2016-02-09 Thread Christophe Leroy
IMMR is now mapped by page tables so it is not anymore necessary to PIN TLBs Signed-off-by: Christophe Leroy --- v2: no change v3: no change v4: no change v5: no change v6: no change v8: no change arch/powerpc/Kconfig.debug | 1 - 1 file changed, 1 deletion(-) diff

[PATCH v8 18/23] powerpc: add inline functions for cache related instructions

2016-02-09 Thread Christophe Leroy
This patch adds inline functions to use dcbz, dcbi, dcbf, dcbst from C functions Signed-off-by: Christophe Leroy --- v2: new v3: no change v4: no change v5: no change v6: no change v8: no change arch/powerpc/include/asm/cache.h | 19 +++ 1 file changed,

[PATCH v8 20/23] powerpc32: move xxxxx_dcache_range() functions inline

2016-02-09 Thread Christophe Leroy
flush/clean/invalidate _dcache_range() functions are all very similar and are quite short. They are mainly used in __dma_sync() perf_event locate them in the top 3 consumming functions during heavy ethernet activity They are good candidate for inlining, as __dma_sync() does almost nothing but

[PATCH v8 13/23] powerpc/8xx: Add missing SPRN defines into reg_8xx.h

2016-02-09 Thread Christophe Leroy
Add missing SPRN defines into reg_8xx.h Some of them are defined in mmu-8xx.h, so we include mmu-8xx.h in reg_8xx.h, for that we remove references to PAGE_SHIFT in mmu-8xx.h to have it self sufficient, as includers of reg_8xx.h don't all include asm/page.h Signed-off-by: Christophe Leroy

[PATCH v8 15/23] powerpc/8xx: remove special handling of CPU6 errata in set_dec()

2016-02-09 Thread Christophe Leroy
CPU6 ERRATA is now handled directly in mtspr(), so we can use the standard set_dec() fonction in all cases. Signed-off-by: Christophe Leroy --- v2: no change v3: no change v4: no change v5: no change v6: no change v8: no change arch/powerpc/include/asm/time.h | 6

[PATCH v8 16/23] powerpc/8xx: rewrite set_context() in C

2016-02-09 Thread Christophe Leroy
There is no real need to have set_context() in assembly. Now that we have mtspr() handling CPU6 ERRATA directly, we can rewrite set_context() in C language for easier maintenance. Signed-off-by: Christophe Leroy --- v2: no change v3: no change v4: no change v5: no change

[PATCH v8 17/23] powerpc/8xx: rewrite flush_instruction_cache() in C

2016-02-09 Thread Christophe Leroy
On PPC8xx, flushing instruction cache is performed by writing in register SPRN_IC_CST. This registers suffers CPU6 ERRATA. The patch rewrites the fonction in C so that CPU6 ERRATA will be handled transparently Signed-off-by: Christophe Leroy --- v2: no change v3: no

[PATCH v8 19/23] powerpc32: Remove clear_pages() and define clear_page() inline

2016-02-09 Thread Christophe Leroy
clear_pages() is never used expect by clear_page, and PPC32 is the only architecture (still) having this function. Neither PPC64 nor any other architecture has it. This patch removes clear_pages() and moves clear_page() function inline (same as PPC64) as it only is a few isns Signed-off-by:

[PATCH v8 21/23] powerpc: Simplify test in __dma_sync()

2016-02-09 Thread Christophe Leroy
This simplification helps the compiler. We now have only one test instead of two, so it reduces the number of branches. Signed-off-by: Christophe Leroy --- v2: new v3: no change v4: no change v5: no change v6: no change v8: no change arch/powerpc/mm/dma-noncoherent.c |

[PATCH v8 23/23] powerpc32: Remove one insn in mulhdu

2016-02-09 Thread Christophe Leroy
Remove one instruction in mulhdu Signed-off-by: Christophe Leroy --- v2: new v3: no change v4: no change v5: no change v6: no change v8: no change arch/powerpc/kernel/misc_32.S | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff --git

[PATCH v8 22/23] powerpc32: small optimisation in flush_icache_range()

2016-02-09 Thread Christophe Leroy
Inlining of _dcache_range() functions has shown that the compiler does the same thing a bit better with one insn less Signed-off-by: Christophe Leroy --- v2: new v3: no change v4: no change v5: no change v6: no change v8: no change arch/powerpc/kernel/misc_32.S | 5

Re: [PATCH v3 0/2] Consolidate redundant register/stack access code

2016-02-09 Thread David Long
On 02/09/2016 04:36 AM, Michael Ellerman wrote: On Tue, 2016-02-09 at 00:38 -0500, David Long wrote: From: "David A. Long" Move duplicate and functionally equivalent code for accessing registers and stack (CONFIG_HAVE_REGS_AND_STACK_ACCESS_API) from arch subdirs into

Re: [PATCH v3 0/2] Consolidate redundant register/stack access code

2016-02-09 Thread David Long
On 02/09/2016 04:45 AM, Ingo Molnar wrote: * Michael Ellerman wrote: On Tue, 2016-02-09 at 00:38 -0500, David Long wrote: From: "David A. Long" Move duplicate and functionally equivalent code for accessing registers and stack

Re: build regression from c153693: Simplify module TOC handling

2016-02-09 Thread Dinar Valeev
On Tue, Feb 9, 2016 at 5:28 PM, Peter Robinson wrote: > Hi Alan, > > Your patch for "powerpc: Simplify module TOC handling" is causing the > Fedora ppc64le to fail to build with depmod failures. Reverting the > commit fixes it for us on rawhide. Anton's patch [1] fixes it.

Re: [PATCH 2/6] ibmvscsi: Add and use enums for valid CRQ header values

2016-02-09 Thread Tyrel Datwyler
On 02/09/2016 09:41 AM, Manoj Kumar wrote: >> Yeah, I can see how that is confusing. Since, all three possible valid >> crq message types have the first bit set I think this was originally a >> cute hack to grab anything that was likely valid. Then in >> ibmvscsi_handle_crq() we explicitly match

Re: build regression from c153693: Simplify module TOC handling

2016-02-09 Thread Michael Ellerman
On Tue, 2016-02-09 at 22:02 +0100, Dinar Valeev wrote: > On Tue, Feb 9, 2016 at 5:28 PM, Peter Robinson wrote: > > Hi Alan, > > > > Your patch for "powerpc: Simplify module TOC handling" is causing the > > Fedora ppc64le to fail to build with depmod failures. Reverting the

Re: [PATCH v7 07/10] ppc64 ftrace: disable profiling for some files

2016-02-09 Thread Michael Ellerman
On Mon, 2016-01-25 at 16:31 +0100, Torsten Duwe wrote: > This patch complements the "notrace" attribute for selected functions. > It adds -mprofile-kernel to the cc flags to be stripped from the command > line for code-patching.o and feature-fixups.o, in addition to "-pg" This could probably be

Re: [RFC PATCH kernel] powerpc/ioda: Set "read" permission when "write" is set

2016-02-09 Thread Alexey Kardashevskiy
On 02/10/2016 01:28 AM, Douglas Miller wrote: We finally got the chance to test it end of last week. I forgot to update everyone Monday. B all appearances, the patch fixes the problem. We did not see any new issues with the patch (vs. same test scenarios without). I'll also update the bugzilla.

Re: powerpc/perf/hv-gpci: Increase request buffer size

2016-02-09 Thread Sukadev Bhattiprolu
Michael Ellerman [m...@ellerman.id.au] wrote: > Here you read from bytes[i] where i can be > 1 (AFAICS). Yes, buffer is large enough and I thought this construct of array was used in a several places. Maybe they are being changed out now (struct pid has one such usage). > > That's fishy at

[PATCH V2 1/1] powerpc/perf/hv-gpci: Increase request buffer size

2016-02-09 Thread Sukadev Bhattiprolu
From f1afe08fbc9797ff63adf03efe564a807a37cfe6 Mon Sep 17 00:00:00 2001 From: Sukadev Bhattiprolu Date: Tue, 9 Feb 2016 02:47:45 -0500 Subject: [PATCH V2 1/1] powerpc/perf/hv-gpci: Increase request buffer size The GPCI hcall allows for a 4K buffer but we limit the

Re: [PATCH v7 06/10] ppc64 ftrace: disable profiling for some functions

2016-02-09 Thread Michael Ellerman
On Mon, 2016-01-25 at 16:31 +0100, Torsten Duwe wrote: > At least POWER7/8 have MMUs that don't completely autoload; > a normal, recoverable memory fault might pass through these functions. > If a dynamic tracer function causes such a fault, any of these functions > being traced with

[PATCH v14 5/9] powerpc/eeh: EEH device for VF

2016-02-09 Thread Gavin Shan
From: Wei Yang VFs and their corresponding pdn are created and released dynamically when their PF's SRIOV capability is enabled and disabled. This creates and releases EEH devices for VFs when creating and releasing their pdn instances, which means EEH devices and pdn

[PATCH v14 0/9] EEH Support for SRIOV VFs

2016-02-09 Thread Gavin Shan
This applies to linux-powerpc-next and additional unmerged patches: [v2,1/4] powerpc/eeh: Fix stale cached primary bus powerpc/eeh: fix incorrect function name in comment [V2] powerpc/powernv: Remove support for p5ioc2 [V7,1/6] powerpc/powernv: don't enable SRIOV when VF BAR has non

[PATCH v14 3/9] powerpc/pci: Remove VFs prior to PF

2016-02-09 Thread Gavin Shan
From: Wei Yang As commit ac205b7bb72f ("PCI: make sriov work with hotplug remove") indicates, VFs which is on the same PCI bus as their PF, should be removed before the PF. Otherwise, we might run into kernel crash at PCI unplugging time. This applies the above

Re: [PATCH v3 13/18] cxl: sysfs support for guests

2016-02-09 Thread Ian Munsie
Excerpts from Frederic Barrat's message of 2016-02-10 02:21:19 +1100: > > Le 08/02/2016 04:02, Stewart Smith a écrit : > > Frederic Barrat writes: > >> --- a/Documentation/ABI/testing/sysfs-class-cxl > >> +++ b/Documentation/ABI/testing/sysfs-class-cxl > >> @@ -183,7

Re: [PATCH v3 05/18] cxl: Rename some bare-metal specific functions

2016-02-09 Thread Ian Munsie
Excerpts from Frederic Barrat's message of 2016-02-07 00:28:52 +1100: > Rename a few functions, mostly prefixed by 'cxl_', to make clear that > the implementation is 'bare-metal' specific. Patch looks fine to me, though the commit message should probably say that you are changing the 'cxl_'

Re: [PATCH v3 07/18] cxl: Update cxl_irq() prototype

2016-02-09 Thread Ian Munsie
> The context parameter when calling cxl_irq() should be strongly typed. Fair enough ;-) Acked-by: Ian Munsie ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev

[PATCH v14 8/9] powerpc/powernv: Support PCI config restore for VFs

2016-02-09 Thread Gavin Shan
From: Wei Yang After PE reset, OPAL API opal_pci_reinit() is called on all devices contained in the PE to reinitialize them. While skiboot is not aware of VFs, we have to implement the function in kernel to reinitialize VFs after reset on PE for VFs. In this patch,

[PATCH v14 7/9] powerpc/powernv: Support EEH reset for VF PE

2016-02-09 Thread Gavin Shan
From: Wei Yang PEs for VFs don't have primary bus. So they have to have their own reset backend, which is used during EEH recovery. The patch implements the reset backend for VF's PE by issuing FLR or AF FLR to the VFs, which are contained in the PE. Signed-off-by:

[PATCH v14 6/9] powerpc/eeh: Create PE for VFs

2016-02-09 Thread Gavin Shan
From: Wei Yang This creates PEs for VFs in the weak function pcibios_bus_add_device(). Those PEs for VFs are identified with newly introduced flag EEH_PE_VF so that we treat them differently during EEH recovery. Signed-off-by: Wei Yang

Re: [PATCH v3 01/18] cxl: Move common code away from bare-metal-specific files

2016-02-09 Thread Ian Munsie
Acked-by: Ian Munsie ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev

Re: [PATCH v3 02/18] cxl: Move bare-metal specific code to specialized files

2016-02-09 Thread Ian Munsie
Acked-by: Ian Munsie ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev

Re: [PATCH v3 04/18] cxl: Introduce implementation-specific API

2016-02-09 Thread Ian Munsie
Acked-by: Ian Munsie ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev

[PATCH] powerpc32: PAGE_EXEC required for inittext

2016-02-09 Thread Christophe Leroy
PAGE_EXEC is required for inittext, otherwise CONFIG_DEBUG_PAGEALLOC ends up with an Oops [0.00] Inode-cache hash table entries: 8192 (order: 1, 32768 bytes) [0.00] Sorting __ex_table... [0.00] bootmem::free_all_bootmem_core nid=0 start=0 end=2000 [0.00] Unable to

Re: [PATCH v3 08/18] cxl: IRQ allocation for guests

2016-02-09 Thread Ian Munsie
Acked-by: Ian Munsie > +/* > + * Look for the interrupt number. > + * On bare-metal, we know the range 0 only contains the PSL > + * interrupt so, we could start counting at range 1 and initialize > + * afu_irq at 1. > + * In a guest, range 0 also

[PATCH v14 1/9] PCI/IOV: Rename and export virtfn_{add, remove}

2016-02-09 Thread Gavin Shan
From: Wei Yang During EEH recovery, hotplug is applied to the devices which don't have drivers or their drivers don't support EEH. However, the hotplug, which was implemented based on PCI bus, can't be applied to VF directly. Instead, we unplug and plug individual PCI

[PATCH v14 2/9] PCI: Add pcibios_bus_add_device() weak function

2016-02-09 Thread Gavin Shan
From: Wei Yang This adds weak function pcibios_bus_add_device() for arch dependent code could do proper setup. For example, powerpc could setup EEH related resources for SRIOV VFs. Signed-off-by: Wei Yang Reviewed-by: Gavin Shan

[PATCH v14 4/9] powerpc/eeh: Cache normal BARs, not windows or IOV BARs

2016-02-09 Thread Gavin Shan
From: Wei Yang This restricts the EEH address cache to use only the first 7 BARs. This makes __eeh_addr_cache_insert_dev() ignore PCI bridge window and IOV BARs. As the result of this change, eeh_addr_cache_get_dev() will return VFs from VF's resource addresses

[PATCH v14 9/9] powerpc/eeh: powerpc/eeh: Support error recovery for VF PE

2016-02-09 Thread Gavin Shan
From: Wei Yang PFs are enumerated on PCI bus, while VFs are created by PF's driver. In EEH recovery, it has two cases: 1. Device and driver is EEH aware, error handlers are called. 2. Device and driver is not EEH aware, un-plug the device and plug it again by

Re: [PATCH v3 03/18] cxl: Define process problem state area at attach time only

2016-02-09 Thread Ian Munsie
Excerpts from Frederic Barrat's message of 2016-02-07 00:28:50 +1100: > Cxl kernel API was defining the process problem state area during > context initialization, making it possible to map the problem state > area before attaching the context. This won't work on a powerVM > guest. So do the

Re: [PATCH v3 06/18] cxl: Isolate a few bare-metal-specific calls

2016-02-09 Thread Ian Munsie
Acked-by: Ian Munsie ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev

  1   2   >