On 29.11.2019 11:22, Jan Beulich wrote:
> On 28.11.2019 17:52, Paul Durrant wrote:
>> --- a/xen/common/grant_table.c
>> +++ b/xen/common/grant_table.c
>> @@ -84,11 +84,40 @@ struct grant_table {
>> struct grant_table_arch arch;
>> };
>>
>> +static int parse_gnttab_limit(const char *param, const char *arg,
>> + unsigned int *valp)
>> +{
>> + const char *e;
>> + unsigned long val;
>> +
>> + val = simple_strtoul(arg, &e, 0);
>> + if ( *e )
>> + return -EINVAL;
>> +
>> + if ( val > INT_MAX )
>> + return -ERANGE;
>> +
>> + return 0;
>> +}
>
> *valp doesn't get written to anymore. With this fixed (and no new
> issues introduced ;-) ) hypervisor side
> Reviewed-by: Jan Beulich <jbeul...@suse.com>
And I guess I should have clarified: I'd be fine adding the missing
assignment while committing, provided the tools side won't require
any changes.
Jan
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
