]> git.baikalelectronics.ru Git - kernel.git/commit
Reduce the number of expensive division instructions done by _parse_integer()
authorDavid Howells <dhowells@redhat.com>
Thu, 9 Feb 2012 15:48:20 +0000 (15:48 +0000)
committerLinus Torvalds <torvalds@linux-foundation.org>
Thu, 9 Feb 2012 18:09:30 +0000 (10:09 -0800)
commit908661c7a226a041f62b702307c79450a26d8b22
treea6ccb2a00ce2e1fd2f3d8facabd0242e61431704
parent76e68e22a5cdf0073c7c2c9d6a002e3f2dac7e04
Reduce the number of expensive division instructions done by _parse_integer()

_parse_integer() does one or two division instructions (which are slow)
per digit parsed to perform the overflow check.

Furthermore, these are particularly expensive examples of division
instruction as the number of clock cycles required to complete them may
go up with the position of the most significant set bit in the dividend:

if (*res > div_u64(ULLONG_MAX - val, base))

which is as maximal as possible.

Worse, on 32-bit arches, more than one of these division instructions
may be required per digit.

So, assuming we don't support a base of more than 16, skip the check if the
top nibble of the result is not set at this point.

Signed-off-by: David Howells <dhowells@redhat.com>
[ Changed it to not dereference the pointer all the time - even if the
  compiler can and does optimize it away, the code just looks cleaner.
  And edited the top nybble test slightly to make the code generated on
  x86-64 better in the loop - test against a hoisted constant instead of
  shifting and testing the result ]
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
lib/kstrtox.c