libfdisk: use reference counting for context

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

libfdisk: extend dump parsing function

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

libfdisk: (dos) fix typo

Reported-by: https://github.com/Mikachu
Signed-off-by: Karel Zak <kzak@redhat.com>

include/optutils: fix typo

Reported-by: Rafael Ferreira <rafael.f.f1@gmail.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

cfdisk: fix typo in the previous commit

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

cfdisk: wrong interpretation of yes in non-english language

Reported-by: tnut@nutyx.org
Signed-off-by: Karel Zak <kzak@redhat.com>

libfdisk: (dos) fix logical partitions add/delete code

If you delete logical partition and then create a new one than fdisk
(and cfdisk) might write EBR to the first sector on the device. That's
wrong of course; because you will lost MBR (primary partitions).

(Probably introduced by commit bcddbe96882b88d53b6bc0495e7322c0820a5122
 where code completely clears EBR stuff.)

Reported-by: Michał Górny <mgorny@gentoo.org>
Signed-off-by: Karel Zak <kzak@redhat.com>

libfdisk: vary basic dump parsing functions

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

libblkid: move string trim function to strutils.h

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

libfdisk: basic fdisk_dump_* functions

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

libfdisk: remove debug dump function, cleanup API

The fdisk_dump_* prefix will be used for sfdisk-like functionality.
The patch also add FDISK_ prefix to fdisk_get_unit() options.

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

mountpoint: clarify that it works on files

fef4dfa174 removed the restriction that the argument must explicitly be
a directory, so let's fix up the documentation to reflect that.

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

mountpoint: drop S_ISDIR check

Files can be mountpoints, too.

libmount: use -t for type.subtype in helpers API

References: https://github.com/karelzak/util-linux/issues/116
Signed-off-by: Karel Zak <kzak@redhat.com>

tests: apply travis clang PATH workaround

As seen in Travis build log the clang is not found, causing one of the
tests to fail.  This change adds a workaround from Travis issue 2607 to
util-linux.

Reference: https://travis-ci.org/karelzak/util-linux/jobs/33056101#L3630
Reference: https://github.com/travis-ci/travis-ci/issues/2607
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

logger: use generic --id=<id>

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

Merge remote-tracking branch 'sami/logger'

libfdisk: fix fdisk_get_optimal_iosize() and update tests

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

tests: fix fdisk/bsd for the two possible sectors/offsets

BSD disklabels depend on the endianess, but also on the architecture as
one can see in include/pt-bsd.h with the BSD_LABELSECTOR and
BSD_LABELOFFSET #define. This cause the testsuite to fail on the
affected architectures as the disk image is not the same as the expected
result.

Commit 180b3a7e tried to fix the endianess, but the special PowerPC case
has been chosen as a reference, so it still fails for example on MIPS BE
or S/390.

This patch fixes the testsuite by converting the md5sums to the expected
values, still depending on the endianess, but also for the two possible
sectors/offsets. This has been tested on Alpha, MIPS, PowerPC and x86-64,
so this should cover all 4 cases.

Cc: Andreas Henriksson <andreas@fatal.se>
Signed-off-by: Aurelien Jarno <aurelien@aurel32.net>

docs: bring the chrt and taskset man pages closer to standard formatting

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

textual: make the license of chrt and taskset slightly more explicit

This makes it match the license of the man pages.
While there, also tweak some other comment lines.

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

docs: make the man-page licenses of chrt and taskset match their intent

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

docs: improve the description of --size in the man page of zramctl

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

docs: mentintion default udp and tcp ports logger is using

The fact that 'syslog tcp/514' does not exist in RFS's, which has lead
to 'syslog-conn 601/tcp' be used in place could be a suprice and should
be told in manual.

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

logger: set function arguments read-only when possible

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

logger: optimize string initializations

Setting whole array to be completely full of nulls cannot be as quick as
making the only significant member of the array when needed.

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

logger: remove openlog(3) options

One variable less, and more importantly bit operations become unnecessary
in if statements.

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

logger: allow use of --id=ppid when logging locally

There is no obvious way to make syslog(3) to print both pid or ppid, so
duplicate the libc syslog() to logger.  Making the ppid printing work
using unix socket has side effect of local becoming capable to use both
rfc format output, which is hopefully seen as good thing.  The
syslog_local() is format wise one-to-one copy with glibc syslog(3)
format.

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

fdisk: add independent debug stuff

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

libfdisk: move fdisk_ask_partnum() to API

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

fdisk: use libfdisk public API only

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

libfdisk: add geometry to API, cleanup

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

cfdisk: use only public libfdisk API

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

libfdisk: fdisk_label_is_labeltype()

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

libfdisk: more docs

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

libfdisk: final parttype API cleanup

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

libfdisk: move get_parttypes to label API

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

libfdisk: (gpt) allow to specify attr bit by API

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

libfdisk: make it possible to get fields for all labes

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

libfdisk: cleanup label geometry API

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

libfdisk: cleanup fdisk_is_label stuff

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

libfdisk: clean up API (alignment.c)

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

libfdisk: clean up API (context.c)

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

cfdisk: add libfdisk independent debug stuff

Now it's possible to use
CFDISK_DEBUG=0xffff ./cfdisk
to debug only cfdisk UI.

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

tests: add debug tests

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

include/debug: allow to speficy empty masknames

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

include/debug: cleanup masks

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

include/debug: make masknames optional

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

libs/debug: accept human readable names for _DEBUG=

For example
$ LIBMOUNT_DEBUG=tab,cache findmnt

to debug only TAB and CACHE subsystem.

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

libfdisk: rename fdisk_column to fdisk_field

.. to make the API and fdisk(s) code more readable.

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

libfdisk: remove dependence on libsmartcols

It's application business to convert libfdisk_table to string.

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

lib/colors: add function to return color from scheme

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

libblkid: (gpt) fix PT offset

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

Fix bash-completion Makemodule options

tests: make ipcs limit calculations work when numbers are large

Test suite  failed when I ran it with kernel 3.16.  Error is the
following.

tests/ts/ipcs/limits2: line 31: [: 18446744073692774399: integer expression expected

Reference: http://lkml.iu.edu/hypermail/linux/kernel/1406.0/01869.html
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

swapon: remove invalid error string print outs

To avoid messages like this:

$ swapon /dev/zram0
swapon: /dev/zram0: read swap header failed: Success

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

textual: remove some inconsistent periods from error messages

While doing so, also improve translatability and some wordings.

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

lscpu: clean up vmware inline asm

This code is not PIC clean which means it fails to build on hardened
32bit x86 systems (i.e. building as PIE).

While here, optimize the existing cpuid logic slightly.

URL: https://bugs.gentoo.org/518936
Signed-off-by: Mike Frysinger <vapier@gentoo.org>

zramctl: improve option combination error messaging

Use of --algorithm or --streams is not expected to work without
reseting device, that happens when --size is specified, so ensure the
former options to be combined with the later.

[kzak@redhat.com: - make it more generic]

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

zramctl: add bash completion script

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

zramctl: cannonicalize devname when no fullpath specified

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

zramctl: improve error message

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

zramctl: fail status printout when device does not exist

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

zramctl: add details to --size option documentation

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

fdisk: update tests and clean up output

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

libblkid: (gpt) use real sector size to set PTMAGIC_OFFSET

So stupid and 5 years old bug...

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

fdisk: always print warning for mis-aligned partitions

This patch restores the original fdisk behavior and always prints
information about misaligned partitions in 'p'print output.

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

cfdisk: move curs_set(1) to ui_end()

This is probably correct to have curs_set() in ui_end(), but according
to Martin it does not work with slang library from signal handler.

Reported-by: Martin Steigerwald <Martin@lichtvoll.de>
Signed-off-by: Karel Zak <kzak@redhat.com>

zramctl: remove extra \n from usage()

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

docs: fix inconsistencies in man pages

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

Merge branch 'master' of https://github.com/entertainyou/util-linux

zramctl: fix typo

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

hexdump: Fix parse format of "byte count without repetition count"

zramctl: fix typo

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

Merge branch 'master' of https://github.com/ottxor/util-linux

docs: improve the wording and formatting of the zramctl man page

Especially fix the mistaken short option -h for --noheadings.

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

zramctl: fix the wording of some message strings

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

tests: kill output message change

Commit 09af3db48e01e5744f15fd8a2395e0fed36526ea changed how a missing
process is reported, which caused kill test to fail when the test suite
is ran as root.

Reference: https://travis-ci.org/karelzak/util-linux/jobs/31312170
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

docs: restore the mention of uhelper= and helper= in man page of umount

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

build-sys: fix double prefix with explict --libdir

* with configure --prefix=$HOME --libdir=$HOME usrlib_execdir
  ends up being ${exec_prefix}/${libbir}, which is double-prefixed

* fix: use explict given ${libbir} if is matches ${prefix}/*

Signed-off-by: Christoph Junghans <ottxor@gentoo.org>

zrmactl: add new command to control /dev/zramN devices

Co-Author: Timofey Titovets <nefelim4ag@gmail.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

lib/ismounted: make it more compatible with lsblk output

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

lib/sysfs: allow to write to sysfs attributes

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

Merge branch 'travis' of git://github.com/kerolasa/lelux-utiliteetit

* 'travis' of git://github.com/kerolasa/lelux-utiliteetit:
  tests: fix rename subdir check that was failing in travis
  build-sys: remove swap-common.h from sys-utils/Makemodule.am

libfdisk: (gpt) add Microsoft Storage Spaces GUID

Storage Spaces is Microsoft's logical volume management.
It has been around since Windows 8.

Cc: Karel Zak <kzak@redhat.com>
Signed-off-by: Sergei Antonov <saproj@gmail.com>

tests: fix rename subdir check that was failing in travis

Apparently some or all travis build servers has a bit older coreutils
version.  The coreutils v8.13 added a feature to readlink that allows
user to specify multiple arguments.  Earlier versions gave exactly the
type of error travis_script log has.

+readlink: extra operand `rename_aa/sublink.2'
+Try `readlink --help' for more information.

The commit 0f85e0535bd4e6d19fadca182e4d57660b6bb57b introduced the travis
readlink issue, and has caused failures for 8 days in row (builds 192 -
213).

Reference: https://travis-ci.org/karelzak/util-linux/builds
Reviewed-by: Bernhard Voelker <mail@bernhard-voelker.de>
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

build-sys: remove swap-common.h from sys-utils/Makemodule.am

The commit 25a365e moved the file to include/

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

fdformat: clean up usage()

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

fdformat: Add new switches -f/--from, -t/--to, -r/--repair

This commit introduces a support for user configurable
from/to track and a basic repair mode for broken floppies.
It also fixes a recently introduced bug that causes
a line breakage when printing the track number.

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

docs: update TODO

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

nsenter: allow to use --set{uid,gid} for all namespaces

Now it's possible to set UID and GID for user namespaces only. This
patch removes this restriction and allow to use --set{uid,gid} in all
cases. The default for user namespaces is still GID=0, UID=0.

Reported-by: Tomas Doran <bobtfish@bobtfish.net>
Signed-off-by: Karel Zak <kzak@redhat.com>

tests: allow non-inotify tailf to keep up

When tailf is not using inotify, it seems to sleep for 0.25s between
checks. Just giving a 0.1s time window between update and removal of the
input file thus gives little chance for tailf to succeed.
Similar between startup and append to file, make sure initial
content is read before appending additional data by bumping the time.

This should possibly be bumped much larger to make sure that
the tailf process actually gets a chance to run at all in the
given time window. Otherwise it might fail on really slow/overloaded
machines.
The drawback would then ofcourse be to increase the time it takes
to run the testsuite.

Signed-off-by: Andreas Henriksson <andreas@fatal.se>

docs: bring the runuser and su man pages closer to standard formatting

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

docs: give the man page of utmpdump the proper User Commands header

Also bring the formatting closer to standard.

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

docs: improve the formatting of the chfn and chsh man pages

Also fix a pasting mistake where the chfn man page suggested
to use ypchsh or lchsh for non-local entries.

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

docs: remove all mention of helper= and uhelper= from umount man page

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

docs: sort the options in the man pages of hwclock and uuidd

Also improve the formatting a bit.

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

docs: bring the rtcwake man page closer to standard formatting

Also sort the options alphabetically, and use the standard comma
instead of the vertical bar to separate short from long option.

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

logger: fix untranslated message

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

logger: require --{server,socket} to remote rfc542/rfc3164 log

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