>>> On 23.08.17 at 11:30, <jgr...@suse.com> wrote:
> On 22/08/17 13:24, Jan Beulich wrote:
>>>>> On 16.08.17 at 14:52, <jgr...@suse.com> wrote:
>>> @@ -176,7 +210,8 @@ int __init parse_bool(const char *s)
>>> !strcmp("on", s) ||
>>> !strcmp("true", s) ||
>>> !strcmp("enable", s) ||
>>> - !strcmp("1", s) )
>>> + !strcmp("1", s) ||
>>> + !*s )
>>> return 1;
>>
>> Careful with this: Taking the "iommu=" example that I've commented
>> on in the other patch already, much depends on what you mean to
>> do about the problem there: "iommu=,..." should not end up
>> meaning "iommu=on,...".
>
> It won't. *s will be ',' in this case.
Right, but as said - much depends on what you mean to do about
the problem in the earlier patch.
Jan
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
