Linux-ARM-Kernel Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Will Deacon <will@kernel.org>
To: Marc Zyngier <maz@kernel.org>
Cc: Neeraj Upadhyay <neeraju@codeaurora.org>,
	linux-arm-kernel@lists.infradead.org
Subject: Re: Query regarding ERRATUM_1418040
Date: Wed, 17 Jun 2020 12:25:42 +0100	[thread overview]
Message-ID: <20200617112542.GB3503@willie-the-truck> (raw)
In-Reply-To: <062be27686369d28bd2054a54c307400@misterjones.org>

On Wed, Jun 17, 2020 at 12:19:16PM +0100, Marc Zyngier wrote:
> On 2020-06-17 09:55, Neeraj Upadhyay wrote:
> > I have query regarding the errata 1418040 [1]. Here, on kernel exit to
> > EL0 64 bit mode, will it always enable ARCH_TIMER_USR_VCT_ACCESS_EN;
> > and override any other erratas, which might require EL0 direct vct
> > access to be disabled for 64 bit also?
> 
> So far, I am not aware of any erratum that would require traps of CNTVCT_EL0
> to EL1 when running AArch64 userspace for CPUs that are affected by
> ARM-1418040. If such an erratum exists, it would have to be handled
> separately.
> 
> > Also, this errata applies
> > mitigation for all CPUs (on return to 32 bit EL0), even if, not all
> > cpus are impacted by the errata?
> 
> Indeed. There isn't much we can do to avoid it here, unless you want to read
> some per-CPU variable that tells you whether the CPU is affected. This would
> add to the cost of the mitigation , and isn't an appealing prospect.

Hmm, but in conjunction with the previous point, doesn't this mean if
some CPUs are affected by an erratum which requires CNTVCT trapping for
AArch64 and others are affected by 1418040, then the former won't actually
be trapped?

Maybe we should preserve ARCH_TIMER_USR_VCT_ACCESS_EN for AArch64 tasks
instead of setting it unconditionally?

Will

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

  reply	other threads:[~2020-06-17 11:25 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-06-17  8:55 Query regarding ERRATUM_1418040 Neeraj Upadhyay
2020-06-17 11:19 ` Marc Zyngier
2020-06-17 11:25   ` Will Deacon [this message]
2020-06-17 13:00     ` Neeraj Upadhyay
2020-06-17 14:29     ` Marc Zyngier
2020-07-01 16:27       ` Will Deacon
2020-07-02  8:19         ` Marc Zyngier
2020-07-06 12:09           ` Will Deacon
2020-07-06 12:27             ` Marc Zyngier
2020-07-06 15:16               ` Will Deacon
2020-07-06 16:05                 ` Marc Zyngier

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=20200617112542.GB3503@willie-the-truck \
    --to=will@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=maz@kernel.org \
    --cc=neeraju@codeaurora.org \
    /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