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

Re: [Xen-devel] [PATCH] common/gnttab: Introduce command line feature controls

>>> On 05.02.18 at 12:55, <andrew.cooper3@xxxxxxxxxx> wrote:
> On 02/02/18 08:51, Jan Beulich wrote:
>>>>> On 01.02.18 at 15:38, <andrew.cooper3@xxxxxxxxxx> wrote:
>>> --- a/docs/misc/xen-command-line.markdown
>>> +++ b/docs/misc/xen-command-line.markdown
>>> @@ -916,6 +916,19 @@ Controls EPT related features.
>>>  Specify which console gdbstub should use. See **console**.
>>> +### gnttab
>>> +> `= List of [ max_ver:<integer>, transitive=<bool> ]`
>> I realize you don't want to change this as people already use it, but
>> I'd still like to give my usual comment: I'd prefer if we could avoid
>> introducing further underscore-containing (sub)options. I really don't
>> understand why everyone does this: Dashes are easier to type on
>> all keyboards I'm aware of, and there's no need to mimic C identifier
>> names for command line options.
> I can introduce a max-ver alias if you insist, but dropping max_ver here
> is going to break users who took this patch for XSA-226.

Hence the way I've worded my reply - I don't mean to insist on
changing what you have, or the introduction of an alias. I merely
wanted to give the comment, in the hope that it helps to avoid
future underscores in command line option names.


Xen-devel mailing list



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