stable.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
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

      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).