[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <86h8o3r5jy.wl-marc.zyngier@arm.com>
Date: Sun, 22 Apr 2018 09:47:29 +0100
From: Marc Zyngier <marc.zyngier@....com>
To: Pavel Machek <pavel@....cz>
Cc: Mark Rutland <mark.rutland@....com>,
<linux-arm-kernel@...ts.infradead.org>, <arnd@...db.de>,
<catalin.marinas@....com>, <cdall@...nel.org>,
<drjones@...hat.com>, <kvmarm@...ts.cs.columbia.edu>,
<linux-arch@...r.kernel.org>, <ramana.radhakrishnan@....com>,
<suzuki.poulose@....com>, <will.deacon@....com>,
<linux-kernel@...r.kernel.org>, <awallis@...eaurora.org>,
<kernel-hardening@...ts.openwall.com>
Subject: Re: [PATCHv3 11/11] arm64: docs: document pointer authentication
On Sun, 22 Apr 2018 09:05:21 +0100,
Pavel Machek wrote:
>
> Hi!
>
> > @@ -205,6 +205,14 @@ Before jumping into the kernel, the following conditions must be met:
> > ICC_SRE_EL2.SRE (bit 0) must be initialised to 0b0.
> > - The DT or ACPI tables must describe a GICv2 interrupt controller.
> >
> > + For CPUs with pointer authentication functionality:
> > + - If EL3 is present:
> > + SCR_EL3.APK (bit 16) must be initialised to 0b1
> > + SCR_EL3.API (bit 17) must be initialised to 0b1
> > + - If the kernel is entered at EL1:
> > + HCR_EL2.APK (bit 40) must be initialised to 0b1
> > + HCR_EL2.API (bit 41) must be initialised to 0b1
> > +
>
> 0b1 is quite confusing way to write 1.
Do you find 0x1 equally confusing?
0bx is a pretty common way of describing bit-fields of a hardware
register. It is also consistent with the rest of this document.
M.
--
Jazz is not dead, it just smell funny.
Powered by blists - more mailing lists