[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 2/2] docs/cpu-features: Update "PAuth" (Pointer Authenticatio
From: |
Kashyap Chamarthy |
Subject: |
Re: [PATCH 2/2] docs/cpu-features: Update "PAuth" (Pointer Authentication) details |
Date: |
Mon, 27 Jan 2025 14:35:35 +0100 |
On Mon, Jan 27, 2025 at 01:06:44PM +0000, Peter Maydell wrote:
> On Mon, 20 Jan 2025 at 09:06, Kashyap Chamarthy <kchamart@redhat.com> wrote:
> >
> > On Sat, Jan 18, 2025 at 10:04:37AM +0000, Marc Zyngier wrote:
> > > On Fri, 17 Jan 2025 19:11:06 +0000,
> > > Kashyap Chamarthy <kchamart@redhat.com> wrote:
> > > > +PAuth (Pointer Authentication) is a security feature in software that
> > > > +was introduced in Armv8.3-A and Armv9.0-A. It aims to protect against
> > >
> > > nit: given that ARMv9.0 is congruent to ARMv8.5 and therefore has all
> > > the ARMv8.5 features, mentioning ARMv8.3 should be enough (but I don't
> > > feel strongly about this). I feel much strongly about the use of
> > > capital letters, but I live in a distant past... ;-)
> >
> > Sure, I can keep it to just v8.3.
> >
> > On capitalization, I don't feel strongly about it, I just followed this
> > commit[1], which explained that the rebranding changed "ARM" to "Arm":
> >
> > 6fe6d6c9a95 (docs: Be consistent about capitalization of 'Arm',
> > 2020-03-09)
> >
> > That's why I went with it. I see you know this by your "distant past"
> > remark :) To match the above, I'll keep the capitalization to "Arm".
>
> We should probably do another fix-capitalization pass on docs/,
> because a bunch more "ARM" uses have crept in since 6fe6d6c9a95.
I can tack it to v2 of this. I see about 22 occurrences in docs/:
$> git grep "ARM " | wc -l
22
> (6fe6d6c9a95's commit message says that architecture names like
> "ARMv8" retain all-caps, but I think that was my personal taste
> creeping in -- at any rate the current Arm ARM uses "Armv8",
> "Armv9", etc. It was right that "ARM926" etc should stay all-caps,
> though.)
Okay, so, architecture names should stick to "Armv8", etc. (I too
would have preferred "ARMv8", but whatever the branding says.)
For all-caps: I assume you mean these should remain as-is: ARM926EJ-S,
ARM1176JZS, ARM1176, etc.
--
/kashyap