linux-stable.git/lib/kstrtox.c, branch v4.13.2

lib/kstrtox.c: use "unsigned int" more

2017-07-10T23:32:34+00:00

gcc does generates stupid code sign extending data back and forth.  Help
by using "unsigned int".

	add/remove: 0/0 grow/shrink: 0/3 up/down: 0/-61 (-61)
	function                                     old     new   delta
	_parse_integer                               128     123      -5

It _still_ does generate useless MOVSX but I don't know how to delete it:

0000000000000070 <_parse_integer>:
			...
  a0:   89 c2                   mov    edx,eax
  a2:   83 e8 30                sub    eax,0x30
  a5:   83 f8 09                cmp    eax,0x9
  a8:   76 11                   jbe    bb <_parse_integer+0x4b>
  aa:   83 ca 20                or     edx,0x20
  ad:   0f be c2      ===>      movsx  eax,dl         <===
			useless
  b0:   8d 50 9f                lea    edx,[rax-0x61]
  b3:   83 fa 05                cmp    edx,0x5

Patch also helps on embedded archs which generally only like "int".  On
arm "and 0xff" is generated which is waste because all values used in
comparisons are positive.

Link: http://lkml.kernel.org/r/20170514194720.GB32563@avx2
Signed-off-by: Alexey Dobriyan 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

lib/kstrtox.c: delete end-of-string test

2017-07-10T23:32:34+00:00

Standard "while (*s)" test is unnecessary because NUL won't pass
valid-digit test anyway.  Save one branch per parsed character.

Link: http://lkml.kernel.org/r/20170514193756.GA32563@avx2
Signed-off-by: Alexey Dobriyan 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

Replace with globally

2016-12-24T19:46:01+00:00

This was entirely automated, using the script by Al:

  PATT='^[[:blank:]]*#[[:blank:]]*include[[:blank:]]*'
  sed -i -e "s!$PATT!#include !" \
        $(git grep -l "$PATT"|grep -v ^include/linux/uaccess.h)

to do the replacement at the end of the merge window.

Requested-by: Al Viro 
Signed-off-by: Linus Torvalds

lib/kstrtox.c: smaller _parse_integer()

2016-10-11T22:06:30+00:00

Set "overflow" bit upon encountering it instead of postponing to the end
of the conversion. Somehow gcc unwedges itself and generates better code:

	$ ./scripts/bloat-o-meter ../vmlinux-000 ../obj/vmlinux
	_parse_integer                      177     139     -38

Inspired by patch from Zhaoxiu Zeng.

Link: http://lkml.kernel.org/r/20160826221920.GA1909@p183.telecom.by
Signed-off-by: Alexey Dobriyan 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

lib: add "on"/"off" support to kstrtobool

2016-03-17T22:09:34+00:00

Add support for "on" and "off" when converting to boolean.

Signed-off-by: Kees Cook 
Cc: Amitkumar Karwar 
Cc: Andy Shevchenko 
Cc: Daniel Borkmann 
Cc: Heiko Carstens 
Cc: Joe Perches 
Cc: Kalle Valo 
Cc: Martin Schwidefsky 
Cc: Michael Ellerman 
Cc: Nishant Sarmukadam 
Cc: Rasmus Villemoes 
Cc: Steve French 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

lib: move strtobool() to kstrtobool()

2016-03-17T22:09:34+00:00

Create the kstrtobool_from_user() helper and move strtobool() logic into
the new kstrtobool() (matching all the other kstrto* functions).
Provides an inline wrapper for existing strtobool() callers.

Signed-off-by: Kees Cook 
Cc: Joe Perches 
Cc: Andy Shevchenko 
Cc: Rasmus Villemoes 
Cc: Daniel Borkmann 
Cc: Amitkumar Karwar 
Cc: Nishant Sarmukadam 
Cc: Kalle Valo 
Cc: Steve French 
Cc: Michael Ellerman 
Cc: Heiko Carstens 
Cc: Martin Schwidefsky 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

kstrto*: accept "-0" for signed conversion

2015-09-10T20:29:01+00:00

strtol(3) et al accept "-0", so should we.

Signed-off-by: Alexey Dobriyan 
Cc: David Howells 
Cc: Jan Kara 
Cc: Joel Becker 
Cc: Mark Fasheh 
Cc: Theodore Ts'o 
Cc: Rasmus Villemoes 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

lib/kstrtox.c: remove redundant cleanup

2014-01-24T00:36:57+00:00

We can't reach the cleanup code unless the flag KSTRTOX_OVERFLOW is not
set, so there's not no point in clearing a bit that we know is not set.

Signed-off-by: Felipe Contreras 
Acked-by: Levente Kurusa 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

kstrto*: add documentation

2012-12-18T01:15:22+00:00

As Bruce Fields pointed out, kstrto* is currently lacking kerneldoc
comments.  This patch adds kerneldoc comments to common variants of
kstrto*: kstrto(u)l, kstrto(u)ll and kstrto(u)int.

Signed-off-by: Eldad Zack 
Cc: J. Bruce Fields 
Cc: Joe Perches 
Cc: Randy Dunlap 
Cc: Alexey Dobriyan 
Cc: Rob Landley 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

lib: reduce the use of module.h wherever possible

2012-03-07T20:04:04+00:00

For files only using THIS_MODULE and/or EXPORT_SYMBOL, map
them onto including export.h -- or if the file isn't even
using those, then just delete the include.  Fix up any implicit
include dependencies that were being masked by module.h along
the way.

Signed-off-by: Paul Gortmaker