[v2 PATCH] cpufreq: powernv: Correctly parse the sign of pstates on POWER8 vs POWER9

2017-12-06 Thread Gautham R. Shenoy
From: "Gautham R. Shenoy" On POWERNV platform, Pstates are 8-bit values. On POWER8 they are negatively numbered while on POWER9 they are positively numbered. Thus, on POWER9, the maximum number of pstates could be as high as 256. The current code interprets pstates as a signed 8-bit value. This

Re: ppc32: semctl fails

2017-12-06 Thread Joakim Tjernlund
On Wed, 2017-12-06 at 23:56 +0100, Andreas Schwab wrote: > > On Dez 06 2017, Joakim Tjernlund wrote: > > > st = semctl(sem, 0, IPC_STAT, &arg); > > This is not a valid use of IPC_STAT. The fourth argument must be a > object of type union semun (not a pointer), with the .buf member > pointi

Re: ppc32: semctl fails

2017-12-06 Thread Andreas Schwab
On Dez 06 2017, Joakim Tjernlund wrote: > st = semctl(sem, 0, IPC_STAT, &arg); This is not a valid use of IPC_STAT. The fourth argument must be a object of type union semun (not a pointer), with the .buf member pointing to the struct semid_ds object. Andreas. -- Andreas Schwab, sch...@li

[PATCH 32/45] tools/testing/selftests: remove duplicate includes

2017-12-06 Thread Pravin Shedge
These duplicate includes have been found with scripts/checkincludes.pl but they have been removed manually to avoid removing false positives. Signed-off-by: Pravin Shedge --- tools/testing/selftests/gpio/gpio-mockup-chardev.c | 1 - tools/testing/selftests/powerpc/pmu/ebb/fork_cleanup_t

Re: [PATCH 0/3] tty/serial/ucc_uart: Adjustments for two functions

2017-12-06 Thread Timur Tabi
On 12/06/2017 03:10 PM, SF Markus Elfring wrote: Three update suggestions were taken into account from static source code analysis. Markus Elfring (3): Delete an error message for a failed memory allocation in ucc_uart_probe() Improve a size determination in ucc_uart_probe() Fix a typo

[PATCH 3/3] tty/serial/ucc_uart: Fix a typo in a comment line

2017-12-06 Thread SF Markus Elfring
From: Markus Elfring Date: Wed, 6 Dec 2017 21:56:28 +0100 Add a missing character in a function name of this description. Signed-off-by: Markus Elfring --- drivers/tty/serial/ucc_uart.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/tty/serial/ucc_uart.c b/drivers/

[PATCH 2/3] tty/serial/ucc_uart: Improve a size determination in ucc_uart_probe()

2017-12-06 Thread SF Markus Elfring
From: Markus Elfring Date: Wed, 6 Dec 2017 21:45:32 +0100 Replace the specification of a data structure by a pointer dereference as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style convention. This issue was dete

[PATCH 1/3] tty/serial/ucc_uart: Delete an error message for a failed memory allocation in ucc_uart_probe()

2017-12-06 Thread SF Markus Elfring
From: Markus Elfring Date: Wed, 6 Dec 2017 21:41:14 +0100 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/tty/serial/ucc_uart.c | 4 +--- 1 file changed, 1 insertion(+),

[PATCH 0/3] tty/serial/ucc_uart: Adjustments for two functions

2017-12-06 Thread SF Markus Elfring
From: Markus Elfring Date: Wed, 6 Dec 2017 22:06:44 +0100 Three update suggestions were taken into account from static source code analysis. Markus Elfring (3): Delete an error message for a failed memory allocation in ucc_uart_probe() Improve a size determination in ucc_uart_probe() Fix a

Re: [PATCH] SB600 for the Nemo board has non-zero devices on non-root bus

2017-12-06 Thread Bjorn Helgaas
On Wed, Dec 06, 2017 at 11:57:03PM +1100, Michael Ellerman wrote: > Christian Zigotzky writes: > ... > > > > Hi Olof, > > > > Many thanks for your patch! :-) The RC2 of kernel 4.15 boots without any > > problems on my P.A. Semi Nemo board (A-EON AmigaOne X1000). I don’t need > > the additional b

Re: [PATCH v2] powerpc/powernv: Add pci_reset_phbs parameter to issue a PHB reset

2017-12-06 Thread Guilherme G. Piccoli
On 11/21/2017 12:35 AM, Balbir Singh wrote: > On Thu, Nov 16, 2017 at 11:14 PM, Guilherme G. Piccoli > wrote: >> On 11/16/2017 01:49 AM, Balbir Singh wrote: >>> On Thu, Oct 26, 2017 at 2:27 AM, Guilherme G. Piccoli >>> wrote: During a kdump kernel boot in PowerPC, we request a reset of the P

[PATCH] [powerpc-next] Fix powerpc64 alignment of .toc section in kernel modules

2017-12-06 Thread Desnes Augusto Nunes do Rosario
powerpc64 gcc can generate code that offsets an address, to access part of an object in memory. If the address is a -mcmodel=medium toc pointer relative address then code like the following is possible. addis r9,r2,var@toc@ha ld r3,var@toc@l(r9) ld r4,(var+8)@toc@l(r9) This works fine so long

ppc32: semctl fails

2017-12-06 Thread Joakim Tjernlund
This test, taken from perl Configure, fails on my ppc32, should it? semctl IPC_STAT failed: errno = Bad Address is what I get, kernel is 4.1.43 - #include #include #include #include #ifndef S_IRUSR # ifdef S_IREAD #define S_IRUSR S_IREAD #define S_IWUSR S_IWRITE #define S

Re: [PATCH v3] ppc64 boot: Wait for boot cpu to show up if nr_cpus limit is about to hit.

2017-12-06 Thread Guilherme G. Piccoli
On 11/29/2017 10:40 AM, Thadeu Lima de Souza Cascardo wrote: > Ping. Any further reviews/comments? Or any chance of this getting > applied soon? > > Thanks. > Cascardo. > Cross-referencing 2 threads on the same topic: https://lists.ozlabs.org/pipermail/linuxppc-dev/2017-November/166328.html L

Re: what is the state about "[v2] ppc64 boot: Wait for boot cpu to show up if nr_cpus limit is about to hit"

2017-12-06 Thread Guilherme G. Piccoli
On 11/28/2017 04:03 AM, Liu ping fan wrote: > Hi, > > I can not find the history about: > https://patchwork.ozlabs.org/patch/577193/ > > > Can we have this patch? > > Thanks, > Pingfan > Cross-referencing 2 threads on the same topic: https://patchwork.ozlabs.org/patch/834860 Cascardo recen

Re: [PATCH] SB600 for the Nemo board has non-zero devices on non-root bus

2017-12-06 Thread Olof Johansson
On Wed, Dec 6, 2017 at 4:44 AM, Michael Ellerman wrote: > Olof Johansson writes: > >> On Sat, Dec 02, 2017 at 01:54:41PM +0100, Christian Zigotzky wrote: >>> On 02 December 2017 at 00:27AM, Bjorn Helgaas wrote: >>> > >>> > Thanks for testing that. I'll merge a similar patch for v4.16. >>> > >>>

Re: Revert "powerpc: Do not call ppc_md.panic in fadump panic notifier"

2017-12-06 Thread Michael Ellerman
On Mon, 2017-12-04 at 05:27:25 UTC, David Gibson wrote: > This reverts commit a3b2cb30f252b21a6f962e0dd107c8b897ca65e4. > > The earlier patch tried to fix problems with panic on powerpc in > certain circumstances, where some output from the generic panic code > was being dropped. > > Unfortunatel

Re: powerpc/perf: Fix oops when grouping different pmu events

2017-12-06 Thread Michael Ellerman
On Thu, 2017-11-30 at 08:33:22 UTC, Ravi Bangoria wrote: > When user tries to group imc (In-Memory Collections) event with > normal event, (sometime) kernel crashes with following log: > > Faulting instruction address: 0x > [link register ] c010ce88 power_check_constraint

Re: [PATCH] SB600 for the Nemo board has non-zero devices on non-root bus

2017-12-06 Thread Michael Ellerman
Christian Zigotzky writes: ... > > Hi Olof, > > Many thanks for your patch! :-) The RC2 of kernel 4.15 boots without any > problems on my P.A. Semi Nemo board (A-EON AmigaOne X1000). I don’t need > the additional boot argument 'pci=pcie_scan_all' anymore. > > Is it possible to merge it via the p

Re: [PATCH] SB600 for the Nemo board has non-zero devices on non-root bus

2017-12-06 Thread Michael Ellerman
Olof Johansson writes: > On Sat, Dec 02, 2017 at 01:54:41PM +0100, Christian Zigotzky wrote: >> On 02 December 2017 at 00:27AM, Bjorn Helgaas wrote: >> > >> > Thanks for testing that. I'll merge a similar patch for v4.16. >> > >> > I don't think using "pci=pcie_scan_all" is really an acceptabl

Re: [PATCH] cpufreq: powernv: Define methods to parse positive & negative pstates

2017-12-06 Thread Michael Ellerman
Gautham R Shenoy writes: > Hi Michael, > > On Wed, Dec 06, 2017 at 09:54:27PM +1100, Michael Ellerman wrote: >> Shilpasri G Bhat writes: >> >> > From: "Gautham R. Shenoy" >> > >> > Pstates are 8bit values but on POWER8 they are negative and on POWER9 >> > they are positive. This patch adds hel

[PATCH] powerpc/xmon: Don't print hashed pointers in xmon

2017-12-06 Thread Michael Ellerman
Since commit ad67b74d2469 ("printk: hash addresses printed with %p") pointers printed with %p are hashed, ie. you don't see the actual pointer value but rather a cryptographic hash of its value. In xmon we want to see the actual pointer values, because xmon is a debugger, so replace %p with %px wh

[PATCH v3 2/2] cxl: read PHB indications from the device tree

2017-12-06 Thread Philippe Bergheaud
Configure the P9 XSL_DSNCTL register with PHB indications found in the device tree, or else use legacy hard-coded values. Signed-off-by: Philippe Bergheaud --- Changelog: v2: New patch. Use the new device tree property "ibm,phb-indications". v3: No change. This patch depends on the following s

[PATCH v3 1/2] powerpc/powernv: Enable tunneled operations

2017-12-06 Thread Philippe Bergheaud
P9 supports PCI tunneled operations (atomics and as_notify). This patch adds support for tunneled operations on powernv, with a new API, to be called by device drivers: pnv_pci_get_tunnel_ind() Tell driver the 16-bit ASN indication used by kernel. pnv_pci_set_tunnel_bar() Tell kernel the Tu

Re: [PATCH] cpufreq: powernv: Define methods to parse positive & negative pstates

2017-12-06 Thread Gautham R Shenoy
Hi Michael, On Wed, Dec 06, 2017 at 09:54:27PM +1100, Michael Ellerman wrote: > Shilpasri G Bhat writes: > > > From: "Gautham R. Shenoy" > > > > Pstates are 8bit values but on POWER8 they are negative and on POWER9 > > they are positive. This patch adds helper routines to differentiate > > the

[PATCH] SB600 for the Nemo board has non-zero devices on non-root bus

2017-12-06 Thread Christian Zigotzky
On 06 December 2017 at 09:37AM, Christian Zigotzky wrote: > On 03 December 2017 at 10:43AM, Christian Zigotzky wrote: > > > > On 3. Dec 2017, at 00:02, Olof Johansson wrote: > >> > >> Typo, should be ';', not ':'. I obviously didn't even try compiling this. :) > >> > >> > >> -Olof > > > > Hi Ol

Re: [PATCH] powerpc/40x: acadia: Fix the 'interrupt-parent' property

2017-12-06 Thread Michael Ellerman
Fabio Estevam writes: > Hi Michael, > > On Mon, Dec 4, 2017 at 2:45 AM, Michael Ellerman wrote: > >> Will pick it up for 4.16. > > Just realized that there is already a fix for this in linux-next: > > commit 3d2d4339cc326c427638daa67e264dd455ee1899 > Author: Geert Uytterhoeven > Date: Fri Jun

Re: [PATCH] cpufreq: powernv: Define methods to parse positive & negative pstates

2017-12-06 Thread Michael Ellerman
Shilpasri G Bhat writes: > From: "Gautham R. Shenoy" > > Pstates are 8bit values but on POWER8 they are negative and on POWER9 > they are positive. This patch adds helper routines to differentiate > the sign to read the correct pstate value. This sounds like it could be a bad bug, but I can't r

Re: [PATCHv4 5/6] symbol lookup: introduce dereference_symbol_descriptor()

2017-12-06 Thread Sergey Senozhatsky
On (12/06/17 11:32), Petr Mladek wrote: [..] > > diff --git a/Documentation/printk-formats.txt > > b/Documentation/printk-formats.txt > > index aa0a776c817a..02745028e909 100644 > > --- a/Documentation/printk-formats.txt > > +++ b/Documentation/printk-formats.txt > > @@ -61,41 +61,31 @@ Symbols/Fu

Re: [PATCHv4 5/6] symbol lookup: introduce dereference_symbol_descriptor()

2017-12-06 Thread Petr Mladek
On Wed 2017-12-06 13:36:49, Sergey Senozhatsky wrote: > Hello, > > so we got a number of build-error reports [somehow I > thought 0day has compile tested the patches already; well, I > was wrong] basically on congifs that have no KALLSYMS. > > > Petr, can we replace 0006 with the following

Re: [linux-next][41e83b9][gcc 4.8.5] make modules fail at net/netfilter/xt_bpf.ko

2017-12-06 Thread Abdul Haleem
On Tue, 2017-12-05 at 18:06 +, Al Viro wrote: > On Tue, Dec 05, 2017 at 08:13:11PM +0530, Abdul Haleem wrote: > > Hi Al Viro, > > > > Today's next kernel build failed with commit 41e83b9: fix "netfilter: > > xt_bpf: Fix XT_BPF_MODE_FD_PINNED > > > > Machine Type: Power 7 > > kernel version: 4

[PATCH] SB600 for the Nemo board has non-zero devices on non-root bus

2017-12-06 Thread Christian Zigotzky
On 03 December 2017 10:43AM, Christian Zigotzky wrote: > > On 3. Dec 2017, at 00:02, Olof Johansson wrote: >> >>> On Sat, Dec 2, 2017 at 3:00 PM, Olof Johansson wrote: >>> >>> The below patch, together with Bjorn's, should do it. Christian, can you test >>> and report back? >>> >>> I'm guessin

[PATCH v3] powerpc/64s: ISAv3 initialize MMU registers before setting partition table

2017-12-06 Thread Nicholas Piggin
kexec can leave MMU registers set when booting into a new kernel, PIDR in particular. The boot sequence does not zero PIDR, so it only gets set when CPUs first switch to a userspace processes (until then it's running a kernel thread with effective PID = 0). This leaves a window where a process tab