From: Greg KH <gregkh@linuxfoundation.org>
To: Greg Hackmann <ghackmann@google.com>
Cc: Mark Rutland <mark.rutland@arm.com>,
stable@vger.kernel.org, mark.brown@linaro.org,
ard.biesheuvel@linaro.org, marc.zyngier@arm.com,
will.deacon@arm.com
Subject: Re: [PATCH v4.9.y 00/27] arm64 meltdown patches
Date: Thu, 5 Apr 2018 21:15:28 +0200 [thread overview]
Message-ID: <20180405191528.GA19589@kroah.com> (raw)
In-Reply-To: <ac09018a-f429-b632-16c0-f7409be2f184@google.com>
On Thu, Apr 05, 2018 at 10:34:30AM -0700, Greg Hackmann wrote:
> On 04/03/2018 04:08 AM, Mark Rutland wrote:
> > Hi Greg,
> >
> > These patches backport KPTI to v4.9.y (based on v4.9.92), providing protection
> > against meltdown on arm64 platforms.
> >
> > I picked up Alex Shi's backport for review and testing, and as I found a couple
> > of issues to fix up, I'm sending this with my Signed-off-by in the chain, with
> > those fixups applied and noted.
> >
> > To the best of my understanding the code is correct, in the context of the
> > v4.9.y kernel, and I've tested the seires on arm64 hardware available to me.
> > i.e. if this didn't have my Signed-off-by it would have my Reviewed-by and
> > Tested-by tags.
> >
> > Are you happy to pick these up for v4.9.93?
> >
> > Thanks,
> > Mark.
> >
> > AKASHI Takahiro (1):
> > module: extend 'rodata=off' boot cmdline parameter to module mappings
> >
> > Jayachandran C (2):
> > arm64: cputype: Add MIDR values for Cavium ThunderX2 CPUs
> > arm64: Turn on KPTI only on CPUs that need it
> >
> > Marc Zyngier (2):
> > arm64: Allow checking of a CPU-local erratum
> > arm64: Force KPTI to be disabled on Cavium ThunderX
> >
> > Mark Rutland (1):
> > arm64: factor out entry stack manipulation
> >
> > Suzuki K Poulose (1):
> > arm64: capabilities: Handle duplicate entries for a capability
> >
> > Will Deacon (20):
> > arm64: mm: Use non-global mappings for kernel space
> > arm64: mm: Move ASID from TTBR0 to TTBR1
> > arm64: mm: Allocate ASIDs in pairs
> > arm64: mm: Add arm64_kernel_unmapped_at_el0 helper
> > arm64: mm: Invalidate both kernel and user ASIDs when performing TLBI
> > arm64: entry: Add exception trampoline page for exceptions from EL0
> > arm64: mm: Map entry trampoline into trampoline and kernel page tables
> > arm64: entry: Explicitly pass exception level to kernel_ventry macro
> > arm64: entry: Hook up entry trampoline to exception vectors
> > arm64: tls: Avoid unconditional zeroing of tpidrro_el0 for native
> > tasks
> > arm64: entry: Add fake CPU feature for unmapping the kernel at EL0
> > arm64: kaslr: Put kernel vectors address in separate data page
> > arm64: use RET instruction for exiting the trampoline
> > arm64: Kconfig: Add CONFIG_UNMAP_KERNEL_AT_EL0
> > arm64: Kconfig: Reword UNMAP_KERNEL_AT_EL0 kconfig entry
> > arm64: Take into account ID_AA64PFR0_EL1.CSV3
> > arm64: kpti: Make use of nG dependent on
> > arm64_kernel_unmapped_at_el0()
> > arm64: kpti: Add ->enable callback to remap swapper using nG mappings
> > arm64: entry: Reword comment about post_ttbr_update_workaround
> > arm64: idmap: Use "awx" flags for .idmap.text .pushsection directives
> >
> > arch/arm64/Kconfig | 12 ++
> > arch/arm64/include/asm/assembler.h | 3 +
> > arch/arm64/include/asm/cpucaps.h | 3 +-
> > arch/arm64/include/asm/cputype.h | 3 +
> > arch/arm64/include/asm/fixmap.h | 6 +
> > arch/arm64/include/asm/mmu.h | 11 ++
> > arch/arm64/include/asm/mmu_context.h | 7 ++
> > arch/arm64/include/asm/pgtable-hwdef.h | 1 +
> > arch/arm64/include/asm/pgtable-prot.h | 35 +++---
> > arch/arm64/include/asm/pgtable.h | 1 +
> > arch/arm64/include/asm/proc-fns.h | 6 -
> > arch/arm64/include/asm/sysreg.h | 1 +
> > arch/arm64/include/asm/tlbflush.h | 16 ++-
> > arch/arm64/kernel/asm-offsets.c | 6 +-
> > arch/arm64/kernel/cpu-reset.S | 2 +-
> > arch/arm64/kernel/cpufeature.c | 135 ++++++++++++++++++---
> > arch/arm64/kernel/entry.S | 188 ++++++++++++++++++++++++----
> > arch/arm64/kernel/head.S | 2 +-
> > arch/arm64/kernel/process.c | 12 +-
> > arch/arm64/kernel/sleep.S | 2 +-
> > arch/arm64/kernel/vmlinux.lds.S | 22 +++-
> > arch/arm64/mm/context.c | 25 ++--
> > arch/arm64/mm/mmu.c | 31 +++++
> > arch/arm64/mm/proc.S | 216 +++++++++++++++++++++++++++++++--
> > include/linux/init.h | 3 +
> > init/main.c | 7 +-
> > kernel/module.c | 20 ++-
> > 27 files changed, 675 insertions(+), 101 deletions(-)
> >
>
> I ran this series on the 1st gen hikey dev board and it works fine for me.
>
> On top of mainline v4.9.92, tip-of-tree AOSP userspace boots to a serial
> shell.
>
> On top of the android-linaro-hikey-4.9 branch on AOSP, it boots to the
> home screen without issues. (android-4.9 has an out-of-tree SW PAN
> backport which I reverted locally for testing purposes.)
>
> So for the series:
>
> Tested-by: Greg Hackmann <ghackmann@google.com>
Great, thanks for testing this and letting me know.
greg k-h
prev parent reply other threads:[~2018-04-05 19:15 UTC|newest]
Thread overview: 63+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-04-03 11:08 [PATCH v4.9.y 00/27] arm64 meltdown patches Mark Rutland
2018-04-03 11:08 ` [PATCH v4.9.y 01/27] arm64: mm: Use non-global mappings for kernel space Mark Rutland
2018-04-05 19:42 ` Patch "arm64: mm: Use non-global mappings for kernel space" has been added to the 4.9-stable tree gregkh
2018-04-03 11:08 ` [PATCH v4.9.y 02/27] arm64: mm: Move ASID from TTBR0 to TTBR1 Mark Rutland
2018-04-05 19:42 ` Patch "arm64: mm: Move ASID from TTBR0 to TTBR1" has been added to the 4.9-stable tree gregkh
2018-04-03 11:08 ` [PATCH v4.9.y 03/27] arm64: mm: Allocate ASIDs in pairs Mark Rutland
2018-04-05 19:42 ` Patch "arm64: mm: Allocate ASIDs in pairs" has been added to the 4.9-stable tree gregkh
2018-04-03 11:09 ` [PATCH v4.9.y 04/27] arm64: mm: Add arm64_kernel_unmapped_at_el0 helper Mark Rutland
2018-04-05 19:42 ` Patch "arm64: mm: Add arm64_kernel_unmapped_at_el0 helper" has been added to the 4.9-stable tree gregkh
2018-04-03 11:09 ` [PATCH v4.9.y 05/27] arm64: mm: Invalidate both kernel and user ASIDs when performing TLBI Mark Rutland
2018-04-05 19:42 ` Patch "arm64: mm: Invalidate both kernel and user ASIDs when performing TLBI" has been added to the 4.9-stable tree gregkh
2018-04-03 11:09 ` [PATCH v4.9.y 06/27] arm64: factor out entry stack manipulation Mark Rutland
2018-04-05 19:42 ` Patch "arm64: factor out entry stack manipulation" has been added to the 4.9-stable tree gregkh
2018-04-03 11:09 ` [PATCH v4.9.y 07/27] module: extend 'rodata=off' boot cmdline parameter to module mappings Mark Rutland
2018-04-05 19:42 ` Patch "module: extend 'rodata=off' boot cmdline parameter to module mappings" has been added to the 4.9-stable tree gregkh
2018-04-03 11:09 ` [PATCH v4.9.y 08/27] arm64: entry: Add exception trampoline page for exceptions from EL0 Mark Rutland
2018-04-05 19:42 ` Patch "arm64: entry: Add exception trampoline page for exceptions from EL0" has been added to the 4.9-stable tree gregkh
2018-04-03 11:09 ` [PATCH v4.9.y 09/27] arm64: mm: Map entry trampoline into trampoline and kernel page tables Mark Rutland
2018-04-03 11:15 ` Mark Rutland
2018-04-05 19:33 ` Greg KH
2018-04-05 19:42 ` Patch "arm64: mm: Map entry trampoline into trampoline and kernel page tables" has been added to the 4.9-stable tree gregkh
2018-04-03 11:09 ` [PATCH v4.9.y 10/27] arm64: entry: Explicitly pass exception level to kernel_ventry macro Mark Rutland
2018-04-05 19:42 ` Patch "arm64: entry: Explicitly pass exception level to kernel_ventry macro" has been added to the 4.9-stable tree gregkh
2018-04-03 11:09 ` [PATCH v4.9.y 11/27] arm64: entry: Hook up entry trampoline to exception vectors Mark Rutland
2018-04-05 19:42 ` Patch "arm64: entry: Hook up entry trampoline to exception vectors" has been added to the 4.9-stable tree gregkh
2018-04-03 11:09 ` [PATCH v4.9.y 12/27] arm64: tls: Avoid unconditional zeroing of tpidrro_el0 for native tasks Mark Rutland
2018-04-05 19:42 ` Patch "arm64: tls: Avoid unconditional zeroing of tpidrro_el0 for native tasks" has been added to the 4.9-stable tree gregkh
2018-04-03 11:09 ` [PATCH v4.9.y 13/27] arm64: entry: Add fake CPU feature for unmapping the kernel at EL0 Mark Rutland
2018-04-05 19:42 ` Patch "arm64: entry: Add fake CPU feature for unmapping the kernel at EL0" has been added to the 4.9-stable tree gregkh
2018-04-03 11:09 ` [PATCH v4.9.y 14/27] arm64: kaslr: Put kernel vectors address in separate data page Mark Rutland
2018-04-05 19:42 ` Patch "arm64: kaslr: Put kernel vectors address in separate data page" has been added to the 4.9-stable tree gregkh
2018-04-03 11:09 ` [PATCH v4.9.y 15/27] arm64: use RET instruction for exiting the trampoline Mark Rutland
2018-04-05 19:42 ` Patch "arm64: use RET instruction for exiting the trampoline" has been added to the 4.9-stable tree gregkh
2018-04-03 11:09 ` [PATCH v4.9.y 16/27] arm64: Kconfig: Add CONFIG_UNMAP_KERNEL_AT_EL0 Mark Rutland
2018-04-05 19:42 ` Patch "arm64: Kconfig: Add CONFIG_UNMAP_KERNEL_AT_EL0" has been added to the 4.9-stable tree gregkh
2018-04-03 11:09 ` [PATCH v4.9.y 17/27] arm64: Kconfig: Reword UNMAP_KERNEL_AT_EL0 kconfig entry Mark Rutland
2018-04-05 19:42 ` Patch "arm64: Kconfig: Reword UNMAP_KERNEL_AT_EL0 kconfig entry" has been added to the 4.9-stable tree gregkh
2018-04-03 11:09 ` [PATCH v4.9.y 18/27] arm64: Take into account ID_AA64PFR0_EL1.CSV3 Mark Rutland
2018-04-05 19:42 ` Patch "arm64: Take into account ID_AA64PFR0_EL1.CSV3" has been added to the 4.9-stable tree gregkh
2018-04-03 11:09 ` [PATCH v4.9.y 19/27] arm64: Allow checking of a CPU-local erratum Mark Rutland
2018-04-05 19:42 ` Patch "arm64: Allow checking of a CPU-local erratum" has been added to the 4.9-stable tree gregkh
2018-04-03 11:09 ` [PATCH v4.9.y 20/27] arm64: capabilities: Handle duplicate entries for a capability Mark Rutland
2018-04-05 19:42 ` Patch "arm64: capabilities: Handle duplicate entries for a capability" has been added to the 4.9-stable tree gregkh
2018-04-03 11:09 ` [PATCH v4.9.y 21/27] arm64: cputype: Add MIDR values for Cavium ThunderX2 CPUs Mark Rutland
2018-04-05 19:42 ` Patch "arm64: cputype: Add MIDR values for Cavium ThunderX2 CPUs" has been added to the 4.9-stable tree gregkh
2018-04-03 11:09 ` [PATCH v4.9.y 22/27] arm64: Turn on KPTI only on CPUs that need it Mark Rutland
2018-04-05 19:42 ` Patch "arm64: Turn on KPTI only on CPUs that need it" has been added to the 4.9-stable tree gregkh
2018-04-03 11:09 ` [PATCH v4.9.y 23/27] arm64: kpti: Make use of nG dependent on arm64_kernel_unmapped_at_el0() Mark Rutland
2018-04-05 19:42 ` Patch "arm64: kpti: Make use of nG dependent on arm64_kernel_unmapped_at_el0()" has been added to the 4.9-stable tree gregkh
2018-04-03 11:09 ` [PATCH v4.9.y 24/27] arm64: kpti: Add ->enable callback to remap swapper using nG mappings Mark Rutland
2018-04-05 19:42 ` Patch "arm64: kpti: Add ->enable callback to remap swapper using nG mappings" has been added to the 4.9-stable tree gregkh
2018-04-03 11:09 ` [PATCH v4.9.y 25/27] arm64: Force KPTI to be disabled on Cavium ThunderX Mark Rutland
2018-04-05 19:42 ` Patch "arm64: Force KPTI to be disabled on Cavium ThunderX" has been added to the 4.9-stable tree gregkh
2018-04-03 11:09 ` [PATCH v4.9.y 26/27] arm64: entry: Reword comment about post_ttbr_update_workaround Mark Rutland
2018-04-05 19:42 ` Patch "arm64: entry: Reword comment about post_ttbr_update_workaround" has been added to the 4.9-stable tree gregkh
2018-04-03 11:09 ` [PATCH v4.9.y 27/27] arm64: idmap: Use "awx" flags for .idmap.text .pushsection directives Mark Rutland
2018-04-05 19:42 ` Patch "arm64: idmap: Use "awx" flags for .idmap.text .pushsection directives" has been added to the 4.9-stable tree gregkh
2018-04-04 15:07 ` [PATCH v4.9.y 00/27] arm64 meltdown patches Greg KH
2018-04-05 10:04 ` Will Deacon
2018-04-05 10:15 ` Mark Rutland
2018-04-05 11:46 ` Will Deacon
2018-04-05 17:34 ` Greg Hackmann
2018-04-05 19:15 ` Greg KH [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20180405191528.GA19589@kroah.com \
--to=gregkh@linuxfoundation.org \
--cc=ard.biesheuvel@linaro.org \
--cc=ghackmann@google.com \
--cc=marc.zyngier@arm.com \
--cc=mark.brown@linaro.org \
--cc=mark.rutland@arm.com \
--cc=stable@vger.kernel.org \
--cc=will.deacon@arm.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).