[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH 2/2] docs: Introduce some hypercall page documentation



>>> On 23.05.19 at 14:02, <andrew.cooper3@xxxxxxxxxx> wrote:
> On 23/05/2019 12:41, Jan Beulich wrote:
>>>>> On 23.05.19 at 13:01, <andrew.cooper3@xxxxxxxxxx> wrote:
>>> On 23/05/2019 11:56, Jan Beulich wrote:
>>>>>>> On 23.05.19 at 12:20, <andrew.cooper3@xxxxxxxxxx> wrote:
>>>>> This also introduced the top-level Guest Documentation section.
>>>>>
>>>>> Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
>>>> Large parts of this are entirely x86-centric, yet hypercalls exist
>>>> for Arm as well. If this is intentional, then I think you should say
>>>> so above.
>>> It is all x86 specific, which is why it is grouped under "x86 guest
>>> documentation".
>> Neither the path nor anything near the top of the added file suggest
>> this is "x86 guest documentation". How is one to make this
>> connection? Or are you referring to the sole entry that ends up in
>> docs/guest-guide/index.rst?
> 
> Yes, and by the way you ask this question, I presume you haven't seen
> how sphinx renders it.

Nor do I think I should need to, for the purpose of reviewing
the change. I'm in particular concerned about further additions
here down the road, when it would then be even less visible
that this is x86-specific documentation.

> Nevertheless, the observation below does warrant some changes here.

Right - hence me mentioning "path" in my original reply.

>> People
>> writing code targeting the hypercall interface, I assume. This
>> includes people who may not at all be familiar with the AT&T
>> peculiarities of the assembly language used (mainly for
>> naming registers). It's easy for the to understand what is
>> meant nevertheless, but to be honest I'd prefer if the SDM /
>> PM register names were used instead, i.e. in particular
>> without the % prefixes (but also omitting the $ on the INT
>> operand).
> 
> While this case is, dropping the AT&T-isms is easy, I'm not convinced
> that will be the case elsewhere.
> 
> Also, I don't want us to be in a case where we develop exclusively with
> AT&T, but have our documentation written in Intel syntax.

For one I don't think documentation and source need to be
aligned in this regard. Aligning our documentation to that
of the CPU vendors seems more important to me. (That
said, in example code I think at least one of the two have
started mixing Intel and AT&T syntax.)

And then - can't we get away without using any assembly
syntax at all in documentation? Surely things can be worded
in a more abstract way (like "INT instruction with an immediate
of 0x82).

Jan



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.