Re: [PATCH v1 3/3] Documentation: riscv: add a section about ISA string ordering in /proc/cpuinfo

From: Conor Dooley
Date: Thu Dec 01 2022 - 03:18:12 EST


On Thu, Dec 01, 2022 at 10:05:32AM +0700, Bagas Sanjaya wrote:
> On Wed, Nov 30, 2022 at 11:41:26PM +0000, Conor Dooley wrote:
> > +#. Single-letter extensions come first, in "canonical order", so
> > + "IMAFDQLCBKJTPVH".
>
> "..., that is ... ."

Hmm, that reads strangely to me. s/that/which/.

>
> > +#. The first letter following the 'Z' conventionally indicates the most
> > + closely related alphabetical extension category, IMAFDQLCBKJTPVH.
> > + If multiple 'Z' extensions are named, they should be ordered first by
> > + category, then alphabetically within a category.
> > +
>
> Did you mean "most closely related alphabetical extension category in
> canonical order"?

I am not 100% sure what you are suggesting a replacement of here. I
think I may reword this as:
For additional standard extensions, the first letter following the 'Z'
conventionally indicates the most closely related alphabetical
extension category. If multiple 'Z' extensions are named, they will
be ordered first by category, in canonical order as listed above, then
alphabetically within a category.

> > +An example string following the order is:
> > + rv64imadc_zifoo_zigoo_zafoo_sbar_scar_zxmbaz_xqux_xrux
> > +
>
> IMO literal code block should be better fit for the example above,
> rather than definition list:

Uh, sure? I'm not sure what impact that has on the output, but I can
switch to a pre-formatted block.

Thanks,
Conor.