Re: [PATCH v2 10/11] man2/futex.2: change limit to use ISO multiple symbol

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

 



Hi Jakub!

On 2/9/23 12:36, Jakub Wilk wrote:
> * Brian Inglis <Brian.Inglis@xxxxxxx>, 2023-02-08 15:33:
>>>> -was not less than 1,000,000,000).
>>>> +was not less than 1G).
>>>
>>> For nanoseconds, I think 1G is a bit weird.
> 
> Not only weird, but also not allowed by SI. From the brochure: "Prefix 
> symbols can neither stand alone nor be attached to the number 1".

That's in section 5.4.7 of the SI Brochure (v9) for those who don't have the context.  However, that section not only forbids 1G, but also 32G or 32Mi.  The point is that you can't attach a prefix to a number without units, because it's a prefix rather than a suffix.  According to the SI, one would have to express that as powers of 10 (or the expanded number if you prefer).

However, since in the case of 32Mi-1, which is 33554431, which is 32 * 2^20 - 1, which is 2^25-1, the most readable version is 32Mi-1, I'm willing to make an exception an divert the SI in that regards.

If someone is familiar with ISO/IEC 80000-1 and could check if it allows that, it would be nice to know.

But in this case, we have a unit, which is seconds, and we're already multiplying it by nano, so G doesn't fit in the rule below.

> 
>> SI actually allows Gns (/nGs?)
> 
> Nope: "Compound prefix symbols, i.e. prefix symbols formed by the 
> juxtaposition of two or more prefix symbols, are not permitted."

Thanks for finding the right quotation.  That was my understanding, but couldn't find it.

This rule I'd rather follow.

Cheers,

Alex

> 

-- 
<http://www.alejandro-colomar.es/>
GPG key fingerprint: A9348594CE31283A826FBDD8D57633D441E25BB5

Attachment: OpenPGP_signature
Description: OpenPGP digital signature


[Index of Archives]     [Kernel Documentation]     [Netdev]     [Linux Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]

  Powered by Linux