swapon: cleanup man page

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

swapon: allow a more flexible swap discard policy

Introduce the necessary changes to swapon(8) allowing a sysadmin to leverage
the new changes introduced to sys_swapon by "swap: discard while swapping
only if SWAP_FLAG_DISCARD_PAGES", therefore allowing a more flexible set of
choices when selection the discard policy for mounted swap areas.
This patch introduces the following optional arguments to the already
existent swapon(8) "--discard" option, in order to allow a discard type to
be selected at swapon time:
 * once    : only single-time area discards are issued. (swapon)
 * pages   : discard freed pages before they are reused.
If no policy is selected both discard types are enabled. (default)

[kzak@redhat.com: - support <policy> argument for short -d option too,
                  - add errx() on unsupported policy name]

Signed-off-by: Rafael Aquini <aquini@redhat.com>
Acked-by: KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

last: mention optional arguments in usage

Reported-by: Bernhard Voelker <mail@bernhard-voelker.de>
Signed-off-by: Karel Zak <kzak@redhat.com>

last: add --present option

The --present option will allow user to easily determine who where logged
in at a given time.

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

last: allow --file to be defined multiple times

This is useful when an admin is trying to find something, and has to
process all available data.

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

last: global variables are initialize automatically to zero

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

last: use as narrow variable scoping as possible

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

last: remove broken code

Comment in the deletion tells everything necessary.

"This doesn't work on modern systems, where only a DNS lookup of the
result from hostname() will get you the domainname.  Remember that
domainname() is the NIS domainname, not DNS.  So basically this whole
piece of code is bullshit."

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

last: prefer enum rather than #definition list

Incremental number lists are more hard to get wrong with enum, and they
are nicer to debug as for example gdb is aware of these symbolic names.

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

last: make switch cases complete, and inform if impossible occurs

Some of the ut_type numbers does not seem to be recognized by last(1) so
they are, at least for now, silently ignored.  See glibc documentation
for information what the ignored EMPTY, INIT_PROCESS, LOGIN_PROCESS, and
ACCOUNTING mean.

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

last: check expected numeric user input is number

This commit also changes the line count to use unsigned integers, as
negative numbers in this context does not make sense.

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

last: use carefulput() for printable character output

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

include: carefulput: print determined char when unprintable char is found

This is done to allow reuse of the functin in last(1).

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

docs: add long options to last.1 manual page

And align the manual page with Documentation/howto-man-page.txt

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

docs: be clear which last(1) is deprecated

The recently merged sysvinit last(1) and lastb(1) are supported.

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

build-sys: complete elvtune removal

The first removal managed to remove only most of the elvtune, this commit
will complete the task.

Original-removal: 7f1fe74248290c50f5447c2c09fcf0f0575a7666
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

docs: update TODO

The minix type fixes took place December 2012.

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

build-sys: remove unnecessary environment variable

The addition to original removal done in commit 952f1d79.

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

pylibmount: improve helps strings formatting

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

pylibmount: add debug messages

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

libmount: cleanup libmnt_fs list after mnt_table_remove_fs()

.. otherwise mnt_free_fs() will try to remove FS from non-existing
list.

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

pylibmount: use libmount reference counting

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

misc: use libmnt_table reference counter

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

libmount: add reference counter to libmnt_table

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

misc: use libmnt_cache reference counting

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

libmount: add reference counting to libmnt_cache

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

fsck: use libmount fs reference counter

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

libmount: add reference counting to libmount_fs

 * mnt_new_fs() returns object with refcount=1
 * mnt_free_fs() does not care about reference counter

 * new functions mnt_ref_fs() and mnt_unref_fs()

 * mnt_table_add_fs() and mnt_table_rem_fs() uses reference counter

 * libmmnt_context uses reference counter for internal FS (as it could be
   shared outside the context)

 * backwardly incompatible change:

- FS could be deallocated after mnt_table_remove_fs()

 * it's recommended to use mnt_unref_fs() after mnt_table_add_fs()

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

pylibmount: remove Context_get_table()

It's too problematic and too low-level to support it Py binding.

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

pylibmount: cleanup cxt usage

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

libmount: add function for context userdata

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

pylibmount: remove unnecessary code

It's enough to call mnt_free_fs() to remove fs from the table.

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

libmount: add mnt_table_is_empty(), improve table list usage

Currently you have to use mnt_table_remove_fs() + mnt_free_fs() to
destroy the list in the table. This is complicated in same situations.
This patch allows to use mnt_free_fs() only.

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

pylibmount: coding style changes, don't touch Context() status

It's better to follow libmnt_context status than allow to play any
nasty games with this important variables in Python.

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

pylibmount: use only public libmount API in tab.c

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

libmount: add mnt_table_{first,last}_fs()

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

libmount: add new function to docs

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

libmount: add mnt_table_{set,get}_userdata()

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

libmount: add mnt_table_with_comments()

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

pylibmount: cleanup *_HELP strings formatting

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

pylibmount: use mnt_fs_set_userdata() rather than ->userdata

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

pylibmount: fs.c - cleanup, remove libmount private stuff

 * remove unnecessary comments
 * cleanup up strings format
 * remove direct access to private libmount stuff

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

pylibmount: remove bindsrc from fs API

It's unnecessary and very low-level.

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

pylibmount: cleanup object names

Cxt -> Context
Tab -> Table

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

pylibmount: coding style changes in tab.c

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

pylibmount: coding style changes in fs.c

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

pylibmount: coding style changes in context.c

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

tests: consolidate Py tests initialization

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

build-sys: install pylibmount into

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

pylibmount: add regression tests

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

pylibmount: add __init__.py

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

pylibmount: basic code

[kzak@redhat.com: - split to more patches
                  - split to more .c files]

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

build-sys: add pylibmount

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

build-sys: remove deprecated elvtune command

The command is deprecated for years and useless with 2.6.x kernels.

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

build-sys: enable mesg(1) by default

It seems that our mesg(1) implementation is compatible with sysvinit,
let's use it by default in mainstream distros.

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

wall: make banner sysvinit compatible

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

wall: allow to specify <message> on command line

wall(1) from sysvinit supports

   # wall I love this company!

semantic, This patch add this functionally to the util-linux wall.

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

last: add link to util-linux to last.1

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

last: coding style clean up

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

last: use xalloc.h, err.h, nls.h and EXIT_ macros

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

last: consolidate btmp/wtmp file paths

 .. so share the paths with another utils.

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

last: add long options, improve usage()

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

build-sys: fix last stuff

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

build-sys: improbe UL_CONFLICT_BUILD macro

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

last: clean up the begin of the file

 .. and improve last(1) stuff in configure script.

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

last: merge sysvinit last/lastb

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

agetty: -L accepts optional argument

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

losetup: use loop-control to explicitly ask for device

Now we use LOOP_CTL_GET_FREE ioctl to ask for free device, for example

  losetup -f foo.img

Unfortunately, losetup(8) allows to ask for specified device

  losetup /dev/loop100 foo.img

and in this case we assume that the device already exists in the
system. This is incorrect, we should be able to use loop-control
LOOP_CTL_ADD ioctl to ask for the specified device.

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

blkdiscard: use fstat() rather than stat() [coverity scan]

 Calling function "open(char const *, int, ...)" that uses "path" after
 a check function. This can cause a time-of-check, time-of-use race
 condition.

.. well, in blkdiscard context it's mostly cosmetic change.

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

losetup: fix loop device name usage

The code is not paranoid enough, loopcxt_set_device() only set the
device name to loopdev struct, but it does not check if the device
really exists.

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

libmount: be robust when work with loopdev backing file paths

It's usually unnecessary as we compare devno and ino, but let's
use absolute paths for situations when it's necessary to compare
paths as strings.

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

losetup: make -j <path> more robust

It's usually unnecessary as we compare devno and ino, but let's
use absolute paths for situations when it's necessary to compare
paths as strings.

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

libmount: canonicalize for conversion from loopdev backing file

  # mount foo.img /mnt
  # umount foo.img
  umount: foo.img: not mounted

The loopdev code (and sysfs backing_file) uses absolute paths, but
libmount does not canonicalize the path before lookup for the backing file.

References: https://bugzilla.redhat.com/show_bug.cgi?id=950497
Signed-off-by: Karel Zak <kzak@redhat.com>

cal: set statically defined data read-only

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

more: make output redirection more efficient

Especially with large inputs the change improves performance
considerably.

old> time more /boot/vmlinuz >/dev/null
real    0m0.224s

new> more /boot/vmlinuz >/dev/null
real    0m0.009s

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

column: use variable lenght printf field width to wprint blanks

This makes program to run a little faster.  My test input show about 20%
speed improvement.

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

libmount: fix typos

Signed-off-by: Ondrej Oprala <ooprala@redhat.com>

more: guarantee space for multibyte

.. to make the code more robust.

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

more: check for buffer size when write multibyte char

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

mkswap: add note about swap header limit to the man page

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

rev: use string printing rather than character output

Fliping a string in memory, and printing it with multibyte output
function makes the command about 1/3 quicker.

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

build-sys: use backticks rather than $() for commands in configure

The configure had both backticks and $() in use.  These command
susbstitutions are mutually interchangeable, so one should pick only one
form.  In this case backticks were favored because they are known with
greater range of shells, making the backticks to be more portable.

References: https://www.gnu.org/software/autoconf/manual/autoconf-2.67/html_node/Shell-Substitutions.html#index-g_t_0024_0028_0040var_007bcommands_007d_0029-1632
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

build-sys: prefer AS_IF rather than shell 'if'

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

build-sys: prefer AS_CASE rather than shell 'case'

References: http://www.gnu.org/savannah-checkouts/gnu/autoconf/manual/autoconf-2.69/html_node/Limitations-of-Builtins.html#case
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

build-sys: use m4 quoting consistently

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

setarch: prefer preprocessor rather than autotools check

It seems to be pointless to spend time in ./configure phase when
preprocessor has to perform #ifndef check anyway.

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

lscpu: fix shadow declaration [smatch scan]

sys-utils/lscpu.c:572:22: warning: symbol 'fd' shadows an earlier one

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

more: fix buffer overflow

The bug has been probably introduced by commit
1ac300932deab8dea2c43050921bbbdb36d62ff1.

Reported-by: "Dr. David Alan Gilbert" <dave@treblig.org>
References: https://bugzilla.novell.com/show_bug.cgi?id=829720
Signed-off-by: Karel Zak <kzak@redhat.com>

setsid: add an option to wait child return value

This change proposed in util-linux maillist by Damien Wyart in behalf of
Daniel Kahn Gillmor.  Sami Kerola changed the patch to use option rather
than makign the default behavior of the command to change.

Reported-by: Damien Wyart <damien.wyart@gmail.com>
Reference: http://markmail.org/message/2z6qh5auk7wjie3l
Bug-Debian: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=495881
CC: Daniel Kahn Gillmor <dkg@fifthhorseman.net>
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

libmount: add missing declarations to libmount.h.in and documentation

Signed-off-by: Ondrej Oprala <ooprala@redhat.com>

libmount: change tailing to trailing in *_tailing_comments functions

Signed-off-by: Ondrej Oprala <ooprala@redhat.com>

dmesg: don't use pager for --follow

References: https://bugzilla.redhat.com/show_bug.cgi?id=984666
Signed-off-by: Karel Zak <kzak@redhat.com>

losetup: set errno for misaligned offsets

References: https://bugs.archlinux.org/task/36189
Signed-off-by: Karel Zak <kzak@redhat.com>

mount: update xfs docs

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

hwclock: remove references to adjtimex from man page

The adjtimex is optional on many distros.

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

sfdisk: don't use word 'disk'.

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

uuidd: add missing initializer [clang-analyzer]

Reported-by: Sami Kerola <kerolasa@iki.fi>
Signed-off-by: Karel Zak <kzak@redhat.com>

libmount: fix memory leak [clang-analyzer]

Reported-by: Sami Kerola <kerolasa@iki.fi>
Signed-off-by: Karel Zak <kzak@redhat.com>

sulogin: fix compiler warning

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

more: use variable lenght printf field width to print blanks

This makes program to run a little faster.

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