Re: [PATCH v2 05/18] powerpc/85xx: Load all early TLB entries at once

On Wed, 2015-10-07 at 17:00 +0300, Laurentiu Tudor wrote: > On 10/07/2015 06:48 AM, Scott Wood wrote: > > Use an AS=1 trampoline TLB entry to allow all normal TLB1 entries to > > be loaded at once. This avoids the need to keep the translation that > > code is executing from in the same TLB entry

Re: [PATCH 1/5] powerpc/eeh: Don't unfreeze PHB PE after reset

On Wed, Oct 07, 2015 at 02:12:05PM +1100, Gavin Shan wrote: >On PowerNV platform, the PE is kept in frozen state until the PE >reset is completed to avoid recursive EEH error caused by MMIO >access during the period of EEH reset. The PE's frozen state is >cleared after BARs of PCI device included

Re: Missing operand for tlbie instruction on Power7

On Wed, Oct 7, 2015 at 5:13 AM, Michael Ellerman wrote: > On Wed, 2015-10-07 at 02:19 -0500, Segher Boessenkool wrote: >> On Wed, Oct 07, 2015 at 05:00:49PM +1100, Michael Ellerman wrote: >> > > It's also worth noting that the __flush_power7 uses tlbiel instead of >> > >

linux-next: build failure after merge of Linus' tree

Hi Linus, After merging Linus' tree, today's linux-next build (powerpc ppc64_defconfig) failed like this: lib/string.c: In function 'strscpy': lib/string.c:209:4: error: implicit declaration of function 'zero_bytemask' [-Werror=implicit-function-declaration] *(unsigned long *)(dest+res) = c

[PATCH FIX] powerpc: discard .exit.data at runtime

.exit.text is discarded at run time and there are some references from that to .exit.data, so we need to discard .exit.data at run time as well. Fixes these errors: `.exit.data' referenced in section `.exit.text' of drivers/built-in.o: defined in discarded section `.exit.data' of

Re: [PATCH v3 3/3] dmaengine: mpc512x: initialize with subsys_initcall()

On 07.10.2015 17:17, Vinod Koul wrote: > On Thu, Sep 24, 2015 at 08:28:57PM +0300, Alexander Popov wrote: >> Initialize Freescale MPC512x DMA driver with subsys_initcall() >> to allow the depending drivers to call dma_request_slave_channel() >> during their probe. > > Why can't we use defered

Re: powerpc: Fix _ALIGN_* errors due to type difference.

Michael Ellerman writes: > On Fri, 2015-02-10 at 14:33:48 UTC, "Aneesh Kumar K.V" wrote: >> This avoid errors like >> >> unsigned int usize = 1 << 30; >> int size = 1 << 30; >> unsigned long addr = 64UL << 30 ; >> >> value =

[PATCH v2] powerpc: Fix checkstop in native_hpte_clear() with lockdep

native_hpte_clear() is called in real mode from two places: - Early in boot during htab initialisation if firmware assisted dump is active. - Late in the kexec path. In both contexts there is no need to disable interrupts are they are already disabled. Furthermore, locking around the tlbie() is

Re: [GIT PULL 0/1] perf/urgent fix

On Wed, 2015-10-07 at 18:33 +0200, Ingo Molnar wrote: > * Arnaldo Carvalho de Melo wrote: > > > > perf/urgent fix: > > > > - Fix build break on (at least) powerpc due to sample_reg_masks, not being > > available

RE: [PATCH v3 1/2] fsl: Add binding for RCPM

> > +++ b/Documentation/devicetree/bindings/soc/fsl/rcpm.txt > > @@ -0,0 +1,63 @@ > > +* Run Control and Power Management > > +--- > > +The RCPM performs all device-level tasks associated with device run > > +control and power management. > > + > > +Required

[PATCH v2 1/8] powerpc/eeh: Don't unfreeze PHB PE after reset

On PowerNV platform, the PE is kept in frozen state until the PE reset is completed to avoid recursive EEH error caused by MMIO access during the period of EEH reset. The PE's frozen state is cleared after BARs of PCI device included in the PE are restored and enabled. However, we needn't clear

[PATCH v2 2/8] powerpc/eeh: More relexed hotplug criterion

Currently, we rely on the existence of struct pci_driver::err_handler to judge if the corresponding PCI device should be unplugged during EEH recovery (partially hotplug case). However, it's not elaborate. some device drivers are implementing part of the EEH error handlers to collect diag-data.

[PATCH v2 3/8] powerpc/eeh: Force reset on fenced PHB

On fenced PHB, the error handlers in the drivers of its subordinate devices could return PCI_ERS_RESULT_CAN_RECOVER, indicating no reset will be issued during the recovery. It's conflicting with the fact that fenced PHB won't be recovered without reset. This limits the return value from the error

[PATCH v2 4/8] powerpc/eeh: More relxed condition for enabled IO path

When one of below two flags or both of them are marked in the PE state, the PE's IO path is regarded as enabled: EEH_STATE_MMIO_ACTIVE or EEH_STATE_MMIO_ENABLED. Signed-off-by: Gavin Shan --- arch/powerpc/kernel/eeh.c | 2 +- 1 file changed, 1 insertion(+), 1

[PATCH v2 6/8] powerpc/powernv: Cleanup on EEH comments

This applies cleanup on eeh-powernv.c, no functional changes: * Remove unnecessary comments and empty line. * Correct inaccurate comments. Signed-off-by: Gavin Shan --- arch/powerpc/platforms/powernv/eeh-powernv.c | 13 ++--- 1 file changed, 2

Re: Missing operand for tlbie instruction on Power7

On Wed, Oct 7, 2015 at 8:10 PM, Michael Ellerman wrote: > On Wed, 2015-10-07 at 10:31 -0400, Josh Boyer wrote: >> On Wed, Oct 7, 2015 at 5:13 AM, Michael Ellerman wrote: >> > On Wed, 2015-10-07 at 02:19 -0500, Segher Boessenkool wrote: >> >> On Wed, Oct

[PATCH v2 7/8] powerpc/powernv: Remove pnv_eeh_cap_start()

This moves the logic of pnv_eeh_cap_start() to pnv_eeh_find_cap() as the function is only called by pnv_eeh_find_cap(). The logic of both functions are pretty simple. No need to have separate functions. Signed-off-by: Gavin Shan ---

[PATCH v2 8/8] powerpc/powernv: Simplify pnv_eeh_set_option()

This simplifies pnv_eeh_set_option() to avoid unnecessary nested if statements, to improve readability. No functional changes. Signed-off-by: Gavin Shan --- arch/powerpc/platforms/powernv/eeh-powernv.c | 51 ++-- 1 file changed, 25

[PATCH v2 0/8] EEH Improvement and Cleanup

The PATCH[1] of this series skips unfreezing PHB PE after reset during recovery because that's unnecessary. PATCH[2] makes the hotplug criterion more realistic by validating all required error handlers. PATCH[3] fixes the issue that reset isn't issued when the error handler (error_detected()) of

Re: [PATCH v2 8/8] powerpc/powernv: Simplify pnv_eeh_set_option()

On 08/10/15 14:58, Gavin Shan wrote: This simplifies pnv_eeh_set_option() to avoid unnecessary nested if statements, to improve readability. No functional changes. Signed-off-by: Gavin Shan Reviewed-by: Andrew Donnellan -- Andrew

Re: [kbuild-all] [PATCH 4/5 v2] pseries/iommu: implement DDW-aware dma_get_page_shift

On Wed, 2015-10-07 at 21:56 +0800, Fengguang Wu wrote: > On Tue, Oct 06, 2015 at 02:39:06PM +1100, Michael Ellerman wrote: > > On Sat, 2015-10-03 at 04:33 +0800, kbuild test robot wrote: > > > Hi Nishanth, > > > > > > [auto build test results on v4.3-rc3 -- if it's inappropriate base, > > >

[PATCH v3 0/2] powerpc/xmon: Paged output for paca display

Changes v2 -> v3: Moved the pagination implementation from xmon.c to nonstdio.c where it's much easier to do and the code is significantly simplified. As it's now trivial to do, add the capability to truncate the output or to stop pagination and dump the rest of the output. Changed function

[PATCH v3 2/2] powerpc/xmon: Paginate kernel log buffer display

The kernel log buffer is often much longer than the size of a terminal so paginate it's output. Signed-off-by: Sam Bobroff --- arch/powerpc/xmon/xmon.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/arch/powerpc/xmon/xmon.c

Re: [PATCH v2 22/25] powerpc32: move xxxxx_dcache_range() functions inline

Le 29/09/2015 02:29, Scott Wood a écrit : On Tue, Sep 22, 2015 at 06:51:13PM +0200, Christophe Leroy wrote: 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

Re: [PATCH v2] barriers: introduce smp_mb__release_acquire and update documentation

On Wed, 2015-10-07 at 08:25 -0700, Paul E. McKenney wrote: > On Wed, Oct 07, 2015 at 02:23:17PM +0100, Will Deacon wrote: > > Hi Peter, > > > > Thanks for the headache ;) > > > > On Wed, Oct 07, 2015 at 01:19:15PM +0200, Peter Zijlstra wrote: > > > On Wed, Oct 07, 2015 at 11:59:28AM +0100, Will

Re: [kbuild-all] [PATCH 4/5 v2] pseries/iommu: implement DDW-aware dma_get_page_shift

On Thu, 2015-10-08 at 09:16 +0800, Fengguang Wu wrote: > On Thu, Oct 08, 2015 at 09:06:03AM +0800, Fengguang Wu wrote: > > On Thu, Oct 08, 2015 at 11:11:59AM +1100, Michael Ellerman wrote: > > > On Wed, 2015-10-07 at 21:56 +0800, Fengguang Wu wrote: > > > > On Tue, Oct 06, 2015 at 02:39:06PM

Re: [PATCH 1/5] powerpc/eeh: Don't unfreeze PHB PE after reset

On Thu, Oct 08, 2015 at 10:20:14AM +1100, Gavin Shan wrote: >On Wed, Oct 07, 2015 at 02:12:05PM +1100, Gavin Shan wrote: >>On PowerNV platform, the PE is kept in frozen state until the PE >>reset is completed to avoid recursive EEH error caused by MMIO >>access during the period of EEH reset. The

Re: [v2,1/2] powerpc/xmon: Paged output for paca display

On Tue, Oct 06, 2015 at 10:05:38PM +1100, Michael Ellerman wrote: > On Fri, 2015-21-08 at 04:24:27 UTC, Sam bobroff wrote: > > The paca display is already more than 24 lines, which can be problematic > > if you have an old school 80x24 terminal, or more likely you are on a > > virtual terminal

Re: Missing operand for tlbie instruction on Power7

On Wed, 2015-10-07 at 20:15 -0400, Josh Boyer wrote: > On Wed, Oct 7, 2015 at 8:10 PM, Michael Ellerman wrote: > > On Wed, 2015-10-07 at 10:31 -0400, Josh Boyer wrote: > >> On Wed, Oct 7, 2015 at 5:13 AM, Michael Ellerman > >> wrote: > >> > On Wed,

Re: [kbuild-all] [PATCH 4/5 v2] pseries/iommu: implement DDW-aware dma_get_page_shift

On Thu, Oct 08, 2015 at 09:06:03AM +0800, Fengguang Wu wrote: > On Thu, Oct 08, 2015 at 11:11:59AM +1100, Michael Ellerman wrote: > > On Wed, 2015-10-07 at 21:56 +0800, Fengguang Wu wrote: > > > On Tue, Oct 06, 2015 at 02:39:06PM +1100, Michael Ellerman wrote: > > > > On Sat, 2015-10-03 at 04:33

Re: Missing operand for tlbie instruction on Power7

On Wed, 2015-10-07 at 10:31 -0400, Josh Boyer wrote: > On Wed, Oct 7, 2015 at 5:13 AM, Michael Ellerman wrote: > > On Wed, 2015-10-07 at 02:19 -0500, Segher Boessenkool wrote: > >> On Wed, Oct 07, 2015 at 05:00:49PM +1100, Michael Ellerman wrote: > >> > > It's also worth

[GIT PULL] strscpy powerpc fix for 3.4

On 10/7/2015 6:44 PM, Stephen Rothwell wrote: Hi Linus, After merging Linus' tree, today's linux-next build (powerpc ppc64_defconfig) failed like this: lib/string.c: In function 'strscpy': lib/string.c:209:4: error: implicit declaration of function 'zero_bytemask'

[PATCH v3 1/2] powerpc/xmon: Paged output for paca display

The paca display is already more than 24 lines, which can be problematic if you have an old school 80x24 terminal, or more likely you are on a virtual terminal which does not scroll for whatever reason. This patch adds a new command ".", which takes a single (hex) numeric argument: lines per

Re: [kbuild-all] [PATCH 4/5 v2] pseries/iommu: implement DDW-aware dma_get_page_shift

On Thu, Oct 08, 2015 at 11:11:59AM +1100, Michael Ellerman wrote: > On Wed, 2015-10-07 at 21:56 +0800, Fengguang Wu wrote: > > On Tue, Oct 06, 2015 at 02:39:06PM +1100, Michael Ellerman wrote: > > > On Sat, 2015-10-03 at 04:33 +0800, kbuild test robot wrote: > > > > Hi Nishanth, > > > > > > > >

Re: [PATCH v2 5/8] powerpc/pseries: Cleanup on pseries_eeh_get_state()

On 08/10/15 14:58, Gavin Shan wrote: This cleans up pseries_eeh_get_state(), no functional changes: * Return EEH_STATE_NOT_SUPPORT early when the 2nd RTAS output argument is zero to avoid nested if statements. * Skip clearing bits in the PE state represented by variable

Re: [PATCH v2 7/8] powerpc/powernv: Remove pnv_eeh_cap_start()

On 08/10/15 14:58, Gavin Shan wrote: This moves the logic of pnv_eeh_cap_start() to pnv_eeh_find_cap() as the function is only called by pnv_eeh_find_cap(). The logic of both functions are pretty simple. No need to have separate functions. Signed-off-by: Gavin Shan

[PATCH v2 5/8] powerpc/pseries: Cleanup on pseries_eeh_get_state()

This cleans up pseries_eeh_get_state(), no functional changes: * Return EEH_STATE_NOT_SUPPORT early when the 2nd RTAS output argument is zero to avoid nested if statements. * Skip clearing bits in the PE state represented by variable "result" to simplify the code. Signed-off-by:

Re: cxl: Fix number of allocated pages in SPA

On Wed, 2015-07-10 at 05:07:40 UTC, Ian Munsie wrote: > From: Christophe Lombard > > The scheduled process area is currently allocated before assigning the > correct maximum processes to the AFU, which will mean we only ever > allocate a fixed number of pages for the

Re: Missing operand for tlbie instruction on Power7

On Wed, 2015-10-07 at 02:19 -0500, Segher Boessenkool wrote: > On Wed, Oct 07, 2015 at 05:00:49PM +1100, Michael Ellerman wrote: > > > It's also worth noting that the __flush_power7 uses tlbiel instead of > > > tlbie. > > > > Yeah that's a good point. It's not clear if the swsusp code wants to a

Re: cxl: Workaround malformed pcie packets on some cards

On Fri, 2015-02-10 at 05:23:33 UTC, Ian Munsie wrote: > From: Philippe Bergheaud > > This works around a pcie host bridge defect on some cards, that can cause > malformed Transaction Layer Packet (TLP) errors to be erroneously reported. > > The upper nibble of the

Re: powerpc: Fix _ALIGN_* errors due to type difference.

On Wed, 2015-10-07 at 14:16 +0530, Aneesh Kumar K.V wrote: > Michael Ellerman writes: > > > On Fri, 2015-02-10 at 14:33:48 UTC, "Aneesh Kumar K.V" wrote: > >> This avoid errors like > >> > >> unsigned int usize = 1 << 30; > >> int size = 1 << 30; > >>

Re: [3/3] cxl: fix leak of ctx->mapping when releasing kernel API contexts

On Wed, 2015-30-09 at 01:58:07 UTC, Andrew Donnellan wrote: > When a context is created via the kernel API, ctx->mapping is allocated > within the kernel and thus needs to be freed when the context is freed. > reclaim_ctx() attempts to do this for contexts with the ctx->kernelapi flag > set, but

Re: [PATCH v2] perf: Fix build break on powerpc due to sample_reg_masks

Em Wed, Oct 07, 2015 at 11:46:38AM +1100, Michael Ellerman escreveu: > On Wed, 2015-09-30 at 16:45 -0300, Arnaldo Carvalho de Melo wrote: > > Em Wed, Sep 30, 2015 at 09:09:09PM +0200, Jiri Olsa escreveu: > > > On Wed, Sep 30, 2015 at 11:28:36AM -0700, Sukadev Bhattiprolu wrote: > > > > From

Re: [kbuild-all] [PATCH 4/5 v2] pseries/iommu: implement DDW-aware dma_get_page_shift

On Tue, Oct 06, 2015 at 02:39:06PM +1100, Michael Ellerman wrote: > On Sat, 2015-10-03 at 04:33 +0800, kbuild test robot wrote: > > Hi Nishanth, > > > > [auto build test results on v4.3-rc3 -- if it's inappropriate base, please > > ignore] > > > > config: powerpc-defconfig (attached as .config)

Re: [PATCH v2 05/18] powerpc/85xx: Load all early TLB entries at once

On 10/07/2015 06:48 AM, Scott Wood wrote: > Use an AS=1 trampoline TLB entry to allow all normal TLB1 entries to > be loaded at once. This avoids the need to keep the translation that > code is executing from in the same TLB entry in the final TLB > configuration as during early boot, which in

Re: [PATCH v3 3/3] dmaengine: mpc512x: initialize with subsys_initcall()

On Thu, Sep 24, 2015 at 08:28:57PM +0300, Alexander Popov wrote: > Initialize Freescale MPC512x DMA driver with subsys_initcall() > to allow the depending drivers to call dma_request_slave_channel() > during their probe. Why can't we use defered probe ? I have been asking people to not move init

Re: Missing operand for tlbie instruction on Power7

On Tue, 2015-10-06 at 11:25 -0700, Laura Abbott wrote: > On 10/05/2015 08:35 PM, Michael Ellerman wrote: > > On Fri, 2015-10-02 at 08:43 -0700, Laura Abbott wrote: > >> Hi, > >> > >> We received a report (https://bugzilla.redhat.com/show_bug.cgi?id=1267395) > >> of bad assembly > >> when

Re: Missing operand for tlbie instruction on Power7

On Wed, Oct 07, 2015 at 05:00:49PM +1100, Michael Ellerman wrote: > > It's also worth noting that the __flush_power7 uses tlbiel instead of tlbie. > > Yeah that's a good point. It's not clear if the swsusp code wants to a local > or > a global invalidate. If I read the code right, this is

Re: [GIT PULL 0/1] perf/urgent fix

* Arnaldo Carvalho de Melo wrote: > From: Arnaldo Carvalho de Melo > > Hi Ingo, > > Please consider pulling, > > - Arnaldo > > The following changes since commit 097f70b3c4d84ffccca15195bdfde3a37c0a7c0f: > > Merge branch 'upstream' of >

[PATCH 1/1] perf tools: Fix build break on powerpc due to sample_reg_masks

From: Sukadev Bhattiprolu perf_regs.c does not get built on Powerpc as CONFIG_PERF_REGS is false. So the weak definition for 'sample_regs_masks' doesn't get picked up. Adding perf_regs.o to util/Build unconditionally, exposes a redefinition error for

Re: [PATCH v9 1/4] perf,kvm/{x86,s390}: Remove dependency on uapi/kvm_perf.h

On 10/6/15 8:25 PM, Hemant Kumar wrote: @@ -358,7 +357,12 @@ static bool handle_end_event(struct perf_kvm_stat *kvm, time_diff = sample->time - time_begin; if (kvm->duration && time_diff > kvm->duration) { - char decode[DECODE_STR_LEN]; + char *decode

[GIT PULL 0/1] perf/urgent fix

From: Arnaldo Carvalho de Melo Hi Ingo, Please consider pulling, - Arnaldo The following changes since commit 097f70b3c4d84ffccca15195bdfde3a37c0a7c0f: Merge branch 'upstream' of git://git.linux-mips.org/pub/scm/ralf/upstream-linus (2015-09-27 18:22:34 -0400) are

Re: Time to remove platforms/cell?

On Mon, 2015-10-05 at 12:27 +0200, Marc Dietrich wrote: > Am Sonntag, 4. Oktober 2015, 12:27:19 schrieb Geoff Levand: > > I just tested the ps3-queue branch (v4.3-rc4 based) of my ps3-linux > > git > > repo and it works OK. > > > > Do you see any output on the display, maybe some penguins at