Fabiano Rosas <[email protected]> writes:

> Markus Armbruster <[email protected]> writes:
>
>> Fabiano Rosas <[email protected]> writes:
>>
>>> The max-bandwidth and avail-switchover-bandwidth parameters are
>>> declared as bytes/second in migration.json and interpreted as such
>>> throughout the code, except in HMP where they're expected to be
>>> provided as mebibytes.
>>>
>>> Fix the HMP code to parse the values as bytes.
>>
>> I wouldn't call this a fix.  The choice of unit was clearly intentional.
>> Doesn't mean we must not change it now.  David, what do you think?
>>
>
> Hm, but if the code uses bytes in arithmetic throughout, there must be
> bugs lingering somewhere. Although I haven't verified that.

I don't think so; the old code parses the argument string into a byte
count.

>                                                             Nonetheless,
> I can use a different terminology, no worries.

Thanks!


Reply via email to