linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: lorenzo.pieralisi@arm.com (Lorenzo Pieralisi)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 0/2] ARM: Do not mess with CCI if booted from HYP
Date: Fri, 23 Sep 2016 11:29:07 +0100	[thread overview]
Message-ID: <20160923102907.GB31120@red-moon> (raw)
In-Reply-To: <1474380833-30121-1-git-send-email-marc.zyngier@arm.com>

Hi Marc,

On Tue, Sep 20, 2016 at 03:13:51PM +0100, Marc Zyngier wrote:
> Some systems (such as the VExpress TC2) are built around a CCI-400,
> which is only accessible from secure mode. This obviously breaks if
> the system is booted in non-secure mode. Detecting non-secure is
> pretty hard to do, unless the system is booted from HYP.
> 
> This small patch series makes sure that:
> - The CCI ports are not accessed when the kernel is booted from HYP
> - MCPM (which depends on CCI) is not enabled on VExpress if booted
> from HYP.
> 
> This allows a multi_v7_defconfig kernel to be booted on a TC2 booting
> from HYP, instead of miserably dying very early on.

As discussed I came up with a different set of patches that
detect CCI ports status as reported by DT and prevent enabling
MCPM if the CCI ports are not under kernel control (eg they are
reported as "disabled" by DT since we booted in HYP).

I will post patches for review and test shortly.

Thanks for fixing it anyway,
Lorenzo

> 
> Marc Zyngier (2):
>   ARM: vexpress: Do not enable MCPM if booted from HYP
>   arm-cci: Do not probe the CCI ports if booted from HYP
> 
>  arch/arm/mach-vexpress/platsmp.c |  8 ++++++--
>  drivers/bus/arm-cci.c            | 11 +++++++++++
>  2 files changed, 17 insertions(+), 2 deletions(-)
> 
> -- 
> 2.1.4
> 

      parent reply	other threads:[~2016-09-23 10:29 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-09-20 14:13 [PATCH 0/2] ARM: Do not mess with CCI if booted from HYP Marc Zyngier
2016-09-20 14:13 ` [PATCH 1/2] ARM: vexpress: Do not enable MCPM " Marc Zyngier
2016-09-20 14:13 ` [PATCH 2/2] arm-cci: Do not probe the CCI ports " Marc Zyngier
2016-09-23 10:29 ` Lorenzo Pieralisi [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=20160923102907.GB31120@red-moon \
    --to=lorenzo.pieralisi@arm.com \
    --cc=linux-arm-kernel@lists.infradead.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;
as well as URLs for NNTP newsgroup(s).