>>> On 06.01.17 at 19:34, <ian.jack...@eu.citrix.com> wrote:
> Jan Beulich writes ("Re: [PATCH v2] ns16550: Add command line parsing
> adjustments"):
>> On 06.01.17 at 17:24, <swapnil.para...@amd.com> wrote:
>> Well, as you may have seen, things are getting complicated here:
>> The two currently-last elements are permitted only with
>> CONFIG_HAS_PCI, so by adding the new fields to the end we'd
>> end up having two syntaxes (one with and one without PCI
>> support). I therefore have to modify my original proposal, and
>> ask for the addition to be done earlier, perhaps - using a
>> separator other than comma (maybe colon or semicolon) - with
>> the [<io-base>|pci|amt] element (as that's really the item
>> (possibly implicitly) specifying the I/O range, which you mean to
>> amend.
>
> Why not introduce the concept of names for these endlessly multiplying
> parameters ?
That's a good ides - we should indeed consider this, but then for all
of the present sub-options (retaining the nameless variant just for
backwards compatibility, and maybe only for a couple of releases.
I'm not sure Swapnil is up to a full overhaul of this parsing ...
Jan
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
