* Re: [1/2] powerpc/xive: Move definition of ESB bits
From: Michael Ellerman @ 2018-01-22 3:34 UTC (permalink / raw)
To: Benjamin Herrenschmidt, linuxppc-dev
In-Reply-To: <20180112023928.10926-1-benh@kernel.crashing.org>
On Fri, 2018-01-12 at 02:39:27 UTC, Benjamin Herrenschmidt wrote:
> >From xive.h to xive-regs.h since it's a HW register definition
> and it can be used from assembly
>
> Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Series applied to powerpc next, thanks.
https://git.kernel.org/powerpc/c/12c1f339cd49119e39063ae67f02d9
cheers
^ permalink raw reply
* Re: powerpc: restore alphabetic order in Kconfig
From: Michael Ellerman @ 2018-01-22 3:34 UTC (permalink / raw)
To: Christophe Leroy, Benjamin Herrenschmidt, Paul Mackerras,
Scott Wood, Balbir Singh
Cc: linuxppc-dev, linux-kernel
In-Reply-To: <20180104153525.A03F56E5E6@localhost.localdomain>
On Thu, 2018-01-04 at 15:35:25 UTC, Christophe Leroy wrote:
> This patch restores the alphabetic order which was broken by
> commit 1e0fc9d1eb2b0 ("powerpc/Kconfig: Enable STRICT_KERNEL_RWX
> for some configs")
>
> Fixes: 1e0fc9d1eb2b0 ("powerpc/Kconfig: Enable STRICT_KERNEL_RWX for some configs")
> Signed-off-by: Christophe Leroy <christophe.leroy@c-s.fr>
> Acked-by: Balbir Singh <bsingharora@gmail.com>
Applied to powerpc next, thanks.
https://git.kernel.org/powerpc/c/4ec591e51a4b0aedb6c7f1a8cd722a
cheers
^ permalink raw reply
* Re: [V7] cxl: Add support for ASB_Notify on POWER9
From: Michael Ellerman @ 2018-01-22 3:34 UTC (permalink / raw)
To: Christophe Lombard, linuxppc-dev, fbarrat, vaibhav,
andrew.donnellan
In-Reply-To: <1515660925-10026-1-git-send-email-clombard@linux.vnet.ibm.com>
On Thu, 2018-01-11 at 08:55:25 UTC, Christophe Lombard wrote:
> The POWER9 core supports a new feature: ASB_Notify which requires the
> support of the Special Purpose Register: TIDR.
>
> The ASB_Notify command, generated by the AFU, will attempt to
> wake-up the host thread identified by the particular LPID:PID:TID.
>
> This patch assign a unique TIDR (thread id) for the current thread which
> will be used in the process element entry.
>
> Signed-off-by: Christophe Lombard <clombard@linux.vnet.ibm.com>
> Reviewed-by: Philippe Bergheaud <felix@linux.vnet.ibm.com>
> Acked-by: Frederic Barrat <fbarrat@linux.vnet.ibm.com>
> Reviewed-by: Vaibhav Jain <vaibhav@linux.vnet.ibm.com>
> Acked-by: Andrew Donnellan <andrew.donnellan@au1.ibm.com>
Applied to powerpc next, thanks.
https://git.kernel.org/powerpc/c/b1db551324f72fa14ad82ca31237a7
cheers
^ permalink raw reply
* Re: [1/2] powerpc/tm: Fix endianness flip on trap
From: Michael Ellerman @ 2018-01-22 3:34 UTC (permalink / raw)
To: Gustavo Romero, linuxppc-dev; +Cc: cyrilbur
In-Reply-To: <1514762446-1323-1-git-send-email-gromero@linux.vnet.ibm.com>
On Sun, 2017-12-31 at 23:20:45 UTC, Gustavo Romero wrote:
> Currently it's possible that a thread on PPC64 LE has its endianness
> flipped inadvertently to Big-Endian resulting in a crash once the process
> is back from the signal handler.
>
> If giveup_all() is called when regs->msr has the bits MSR.FP and MSR.VEC
> disabled (and hence MSR.VSX disabled too) it returns without calling
> check_if_tm_restore_required() which copies regs->msr to ckpt_regs->msr if
> the process caught a signal whilst in transactional mode. Then once in
> setup_tm_sigcontexts() MSR from ckpt_regs.msr is used, but since
> check_if_tm_restore_required() was not called previuosly, gp_regs[PT_MSR]
> gets a copy of invalid MSR bits as MSR in ckpt_regs was not updated from
> regs->msr and so is zeroed. Later when leaving the signal handler once in
> sys_rt_sigreturn() the TS bits of gp_regs[PT_MSR] are checked to determine
> if restore_tm_sigcontexts() must be called to pull in the correct MSR state
> into the user context. Because TS bits are zeroed
> restore_tm_sigcontexts() is never called and MSR restored from the user
> context on returning from the signal handler has the MSR.LE (the endianness
> bit) forced to zero (Big-Endian). That leads, for instance, to 'nop' being
> treated as an illegal instruction in the following sequence:
>
> tbegin.
> beq 1f
> trap
> tend.
> 1: nop
>
> on PPC64 LE machines and the process dies just after returning from the
> signal handler.
>
> PPC64 BE is also affected but in a subtle way since forcing Big-Endian on
> a BE machine does not change the endianness.
>
> This commit fixes the issue described above by ensuring that once in
> setup_tm_sigcontexts() the MSR used is from regs->msr instead of from
> ckpt_regs->msr and by ensuring that we pull in only the MSR.FP, MSR.VEC,
> and MSR.VSX bits from ckpt_regs->msr.
>
> The fix was tested both on LE and BE machines and no regression regarding
> the powerpc/tm selftests was observed.
>
> Signed-off-by: Gustavo Romero <gromero@linux.vnet.ibm.com>
Series applied to powerpc next, thanks.
https://git.kernel.org/powerpc/c/1c200e63d055ec0125e44a5e386b9b
cheers
^ permalink raw reply
* Re: [1/2] powerpc: Fix old-style function definition
From: Michael Ellerman @ 2018-01-22 3:34 UTC (permalink / raw)
To: Mathieu Malaterre
Cc: Mathieu Malaterre, linux-kernel, Paul Mackerras, linuxppc-dev
In-Reply-To: <20171226141233.19444-1-malat@debian.org>
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain, Size: 573 bytes --]
On Tue, 2017-12-26 at 14:12:33 UTC, Mathieu Malaterre wrote:
> Fix warnings such as:
>
> arch/powerpc/platforms/powermac/backlight.c: In function ‘pmac_backlight_get_legacy_brightness’:
> arch/powerpc/platforms/powermac/backlight.c:189:5: error: old-style function definition [-Werror=old-style-definition]
> int pmac_backlight_get_legacy_brightness()
> ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>
> Signed-off-by: Mathieu Malaterre <malat@debian.org>
Series applied to powerpc next, thanks.
https://git.kernel.org/powerpc/c/4a7b8a499724c98493023e1e3009aa
cheers
^ permalink raw reply
* Re: powerpc/xmon: Do not compute/store the major opcode
From: Michael Ellerman @ 2018-01-22 3:34 UTC (permalink / raw)
To: Mathieu Malaterre
Cc: Mathieu Malaterre, linux-kernel, Paul Mackerras, linuxppc-dev
In-Reply-To: <20171226132548.22177-1-malat@debian.org>
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain, Size: 671 bytes --]
On Tue, 2017-12-26 at 13:25:47 UTC, Mathieu Malaterre wrote:
> In commit 5b102782c7f4 ("powerpc/xmon: Enable disassembly files (compilation
> changes)") usage of variable `op` has been removed. Completely remove opcode
> computation since not used anymore.
>
> Fix fatal warning:
>
> arch/powerpc/xmon/ppc-dis.c: In function ‘lookup_powerpc’:
> arch/powerpc/xmon/ppc-dis.c:96:17: error: variable ‘op’ set but not used [-Werror=unused-but-set-variable]
> unsigned long op;
> ^~
>
> Signed-off-by: Mathieu Malaterre <malat@debian.org>
Applied to powerpc next, thanks.
https://git.kernel.org/powerpc/c/104d55ae4df928c205268c1eb83b80
cheers
^ permalink raw reply
* Re: powerpc/xive: Properly use static keyword for inline function
From: Michael Ellerman @ 2018-01-22 3:34 UTC (permalink / raw)
To: Mathieu Malaterre
Cc: Mathieu Malaterre, linux-kernel, Paul Mackerras,
Cédric Le Goater, linuxppc-dev
In-Reply-To: <20171226130020.23626-1-malat@debian.org>
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain, Size: 609 bytes --]
On Tue, 2017-12-26 at 13:00:17 UTC, Mathieu Malaterre wrote:
> Fix fatal warning during compilation:
>
> In file included from arch/powerpc/xmon/xmon.c:54:0:
> ./arch/powerpc/include/asm/xive.h:157:20: error: no previous prototype for ‘xive_smp_prepare_cpu’ [-Werror=missing-prototypes]
> extern inline int xive_smp_prepare_cpu(unsigned int cpu) { return -EINVAL; }
> ^
>
> Signed-off-by: Mathieu Malaterre <malat@debian.org>
> Reviewed-by: Cédric Le Goater <clg@kaod.org>
Applied to powerpc next, thanks.
https://git.kernel.org/powerpc/c/38833faa112c10dd6e3d888f8b53de
cheers
^ permalink raw reply
* Re: [1/2] powerpc: System reset avoid interleaving oops using die synchronisation
From: Michael Ellerman @ 2018-01-22 3:34 UTC (permalink / raw)
To: Nicholas Piggin, linuxppc-dev; +Cc: Nicholas Piggin, David Gibson
In-Reply-To: <20171223164923.10587-2-npiggin@gmail.com>
On Sat, 2017-12-23 at 16:49:22 UTC, Nicholas Piggin wrote:
> The die() oops path contains a serializing lock to prevent oops
> messages from being interleaved. In the case of a system reset
> initiated oops (e.g., qemu nmi command), __die was being called
> which lacks that synchronisation and oops reports could be
> interleaved across CPUs.
>
> A recent patch 4388c9b3a6ee7 ("powerpc: Do not send system reset
> request through the oops path") changed this to __die to avoid
> the debugger() call, but there is no real harm to calling it twice
> if the first time fell through. So go back to using die() here.
> This was observed to fix the problem.
>
> Fixes: 4388c9b3a6ee7 ("powerpc: Do not send system reset request through the oops path")
> Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
> Reviewed-by: David Gibson <david@gibson.dropbear.id.au>
Applied to powerpc next, thanks.
https://git.kernel.org/powerpc/c/4552d128c26e0f0f27a5bd2fadc240
cheers
^ permalink raw reply
* Re: [v3] powerpc/64s: Improve local TLB flush for boot and MCE on POWER9
From: Michael Ellerman @ 2018-01-22 3:34 UTC (permalink / raw)
To: Nicholas Piggin, linuxppc-dev; +Cc: Aneesh Kumar K . V, Nicholas Piggin
In-Reply-To: <20171223151550.30612-1-npiggin@gmail.com>
On Sat, 2017-12-23 at 15:15:50 UTC, Nicholas Piggin wrote:
> There are several cases outside the normal address space management
> where a CPU's entire local TLB is to be flushed:
>
> 1. Booting the kernel, in case something has left stale entries in
> the TLB (e.g., kexec).
>
> 2. Machine check, to clean corrupted TLB entries.
...
>
> The TLB flush is no longer called when restoring from deep idle states.
> This was not be done as a separate step because booting secondaries
> uses the same cpu_restore as idle restore, which needs the TLB flush.
>
> Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
Applied to powerpc next, thanks.
https://git.kernel.org/powerpc/c/d4748276ae14ce951a3254852dddc3
cheers
^ permalink raw reply
* Re: [v10, 02/17] powerpc/64: Add #defines for paca->soft_enabled flags
From: Michael Ellerman @ 2018-01-22 3:34 UTC (permalink / raw)
To: Madhavan Srinivasan
Cc: Madhavan Srinivasan, npiggin, paulus, anton, linuxppc-dev
In-Reply-To: <1513742157-28768-3-git-send-email-maddy@linux.vnet.ibm.com>
On Wed, 2017-12-20 at 03:55:42 UTC, Madhavan Srinivasan wrote:
> Two #defines, IRQ_ENABLED and IRQ_DISABLED, are added to be used when
> updating paca->soft_enabled. Replace the hardcoded values used when
> updating paca->soft_enabled with IRQ_[EN/DIS]ABLED. No logic change.
>
> Signed-off-by: Madhavan Srinivasan <maddy@linux.vnet.ibm.com>
Patchs 2-17 applied to powerpc next, thanks.
https://git.kernel.org/powerpc/c/c2e480ba822718190e58849b79a76d
cheers
^ permalink raw reply
* Re: [1/9] powerpc/powernv: Remove real mode access limit for early allocations
From: Michael Ellerman @ 2018-01-22 3:34 UTC (permalink / raw)
To: Nicholas Piggin, linuxppc-dev; +Cc: Aneesh Kumar K . V, Nicholas Piggin
In-Reply-To: <20171222111716.13101-2-npiggin@gmail.com>
On Fri, 2017-12-22 at 11:17:08 UTC, Nicholas Piggin wrote:
> This removes the RMA limit on powernv platform, which constrains
> early allocations such as PACAs and stacks. There are still other
> restrictions that must be followed, such as bolted SLB limits, but
> real mode addressing has no constraints.
>
> Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
Patches 1-6 applied to powerpc next, thanks.
https://git.kernel.org/powerpc/c/1513c33d7174e87a079cfa2666cb9a
cheers
^ permalink raw reply
* Re: powerpc/pseries/cpuidle: add polling idle for shared processor guests
From: Michael Ellerman @ 2018-01-22 3:34 UTC (permalink / raw)
To: Nicholas Piggin, linuxppc-dev; +Cc: kvm-ppc, Nicholas Piggin, linux-pm
In-Reply-To: <20171010071109.31207-1-npiggin@gmail.com>
On Tue, 2017-10-10 at 07:11:09 UTC, Nicholas Piggin wrote:
> For shared processor guests (e.g., KVM), add an idle polling mode rather
> than immediately returning to the hypervisor when the guest CPU goes
> idle.
>
> Test setup is a 2 socket POWER9 with 4 guests running, each with vCPUs
> equal to 1/2 of real of CPUs. Saturated each guest with tbench. Using
> polling idle gives about 1.4x throughput.
>
> Kernel compile speed was not changed significantly.
>
> Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
Applied to powerpc next, thanks.
https://git.kernel.org/powerpc/c/f2ac428e0edabbca41b9dfe9473a90
cheers
^ permalink raw reply
* Re: powerpc: dts: Remove leading 0x and 0s from bindings notation
From: Michael Ellerman @ 2018-01-22 3:34 UTC (permalink / raw)
To: Mathieu Malaterre, Rob Herring
Cc: Mark Rutland, devicetree, Mathieu Malaterre, linux-kernel,
Masahiro Yamada, Paul Mackerras, Andrew Morton, linuxppc-dev
In-Reply-To: <20171214165413.28163-1-malat@debian.org>
On Thu, 2017-12-14 at 16:54:00 UTC, Mathieu Malaterre wrote:
> Improve the DTS files by removing all the leading "0x" and zeros to fix the
> following dtc warnings:
>
> Warning (unit_address_format): Node /XXX unit name should not have leading "0x"
>
> and
>
> Warning (unit_address_format): Node /XXX unit name should not have leading 0s
>
> Converted using the following command:
>
> find . -type f \( -iname *.dts -o -iname *.dtsi \) -exec sed -E -i -e "s/@0x([0-9a-fA-F\.]+)\s?\{/@\L\1 \{/g" -e "s/@0+([0-9a-fA-F\.]+)\s?\{/@\L\1 \{/g" {} +
>
> For simplicity, two sed expressions were used to solve each warnings separately.
>
> To make the regex expression more robust a few other issues were resolved,
> namely setting unit-address to lower case, and adding a whitespace before the
> the opening curly brace:
>
> https://elinux.org/Device_Tree_Linux#Linux_conventions
>
> This is a follow up to commit 4c9847b7375a ("dt-bindings: Remove leading 0x from bindings notation")
>
> Reported-by: David Daney <ddaney@caviumnetworks.com>
> Suggested-by: Rob Herring <robh@kernel.org>
> Signed-off-by: Mathieu Malaterre <malat@debian.org>
Applied to powerpc next, thanks.
https://git.kernel.org/powerpc/c/600ecc1936be075f611f299755e2de
cheers
^ permalink raw reply
* Re: [v2] platform/powernv: Add debugfs interface for imc-mode and imc-command
From: Michael Ellerman @ 2018-01-22 3:34 UTC (permalink / raw)
To: Anju T Sudhakar; +Cc: maddy, linuxppc-dev, anju
In-Reply-To: <1513145394-6100-1-git-send-email-anju@linux.vnet.ibm.com>
On Wed, 2017-12-13 at 06:09:54 UTC, Anju T Sudhakar wrote:
> In memory Collection (IMC) counter pmu driver controls the ucode's execution
> state. At the system boot, IMC perf driver pause the ucode. Ucode state is
> changed to "running" only when any of the nest units are monitored or profiled
> using perf tool.
...
>
> Signed-off-by: Anju T Sudhakar <anju@linux.vnet.ibm.com>
Applied to powerpc next, thanks.
https://git.kernel.org/powerpc/c/684d984038aa2b8ec3ab4b9d314257
cheers
^ permalink raw reply
* Re: [1/3] powerpc/perf: Remove thread_imc_pmu global variable from
From: Michael Ellerman @ 2018-01-22 3:34 UTC (permalink / raw)
To: Anju T Sudhakar; +Cc: maddy, linuxppc-dev, anju
In-Reply-To: <1512971917-23851-2-git-send-email-anju@linux.vnet.ibm.com>
On Mon, 2017-12-11 at 05:58:35 UTC, Anju T Sudhakar wrote:
> Remove the global variable 'thread_imc_pmu', since it is not used in the code.
>
> Signed-off-by: Anju T Sudhakar <anju@linux.vnet.ibm.com>
> Reviewed-by: madhavan Srinivasan <maddy@linux.vnet.ibm.com>
Series applied to powerpc next, thanks.
https://git.kernel.org/powerpc/c/e7673818d9510043f11502e55afee5
cheers
^ permalink raw reply
* Re: [V2] powerpc/mm: Invalidate subpage_prot() system call on radix platforms
From: Michael Ellerman @ 2018-01-22 3:34 UTC (permalink / raw)
To: Anshuman Khandual, linuxppc-dev; +Cc: aneesh.kumar
In-Reply-To: <20171204054922.16075-1-khandual@linux.vnet.ibm.com>
On Mon, 2017-12-04 at 05:49:22 UTC, Anshuman Khandual wrote:
> Radix enabled platforms don't support subpage_prot() system calls. But
> at present the system call goes through without an error and fails
> later on while validating expected subpage accesses. Lets not allow
> the system call on powerpc radix platforms to begin with to prevent
> this confusion in user space.
>
> Signed-off-by: Anshuman Khandual <khandual@linux.vnet.ibm.com>
Applied to powerpc next, thanks.
https://git.kernel.org/powerpc/c/5b2b80714796073ec9e196c7aeec59
cheers
^ permalink raw reply
* Re: [V6, 1/4] powerpc/firmware: Add definitions for new drc-info firmware feature
From: Michael Ellerman @ 2018-01-22 3:34 UTC (permalink / raw)
To: Michael Bringmann, linuxppc-dev; +Cc: nfont, Michael Bringmann
In-Reply-To: <1d7f84eb-defc-a1ae-fd8b-d4c00a16aedb@linux.vnet.ibm.com>
On Fri, 2017-12-01 at 23:19:40 UTC, Michael Bringmann wrote:
> Firmware Features: Define new bit flag representing the presence of
> new device tree property "ibm,drc-info". The flag is used to tell
> the front end processor whether the Linux kernel supports the new
> property, and by the front end processor to tell the Linux kernel
> that the new property is present in the device tree.
>
> Signed-off-by: Michael Bringmann <mwb@linux.vnet.ibm.com>
Series applied to powerpc next, thanks.
https://git.kernel.org/powerpc/c/3f38000eda485f466153d37c4f9201
cheers
^ permalink raw reply
* Re: powerpc/radix: Remove trace_tlbie call from radix__flush_tlb_all
From: Michael Ellerman @ 2018-01-22 3:34 UTC (permalink / raw)
To: Mahesh J Salgaonkar, linuxppc-dev; +Cc: Aneesh Kumar K.V
In-Reply-To: <151203273555.16587.12957613472048945542.stgit@jupiter.in.ibm.com>
On Thu, 2017-11-30 at 09:05:54 UTC, Mahesh J Salgaonkar wrote:
> From: Mahesh Salgaonkar <mahesh@linux.vnet.ibm.com>
>
> radix__flush_tlb_all() is called only in kexec path in real mode and any
> tracepoints at this stage will make kexec to fail if enabled.
>
> To verify enable tlbie trace before kexec.
>
> $ echo 1 > /sys/kernel/debug/tracing/events/powerpc/tlbie/enable
> == kexec into new kernel and kexec fails.
>
> Fix this by not calling trace_tlbie from radix__flush_tlb_all().
>
> Fixes: 0428491cba92 ("powerpc/mm: Trace tlbie(l) instructions")
> Signed-off-by: Mahesh Salgaonkar <mahesh@linux.vnet.ibm.com>
> Acked-by: Balbir Singh <bsingharora@gmail.com>
Applied to powerpc next, thanks.
https://git.kernel.org/powerpc/c/8d81296cfcce89013a714feb8d2500
cheers
^ permalink raw reply
* Re: selftest/powerpc: Add additional option to mmap_bench test
From: Michael Ellerman @ 2018-01-22 3:34 UTC (permalink / raw)
To: Aneesh Kumar K.V, benh, paulus; +Cc: linuxppc-dev, Aneesh Kumar K.V
In-Reply-To: <20171128083639.20561-1-aneesh.kumar@linux.vnet.ibm.com>
On Tue, 2017-11-28 at 08:36:39 UTC, "Aneesh Kumar K.V" wrote:
> This patch adds --pgfault and --iterations options to mmap_bench test. With
> --pgfault we touch every page mapped. This helps in measuring impact in the
> page fault path with a patch series.
>
> Signed-off-by: Aneesh Kumar K.V <aneesh.kumar@linux.vnet.ibm.com>
Applied to powerpc next, thanks.
https://git.kernel.org/powerpc/c/be68fb64f763b7b6ddb202e0a931f4
cheers
^ permalink raw reply
* Re: powerpc/hash: Skip non initialized page size in init_hpte_page_sizes
From: Michael Ellerman @ 2018-01-22 3:34 UTC (permalink / raw)
To: Aneesh Kumar K.V, benh, paulus; +Cc: linuxppc-dev, Aneesh Kumar K.V
In-Reply-To: <20171128083440.18688-1-aneesh.kumar@linux.vnet.ibm.com>
On Tue, 2017-11-28 at 08:34:40 UTC, "Aneesh Kumar K.V" wrote:
> One of the easiest way to test config with 4K HPTE is to disable 64K hardware
> page size like below.
>
> int __init htab_dt_scan_page_sizes(unsigned long node,
>
> size -= 3; prop += 3;
> base_idx = get_idx_from_shift(base_shift);
> - if (base_idx < 0) {
> + if (base_idx < 0 || base_idx == MMU_PAGE_64K) {
> /* skip the pte encoding also */
> prop += lpnum * 2; size -= lpnum * 2;
>
> But then this results in error in other part of the code such as MPSS parsing
> where we look at 4K base page size and 64K actual page size support.
>
> This patch fix MPSS parsing by ignoring the actual page sizes marked
> unsupported. In reality this can happen only with a corrupt device tree. But it
> is good to tighten the error check.
>
> Signed-off-by: Aneesh Kumar K.V <aneesh.kumar@linux.vnet.ibm.com>
Applied to powerpc next, thanks.
https://git.kernel.org/powerpc/c/10527e808123f4b12db604993638b3
cheers
^ permalink raw reply
* Re: [1/2] powerpc/lib/code-patching: refactor patch_instruction()
From: Michael Ellerman @ 2018-01-22 3:34 UTC (permalink / raw)
To: Christophe Leroy, Benjamin Herrenschmidt, Paul Mackerras,
Scott Wood
Cc: linuxppc-dev, linux-kernel
In-Reply-To: <8703033cf68861aba6424b8c13694e9823f50f98.1511506950.git.christophe.leroy@c-s.fr>
On Fri, 2017-11-24 at 07:31:07 UTC, Christophe Leroy wrote:
> patch_instruction() uses almost the same sequence as
> __patch_instruction()
>
> This patch refactor it so that patch_instruction() uses
> __patch_instruction() instead of duplicating code.
>
> Signed-off-by: Christophe Leroy <christophe.leroy@c-s.fr>
> Acked-by: Balbir Singh <bsingharora@gmail.com>
Series applied to powerpc next, thanks.
https://git.kernel.org/powerpc/c/8cf4c05712f04a405f0dacebcca8f0
cheers
^ permalink raw reply
* Re: [v3] powerpc/powernv: Add ppc_pci_reset_phbs parameter to issue a PHB reset
From: Michael Ellerman @ 2018-01-22 3:34 UTC (permalink / raw)
To: Guilherme G. Piccoli, linuxppc-dev; +Cc: paulus, gpiccoli
In-Reply-To: <20171117185859.2756-1-gpiccoli@linux.vnet.ibm.com>
On Fri, 2017-11-17 at 18:58:59 UTC, "Guilherme G. Piccoli" wrote:
> During a kdump kernel boot in PowerPC, we request a reset of the PHBs
> to the FW. It makes sense, since if we are booting a kdump kernel it
> means we had some trouble before and we cannot rely in the adapters'
> health; they could be in a bad state, hence the reset is needed.
>
> But this reset is useful not only in kdump - there are situations,
> specially when debugging drivers, that we could break an adapter in
> a way it requires such reset. One can tell to just go ahead and
> reboot the machine, but happens that many times doing kexec is much
> faster, and so preferable than a full power cycle.
>
> This patch adds the ppc_pci_reset_phbs parameter to perform such reset.
>
> Signed-off-by: Guilherme G. Piccoli <gpiccoli@linux.vnet.ibm.com>
Applied to powerpc next, thanks.
https://git.kernel.org/powerpc/c/45baee1416a0086dccaa8a94c70e67
cheers
^ permalink raw reply
* Re: [4/4] cpuidle/powernv: avoid double irq enable coming out of idle
From: Michael Ellerman @ 2018-01-22 3:34 UTC (permalink / raw)
To: Nicholas Piggin, linuxppc-dev; +Cc: Nicholas Piggin
In-Reply-To: <20171116160052.18672-5-npiggin@gmail.com>
On Thu, 2017-11-16 at 16:00:52 UTC, Nicholas Piggin wrote:
> Since e1689795a7 ("cpuidle: Add common time keeping and irq enabling"),
> cpuidle drivers are expected to return from ->enter with irqs disabled.
>
> Update the cpuidle-powernv snooze and cede loops to disable irqs before
> returning.
>
> Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
Applied to powerpc next, thanks.
https://git.kernel.org/powerpc/c/ced54c08d8e4060d59c10629ea5a4c
cheers
^ permalink raw reply
* Re: [3/4] cpuidle/powernv: avoid double irq enable coming out of idle
From: Michael Ellerman @ 2018-01-22 3:34 UTC (permalink / raw)
To: Nicholas Piggin, linuxppc-dev; +Cc: Nicholas Piggin
In-Reply-To: <20171116160052.18672-4-npiggin@gmail.com>
On Thu, 2017-11-16 at 16:00:51 UTC, Nicholas Piggin wrote:
> Since e1689795a7 ("cpuidle: Add common time keeping and irq enabling"),
> cpuidle drivers are expected to return from ->enter with irqs disabled.
>
> Update the cpuidle-powernv snooze loop to disable irqs before returning.
>
> Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
Applied to powerpc next, thanks.
https://git.kernel.org/powerpc/c/f1343d044629f11e7d63ef1a07edb1
cheers
^ permalink raw reply
* Re: [1/4] powerpc: define __ARCH_IRQ_EXIT_IRQS_DISABLED
From: Michael Ellerman @ 2018-01-22 3:34 UTC (permalink / raw)
To: Nicholas Piggin, linuxppc-dev; +Cc: Nicholas Piggin
In-Reply-To: <20171116160052.18672-2-npiggin@gmail.com>
On Thu, 2017-11-16 at 16:00:49 UTC, Nicholas Piggin wrote:
> powerpc calls irq_exit() with local irqs disabled, therefore it
> can define __ARCH_IRQ_EXIT_IRQS_DISABLED.
>
> Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
Applied to powerpc next, thanks.
https://git.kernel.org/powerpc/c/c16bee4bded5449ec3b3ec73579ba2
cheers
^ permalink raw reply
page: next (older) | prev (newer) | latest
- recent:[subjects (threaded)|topics (new)|topics (active)]
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox