Re: [PATCH] powerpc: Inline setup_kup()

2020-12-14 Thread Christophe Leroy
Le 15/12/2020 à 02:42, Michael Ellerman a écrit : Christophe Leroy writes: Le 14/12/2020 à 13:30, Michael Ellerman a écrit : setup_kup() is used by both 64-bit and 32-bit code. However on 64-bit it must not be __init, because it's used for CPU hotplug, whereas on 32-bit it should be __init

[PATCH] powerpc/boot: Fix build of dts/fsl

2020-12-14 Thread Michael Ellerman
The lkp robot reported that some configs fail to build, for example mpc85xx_smp_defconfig, with: cc1: fatal error: opening output file arch/powerpc/boot/dts/fsl/.mpc8540ads.dtb.dts.tmp: No such file or directory This bisects to: cc8a51ca6f05 ("kbuild: always create directories of targets")

Re: [PATCH] KVM: PPC: fix comparison to bool warning

2020-12-14 Thread Paul Mackerras
On Sat, Nov 07, 2020 at 11:49:38PM +0800, xiakaixu1...@gmail.com wrote: > From: Kaixu Xia > > Fix the following coccicheck warning: > > ./arch/powerpc/kvm/booke.c:503:6-16: WARNING: Comparison to bool > ./arch/powerpc/kvm/booke.c:505:6-17: WARNING: Comparison to bool >

Re: [PATCH] KVM: PPC: Book3S: Assign boolean values to a bool variable

2020-12-14 Thread Paul Mackerras
On Sat, Nov 07, 2020 at 02:26:22PM +0800, xiakaixu1...@gmail.com wrote: > From: Kaixu Xia > > Fix the following coccinelle warnings: > > ./arch/powerpc/kvm/book3s_xics.c:476:3-15: WARNING: Assignment of 0/1 to bool > variable > ./arch/powerpc/kvm/book3s_xics.c:504:3-15: WARNING: Assignment of

Re: [PATCH v2 1/1] powerpc/kvm: Fix mask size for emulated msgsndp

2020-12-14 Thread Paul Mackerras
On Tue, Dec 08, 2020 at 06:57:08PM -0300, Leonardo Bras wrote: > According to ISAv3.1 and ISAv3.0b, the msgsndp is described to split RB in: > msgtype <- (RB) 32:36 > payload <- (RB) 37:63 > t <- (RB) 57:63 > > The current way of getting 'msgtype', and 't' is missing their MSB: > msgtype:

Re: [PATCH] powerpc: Inline setup_kup()

2020-12-14 Thread Michael Ellerman
Christophe Leroy writes: > Le 14/12/2020 à 13:30, Michael Ellerman a écrit : >> setup_kup() is used by both 64-bit and 32-bit code. However on 64-bit >> it must not be __init, because it's used for CPU hotplug, whereas on >> 32-bit it should be __init because it calls setup_kuap/kuep() which >>

[powerpc:next] BUILD SUCCESS 44b4c4450f8d31296ba633d74be753a85fd627bd

2020-12-14 Thread kernel test robot
allyesconfig powerpc allmodconfig powerpc allnoconfig i386 randconfig-a001-20201214 i386 randconfig-a004-20201214 i386 randconfig-a003-20201214 i386 randconfig-a002-20201214 i386

[powerpc:merge] BUILD SUCCESS 3a6a31edf52760dbc480853232f7bae9867f8e72

2020-12-14 Thread kernel test robot
allyesconfig mips allmodconfig powerpc allyesconfig powerpc allmodconfig powerpc allnoconfig i386 randconfig-a001-20201214 i386

[powerpc:next-test] BUILD SUCCESS bd9161ccbf25eef78b073c239621dfd298222a64

2020-12-14 Thread kernel test robot
allyesconfig mips allmodconfig powerpc allyesconfig powerpc allmodconfig powerpc allnoconfig i386 randconfig-a001-20201214 i386 randconfig

[PATCH] selftests/powerpc: Add test to check if TM is disabled when it must be

2020-12-14 Thread Gustavo Romero
Add a TM test to check that when TM is not advertised by the OS (is disabled) a transaction can not really be started and generates a SIGILL, which is the right behavior in that case. Signed-off-by: Gustavo Romero --- tools/testing/selftests/powerpc/tm/.gitignore | 1 +

Re: [PATCH 2/3] kbuild: LD_VERSION redenomination

2020-12-14 Thread Will Deacon
On Sun, Dec 13, 2020 at 01:54:30AM +0900, Masahiro Yamada wrote: > Commit ccbef1674a15 ("Kbuild, lto: add ld-version and ld-ifversion > macros") introduced scripts/ld-version.sh for GCC LTO. > > At that time, this script handled 5 version fields because GCC LTO > needed the downstream binutils.

Re: [PATCH v12 00/31] Speculative page faults

2020-12-14 Thread Joel Fernandes
On Mon, Dec 14, 2020 at 10:36:29AM +0100, Laurent Dufour wrote: > Le 14/12/2020 à 03:03, Joel Fernandes a écrit : > > On Tue, Jul 07, 2020 at 01:31:37PM +0800, Chinwen Chang wrote: > > [..] > > > > > Hi Laurent, > > > > > > > > > > We merged SPF v11 and some patches from v12 into our platforms.

Re: [RFC HACK PATCH] PCI: dwc: layerscape: Hack around enumeration problems with Honeycomb LX2K

2020-12-14 Thread Rob Herring
On Mon, Dec 14, 2020 at 4:43 AM Daniel Thompson wrote: > > On Fri, Dec 11, 2020 at 05:05:58PM +, Daniel Thompson wrote: > > On Fri, Dec 11, 2020 at 08:37:40AM -0600, Rob Herring wrote: > > > On Fri, Dec 11, 2020 at 6:15 AM Daniel Thompson > > > > BTW I noticed many other pcie-designware

[PATCH -next] tty/hvc/hvsi_lib: convert comma to semicolon

2020-12-14 Thread Zheng Yongjun
Replace a comma between expression statements by a semicolon. Signed-off-by: Zheng Yongjun --- drivers/tty/hvc/hvsi_lib.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/tty/hvc/hvsi_lib.c b/drivers/tty/hvc/hvsi_lib.c index 09289c8154ae..8c4ad495f819 100644 ---

Re: [PATCH] powerpc: Inline setup_kup()

2020-12-14 Thread Christophe Leroy
Le 14/12/2020 à 13:30, Michael Ellerman a écrit : setup_kup() is used by both 64-bit and 32-bit code. However on 64-bit it must not be __init, because it's used for CPU hotplug, whereas on 32-bit it should be __init because it calls setup_kuap/kuep() which are __init. We worked around that

[PATCH] powerpc: Inline setup_kup()

2020-12-14 Thread Michael Ellerman
setup_kup() is used by both 64-bit and 32-bit code. However on 64-bit it must not be __init, because it's used for CPU hotplug, whereas on 32-bit it should be __init because it calls setup_kuap/kuep() which are __init. We worked around that problem in the past by marking it __ref, see commit

Re: [PATCH 2/2] powerpc/64s: Trim offlined CPUs from mm_cpumasks

2020-12-14 Thread Nicholas Piggin
Excerpts from Michael Ellerman's message of December 14, 2020 8:43 pm: > Nicholas Piggin writes: >> Excerpts from Geert Uytterhoeven's message of December 10, 2020 7:06 pm: >>> Hi Nicholas, >>> >>> On Fri, Nov 20, 2020 at 4:01 AM Nicholas Piggin wrote: When offlining a CPU,

Re: [PATCH v2] powerpc/book3s/kup: Mark the kuap/keup function non __init

2020-12-14 Thread Sachin Sant
> On 14-Dec-2020, at 1:31 PM, Aneesh Kumar K.V > wrote: > > The kernel call these functions on cpu online and hence they should > not be marked __init. > > Fixes: 3b47b7549ead ("powerpc/book3s64/kuap: Move KUAP related function > outside radix") > Signed-off-by: Aneesh Kumar K.V > — This

Re: [RFC HACK PATCH] PCI: dwc: layerscape: Hack around enumeration problems with Honeycomb LX2K

2020-12-14 Thread Daniel Thompson
On Fri, Dec 11, 2020 at 05:05:58PM +, Daniel Thompson wrote: > On Fri, Dec 11, 2020 at 08:37:40AM -0600, Rob Herring wrote: > > On Fri, Dec 11, 2020 at 6:15 AM Daniel Thompson > > > BTW I noticed many other pcie-designware drivers take advantage > > > of a function called

Re: [PATCH 2/2] powerpc/64s: Trim offlined CPUs from mm_cpumasks

2020-12-14 Thread Michael Ellerman
Nicholas Piggin writes: > Excerpts from Geert Uytterhoeven's message of December 10, 2020 7:06 pm: >> Hi Nicholas, >> >> On Fri, Nov 20, 2020 at 4:01 AM Nicholas Piggin wrote: >>> >>> When offlining a CPU, powerpc/64s does not flush TLBs, rather it just >>> leaves the CPU set in mm_cpumasks, so

Re: [PATCH v12 00/31] Speculative page faults

2020-12-14 Thread Laurent Dufour
Le 14/12/2020 à 03:03, Joel Fernandes a écrit : On Tue, Jul 07, 2020 at 01:31:37PM +0800, Chinwen Chang wrote: [..] Hi Laurent, We merged SPF v11 and some patches from v12 into our platforms. After several experiments, we observed SPF has obvious improvements on the launch time of

Re: [PATCH v1 1/2] KVM: PPC: Book3S HV: Add support for H_RPT_INVALIDATE (nested case only)

2020-12-14 Thread David Gibson
On Fri, Dec 11, 2020 at 04:03:36PM +0530, Bharata B Rao wrote: > On Mon, Oct 19, 2020 at 04:56:41PM +0530, Bharata B Rao wrote: > > Implements H_RPT_INVALIDATE hcall and supports only nested case > > currently. > > > > A KVM capability KVM_CAP_RPT_INVALIDATE is added to indicate the > > support

Re: [PATCH v1 1/2] KVM: PPC: Book3S HV: Add support for H_RPT_INVALIDATE (nested case only)

2020-12-14 Thread David Gibson
On Fri, Dec 11, 2020 at 04:27:44PM +1100, Paul Mackerras wrote: > On Fri, Dec 11, 2020 at 12:16:39PM +1100, David Gibson wrote: > > On Thu, Dec 10, 2020 at 09:54:18AM +0530, Bharata B Rao wrote: > > > On Wed, Dec 09, 2020 at 03:15:42PM +1100, Paul Mackerras wrote: > > > > On Mon, Oct 19, 2020 at

[PATCH kernel v3] powerpc/kuap: Restore AMR after replaying soft interrupts

2020-12-14 Thread Alexey Kardashevskiy
Since de78a9c "powerpc: Add a framework for Kernel Userspace Access Protection", user access helpers call user_{read|write}_access_{begin|end} when user space access is allowed. 890274c "powerpc/64s: Implement KUAP for Radix MMU" made the mentioned helpers program a AMR special register to allow

[PATCH v2] powerpc/book3s/kup: Mark the kuap/keup function non __init

2020-12-14 Thread Aneesh Kumar K.V
The kernel call these functions on cpu online and hence they should not be marked __init. Fixes: 3b47b7549ead ("powerpc/book3s64/kuap: Move KUAP related function outside radix") Signed-off-by: Aneesh Kumar K.V --- arch/powerpc/mm/book3s64/pkeys.c | 4 ++-- 1 file changed, 2 insertions(+), 2