Merge branch 'sys-utils-again' of https://github.com/kerolasa/lelux-utiliteetit

* 'sys-utils-again' of https://github.com/kerolasa/lelux-utiliteetit:
  cytune: fix printf type warning
  docs: mention long options in cytune.8
  cytune: coding style fixes
  cytune: refactor main(), new function query_tty_stats()
  cytune: use libc error printing facilities
  cytune: check numeric user inputs
  cytune: add long options and usage()
  docs: restructure flock.1 manual page
  flock: simplify strtotimeval()
  lib: [strutils] add strtod_or_err() function
  flock: use strutils.h to check numeric user input
  flock: use sysexit.h for all exit values
  flock: use function attributes
  flock: align with howto-usage-function.txt
  flock: use libc error printing facilities
  flock: move long_options struct to function scope
  flock: fix coding style
  docs: align fallocate.1 with howto-man-page.txt

lsblk: add udev support

[kzak@redhat.com: - enable udev support by default
                  - don't check for libudev.h
                  - minor udev code refactoring in lsblk.c]

Signed-off-by: Ilias Mamedov <arknir@yandex.ru>
Signed-off-by: Karel Zak <kzak@redhat.com>

docs: add prlimit to the TODO file

Signed-off-by: Karel Zak <kzak@redhat.com>

umount: add warning

It's better to inform users about reason why their request does not
makes sense :-)

Signed-off-by: Karel Zak <kzak@redhat.com>

umount: use new lib/loopdev.c code

Signed-off-by: Karel Zak <kzak@redhat.com>

mount: fix loopdev deinitialization

Signed-off-by: Karel Zak <kzak@redhat.com>

libmount: support -o loop=<devname>

Signed-off-by: Karel Zak <kzak@redhat.com>

mount: use new lib/loopdev.c code

Signed-off-by: Karel Zak <kzak@redhat.com>

include,xalloc: fix whitespace to be consistent

We use spaces for the rest of this file, so the random tabs within
xstrdup stand out.

Signed-off-by: Dave Reisner <dreisner@archlinux.org>

include,xalloc: check for NULL before calling strdup

This fixes a segfault in mount (and possibly elsewhere) when invoked
without a -t parameter.

Broken in 7ef9fd7 when the common xalloc.h libs were introduced.

Signed-off-by: Dave Reisner <dreisner@archlinux.org>

tests: add missing file

Signed-off-by: Karel Zak <kzak@redhat.com>

libmount: use loopdev debug facility

Signed-off-by: Karel Zak <kzak@redhat.com>

lib,loopdev: add debugu messages

Signed-off-by: Karel Zak <kzak@redhat.com>

cytune: fix printf type warning

conversion specifies type 'unsigned long' but the argument has type 'unsigned long *'

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

docs: mention long options in cytune.8

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

cytune: coding style fixes

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

cytune: refactor main(), new function query_tty_stats()

This commit also moves global variables to the beginning of the
file, or to function scope where they belong.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

cytune: use libc error printing facilities

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

cytune: check numeric user inputs

Checks use strtol_or_err() from strutils.h

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

cytune: add long options and usage()

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

docs: restructure flock.1 manual page

The commit aligns manual page with howto-man-page.txt, and adds example
section which I hope makes usage of this command easier.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

flock: simplify strtotimeval()

Retire private string to number conversion and use strtod_or_err() instead.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

lib: [strutils] add strtod_or_err() function

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

flock: use strutils.h to check numeric user input

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

flock: use sysexit.h for all exit values

Additionally enhance readability of complex double "if shorthand's" by
making the segment to be few normal "if's".

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

flock: use function attributes

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

flock: align with howto-usage-function.txt

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

flock: use libc error printing facilities

Retire global progname variable and use warn{,x} and err{,x}
functions. As a side effect of using err function the occurences of
err variable had to be converted to referrals to errno variable.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

flock: move long_options struct to function scope

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

flock: fix coding style

Sanitize indentation, spacing, brace positions, comment line lenghts and
positions etc. This commit does not change a thing in code.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

docs: align fallocate.1 with howto-man-page.txt

Add also earlierly missing --version to the manual.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

tests: add loopdev libmount test

Signed-off-by: Karel Zak <kzak@redhat.com>

libmount: support {offset,sizelimit}= foor loopdevs

Signed-off-by: Karel Zak <kzak@redhat.com>

libmount: add function to parse offsets/sizes

ipcrm: fix wrong subject in error messages for -m, -q, -s id options

This makes error messages print 'id' instead of 'key' using id options.

Signed-off-by: Francesco Cosoleto <cosoleto@gmail.com>

ipcrm: use ternary operator in "(key|id) failed" message

Signed-off-by: Francesco Cosoleto <cosoleto@gmail.com>

ipcrm: add static keyword to some functions

Signed-off-by: Francesco Cosoleto <cosoleto@gmail.com>

misc: use %m in format string instead of %s and strerror(errno)

Signed-off-by: Petr Uzel <petr.uzel@suse.cz>

docs: update TODO

TODO items already done by Sami Kerola

Signed-off-by: Petr Uzel <petr.uzel@suse.cz>

sfdisk: warn if partition exceeds partition table limits

Warn the user if the partition to be created exceeds limits
imposed by the DOS partition table, which is:
* partition has to start on sector < 2^32
* partition size has to be < 2^32 sectors

For 512-byte logical sector size, these limits are ~2.2 TiB.

Signed-off-by: Petr Uzel <petr.uzel@suse.cz>

sfdisk: use unsigned long long to internally represent sector number

Unsigned long is only 4 bytes long on i586, which is not enough to
represent sector number on todays large disks. Use unsigned long long,
which is 8 bytes long, to store the sector numbers internally, so that
we could later make some sanity checks and warn the user if the sector
numbers exceed the limits imposed by DOS partition table format.

Signed-off-by: Petr Uzel <petr.uzel@suse.cz>

sfdisk: introduce get_ull() function

Signed-off-by: Petr Uzel <petr.uzel@suse.cz>

sfdisk: get_disksize() returns long long

unsigned long is 4 bytes long on i586, which is not enough for big
HDD's with 512B sectors. Use unsigned long long, which is 8 bytes.

Signed-off-by: Petr Uzel <petr.uzel@suse.cz>

sfdisk: make the cylinder boundary check less fatal

If the specified format is not cylinders, make the cylinder boundary
check only print a warning and proceed anyways.

Signed-off-by: Petr Uzel <petr.uzel@suse.cz>

sfdisk: fix weird indentation in msdos_partition()

Signed-off-by: Petr Uzel <petr.uzel@suse.cz>

sfdisk: file descriptor is of int type

Signed-off-by: Petr Uzel <petr.uzel@suse.cz>

blkid: fix unfinished sentence for -l option in blkid.8

Signed-off-by: Dave Reisner <dreisner@archlinux.org>

uuidd: Add missing break to option case statement

Specifying the "-n" option to uuidd would incorrectly
fall through to the "-p" case, and assign that number to
the pidfile_path.

Signed-off-by: Eric Sandeen <sandeen@redhat.com>

libmount: fix use of logical '&&' with constant operand

Found by clang.

Signed-off-by: Francesco Cosoleto <cosoleto@gmail.com>

lib,loopdev: fix use of logical '&&' with constant operand

Found by clang.

Signed-off-by: Francesco Cosoleto <cosoleto@gmail.com>

flock: make flock(1) work on NFSv4

To pleace an exclusive lock on a file, NFSv4 requires
the file to be opened RW because of the emulation of flock()
by fcntl():

http://www.spinics.net/lists/linux-nfs/msg18502.html

So instead of O_RDONLY, open the file in O_RDWR if access()
indicates it is possible (unless shared lock is requested).

From: Michal Kubecek <mkubecek@suse.cz>
Signed-off-by: Petr Uzel <petr.uzel@suse.cz>

build-sys: add path.h to Makefile.am

Signed-off-by: Karel Zak <kzak@redhat.com>

lscpu: stricter command line parsing

Disallow superfluous commands for lscpu like e.g. "lscpu bla" and let it
fail print the help text instead.

Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>

lscpu: add --offline option

Implement "--offline" option which only prints offline cpus. As a side effect
we can get rid of the internal "allcpus" flag, since if we want to print
informations for online and offline cpus we simply set both flags.

When reading sysfs attributes of cpus this is now done for all cpus, since
e.g. the topology informations of the online cpus may influence the
topology informations of the offline cpus. This mainly because online cpus
may contain masks which include offline cpus while offline cpus have a
missing topology directory.

Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>

lscpu: fix -e output

The modifier mod->allcpus must be set earlier and also must be used
earlier. The current code only reads sysfs attributes from online
cpus but skips offline cpus.
So initialize mod->allcpus earlier.

Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>

chcpu: provide better user feedback

Instead of printing error messages like "I/O resource busy" which are
supplied by strerror, give better feedback if the reason of failure
is known.
E.g. taking the last cpu offline cannot succeed, therefore print a
message that gives this "hint".

Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>

lib,path: use write_all()

From: Heiko Carstens <heiko.carstens@de.ibm.com>

Since write() doesn't necessarily write the complete buffer with
one call we better use write_all() which takes care of this.

Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>

chcpu: convert to use lib/path.c

Use the common path access functions. In order to simplify chcpu also implement
and use path_writestr() which writes a string to the path specified.

Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>

lib,path: move path access functions from lscpu into lib/path.c

A couple of these functions already have been copied to chcpu.c,
so it makes sense to move these functions into an own file.

Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>

setarch: minor changes to usage()

Signed-off-by: Karel Zak <kzak@redhat.com>

pivot_root: minor changes to usage()

Signed-off-by: Karel Zak <kzak@redhat.com>

ipcs: minor changes to usage()

Signed-off-by: Karel Zak <kzak@redhat.com>

ctrlaltdel: minor changes to usage()

Signed-off-by: Karel Zak <kzak@redhat.com>

arch: minor changes to usage()

Signed-off-by: Karel Zak <kzak@redhat.com>

Merge branch 'sys-utils' of https://github.com/kerolasa/lelux-utiliteetit

* 'sys-utils' of https://github.com/kerolasa/lelux-utiliteetit: (29 commits)
  docs: add non-return function and if shorthand tips
  build-sys: fixes to USAGE_* macros
  ipcrm: check IPC syscalls
  ipcrm: add --verbose option
  ipcmk: allow high speed ipc creation
  ipcrm: add --all option
  docs: add long options to ipcs.1 man page
  docs: add long options to ipcrm.1 man page
  docs: add long options to ipcmk.1 man page
  docs: add --version to setarch.8
  docs: mention long options in ctrlaltdel.8
  ctrlaltdel: add version & help options
  docs: mention long options in pivot_root.8
  pivot_root: add version & help option
  ipcs: comment & white space clean up
  ipcs: include-what-you-use header check
  ipcs: add long options
  ipcrm: include-what-you-use header check
  ipcrm: refactor new and old main to share code
  ipcrm: exit if unknown error occurs
  ...

mount: use common libs

Get rid of the local xmalloc.[c/h] files by using the global xalloc and
strutils libraries.

Signed-off-by: Davidlohr Bueso <dave@gnu.org>

lscpu: add column names to --help

 * use uppercase by default (only -p uses lowercase)
 * a little cleanup usage()

Signed-off-by: Karel Zak <kzak@redhat.com>

lscpu: gettextize two columns

Signed-off-by: Karel Zak <kzak@redhat.com>

agetty: test ECHO on c_lflag

Reported-by: xinglp <xinglp@gmail.com>
Addresses: http://bugzilla.redhat.com/show_bug.cgi?id=739522
Signed-off-by: Karel Zak <kzak@redhat.com>

docs: add non-return function and if shorthand tips

Non-return functions should not be combined with `else' clause.

The if shorthands `var = e ? t : f;' need to fit to single line,
and if that does not look good use normal "if else" syntax.

Both tips are mentioned in email bellow.

http://www.spinics.net/lists/util-linux-ng/msg05152.html

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

build-sys: fixes to USAGE_* macros

The USAGE_BEGIN_TAIL is removed as unnecessary.

In between command specific options and --help & --version
USAGE_SEPARATOR is inserted. For now the separator is empty line.

The USAGE_MAN_TAIL is changed to take an argument.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

ipcrm: check IPC syscalls

It's not enough to check errno for errors as the variable is not
reset, we also need to check the last syscall return value to
verify a problem. This addresses bogus msgqueue errors when
deleting keys.

Signed-off-by: Davidlohr Bueso <dave@gnu.org>
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

ipcrm: add --verbose option

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

ipcmk: allow high speed ipc creation

This commit will allow user to create quickly plenty of ipc
resources. Earlier the line bellow gave no satisfaction.

for i in $(seq 0 42); do ipcmk -Q; done

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

ipcrm: add --all option

An --all option will remove all ipc entries. The option takes
optional resource argument, which limits the removal to be
applied only the given resource entries.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

docs: add long options to ipcs.1 man page

Includes few style fixes from Documentation/howto-man-page.txt
and align information by what is being said by The Open Group.

http://pubs.opengroup.org/onlinepubs/009604599/utilities/ipcs.html

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

docs: add long options to ipcrm.1 man page

Includes few style fixes from Documentation/howto-man-page.txt

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

docs: add long options to ipcmk.1 man page

Includes few style fixes from Documentation/howto-man-page.txt

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

docs: add --version to setarch.8

Includes few style fixes from Documentation/howto-man-page.txt

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

docs: mention long options in ctrlaltdel.8

Includes other manual page maintenance fixes.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

ctrlaltdel: add version & help options

Including other necessary changes to usage().

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

docs: mention long options in pivot_root.8

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

pivot_root: add version & help option

Including other necessary changes to usage().

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

ipcs: comment & white space clean up

Add to multiline comments to have left side stars, convert spaces
to tabs and indent preprocessor directives

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

ipcs: include-what-you-use header check

Two includes added & a sort to alphabetical order.

ipcs.c should add these lines:
 #include <features.h>                   // for __GLIBC__
 #include <stddef.h>                     // for size_t

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

ipcs: add long options

Includes necessary usage() changes.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

ipcrm: include-what-you-use header check

Three removes & a sort to alphabetical order.

ipcrm.c should remove these lines:
- #include <ctype.h>  // lines 29-29
- #include <sys/types.h>  // lines 18-18
- #include <unistd.h>  // lines 27-27

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

ipcrm: refactor new and old main to share code

The code is now much more tidy, and as a bonus old main has
shares error printing with the `new' code.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

ipcrm: exit if unknown error occurs

Previously for instance lack of memory space caused unknown
reason to be printed, and ipcrm tried to continue.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

mount: check for target before source on remount

Addresses: http://bugzilla.redhat.com/show_bug.cgi?id=737091
Reported-by: Eric Paris <eparis@redhat.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

lib: do not attempt to close(0) in sysfs_deinit()

If the 'open' fails we 'goto err' which runs 'sysfs_deinit()' on a 'cxt'
which hasn't been fully initialised. The 'dir_fd' is still 0, so
sysfs_deinit calls "close(0)".

Addresses: https://bugzilla.novell.com/show_bug.cgi?id=714151
Reported-by: Diego Ercolani <diego.ercolani@gmail.com>
Analysed-by: Neil Brown <nfbrown@suse.com>
Signed-off-by: Petr Uzel <petr.uzel@suse.cz>

chfn: remove non-ANSI definitions [smatch scan]

Signed-off-by: Karel Zak <kzak@redhat.com>

build-sys: add err and errx to smatch no_return_funcs

Signed-off-by: Karel Zak <kzak@redhat.com>

build-sys: add smatch data

Signed-off-by: Karel Zak <kzak@redhat.com>

ipcrm: add long options

With long options usage function had to be changed. The change
also takes libc error printing facilities to use, primarily to
get rid of execname & progname variables.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

ipcmk: include-what-you-use header check

Three removes & a sort to alphabetical order.

ipcmk.c should remove these lines:
- #include <string.h>  // lines 24-24
- #include <sys/types.h>  // lines 30-30
- #include <unistd.h>  // lines 29-29

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

ipcmk: remove camel casing

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

ipcmk: validate numeric option arguments

Use strtoul_or_err() instead of atoi().

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

ipcmk: remove useless code

Checking same return value twice does not make command any
better. Secondly the program_invocation_short_name is known to
work, so global progname variable does not add anything extra.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

ipcmk: add long options & fix usage()

Signed-off-by: Sami Kerola <kerolasa@iki.fi>