Re: [kernel] powerpc/powernv/ioda2: Fix calculation for memory allocated for TCE table

2015-07-30 Thread Michael Ellerman
On Mon, 2015-20-07 at 10:45:51 UTC, Alexey Kardashevskiy wrote: > The existing code stores the amount of memory allocated for a TCE table. > At the moment it uses @offset which is a virtual offset in the TCE table > which is only correct for a one level tables and it does not include > memory alloc

Re: powerpc/eeh-powernv: Fix unbalanced IRQ warning

2015-07-30 Thread Michael Ellerman
On Thu, 2015-30-07 at 06:53:54 UTC, Alistair Popple wrote: > pnv_eeh_next_error() re-enables the eeh opal event interrupt but it > gets called from a loop if there are more outstanding events to > process, resulting in a warning due to enabling an already enabled > interrupt. Instead the interrupt

[git pull] Please pull powerpc/linux.git powerpc-4.2-3 tag

2015-08-02 Thread Michael Ellerman
Hi Linus, Please pull some more powerpc fixes for 4.2: The following changes since commit c5dfd654d0ec0a28fe81e7bd4d4fd984a9855e09: Merge git://git.kernel.org/pub/scm/linux/kernel/git/davem/net (2015-07-22 14:45:25 -0700) are available in the git repository at: git://git.kernel.org/pub/sc

Re: [PATCH] cxl: Fix ambiguous else warnings

2015-08-02 Thread Michael Ellerman
On Mon, 2015-08-03 at 10:36 +1000, Daniel Axtens wrote: > Hi Andreas, > > On Fri, 2015-07-31 at 11:16 +0200, Andreas Schwab wrote: > > Daniel Axtens writes: > > > > > Every time I build cxl I see the following warnings: > > > > > > /scratch/dja/linux-capi/drivers/misc/cxl/pci.c: In function > >

Re: [v2] powerpc: Use hardware RNG for arch_get_random_seed_* not arch_get_random_*

2015-08-02 Thread Michael Ellerman
On Fri, 2015-17-07 at 10:11:43 UTC, Paul Mackerras wrote: > The hardware RNG on POWER8 and POWER7+ can be relatively slow, since > it can only supply one 64-bit value per microsecond. Currently we > read it in arch_get_random_long(), but that slows down reading from > /dev/urandom since the code i

Re: powerpc/rtas: Introduce rtas_get_sensor_fast() for IRQ handlers

2015-08-02 Thread Michael Ellerman
On Fri, 2015-17-07 at 10:46:58 UTC, Thomas Huth wrote: > The EPOW interrupt handler uses rtas_get_sensor(), which in turn > uses rtas_busy_delay() to wait for RTAS becoming ready in case it > is necessary. But rtas_busy_delay() is annotated with might_sleep() > and thus may not be used by interrupt

Re: [v3, 1/8] powerpc/perf/hv-24x7: Whitespace - fix parameter alignment

2015-08-02 Thread Michael Ellerman
On Wed, 2015-15-07 at 03:01:48 UTC, Sukadev Bhattiprolu wrote: > Fix parameter alignment to be consistent with coding style. > > Signed-off-by: Sukadev Bhattiprolu Applied to powerpc next, thanks. https://git.kernel.org/powerpc/c/40386217cd7bc38908d6 cheers

Re: [v3, 2/8] powerpc/perf/hv-24x7: Simplify extracting counter from result buffer

2015-08-02 Thread Michael Ellerman
On Wed, 2015-15-07 at 03:01:49 UTC, Sukadev Bhattiprolu wrote: > Simplify code that extracts a 24x7 counter from the HCALL's result buffer. > > Suggested-by: Joe Perches > Signed-off-by: Sukadev Bhattiprolu Applied to powerpc next, thanks. https://git.kernel.org/powerpc/c/465345ca387ed491c467

Re: powerpc/rtas: Replace magic values with defines

2015-08-02 Thread Michael Ellerman
On Wed, 2015-22-07 at 16:56:47 UTC, Thomas Huth wrote: > rtas.h already has some nice #defines for RTAS return status > codes - let's use them instead of hard-coded "magic" values! > > Signed-off-by: Thomas Huth > Reviewed-by: Tyrel Datwyler Applied to powerpc next, thanks. https://git.kernel.

Re: [2/5] powerpc/perf: Change type of the bhrb_users variable

2015-08-02 Thread Michael Ellerman
On Tue, 2015-30-06 at 08:20:28 UTC, Anshuman Khandual wrote: > This patch just changes data type of bhrb_users variable from > int to unsigned int because it never contains a negative value. > > Reported-by: Daniel Axtens > Signed-off-by: Anshuman Khandual Applied to powerpc next, thanks. http

Re: [PATCH] cxl: Fix ambiguous else warnings

2015-08-02 Thread Michael Ellerman
On Mon, 2015-08-03 at 10:47 +1000, Daniel Axtens wrote: > > This should be a static inline, not a #define. > > > I'm changing them to static inlines at the moment. Would you prefer a v3 > of the EEH patches, or a new patch on top? Without looking closer I think a v3. ie. before you change the lo

Re: [PATCH v2] arch: use WRITE_ONCE/READ_ONCE in smp_store_release/smp_load_acquire

2015-08-02 Thread Michael Ellerman
t; + typeof(*p) ___p1 = READ_ONCE(*p); \ > compiletime_assert_atomic_type(*p); \ > smp_lwsync(); \ > ___p1;

Re: [PATCH] powerpc/hvsi: Fix endianness issues in the HVSI driver

2015-08-03 Thread Michael Ellerman
On Fri, 2015-07-31 at 11:29 +0200, Laurent Dufour wrote: > This patch fixes several endianness issues detected when running the HVSI > driver in little endian mode. > > These issues are raised in little endian mode because the data exchanged in > memory between the kernel and the hypervisor has to

Re: [V3] powerpc/irq: Enable some more exceptions in /proc/interrupts interface

2015-08-04 Thread Michael Ellerman
On Mon, 2015-13-07 at 08:16:06 UTC, Anshuman Khandual wrote: > This patch enables facility unavailable exceptions for generic facility, > FPU, ALTIVEC and VSX in /proc/interrupts listing by incrementing their > newly added IRQ statistical counters as and when these exceptions happen. > This also ad

Re: powerpc: Add an inline function to update HID0

2015-08-04 Thread Michael Ellerman
On Tue, 2015-04-08 at 08:30:58 UTC, "Gautham R. Shenoy" wrote: > Section 3.7 of Version 1.2 of the Power8 Processor User's Manual > prescribes that updates to HID0 be preceded by a SYNC instruction and > followed by an ISYNC instruction (Page 91). > > Create a function name update_hid0() which fol

Re: powerpc: Add an inline function to update HID0

2015-08-04 Thread Michael Ellerman
On Tue, 2015-08-04 at 19:36 +0530, Madhavan Srinivasan wrote: > > On Tuesday 04 August 2015 03:38 PM, Michael Ellerman wrote: > > On Tue, 2015-04-08 at 08:30:58 UTC, "Gautham R. Shenoy" wrote: > >> Section 3.7 of Version 1.2 of the Power8 Processor User's Ma

Re: windfarm: decrement client count when unregistering

2015-08-04 Thread Michael Ellerman
On Fri, 2015-31-07 at 12:08:58 UTC, Paul Bolle wrote: > wf_unregister_client() increments the client count when a client > unregisters. That is obviously incorrect. Decrement that client count > instead. > > Fixes: 75722d3992f5 ("[PATCH] ppc64: Thermal control for SMU based machines") > > Signed-

[PATCH] powerpc/vdso: Emit GNU & SysV hashes

2015-08-06 Thread Michael Ellerman
every DSO has a GNU hash section, then libc can avoid calculating SysV hashes at all. Both musl and glibc appear to have this optimization. Signed-off-by: Michael Ellerman --- arch/powerpc/kernel/vdso32/Makefile | 2 +- arch/powerpc/kernel/vdso64/Makefile | 2 +- 2 files changed, 2 insertions

[PATCH 1/5] powerpc/mm: Fix pte_pagesize_index() crash on 4K w/64K hash

2015-08-06 Thread Michael Ellerman
s configuration, otherwise we use the slice mask. Fixes: 7aa0727f3302 ("powerpc/mm: Increase the slice range to 64TB") Reported-by: Cyril Bur Signed-off-by: Michael Ellerman --- arch/powerpc/include/asm/pgtable-ppc64.h | 14 +- 1 file changed, 13 insertions(+), 1 deletion(-)

[PATCH 2/5] powerpc/cell: Drop support for 64K local store on 4K kernels

2015-08-06 Thread Michael Ellerman
. However these days the number of Cell users is approaching zero, and supporting this option adds unpleasant complexity to the memory management code. So drop the option, CONFIG_SPU_FS_64K_LS, and all related code. Signed-off-by: Michael Ellerman --- arch/powerpc/include/asm/spu_csa.h

[PATCH 3/5] powerpc/mm: Drop the 64K on 4K version of pte_pagesize_index()

2015-08-06 Thread Michael Ellerman
pte_pagesize_index() and crucially __real_pte, which means this defintion can never be used. Signed-off-by: Michael Ellerman --- arch/powerpc/include/asm/pgtable-ppc64.h | 12 1 file changed, 12 deletions(-) diff --git a/arch/powerpc/include/asm/pgtable-ppc64.h b/arch/powerpc/include/asm/pgtable

[PATCH 4/5] powerpc/mm: Simplify page size kconfig dependencies

2015-08-06 Thread Michael Ellerman
pse them into a single 'depends' by combining them with &&. Signed-off-by: Michael Ellerman --- arch/powerpc/Kconfig | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig index 5ef27113b898..3a4ba2809201 100644

[PATCH 5/5] powerpc/mm: Drop CONFIG_PPC_HAS_HASH_64K

2015-08-06 Thread Michael Ellerman
uilt for PPC_STD_MMU_64, meaning it's equivalent to PPC_64K_PAGES. So replace all uses and drop it. Signed-off-by: Michael Ellerman --- arch/powerpc/Kconfig| 6 -- arch/powerpc/mm/hash_low_64.S | 4 ++-- arch/powerpc/mm/hash_utils_64.c | 12 ++-- 3 files changed,

Re: 4.1-rc6: ATA link is slow to respond, please be patient

2015-08-09 Thread Michael Ellerman
On Sat, 2015-08-08 at 21:17 -0700, Christian Kujau wrote: > [Adding linux-...@vger.kernel.org] > > On Fri, 7 Aug 2015, Christian Kujau wrote: > > this PowerBook G4 was running 3.16 for a while but now I wanted to upgrade > > to latest mainline. However, during bootup the following happens: > > >

Re: [3/5] drivers/tty: make hvc_console.c explicitly non-modular

2015-08-09 Thread Michael Ellerman
nit, > so we dont have init ordering concerns with this commit. > > Cc: Greg Kroah-Hartman > Cc: Jiri Slaby > Cc: linuxppc-dev@lists.ozlabs.org > Signed-off-by: Paul Gortmaker Seems sane to me, possibly back in the deep dark past it could be a module, but no point keeping t

Re: [v2,2/2] powerpc/powernv: Pull all HMI events before panic.

2015-08-10 Thread Michael Ellerman
On Tue, 2015-05-05 at 08:05:43 UTC, Mahesh Salgaonkar wrote: > From: Mahesh Salgaonkar > > In the event of unrecovered HMI the existing code panics as soon as > it receives the first unrecovered HMI event. This makes host to report > partial information about HMIs before panic. There may be more

Re: [v2] powerpc/ftrace: add powerpc timebase as a trace clock source

2015-08-10 Thread Michael Ellerman
On Fri, 2015-24-04 at 08:54:44 UTC, "Naveen N. Rao" wrote: > Add a new powerpc-specific trace clock using the timebase register, > similar to x86-tsc. This gives us > - a fast, monotonic, hardware clock source for trace entries, and > - a clock that can be used to correlate events across cpus as we

Re: powerpc/4xx: Fix return value check in hsta_msi_probe()

2015-08-10 Thread Michael Ellerman
On Thu, 2015-16-04 at 12:18:50 UTC, weiyj...@163.com wrote: > From: Wei Yongjun > > In case of error, the functions platform_get_resource() and kmalloc() > returns NULL not ERR_PTR(). The IS_ERR() test in the return value check > should be replaced with NULL test. > > Signed-off-by: Wei Yongjun

Re: cxl: Don't ignore add_process_element result when attaching context

2015-08-10 Thread Michael Ellerman
On Wed, 2015-29-07 at 04:07:22 UTC, Daniel Axtens wrote: > Previously, when attaching a context in dedicated mode, we ignored > the result of add_process_element, which could potentially fail. > > If add_process_element returns and error, pass it back to the caller. > > Signed-off-by: Daniel Axte

Re: powernv: Invoke opal_cec_reboot2() on unrecoverable HMI.

2015-08-10 Thread Michael Ellerman
On Tue, 2015-04-08 at 11:18:56 UTC, Mahesh Salgaonkar wrote: > From: Mahesh Salgaonkar > > Invoke new opal_cec_reboot2() call with reboot type > OPAL_REBOOT_PLATFORM_ERROR (for unrecoverable HMI interrupts) to inform > BMC/OCC about this error, so that BMC can collect relevant data for error > an

Re: windfarm: remove three exported but unused functions

2015-08-10 Thread Michael Ellerman
On Fri, 2015-31-07 at 12:14:20 UTC, Paul Bolle wrote: > wf_find_control(), wf_find_sensor(), and wf_is_overtemp() are exported > but unused. Remove these three functions. > > Signed-off-by: Paul Bolle Applied to powerpc next, thanks. https://git.kernel.org/powerpc/c/a368c29cf105485d2c34 cheers

Re: powerpc: Remove redundant breaks

2015-08-10 Thread Michael Ellerman
On Mon, 2015-29-06 at 21:30:39 UTC, Joe Perches wrote: > break; break; isn't useful. > > Remove one. > > Signed-off-by: Joe Perches Applied to powerpc next, thanks. https://git.kernel.org/powerpc/c/a825ac078b50266fb091 cheers ___ Linuxppc-dev mailin

Re: powerpc: pci: use %pR for printing struct resource

2015-08-10 Thread Michael Ellerman
On Fri, 2015-12-06 at 02:26:37 UTC, Kevin Hao wrote: > Use %pR to simplify the debug code. This also make the debug info more > readable. > > Signed-off-by: Kevin Hao Applied to powerpc next, thanks. https://git.kernel.org/powerpc/c/ae2a84b4074cff81957b cheers _

Re: [v2, 1/2] powerpc/powernv: display reason for Malfunction Alert HMI.

2015-08-10 Thread Michael Ellerman
On Tue, 2015-05-05 at 08:04:58 UTC, Mahesh Salgaonkar wrote: > From: Mahesh Salgaonkar > > The V2 version of HMI event now carries additional information for > Malfunction Alert. It now contains error information about CORE and NX > checkstop. This patch checks and displays the check stop reason

Re: windfarm: make wf_critical_overtemp() static

2015-08-10 Thread Michael Ellerman
On Fri, 2015-31-07 at 12:12:20 UTC, Paul Bolle wrote: > wf_critical_overtemp() is exported. But nothing uses that export. > That's unsurprising because there's no header that defines it. Stop > exporting that function and make it static. > > Signed-off-by: Paul Bolle Applied to powerpc next, tha

Re: powernv: Invoke opal_cec_reboot2() on unrecoverable machine check errors.

2015-08-10 Thread Michael Ellerman
On Fri, 2015-31-07 at 15:54:38 UTC, Mahesh Salgaonkar wrote: > From: Mahesh Salgaonkar > > On non-recoverable MCE errors in kernel space, Linux kernel panics > and system reboots. On BMC based system opal-prd runs as a daemon > in the host. Hence, kernel crash may prevent opal-prd to detect and >

Re: windfarm: decrement client count when unregistering

2015-08-10 Thread Michael Ellerman
On Fri, 2015-31-07 at 12:08:58 UTC, Paul Bolle wrote: > wf_unregister_client() increments the client count when a client > unregisters. That is obviously incorrect. Decrement that client count > instead. > > Fixes: 75722d3992f5 ("[PATCH] ppc64: Thermal control for SMU based machines") > > Signed-

Re: [PATCH] powerpc/prom: Use DRCONF flags while processing detected LMBs

2015-08-10 Thread Michael Ellerman
On Fri, 2015-08-07 at 07:49 +0530, Madhavan Srinivasan wrote: > > On Thursday 06 August 2015 06:35 PM, Anshuman Khandual wrote: > > This patch just replaces hard coded values with existing > > Please drop "This patch just" and start with "Replace hard ..." > > https://www.kernel.org/doc/Docu

Re: [3/4] cxl: sparse: Flag iomem pointers properly

2015-08-11 Thread Michael Ellerman
On Fri, 2015-07-08 at 03:18:19 UTC, Daniel Axtens wrote: > Sparse identifies the following address space issues: > /scratch/dja/linux-capi/drivers/misc/cxl/vphb.c:125:17: warning: incorrect > type in assignment (different address spaces) > /scratch/dja/linux-capi/drivers/misc/cxl/vphb.c:125:17:

Re: [4/8] powerpc/slb: Add some helper functions to improve modularization

2015-08-11 Thread Michael Ellerman
On Wed, 2015-29-07 at 07:10:01 UTC, Anshuman Khandual wrote: > This patch adds the following six helper functions to help improve > modularization and readability of the code. > > (1) slb_invalidate_all: Invalidates the entire SLB > (2) slb_invalidate: Invalidates SLB entri

Re: [5/8] powerpc/slb: Add documentation to runtime patching of SLB encoding

2015-08-11 Thread Michael Ellerman
On Wed, 2015-29-07 at 07:10:02 UTC, Anshuman Khandual wrote: > This patch adds some documentation to 'patch_slb_encoding' function > explaining about how it clears the existing immediate value in the > given instruction and inserts a new one there. > > Signed-off-by: Anshuman Khandual > --- > ar

Re: [PATCH 8/8] powerpc/xmon: Add some more elements to the existing PACA dump list

2015-08-11 Thread Michael Ellerman
On Wed, 2015-07-29 at 12:40 +0530, Anshuman Khandual wrote: > This patch adds a set of new elements to the existing PACA dump list > inside an xmon session which can be listed below improving the overall > xmon debug support. > > (1) hmi_event_available > (2) dscr_default > (3) vmalloc_sllp > (4)

[PATCH] powerpc/xmon: Allow limiting the size of the paca display

2015-08-11 Thread Michael Ellerman
ent = yes online = yes lock_token = 0x8000(0xa) paca_index = 0x3 (0x8) Signed-off-by: Michael Ellerman --- arch/powerpc/xmon/xmon.c | 23 +++ 1 file changed, 15 insertions(+), 8 deletions(-) diff --git a/arc

Re: [PATCH] powerpc/xmon: Allow limiting the size of the paca display

2015-08-12 Thread Michael Ellerman
On Wed, 2015-08-12 at 13:24 +0530, Anshuman Khandual wrote: > On 08/12/2015 12:27 PM, Michael Ellerman wrote: > > @@ -2090,9 +2092,12 @@ static void dump_one_paca(int cpu) > > printf(" %-*s = %s\n", 16, "present", cpu_present(cpu) ? "yes" : &quo

[PATCH v2] powerpc/xmon: Allow limiting the size of the paca display

2015-08-12 Thread Michael Ellerman
ent = yes online = yes lock_token = 0x8000(0xa) paca_index = 0x3 (0x8) kernel_toc = 0xc1230300 (0x10) Signed-off-by: Michael Ellerman --- v2: Honour the limit even for the initial lines, and change the

Re: [2/8] powerpc/slb: Rename all the 'slot' occurrences to 'entry'

2015-08-12 Thread Michael Ellerman
On Wed, 2015-29-07 at 07:09:59 UTC, Anshuman Khandual wrote: > These are essentially SLB individual slots with entries what we are > dealing with in these functions. Usage of both 'entry' and 'slot' > synonyms makes it real confusing sometimes. This patch makes it > uniform across the file by repla

Re: [1/8] powerpc/slb: Remove a duplicate extern variable

2015-08-12 Thread Michael Ellerman
On Wed, 2015-29-07 at 07:09:58 UTC, Anshuman Khandual wrote: > This patch just removes one redundant entry for one extern variable > 'slb_compare_rr_to_size' from the scope. This patch does not change > any functionality. > > Signed-off-by: Anshuman Khandual Applied to powerpc next, thanks. htt

Re: [2/4] cxl: sparse: Make declarations static

2015-08-12 Thread Michael Ellerman
On Fri, 2015-07-08 at 03:18:18 UTC, Daniel Axtens wrote: > A few declarations were identified by sparse as needing to be static: > > /scratch/dja/linux-capi/drivers/misc/cxl/irq.c:408:6: warning: symbol > 'afu_irq_name_free' was not declared. Should it be static? > /scratch/dja/linux-capi/drivers

Re: [5/8] powerpc/slb: Add documentation to runtime patching of SLB encoding

2015-08-12 Thread Michael Ellerman
On Wed, 2015-29-07 at 07:10:02 UTC, Anshuman Khandual wrote: > This patch adds some documentation to 'patch_slb_encoding' function > explaining about how it clears the existing immediate value in the > given instruction and inserts a new one there. > > Signed-off-by: Anshuman Khandual Applied to

Re: [7/8] powerpc/xmon: Drop the 'valid' variable completely in 'dump_segments'

2015-08-12 Thread Michael Ellerman
On Wed, 2015-29-07 at 07:10:04 UTC, Anshuman Khandual wrote: > Value of the 'valid' variable is zero when 'esid' is zero and it does > not matter when 'esid' is non-zero. The variable 'valid' can be dropped > from the function 'dump_segments' by checking for validity of 'esid' > inside the nested c

Re: powerpc/prom: Use DRCONF flags while processing detected LMBs

2015-08-12 Thread Michael Ellerman
On Thu, 2015-06-08 at 13:05:07 UTC, Anshuman Khandual wrote: > This patch just replaces hard coded values with existing > DRCONF flags while procesing detected LMBs from the device > tree. This does not change any functionality. > > Signed-off-by: Anshuman Khandual Applied to powerpc next, thank

Re: [4/4] cxl: sparse: Silence iomem warning in debugfs file creation

2015-08-12 Thread Michael Ellerman
On Fri, 2015-07-08 at 03:18:20 UTC, Daniel Axtens wrote: > An IO address, tagged with __iomem, is passed to debugfs_create_file > as private data. This requires that it be cast to void *. The cast > creates a sparse warning: > /scratch/dja/linux-capi/drivers/misc/cxl/debugfs.c:51:57: warning: cast

Re: [1/4] cxl: Compile with -Werror

2015-08-12 Thread Michael Ellerman
On Fri, 2015-07-08 at 03:18:17 UTC, Daniel Axtens wrote: > It's a good idea, and it brings us in line with the rest of arch/powerpc. > > Signed-off-by: Daniel Axtens > Acked-by: Michael Neuling Applied to powerpc next, thanks. https://git.kernel.org/powerpc/c/d3d73f4b38a8ece19846 cheers _

Re: [6/8] powerpc/prom: Simplify the logic while fetching SLB size

2015-08-12 Thread Michael Ellerman
On Wed, 2015-29-07 at 07:10:03 UTC, Anshuman Khandual wrote: > This patch just simplifies the existing code logic while fetching > the SLB size property from the device tree. This also changes the > function name from check_cpu_slb_size to init_mmu_slb_size as > it just initializes the mmu_slb_size

[PATCH v2] powerpc/slb: Define an enum for the bolted indexes

2015-08-13 Thread Michael Ellerman
From: Anshuman Khandual This patch defines macros for the three bolted SLB indexes we use. Switch the functions that take the indexes as an argument to use the enum. Signed-off-by: Anshuman Khandual Signed-off-by: Michael Ellerman --- v2: Use index rather than slot as that's what th

[PATCH] powerpc/slb: Use a local to avoid multiple calls to get_slb_shadow()

2015-08-13 Thread Michael Ellerman
For no reason other than it looks ugly. Signed-off-by: Michael Ellerman --- arch/powerpc/mm/slb.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/arch/powerpc/mm/slb.c b/arch/powerpc/mm/slb.c index 0c7115fd314b..515730e499fe 100644 --- a/arch/powerpc/mm/slb.c

Re: RFC: Reducing the number of non volatile GPRs in the ppc64 kernel

2015-08-13 Thread Michael Ellerman
On Wed, 2015-08-05 at 14:03 +1000, Anton Blanchard wrote: > Hi, > > While looking at traces of kernel workloads, I noticed places where gcc > used a large number of non volatiles. Some of these functions > did very little work, and we spent most of our time saving the > non volatiles to the stack

Re: [PATCH] kvm:powerpc:Fix return statements for wrapper functions in the file book3s_64_mmu_hv.c

2015-08-13 Thread Michael Ellerman
On Wed, 2015-08-12 at 21:06 +0200, Alexander Graf wrote: > > On 10.08.15 17:27, Nicholas Krause wrote: > > This fixes the wrapper functions kvm_umap_hva_hv and the function > > kvm_unmap_hav_range_hv to return the return value of the function > > kvm_handle_hva or kvm_handle_hva_range that they ar

Re: [V3] powerpc/irq: Enable some more exceptions in /proc/interrupts interface

2015-08-13 Thread Michael Ellerman
On Thu, 2015-08-06 at 18:54 +0530, Anshuman Khandual wrote: > On 08/04/2015 03:27 PM, Michael Ellerman wrote: > > On Mon, 2015-13-07 at 08:16:06 UTC, Anshuman Khandual wrote: > >> This patch enables facility unavailable exceptions for generic facility, > >> FPU, ALTIVEC

Re: [v2,5/5] powerpc/pseries: re-use code from of_helpers module

2015-08-14 Thread Michael Ellerman
On Tue, 2015-11-08 at 11:23:09 UTC, Andy Shevchenko wrote: > The derive_parent() has similar semantics to what we have in newly introduced > of_helpers module. The replacement reduces code base and propagates the actual > error code to the caller. > > Signed-off-by: Andy Shevchenko > --- > arch/

Re: [PATCH v5 11/11] cxl: Add CONFIG_CXL_EEH symbol

2015-08-16 Thread Michael Ellerman
On Fri, 2015-08-14 at 17:41 +1000, Daniel Axtens wrote: > CONFIG_CXL_EEH is for CXL's EEH related code. > > Other drivers can depend on or #ifdef on this symbol to configure > PERST behaviour, allowing CXL to participate in the EEH process. > > Reviewed-by: Cyril Bur > Signed-off-by: Daniel Axte

Re: [v5, 01/11] cxl: Convert MMIO read/write macros to inline functions

2015-08-17 Thread Michael Ellerman
On Fri, 2015-14-08 at 07:41:17 UTC, Daniel Axtens wrote: > We're about to make these more complex, so make them functions > first. > > Signed-off-by: Daniel Axtens Series applied to powerpc next, thanks. https://git.kernel.org/powerpc/c/588b34be20bc3dd7441c cheers _

Re: [RESEND] cxl:Plug irq_bitmap getting leaked in cxl_context

2015-08-17 Thread Michael Ellerman
On Fri, 2015-14-08 at 06:58:38 UTC, Vaibhav Jain wrote: > This patch plugs the leak of irq_bitmap, allocated as part of > initialization of cxl_context struct; during the call to > afu_allocate_irqs. The bitmap is now release during the call to function > afu_release_irqs. > > Reported-by: Matthew

Re: [v3] powerpc: Add an inline function to update POWER8 HID0

2015-08-17 Thread Michael Ellerman
On Wed, 2015-05-08 at 07:08:31 UTC, "Gautham R. Shenoy" wrote: > Section 3.7 of Version 1.2 of the Power8 Processor User's Manual > prescribes that updates to HID0 be preceded by a SYNC instruction and > followed by an ISYNC instruction (Page 91). > > Create an inline function name update_power8_h

Re: powerpc/eeh: Probe after unbalanced kref check

2015-08-17 Thread Michael Ellerman
On Fri, 2015-14-08 at 06:03:19 UTC, Daniel Axtens wrote: > In the complete hotplug case, EEH PEs are supposed to be released > and set to NULL. Normally, this is done by eeh_remove_device(), > which is called from pcibios_release_device(). > > However, if something is holding a kref to the device,

Re: [PATCH 2/2] ppc64: Add a flag to tell the kernel it's booting from kexec

2015-08-17 Thread Michael Ellerman
On Mon, 2015-08-17 at 13:59 -0500, Scott Wood wrote: > On Sat, 2015-07-18 at 14:57 -0500, Scott Wood wrote: > > It needs to know this because the SMP release mechanism for Freescale > > book3e is different from when booting with normal hardware. In theory > > we could simulate the normal spin tabl

Re: [PATCH RESEND] powerpc/numa: initialize distance lookup table from drconf path

2015-08-17 Thread Michael Ellerman
On Tue, 2015-08-11 at 15:25 +0530, Nikunj A Dadhania wrote: > Hi Michael, > > Nikunj A Dadhania writes: > > In some situations, a NUMA guest that supports > > ibm,dynamic-memory-reconfiguration node will end up having flat NUMA > > distances between nodes. This is because of two problems in the >

Re: [PATCH 2/2] ppc64: Add a flag to tell the kernel it's booting from kexec

2015-08-17 Thread Michael Ellerman
On Mon, 2015-08-17 at 22:30 -0500, Scott Wood wrote: > On Tue, 2015-08-18 at 12:51 +1000, Michael Ellerman wrote: > > On Mon, 2015-08-17 at 13:59 -0500, Scott Wood wrote: > > > On Sat, 2015-07-18 at 14:57 -0500, Scott Wood wrote: > > > > It needs to know this because

Re: [RFC,14/17] powerpc/book3e-64/kexec: Enable SMP release

2015-08-17 Thread Michael Ellerman
On Sat, 2015-18-07 at 20:08:51 UTC, Scott Wood wrote: > booted_from_exec is similar to __run_at_load, except that it is set for ^ missing k. Also do you mind using __booted_from_kexec to keep the naming similar to the other variables down there, and also make it clear i

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

2015-08-17 Thread Michael Ellerman
0x0 (0x2ba) > irq_happened = 0x1 (0x2bb) > io_sync = 0x0 (0x2bc) > irq_work_pending = 0x0 (0x2bd) > nap_state_lost = 0x0 (0x2be) > 0:mon> > > (Based

Re: [PATCH] cxl: Allow release of contexts which have been OPENED but not STARTED

2015-08-18 Thread Michael Ellerman
On Tue, 2015-08-18 at 16:30 +1000, Andrew Donnellan wrote: > If we open a context but do not start it (either because we do not attempt > to start it, or because it fails to start for some reason), we are left > with a context in state OPENED. Previously, cxl_release_context() only > allowed releas

Re: [v8,3/3] leds/powernv: Add driver for PowerNV platform

2015-08-18 Thread Michael Ellerman
On Sat, 2015-25-07 at 05:21:10 UTC, Vasant Hegde wrote: > This patch implements LED driver for PowerNV platform using the existing > generic LED class framework. > > PowerNV platform has below type of LEDs: > - System attention > Indicates there is a problem with the system that needs atte

Re: [05/27] macintosh: therm_windtunnel: Export I2C module alias information

2015-08-18 Thread Michael Ellerman
On Thu, 2015-30-07 at 16:18:30 UTC, Javier Martinez Canillas wrote: > The I2C core always reports the MODALIAS uevent as "i2c: regardless if the driver was matched using the I2C id_table or the > of_match_table. So the driver needs to export the I2C table and this > be built into the module or udev

Re: [v2, 1/2] Move the pt_regs_offset struct definition from arch to common include file

2015-08-18 Thread Michael Ellerman
unlikely that definition would ever need to be > changed regardless of architecture so lets move it into > include/linux/ptrace.h, along with macros commonly used to access it. Thanks for cleaning this up. Tested successfully on powerpc. Acked-by: Michael Ellerman cheers ___

Re: [PATCH 1/2] PCI: Make pci_msi_setup_pci_dev() non-static for use by arch code

2015-08-18 Thread Michael Ellerman
Hi Guilherme, Thanks for the patches. On Tue, 2015-08-18 at 18:13 -0300, Guilherme G. Piccoli wrote: > Commit 1851617cd2 ("PCI/MSI: Disable MSI at enumeration even if kernel > doesn't support MSI") changed the location of the code that disables > MSI/MSI-X interrupts at PCI probe time in devices

Re: [05/27] macintosh: therm_windtunnel: Export I2C module alias information

2015-08-18 Thread Michael Ellerman
On Tue, 2015-08-18 at 12:35 +0200, Javier Martinez Canillas wrote: > Hello Michael, > > On 08/18/2015 12:24 PM, Michael Ellerman wrote: > > On Thu, 2015-30-07 at 16:18:30 UTC, Javier Martinez Canillas wrote: > >> The I2C core always reports the MODALIAS uevent as "i2c

Re: [PATCH 2/2] cxl: add set/get private data to context struct

2015-08-18 Thread Michael Ellerman
On Wed, 2015-08-19 at 14:19 +1000, Ian Munsie wrote: > From: Michael Neuling > > This provides AFU drivers a means to associate private data with a cxl > context. This is particularly intended for make the new callbacks for > driver specific events easier for AFU drivers to use, as they can easil

Re: [PATCH 1/2] cxl: Add mechanism for delivering AFU driver specific events

2015-08-18 Thread Michael Ellerman
On Wed, 2015-08-19 at 14:19 +1000, Ian Munsie wrote: > From: Ian Munsie > > This adds an afu_driver_ops structure with event_pending and > deliver_event callbacks. An AFU driver can fill these out and associate > it with a context to enable passing custom AFU specific events to > userspace. What

Re: [PATCH 2/2] cxl: add set/get private data to context struct

2015-08-18 Thread Michael Ellerman
On Wed, 2015-08-19 at 15:12 +1000, Ian Munsie wrote: > Excerpts from Michael Ellerman's message of 2015-08-19 14:49:30 +1000: > > Do we really need the accessors? They don't buy anything I can see over just > > using ctx->priv directly. > > The reasoning there is because we don't currently expose

Re: [RESEND] powerpc/numa: initialize distance lookup table from drconf path

2015-08-19 Thread Michael Ellerman
On Thu, 2015-02-07 at 05:39:01 UTC, Nikunj A Dadhania wrote: > In some situations, a NUMA guest that supports > ibm,dynamic-memory-reconfiguration node will end up having flat NUMA > distances between nodes. This is because of two problems in the > current code. > > 1) Different representations of

Re: [06/31] arch/powerpc/pseries: use kmemdup rather than duplicating its implementation

2015-08-19 Thread Michael Ellerman
On Fri, 2015-07-08 at 07:59:12 UTC, Andrzej Hajda wrote: > The patch was generated using fixed coccinelle semantic patch > scripts/coccinelle/api/memdup.cocci [1]. > > [1]: http://permalink.gmane.org/gmane.linux.kernel/2014320 > > Signed-off-by: Andrzej Hajda > Reviewed-by: Nathan Fontenot App

Re: powerpc/eeh: Disable automatically blocked PCI config

2015-08-19 Thread Michael Ellerman
On Wed, 2015-29-07 at 23:26:51 UTC, Gavin Shan wrote: > pcibios_set_pcie_reset_state() could be called to complete > reset request when passing through PCI device, flag > EEH_PE_ISOLATED is set before saving the PCI config sapce. > On some Broadcom adapters, EEH_PE_CFG_BLOCKED is automatically > se

Re: nvram: print no error message when nvram is not set as pstore backend

2015-08-19 Thread Michael Ellerman
On Mon, 2015-11-05 at 08:23:43 UTC, Hari Bathini wrote: > Pstore only supports one backend at a time. The preferred > pstore backend is set by passing the pstore.backend= > argument to the kernel at boot time. Currently, while trying > to register with pstore, nvram throws an error message even > w

Re: powerpc/powernv: move dma_get_required_mask from pnv_phb to pci_controller_ops

2015-08-19 Thread Michael Ellerman
On Fri, 2015-07-08 at 03:45:54 UTC, Andrew Donnellan wrote: > Simplify the dma_get_required_mask call chain by moving it from pnv_phb to > pci_controller_ops, similar to commit 763d2d8df1ee2b92ff09c > ("powerpc/powernv: Move dma_set_mask from pnv_phb to pci_controller_ops"). > > Previous call chai

Re: [1/5] powerpc/mm: Fix pte_pagesize_index() crash on 4K w/64K hash

2015-08-19 Thread Michael Ellerman
On Fri, 2015-07-08 at 06:19:43 UTC, Michael Ellerman wrote: > The powerpc kernel can be built to have either a 4K PAGE_SIZE or a 64K > PAGE_SIZE. ... > This fix is ugly, but simple. When we're called for a kernel address we > return 4K, which is always correct in this configurat

Re: [v1,1/1] mpc512x: silence a USB Kconfig dependency warning

2015-08-19 Thread Michael Ellerman
On Mon, 2013-03-06 at 11:18:06 UTC, Gerhard Sittig wrote: > the PPC_MPC512x config automatically selected USB_EHCI_BIG_ENDIAN_* > switches, which made Kconfig warn about "unmet direct dependencies": > > scripts/kconfig/conf --silentoldconfig Kconfig > warning: (PPC_MPC512x && 440EPX) selects U

Re: powerpc: Export include/uapi/asm/eeh.h

2015-08-19 Thread Michael Ellerman
On Tue, 2015-18-08 at 00:57:40 UTC, Gavin Shan wrote: > This adds include/uapi/asm/eeh.h to kbuild so that the header > file will be exported automatically with below command. The > header file was added by commit ed3e81f ("powerpc/eeh: Move PE > state constants around") > >make INSTALL_HDR_PA

Re: cxl: Add alternate MMIO error handling

2015-08-19 Thread Michael Ellerman
On Thu, 2015-23-07 at 06:43:56 UTC, Ian Munsie wrote: > From: Ian Munsie > > userspace programs using cxl currently have to use two strategies for > dealing with MMIO errors simultaneously. They have to check every read > for a return of all Fs in case the adapter has gone away and the kernel > h

Re: [RESEND] powerpc/configs: enable RTC class support

2015-08-19 Thread Michael Ellerman
On Fri, 2015-03-07 at 05:11:20 UTC, Vaibhav Jain wrote: > A working rtc kernel driver is needed so that hwclock can synchronize > system clock to rtc during shutdown/boot. We already have a powernv > platform rtc driver located at drivers/rtc/rtc-opal.c. However it depends > on CONFIG_RTC_CLASS whi

Re: [05/31] arch/powerpc/nvram: use kmemdup rather than duplicating its implementation

2015-08-19 Thread Michael Ellerman
On Fri, 2015-07-08 at 07:59:11 UTC, Andrzej Hajda wrote: > The patch was generated using fixed coccinelle semantic patch > scripts/coccinelle/api/memdup.cocci [1]. > > [1]: http://permalink.gmane.org/gmane.linux.kernel/2014320 > > Signed-off-by: Andrzej Hajda > Reviewed-by: Nathan Fontenot App

Re: Ping Re: [PATCH 0/8] math-emu: Update kernel math-emu code from current glibc soft-fp

2015-08-19 Thread Michael Ellerman
On Wed, 2015-08-19 at 14:39 +, Joseph Myers wrote: > I'd like to ping this patch series, not having seen any comments on it. > > [PATCH 0/8] math-emu: Update kernel math-emu code from current glibc soft-fp > https://lkml.org/lkml/2015/7/2/394 > > [PATCH 1/8] math-emu: Move math-emu to math-em

Re: [PATCH v2 1/2] PCI: Make pci_msi_setup_pci_dev() non-static for use by arch code

2015-08-19 Thread Michael Ellerman
On Wed, 2015-08-19 at 15:45 -0300, Guilherme G. Piccoli wrote: > Thanks very much for your suggestions Michael. I agree with them all, > so I'm sending the patch v2 (see below). > > About the relevant mailing lists, I already sent to the linux-pci and > already cc'ed Bjorn Helgaas - the problem is

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

2015-08-19 Thread Michael Ellerman
On Thu, 2015-08-20 at 10:42 +1000, Sam Bobroff wrote: > On Tue, Aug 18, 2015 at 04:26:32PM +1000, Michael Ellerman wrote: > > On Fri, 2015-14-08 at 02:55:14 UTC, Sam bobroff wrote: > > > The paca display is already more than 24 lines, which can be problematic > > > if y

Re: provide more common DMA API functions V2

2015-08-19 Thread Michael Ellerman
On Thu, 2015-08-20 at 09:52 +1000, Stephen Rothwell wrote: > Hi Andrew (sorry, I can't tell who made the incorrect statement below > that I am replying to), > > On Wed, 19 Aug 2015 14:36:56 -0700 Andrew Morton > wrote: > > > > On Wed, 19 Aug 2015 10:08:14 +0200 Christoph Hellwig wrote: > > > >

Re: [PATCH] powerpc/hvsi: Fix endianness issues in the HVSI driver

2015-08-19 Thread Michael Ellerman
On Wed, 2015-08-19 at 23:53 +0200, Laurent Dufour wrote: > On 04/08/2015 02:51, Michael Ellerman wrote: > > On Fri, 2015-07-31 at 11:29 +0200, Laurent Dufour wrote: > >> This patch fixes several endianness issues detected when running the HVSI > >> driver in little e

Re: [PATCH] powerpc/hvsi: Fix endianness issues in the HVSI driver

2015-08-19 Thread Michael Ellerman
On Thu, 2015-08-20 at 11:40 +1000, Michael Ellerman wrote: > On Wed, 2015-08-19 at 23:53 +0200, Laurent Dufour wrote: > > On 04/08/2015 02:51, Michael Ellerman wrote: > > > On Fri, 2015-07-31 at 11:29 +0200, Laurent Dufour wrote: > > >> This patch fixes several e

Re: [RFC PATCH 14/17] powerpc/book3e-64/kexec: Enable SMP release

2015-08-19 Thread Michael Ellerman
Hi Scott, Sorry for the delay. So I'm back to square one on this patch. On Sat, 2015-07-18 at 15:08 -0500, Scott Wood wrote: > booted_from_exec is similar to __run_at_load, except that it is set for > regular kexec as well as kdump. > > The flag is needed because the SMP release mechanism for FS

Re: powerpc/powernv: Fix the log message when disabling VF

2015-08-20 Thread Michael Ellerman
On Mon, 2015-20-07 at 10:14:58 UTC, Wei Yang wrote: > On powernv platform, IOV BAR would be shifted if necessary. While the log > message is not correct when disabling VFs. > > This patch fixes this by print correct message based on the offset value. > > Signed-off-by: Wei Yang Applied to power

Re: [V2] cxl: Allow release of contexts which have been OPENED but not STARTED

2015-08-20 Thread Michael Ellerman
On Tue, 2015-18-08 at 23:27:18 UTC, Andrew Donnellan wrote: > If we open a context but do not start it (either because we do not attempt > to start it, or because it fails to start for some reason), we are left > with a context in state OPENED. Previously, cxl_release_context() only > allowed relea

Re: [v10,3/3] leds/powernv: Add driver for PowerNV platform

2015-08-20 Thread Michael Ellerman
On Wed, 2015-19-08 at 16:49:54 UTC, Vasant Hegde wrote: > This patch implements LED driver for PowerNV platform using the existing > generic LED class framework. > > PowerNV platform has below type of LEDs: > - System attention > Indicates there is a problem with the system that needs atte

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