Re: [PATCH 1/2] tools include: Sync byteorder/generic.h

2016-06-15 Thread Jiri Olsa
On Thu, Jun 16, 2016 at 01:32:08AM +, He Kuang wrote: > From: Wang Nan > > This patch copies "include/linux/byteorder/generic.h" to > "tools/include/linux/byteorder/generic.h" to enable other libraries to > use macros in it. it's not the file copied, as the changelog suggest, just several ma

Re: [PATCH 2/2] tools include: Fix wrong macro definitions for cpu_to_le* for big endian

2016-06-15 Thread Jiri Olsa
On Thu, Jun 16, 2016 at 01:32:09AM +, He Kuang wrote: > From: Wang Nan > > The cpu_to_le* macros in kernel.h are defined without considering > endianese. This patch includes "byteoder/generic.h" instead to fix the > bug, and removes redundant le64_to_cpu definition in intel-bts.c. > > Signed

Re: [PATCH 2/2] tools include: Fix wrong macro definitions for cpu_to_le* for big endian

2016-06-15 Thread kbuild test robot
Hi, [auto build test ERROR on tip/perf/core] [also build test ERROR on v4.7-rc3 next-20160615] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/He-Kuang/tools-include-Fix-wrong-macro-definitions

[PATCH v3 2/2] cpufreq: qoriq: Don't look at clock implementation details

2016-06-15 Thread Scott Wood
From: Scott Wood Get the CPU clock's potential parent clocks from the clock interface itself, rather than manually parsing the clocks property to find a phandle, looking at the clock-names property of that, and assuming that those are valid parent clocks for the cpu clock. This is necessary now

[PATCH v3 1/2] clk: Add consumer APIs for discovering possible parent clocks

2016-06-15 Thread Scott Wood
From: Scott Wood Commit fc4a05d4b0eb ("clk: Remove unused provider APIs") removed __clk_get_num_parents() and clk_hw_get_parent_by_index(), leaving only true provider API versions that operate on struct clk_hw. qoriq-cpufreq needs these functions in order to determine the options it has for call

Re: kernel bug in "Drop WIMG in favour of new constants"?

2016-06-15 Thread Darrick J. Wong
On Thu, Jun 16, 2016 at 03:23:47PM +1000, Michael Ellerman wrote: > On Wed, 2016-06-15 at 21:33 -0700, Darrick J. Wong wrote: > > > Hi Aneesh, > > > > I noticed when trying out 4.7-rc3 on qemu-2.5 that the kernel no longer > > boots. 4.6 booted just fine, so I bisected the kernel to the commit >

[PATCH][v3] Add support for power9 in ibm_architecture_vec

2016-06-15 Thread Balbir Singh
The PVR list has been updated and IBM_ARCH_VEC_NRCORES_OFFSET. This provides the cpu versions supported to the hypervisor and in this case tells the hypervisor that the guest supports ISA 3.0 and Power9. Signed-off-by: Balbir Singh --- Changelog v3: Move the changelog to below "---"

Re: [PATCH v2 0/9] kexec_file_load implementation for PowerPC

2016-06-15 Thread Michael Ellerman
On Tue, 2016-06-14 at 11:59 -0300, Thiago Jung Bauermann wrote: > Hello, > > This patch series implements the kexec_file_load system call on PowerPC. Can you tell me what this syscall does and why I would want it? cheers ___ Linuxppc-dev mailing list

Re: [PATCH][V2] Add support for power9 in ibm_architecture_vec

2016-06-15 Thread Balbir Singh
On 16/06/16 15:31, Michael Ellerman wrote: >> Subject: [PATCH][V2] Add support for power9 in ibm_architecture_vec > > Can you use "[PATCH v2]", my scripts don't grok your format. > > On Thu, 2016-06-16 at 14:54 +1000, Balbir Singh wrote: >> >> On 16/06/16 10:09, Balbir Singh wrote: > >>> Yes,

Re: [PATCH][V2] Add support for power9 in ibm_architecture_vec

2016-06-15 Thread Michael Ellerman
> Subject: [PATCH][V2] Add support for power9 in ibm_architecture_vec Can you use "[PATCH v2]", my scripts don't grok your format. On Thu, 2016-06-16 at 14:54 +1000, Balbir Singh wrote: > > On 16/06/16 10:09, Balbir Singh wrote: > > Yes, my bad.. I'll respin. I thought I had it, but I clearly d

Re: kernel bug in "Drop WIMG in favour of new constants"?

2016-06-15 Thread Balbir Singh
On 16/06/16 14:33, Darrick J. Wong wrote: > Hi Aneesh, > > I noticed when trying out 4.7-rc3 on qemu-2.5 that the kernel no longer > boots. 4.6 booted just fine, so I bisected the kernel to the commit > 30bda41aba4efb2370c97e2cbe7385de93ccc372, which is "powerpc/mm: Drop WIMG in > favour of new

Re: kernel bug in "Drop WIMG in favour of new constants"?

2016-06-15 Thread Michael Ellerman
On Wed, 2016-06-15 at 21:33 -0700, Darrick J. Wong wrote: > Hi Aneesh, > > I noticed when trying out 4.7-rc3 on qemu-2.5 that the kernel no longer > boots. 4.6 booted just fine, so I bisected the kernel to the commit > 30bda41aba4efb2370c97e2cbe7385de93ccc372, which is "powerpc/mm: Drop WIMG in

Re: [PATCH] powerpc/mm/radix: Update Radix tree size as per ISA 3.0

2016-06-15 Thread Balbir Singh
On 16/06/16 13:21, Aneesh Kumar K.V wrote: > ISA 3.0 updated it to be encoded as Radix tree size = 2^(RTS + 31). We > have it encoded as 2^(RTS + 28). Add a helper with the correct encoding > and use it instead of opencoding. > > Fixes commit 2bfd65e45e87 ("powerpc/mm/radix: Add radix callbacks

[PATCH][V2] Add support for power9 in ibm_architecture_vec

2016-06-15 Thread Balbir Singh
On 16/06/16 10:09, Balbir Singh wrote: > Yes, my bad.. I'll respin. I thought I had it, but I clearly did not > V2: Update the vector length for OV1 after adding 3.0 support The PVR list has been updated and IBM_ARCH_VEC_NRCORES_OFFSET. This provides the cpu version supported to the hypervisor

kernel bug in "Drop WIMG in favour of new constants"?

2016-06-15 Thread Darrick J. Wong
Hi Aneesh, I noticed when trying out 4.7-rc3 on qemu-2.5 that the kernel no longer boots. 4.6 booted just fine, so I bisected the kernel to the commit 30bda41aba4efb2370c97e2cbe7385de93ccc372, which is "powerpc/mm: Drop WIMG in favour of new constants". The changelog suggests that the KVM change

Re: [PATCH] tools/perf: fix the word selected in find_*_bit

2016-06-15 Thread Hekuang
在 2016/6/16 5:29, Arnaldo Carvalho de Melo 写道: Em Thu, Jun 16, 2016 at 12:11:04AM +0300, Yury Norov escreveu: On Wed, Jun 15, 2016 at 10:51:27PM +0300, Yury Norov wrote: Maybe there already is some macro doing the conversion for you... yes it is, cpu_to_le64() is what you want Beware that t

[PATCH 2/2] tools include: Fix wrong macro definitions for cpu_to_le* for big endian

2016-06-15 Thread He Kuang
From: Wang Nan The cpu_to_le* macros in kernel.h are defined without considering endianese. This patch includes "byteoder/generic.h" instead to fix the bug, and removes redundant le64_to_cpu definition in intel-bts.c. Signed-off-by: Wang Nan Signed-off-by: He Kuang --- tools/include/linux/ker

[PATCH 1/2] tools include: Sync byteorder/generic.h

2016-06-15 Thread He Kuang
From: Wang Nan This patch copies "include/linux/byteorder/generic.h" to "tools/include/linux/byteorder/generic.h" to enable other libraries to use macros in it. tools/perf/MANIFEST is also updated for 'make perf-*-src-pkg'. Signed-off-by: Wang Nan Signed-off-by: He Kuang --- tools/include/li

[PATCH] tools include: Fix wrong macro definitions for cpu_to_le* for big endian

2016-06-15 Thread He Kuang
From: Wang Nan The cpu_to_le* macros in kernel.h are defined without considering endianese. This patch includes "byteoder/generic.h" instead to fix the bug, and removes redundant le64_to_cpu definition in intel-bts.c. Signed-off-by: Wang Nan Signed-off-by: He Kuang --- tools/include/linux/ker

[PATCH] tools include: Fix wrong macro definitions for cpu_to_le* for big endian

2016-06-15 Thread He Kuang
From: Wang Nan The cpu_to_le* macros in kernel.h are defined without considering endianese. This patch includes "byteoder/generic.h" instead to fix the bug, and removes redundant le64_to_cpu definition in intel-bts.c. Signed-off-by: Wang Nan Signed-off-by: He Kuang --- tools/include/linux/ker

[PATCH] powerpc/mm/radix: Update Radix tree size as per ISA 3.0

2016-06-15 Thread Aneesh Kumar K.V
ISA 3.0 updated it to be encoded as Radix tree size = 2^(RTS + 31). We have it encoded as 2^(RTS + 28). Add a helper with the correct encoding and use it instead of opencoding. Fixes commit 2bfd65e45e87 ("powerpc/mm/radix: Add radix callbacks for early init routine ") Signed-off-by: Aneesh Kumar

Re: [PATCH v2 2/9] kexec_file: Generalize kexec_add_buffer.

2016-06-15 Thread Dave Young
On 06/15/16 at 01:21pm, Thiago Jung Bauermann wrote: > Hello Dave, > > Am Mittwoch, 15 Juni 2016, 15:33:02 schrieb Dave Young: > > > @@ -472,14 +498,16 @@ int kexec_add_buffer(struct kimage *image, char > > > *buffer, unsigned long bufsz,> > > > kbuf->top_down = top_down; > > > > > > /* Wa

Re: [PATCH] Add support for power9 in ibm_architecture_vec

2016-06-15 Thread Balbir Singh
On 16/06/16 01:58, Aneesh Kumar K.V wrote: > Balbir Singh writes: > >> On Wed, Jun 15, 2016 at 9:17 PM, Aneesh Kumar K.V >> wrote: >>> Shouldn't this be 125 + 16 ? >> >> Actually the previous patch broke it and Ben fixed it, it should have >> been 133 after adding POWER8NVL. My patch adds 16+1

[PATCH 3/3] cxlflash: Shutdown notify support for CXL Flash cards

2016-06-15 Thread Uma Krishnan
Some CXL Flash cards need notification of device shutdown in order to flush pending I/Os. A PCI notification hook for shutdown has been added where the driver notifies the card and returns. When the device is removed in the PCI remove path, notification code will wait for shutdown processing to co

[PATCH 2/3] cxlflash: Add device dependent flags

2016-06-15 Thread Uma Krishnan
Device dependent flags are needed to support functions that are specific to a particular device. One such case is - some CXL Flash cards need to be notified of device shutdown. For other CXL devices, this feature does not prove to be useful yet. Such distinct features need to be identified in the

[PATCH 1/3] cxlflash: Fix to drain operations from previous reset

2016-06-15 Thread Uma Krishnan
From: "Manoj N. Kumar" While running 'sg_reset -H' in a loop with a user-space application active, hit the following exception: cpu 0x2: Vector: 300 (Data Access) pc: : afu_attach+0x50/0x240 [cxlflash] lr: : cxlflash_afu_recover+0x3dc/0x7d0 [cxlflash] pid = 20365, comm = run_block_

[PATCH 0/3] cxlflash: Shutdown notification and reset patch

2016-06-15 Thread Uma Krishnan
This patch set contains support to notify CXL Flash devices of an impending shutdown and a fix to drain operations prior to a reset. This series is intended for 4.8 and is bisectable. Manoj N. Kumar (1): cxlflash: Fix to drain operations from previous reset Uma Krishnan (2): cxlflash: Add de

Re: [PATCH 0/1] cxlflash: Fix in EEH recovery

2016-06-15 Thread Uma Krishnan
Please ignore this email. It was sent in error. Sorry for the confusion. On 6/15/2016 6:46 PM, Uma Krishnan wrote: This patch addresses a deadlock issue seen during EEH recovery and is intended for 4.7. Manoj N. Kumar (1): cxlflash: Fix to resolve dead-lock during EEH recovery drivers/scsi/

[PATCH 0/1] cxlflash: Fix in EEH recovery

2016-06-15 Thread Uma Krishnan
This patch addresses a deadlock issue seen during EEH recovery and is intended for 4.7. Manoj N. Kumar (1): cxlflash: Fix to resolve dead-lock during EEH recovery drivers/scsi/cxlflash/superpipe.c | 15 +++ 1 file changed, 15 insertions(+) -- 2.1.0 __

Re: [PATCH] cxl: Make vPHB device node match adapter's

2016-06-15 Thread Ian Munsie
This could probably use a description in the commit message, perhaps including output showing the before/after difference this makes to lsvpd, but otherwise it looks fine to me. @Mikey - this look OK to you? Acked-by: Ian Munsie Excerpts from Frederic Barrat's message of 2016-06-15 16:42:16 +02

Re: [PATCH] tools/perf: fix the word selected in find_*_bit

2016-06-15 Thread Yury Norov
On Wed, Jun 15, 2016 at 10:51:27PM +0300, Yury Norov wrote: > Hi Madhavan, > > On Wed, Jun 15, 2016 at 05:12:53PM +0530, Madhavan Srinivasan wrote: > > When decoding the perf_regs mask in regs_dump__printf(), > > we loop through the mask using find_first_bit and find_next_bit functions. > > And ma

Re: [PATCH] tools/perf: fix the word selected in find_*_bit

2016-06-15 Thread Yury Norov
Hi Madhavan, On Wed, Jun 15, 2016 at 05:12:53PM +0530, Madhavan Srinivasan wrote: > When decoding the perf_regs mask in regs_dump__printf(), > we loop through the mask using find_first_bit and find_next_bit functions. > And mask is of type "u64". But "u64" is send as a "unsigned long *" to > lib f

[PATCH] Remove a tiny memory leak in drivers/of

2016-06-15 Thread Mathieu Malaterre
On my PowerMac device-tree would generate a duplicate name: [0.023043] device-tree: Duplicate name in PowerPC,G4@0, renamed to "l2-cache#1" in this case a newly allocated name is generated by `safe_name`. However in this case it is never deallocated. The bug was found using kmemleak reporte

Re: [PATCH] tools/perf: fix the word selected in find_*_bit

2016-06-15 Thread Arnaldo Carvalho de Melo
Em Thu, Jun 16, 2016 at 12:11:04AM +0300, Yury Norov escreveu: > On Wed, Jun 15, 2016 at 10:51:27PM +0300, Yury Norov wrote: > > Maybe there already is some macro doing the conversion for you... > > yes it is, cpu_to_le64() is what you want Beware that the cpu_to_le64() in tools/perf is bogus, we

[PATCH v2] of: fix memory leak related to safe_name()

2016-06-15 Thread Frank Rowand
From: Frank Rowand Fix a memory leak resulting from memory allocation in safe_name(). This patch fixes all call sites of safe_name(). Mathieu Malaterre reported the memory leak on boot: On my PowerMac device-tree would generate a duplicate name: [0.023043] device-tree: Duplicate name in Po

Re: [PATCH] of: fix memory leak related to safe_name()

2016-06-15 Thread Frank Rowand
On 06/15/16 12:12, Rob Herring wrote: > On Wed, Jun 15, 2016 at 1:13 PM, Frank Rowand wrote: >> From: Frank Rowand >> >> Fix a memory leak resulting from memory allocation in safe_name(). >> This patch fixes all call sites of safe_name(). >> >> Mathieu Malaterre reported the memory leak on boot:

[PATCH v2] powerpc/pseries: start rtasd before PCI probing

2016-06-15 Thread Greg Kurz
A strange behaviour is observed when comparing PCI hotplug in QEMU, between x86 and pseries. If you consider the following steps: - start a VM - add a PCI device via the QEMU monitor before the rtasd has started (for example starting the VM in paused state, or hotplug during FW or boot loader)

Re: [PATCH] cxl: Make vPHB device node match adapter's

2016-06-15 Thread Matthew R. Ochs
> On Jun 15, 2016, at 9:42 AM, Frederic Barrat > wrote: > > Tested by cxlflash on bare-metal and powerVM. > > Signed-off-by: Frederic Barrat Reviewed-by: Matthew R. Ochs ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozl

Re: [PATCH 1/2] workqueue: Move wq_update_unbound_numa() to the beginning of CPU_ONLINE

2016-06-15 Thread Gautham R Shenoy
Hello Tejun, On Wed, Jun 15, 2016 at 11:53:50AM -0400, Tejun Heo wrote: > Hello, > > On Tue, Jun 07, 2016 at 08:44:02PM +0530, Gautham R. Shenoy wrote: > > Currently in the CPU_ONLINE workqueue handler, the > > restore_unbound_workers_cpumask() will never call > > set_cpus_allowed_ptr() for a new

Re: [PATCH] of: fix memory leak related to safe_name()

2016-06-15 Thread Rob Herring
On Wed, Jun 15, 2016 at 1:13 PM, Frank Rowand wrote: > From: Frank Rowand > > Fix a memory leak resulting from memory allocation in safe_name(). > This patch fixes all call sites of safe_name(). > > Mathieu Malaterre reported the memory leak on boot: > > On my PowerMac device-tree would generate

[PATCH] of: fix memory leak related to safe_name()

2016-06-15 Thread Frank Rowand
From: Frank Rowand Fix a memory leak resulting from memory allocation in safe_name(). This patch fixes all call sites of safe_name(). Mathieu Malaterre reported the memory leak on boot: On my PowerMac device-tree would generate a duplicate name: [0.023043] device-tree: Duplicate name in Po

Re: [PATCH] Remove a tiny memory leak in drivers/of

2016-06-15 Thread Frank Rowand
Mathieu, On 06/15/16 08:01, Mathieu Malaterre wrote: > On my PowerMac device-tree would generate a duplicate name: > > [0.023043] device-tree: Duplicate name in PowerPC,G4@0, renamed to > "l2-cache#1" > > in this case a newly allocated name is generated by `safe_name`. However > in this cas

Re: [PATCH 8/8] dmaengine: Remove site specific OOM error messages on kzalloc

2016-06-15 Thread Vinod Koul
On Wed, Jun 08, 2016 at 02:12:27PM +0200, Linus Walleij wrote: > On Tue, Jun 7, 2016 at 7:38 PM, Peter Griffin > wrote: > > > If kzalloc() fails it will issue it's own error message including > > a dump_stack(). So remove the site specific error messages. > > > > Signed-off-by: Peter Griffin >

Re: [PATCH v2 2/9] kexec_file: Generalize kexec_add_buffer.

2016-06-15 Thread Thiago Jung Bauermann
Hello Dave, Am Mittwoch, 15 Juni 2016, 15:33:02 schrieb Dave Young: > > @@ -472,14 +498,16 @@ int kexec_add_buffer(struct kimage *image, char > > *buffer, unsigned long bufsz,> > > kbuf->top_down = top_down; > > > > /* Walk the RAM ranges and allocate a suitable range for the buffer

[PATCH V3] powerpc/mm: Clear top 16 bits of va only on older cpus

2016-06-15 Thread Aneesh Kumar K.V
As per ISA, we need to do this only for architecture version 2.02 and earlier. This continued to work even for 2.07. But let's not do this for anything after 2.02. ISA 3.0 requires these top bits to be not cleared. Signed-off-by: Aneesh Kumar K.V --- Changes from v2: * Update commit message * Add

Re: [PATCH 2/2] workqueue:Fix affinity of an unbound worker of a node with 1 online CPU

2016-06-15 Thread Tejun Heo
Hello, On Wed, Jun 15, 2016 at 03:14:15PM +0200, Peter Zijlstra wrote: > On Wed, Jun 15, 2016 at 06:20:33PM +0530, Gautham R Shenoy wrote: > > We will no longer have the optimization in > > restore_unbound_workers_cpumask() but I suppose we don't lose much by > > resetting the affinity every time

Re: [PATCH] Add support for power9 in ibm_architecture_vec

2016-06-15 Thread Aneesh Kumar K.V
Balbir Singh writes: > On Wed, Jun 15, 2016 at 9:17 PM, Aneesh Kumar K.V > wrote: >> Shouldn't this be 125 + 16 ? > > Actually the previous patch broke it and Ben fixed it, it should have > been 133 after adding POWER8NVL. My patch adds 16+1 (1 for the > OV1_PPC_3_00) > > 133+17 = 150 > > Michae

Re: [PATCH 1/2] workqueue: Move wq_update_unbound_numa() to the beginning of CPU_ONLINE

2016-06-15 Thread Tejun Heo
Hello, On Tue, Jun 07, 2016 at 08:44:02PM +0530, Gautham R. Shenoy wrote: > Currently in the CPU_ONLINE workqueue handler, the > restore_unbound_workers_cpumask() will never call > set_cpus_allowed_ptr() for a newly created unbound worker thread. Hmmm... did you actually verify that this happens?

[PATCH] cxl: Make vPHB device node match adapter's

2016-06-15 Thread Frederic Barrat
Tested by cxlflash on bare-metal and powerVM. Signed-off-by: Frederic Barrat --- drivers/misc/cxl/vphb.c | 21 ++--- 1 file changed, 10 insertions(+), 11 deletions(-) diff --git a/drivers/misc/cxl/vphb.c b/drivers/misc/cxl/vphb.c index cdc7723..012b6aa 100644 --- a/drivers/misc/

Re: [PATCH] tools/perf: fix the word selected in find_*_bit

2016-06-15 Thread George Spelvin
Madhavan Srinivasan wrote: > +#if (__BYTE_ORDER == __BIG_ENDIAN) && (BITS_PER_LONG != 64) > + tmp = addr[(((nbits - 1)/BITS_PER_LONG) - (start / BITS_PER_LONG))] > + ^ invert; > +#else > tmp = addr[start / BITS_PER_LONG] ^ invert

Re: [PATCH] hsi: Build hsi_boardinfo.c into hsi core if enabled

2016-06-15 Thread Sebastian Reichel
Hi Andrew, On Tue, Jun 14, 2016 at 11:13:04AM -0500, Andrew F. Davis wrote: > If the HSI core is built as a module hsi_boardinfo may still > be built-in as its Kconfig type is bool, which can cause build > issues. Fix this by building this code into the HSI core when > enabled. > > Reported-by: k

Re: [PATCH 2/2] workqueue:Fix affinity of an unbound worker of a node with 1 online CPU

2016-06-15 Thread Peter Zijlstra
On Wed, Jun 15, 2016 at 06:20:33PM +0530, Gautham R Shenoy wrote: > We will no longer have the optimization in > restore_unbound_workers_cpumask() but I suppose we don't lose much by > resetting the affinity every time a CPU in the pool->attr->cpumask > comes online. Right; optimizing hotplug real

Re: [v6, 08/11] powerpc/powernv: Add platform support for stop instruction

2016-06-15 Thread Shreyas B Prabhu
Hi Michael, On 06/15/2016 04:44 PM, Michael Ellerman wrote: > Hi Shreyas, > > Comments inline ... > > On Wed, 2016-08-06 at 16:54:28 UTC, "Shreyas B. Prabhu" wrote: >> POWER ISA v3 defines a new idle processor core mechanism. In summary, >> a) new instruction named stop is added. This instructi

Re: [PATCH 2/2] workqueue:Fix affinity of an unbound worker of a node with 1 online CPU

2016-06-15 Thread Gautham R Shenoy
On Wed, Jun 15, 2016 at 01:32:49PM +0200, Peter Zijlstra wrote: > On Wed, Jun 15, 2016 at 03:49:36PM +0530, Gautham R Shenoy wrote: > > > Also, with the first patch in the series (which ensures that > > restore_unbound_workers are called *after* the new workers for the > > newly onlined CPUs are c

Re: [v4] powerpc: Define and use PPC64_ELF_ABI_v2/v1

2016-06-15 Thread Michael Ellerman
On Mon, 2016-06-06 at 16:56:10 UTC, "Naveen N. Rao" wrote: > From: Michael Ellerman > > We're approaching 20 locations where we need to check for ELF ABI v2. > That's fine, except the logic is a bit awkward, because we have to check > that _CALL_ELF is defined and then what its value is. > > So

Re: powerpc/nvram: remove unused pstore headers

2016-06-15 Thread Michael Ellerman
On Wed, 2016-08-06 at 10:02:37 UTC, Geliang Tang wrote: > Since the pstore code has moved away from nvram.c, remove unused > pstore headers pstore.h and kmsg_dump.h. > > Signed-off-by: Geliang Tang Applied to powerpc next, thanks. https://git.kernel.org/powerpc/c/1e61423fb1b6a814e50ed7845c che

Re: powerpc32: get rid of sub_reloc_offset()

2016-06-15 Thread Michael Ellerman
On Mon, 2016-23-05 at 08:45:49 UTC, Christophe Leroy wrote: > sub_reloc_offset() has not been used since > commit 917f0af9e5a9 ("powerpc: Remove arch/ppc and include/asm-ppc") > which removed include/asm-ppc/prom.h > > Signed-off-by: Christophe Leroy Applied to powerpc next, thanks. https://git

Re: powerpc/pseries: fix trivial typo in function name

2016-06-15 Thread Michael Ellerman
On Fri, 2016-20-05 at 14:18:57 UTC, Greg Kurz wrote: > Signed-off-by: Greg Kurz Applied to powerpc next, thanks. https://git.kernel.org/powerpc/c/6e45273eacc829a44fae1d3df1 cheers ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lis

Re: [2/3] powerpc: Avoid load hit store in setup_sigcontext()

2016-06-15 Thread Michael Ellerman
On Sun, 2016-29-05 at 12:03:51 UTC, Anton Blanchard wrote: > From: Anton Blanchard > > In setup_sigcontext(), we set current->thread.vrsave then use it > straight after. Since current is hidden from the compiler via inline > assembly, it cannot optimise this and we end up with a load hit store. >

Re: [2/2] powerpc: Align hot loops of some string functions

2016-06-15 Thread Michael Ellerman
On Wed, 2016-25-05 at 22:39:55 UTC, Unknown sender due to SPF wrote: > Align the hot loops in our assembly implementation of strncpy(), > strncmp() and memchr(). > > Signed-off-by: Anton Blanchard Applied to powerpc next, thanks. https://git.kernel.org/powerpc/c/87a156fb18fe15d012c3db506b chee

Re: [1/5] powerpc/pseries: Drop support for MPIC in pseries

2016-06-15 Thread Michael Ellerman
On Mon, 2016-30-05 at 06:18:11 UTC, Rashmica Gupta wrote: > MPIC was only used by Power3 which is now unsupported, so drop support > for MPIC. XICS is now the only supported interrupt controller for > pSeries so make the XICS functions generic. > > Signed-off-by: Rashmica Gupta Series applied to

Re: [1/3] powerpc: Avoid load hit store in __giveup_fpu() and __giveup_altivec()

2016-06-15 Thread Michael Ellerman
On Sun, 2016-29-05 at 12:03:50 UTC, Anton Blanchard wrote: > From: Anton Blanchard > > In both __giveup_fpu() and __giveup_altivec() we make two modifications > to tsk->thread.regs->msr. gcc decides to do a read/modify/write of > each change, so we end up with a load hit store: > > ld

Re: [1/2] powerpc: Remove assembly versions of strcpy, strcat, strlen and strcmp

2016-06-15 Thread Michael Ellerman
On Wed, 2016-25-05 at 22:38:13 UTC, Unknown sender due to SPF wrote: > A number of our assembly implementations of string functions do not > align their hot loops. I was going to align them manually, but I > realised that they are are almost instruction for instruction > identical to what gcc produ

Re: [V2, 07/10] powerpc/hash: Use the correct ppp mask when updating hpte

2016-06-15 Thread Michael Ellerman
On Wed, 2016-08-06 at 14:25:55 UTC, "Aneesh Kumar K.V" wrote: > With commit: e58e87adc8bf9 ("powerpc/mm: Update _PAGE_KERNEL_RO") we > now use all the three PPP bits. The top bit is now used to have a > PPP value of 0b110 which will be mapped to kernel read only. When > updating the hpte entry use

Re: [V2, 03/10] powerpc/mm/radix: Flush page walk cache when freeing page table

2016-06-15 Thread Michael Ellerman
On Wed, 2016-08-06 at 14:25:51 UTC, "Aneesh Kumar K.V" wrote: > Even though a tlb_flush() does a flush with invalidate all cache, > we can end up doing an RCU page table free before calling tlb_flush(). > That means we can have page walk cache entries even after we free the > page table pages. This

Re: [V2, 02/10] powerpc/mm/radix: Update to tlb functions ric argument

2016-06-15 Thread Michael Ellerman
On Wed, 2016-08-06 at 14:25:50 UTC, "Aneesh Kumar K.V" wrote: > Radix invalidate control (RIC) is used to control which cache to flush > using tlb instructions. When doing a PID flush, we currently flush > everything including page walk cache. For address range flush, we flush > only the TLB. In th

Re: [PATCH] Add support for power9 in ibm_architecture_vec

2016-06-15 Thread Balbir Singh
On Wed, Jun 15, 2016 at 9:17 PM, Aneesh Kumar K.V wrote: > Shouldn't this be 125 + 16 ? Actually the previous patch broke it and Ben fixed it, it should have been 133 after adding POWER8NVL. My patch adds 16+1 (1 for the OV1_PPC_3_00) 133+17 = 150 Michael, can you please check. I did a quick ch

[PATCH] tools/perf: fix the word selected in find_*_bit

2016-06-15 Thread Madhavan Srinivasan
When decoding the perf_regs mask in regs_dump__printf(), we loop through the mask using find_first_bit and find_next_bit functions. And mask is of type "u64". But "u64" is send as a "unsigned long *" to lib functions along with sizeof(). While the exisitng code works fine in most of the case, when

Re: [PATCH 2/2] workqueue:Fix affinity of an unbound worker of a node with 1 online CPU

2016-06-15 Thread Peter Zijlstra
On Wed, Jun 15, 2016 at 03:49:36PM +0530, Gautham R Shenoy wrote: > Also, with the first patch in the series (which ensures that > restore_unbound_workers are called *after* the new workers for the > newly onlined CPUs are created) and without this one, you can > reproduce this WARN_ON on both x86

Re: [PATCH] Add support for power9 in ibm_architecture_vec

2016-06-15 Thread Aneesh Kumar K.V
Balbir Singh writes: > The PVR list has been updated and IBM_ARCH_VEC_NRCORES_OFFSET. > > Signed-off-by: Balbir Singh > --- > arch/powerpc/include/asm/prom.h | 5 - > arch/powerpc/kernel/prom_init.c | 5 - > 2 files changed, 8 insertions(+), 2 deletions(-) > > diff --git a/arch/powerpc/

Re: [v6, 08/11] powerpc/powernv: Add platform support for stop instruction

2016-06-15 Thread Michael Ellerman
Hi Shreyas, Comments inline ... On Wed, 2016-08-06 at 16:54:28 UTC, "Shreyas B. Prabhu" wrote: > POWER ISA v3 defines a new idle processor core mechanism. In summary, > a) new instruction named stop is added. This instruction replaces > instructions like nap, sleep, rvwinkle. > b) new per

Re: [PATCH 2/2] workqueue:Fix affinity of an unbound worker of a node with 1 online CPU

2016-06-15 Thread Gautham R Shenoy
Hi Peter, On Tue, Jun 14, 2016 at 01:22:34PM +0200, Peter Zijlstra wrote: > On Tue, Jun 07, 2016 at 08:44:03PM +0530, Gautham R. Shenoy wrote: > > I'm still puzzled why we don't see this on x86. Afaict there's nothing > PPC specific about this. You are right. On PPC, at boot time we hit the WARN

[PATCH 4/4] powerpc/kvm/stats: Implement existing and add new halt polling vcpu stats

2016-06-15 Thread Suraj Jitindar Singh
vcpu stats are used to collect information about a vcpu which can be viewed in the debugfs. For example halt_attempted_poll and halt_successful_poll are used to keep track of the number of times the vcpu attempts to and successfully polls. These stats are currently not used on powerpc. Implement i

[PATCH 3/4] kvm/stats: Add provisioning for 64-bit vcpu statistics

2016-06-15 Thread Suraj Jitindar Singh
vcpus have statistics associated with them which can be viewed within the debugfs. Currently it is assumed within the vcpu_stat_get() and vcpu_stat_get_per_vm() functions that all of these statistics are represented as 32-bit numbers. The next patch adds some 64-bit statistics, so add provisioning

[PATCH 2/4] kvm/ppc/book3s_hv: Implement halt polling in the kvm_hv kernel module

2016-06-15 Thread Suraj Jitindar Singh
This patch introduces new halt polling functionality into the kvm_hv kernel module. When a vcore is idle it will poll for some period of time before scheduling itself out. When all of the runnable vcpus on a vcore have ceded (and thus the vcore is idle) we schedule ourselves out to allow something

[PATCH 1/4] kvm/ppc/book3s_hv: Change vcore element runnable_threads from linked-list to array

2016-06-15 Thread Suraj Jitindar Singh
The struct kvmppc_vcore is a structure used to store various information about a virtual core for a kvm guest. The runnable_threads element of the struct provides a list of all of the currently runnable vcpus on the core (those in the KVMPPC_VCPU_RUNNABLE state). The previous implementation of this

[PATCH] Add support for power9 in ibm_architecture_vec

2016-06-15 Thread Balbir Singh
The PVR list has been updated and IBM_ARCH_VEC_NRCORES_OFFSET. Signed-off-by: Balbir Singh --- arch/powerpc/include/asm/prom.h | 5 - arch/powerpc/kernel/prom_init.c | 5 - 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/arch/powerpc/include/asm/prom.h b/arch/powerpc/inc

Re: [PATCH 2/2] powerpc: tm: Always reclaim in start_thread() for exec() class syscalls

2016-06-15 Thread kbuild test robot
Hi, [auto build test ERROR on powerpc/next] [also build test ERROR on v4.7-rc3 next-20160615] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Cyril-Bur/selftests-powerpc-exec-with-suspended

Re: [PATCH v2 2/9] kexec_file: Generalize kexec_add_buffer.

2016-06-15 Thread Dave Young
Hi, Thiago On 06/14/16 at 11:59am, Thiago Jung Bauermann wrote: > Allow architectures to specify different memory walking functions for > kexec_add_buffer. Intel uses iomem to track reserved memory ranges, > but PowerPC uses the memblock subsystem. > > Signed-off-by: Thiago Jung Bauermann > Cc:

Re: [V2][PATCH] New MMU feature for new hpte format in ISA3

2016-06-15 Thread Balbir Singh
On 15/06/16 15:42, Aneesh Kumar K.V wrote: > Balbir Singh writes: > >> On 15/06/16 10:38, Balbir Singh wrote: >>> >>> Thanks for a review, I'll do a V2 with some changes >>> >>> Balbir >>> >> >> Michael, please review and check if you like the #ifdef, >> since we are crossing the 32 bit boundar