libblkid: use stuff from sysfs.h and at.h

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

lsblk: use generic sysfs functions

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

lib: add generic sysfs utils

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

libblkid: use cached buffers for nested PT probing

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

libblkid: make whole disk probing more robust

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

tests: update for recent fdisk changes

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

libblkid: cleanup prober initialization

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

libblkid: cleanup flags

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

mount: remove note about obsolete volume_id library from man page

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

blkid: don't read past end of FAT32 cluster chain

When looking for a volume label on a FAT32 filesystem, blkid does not
correctly handle special cluster values in the range 0x0FFFFFF8 to
0x0FFFFFFF, which mark the last cluster in a chain. As a result, it
begins to read [what it treats as] FAT entries from past the end of the
FAT. Depending on the data read, it may then try to parse random data
from the filesystem (including user files, free space, and other
directories) as though it were part of the root directory.

Because parsing stops early when a volume label is found, the problem
only occurs on filesystems without a volume label. When it occurs, it
may result in a long sequence of lseek and read calls; on an older IDE
drive with a 2 GB FAT32 partition, this amounted to around 3 seconds of
disk I/O.

After patching, blkid stop parsing as soon as it reaches a cluster value
greater than or equal to the number of entries in the FAT.

[kzak@redhat.com: - add le32_to_cpu()]

Signed-off-by: John Lindgren <john.lindgren@tds.net>

tests: add mount(8) test for fstab entries without options

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

mount: -a segfaults when 4th field is omitted (mount options)

 # echo 'tmpd /tmp/x tmpfs' >> /etc/fstab
 # mkdir /tmp/x
 # mount -a
 segfault

Reported-by: Mike Frysinger <vapier@gentoo.org>
Signed-off-by: Karel Zak <kzak@redhat.com>

taskset: make threads aware

Add a new '-a' option to view/modify the CPU affinity for an entire
group of threads belonging to a given PID.  We create two new
functions, print_affinity() and do_taskset() for code simplification.

Example:

zeus@jilguero:~/src/util-linux/schedutils$ ./taskset -a -p 01 3142
pid 3142's current affinity mask: 2
pid 3142's new affinity mask: 1
pid 3164's current affinity mask: 2
pid 3164's new affinity mask: 1
pid 854's current affinity mask: 2
pid 854's new affinity mask: 1

[kzak@redhat.com: - clean up
                  - move variables to struct taskset]

Signed-off-by: Davidlohr Bueso <dave@gnu.org>
Tested-by: Jonathan Gonzalez <zeus@gnu.org>
Signed-off-by: Karel Zak <kzak@redhat.com>

taskset: use xalloc lib

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

fdisk: accept digits-space-suffix format

This is for users who are used to separate with a space numbers from units.

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

chrt.1: fix grammar

Corrent the English grammar for the new -a option.

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

fdisk: print logical partition number while adding it

[kzak@redhat.com: - merged two Francesco's patches]

Signed-off-by: Karel Zak <kzak@redhat.com>
Signed-off-by: Francesco Cosoleto <cosoleto@gmail.com>

fdisk: quit from partition type dialog after invalid response

This gets rid of the while() loop and fixes indentation.

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

fdisk: add a default response for the partition type dialog

The default response is 'p' until three primary partitions are created,
then 'e'.

The original idea is from Karel Zak.

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

fdisk: change primary or extended partition type dialog

Mainly this appends a colon for the prompt, reverses order of the list,
reports the number of primary partitions used and unused.

Old version:
 Command action
    e   extended
    p   primary partition (1-4)
 p

New version:
 Partition type:
    p   primary (0 primary, 0 extended, 4 free)
    e   extended
 Select: p

Old version:
 Command action
    l   logical (5 or over)
    p   primary partition (1-4)
 l

New version:
 Partition type:
    p   primary (0 primary, 1 extended, 3 free)
    l   logical (numbered from 5)
 Select: l

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

findmnt: add columns description to the --help output

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

partx:  add --pairs to output in key="value" format

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

findmnt: add --pairs to output in key="value" format

... usable in scripts, for example:

findmnt --pairs -o "SOURCE,TARGET,FSTYPE" $1 | while read line; do
        eval $line
        echo "$SOURCE is mounted on $TARGET [$FSTYPE]"
done

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

findmnt: fix leak

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

lsblk: add --pairs to output in key="value" format

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

lib: [tt.c] support key="value" output format

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

sfdisk: returns non-zero on failed BLKRRPART ioctl

Make sfdisk(8) return non-zero value on failed BLKRRPART ioctl().
This will happen for either partition table re-reads after partition
changes or on explicit use of -R switch.

The function reread_disk_partition() was changed from 'void' to 'int'
and now returns a result to let the calling functions decide to exit(1).

Signed-off-by: Andrea Galbusera <gizero@gmail.com>

libblkid: ignore hfsplus superblocks with blocksize < 512

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

libblkid: validate hfs blocksize != 0

Validate that blocksize != 0 as a blocksize == 0 can cause
a division-by-zero error.

Signed-off-by: Timo Warns <warns@pre-sense.de>

docs: update TODO file

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

addpart: multiplication on 512 deleted

Multiplication on 512 now placed in partx/kpartx.h in '<< 9' form.
Double moutiplication causes an error.

chrt: allow to use --all-tasks when retrieve info

master thread:
 $ chrt --pid $(pidof firefox)

all threads:
 $ chrt --all-tasks --pid $(pidof firefox)

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

chrt: make threads aware

Currently this program works only with the master thread. Add a '-t'
option to propagate changes to the entire group of threads.

Example:

root@offworld:~/projects/util-linux/schedutils# ls /proc/2111/task/
2111  2112  2119  2121  2138  2139  2159  2160
root@offworld:~/projects/util-linux/schedutils# ./chrt -p 2111
pid 2111's current scheduling policy: SCHED_RR
pid 2111's current scheduling priority: 3
root@offworld:~/projects/util-linux/schedutils# ./chrt -t -p 2 2111
root@offworld:~/projects/util-linux/schedutils# ./chrt -p 2112
pid 2112's current scheduling policy: SCHED_RR
pid 2112's current scheduling priority: 2
root@offworld:~/projects/util-linux/schedutils# ./chrt -p 2111
pid 2111's current scheduling policy: SCHED_RR
pid 2111's current scheduling priority: 2

[kzak@redhat.com: - rename -t/--thread to -a/--all-tasks]

Signed-off-by: Davidlohr Bueso <dave@gnu.org>
Signed-off-by: Karel Zak <kzak@redhat.com>

procutils: include missing header

To be able to use DIR we need to include the dirent.h header. This fixes issues such as:

../include/procutils.h:5: error: syntax error before "DIR"
../include/procutils.h:5: warning: no semicolon at end of struct or union

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

lsblk: add support for discard topology (-D option)

I got tired of poking around in sysfs to find the discard topology.
Here's a patch against lsblk that adds a -D option to present this
information in a human-readable form:

NAME                  DISC-ALN DISC-GRAN DISC-MAX DISC-ZERO
sda                          0        0B       0B         0
└─sda1                       0        0B       0B         0
sdb                          0      512B       2G         1
└─sdb1                       0      512B       2G         1

Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

lib: [procutils.c]: add missing files. Sorry.

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

lib: [procutils.c]: general purpose procfs parsing functions

We include the following functions as a first approach:

  - proc_open_tasks(): allocate resources and setup basic data
  - proc_next_tid(): iterate over the thread group
  - proc_close_tasks(): free used resources

[kzak@redhat.com: - free() on error in proc_open_tasks(),
                  - always call free() in proc_close_tasks()
                  - ignore non-digit dirs and return 1 on end-of-dir
                    in proc_next_tid()]

Signed-off-by: Davidlohr Bueso <dave@gnu.org>
Signed-off-by: Karel Zak <kzak@redhat.com>

mkfs.minix: remove die()

Get rid of this function and use errx(3) instead. This patch also
introduces a mkfs.h header and defines general purpose mkfs related
exit codes.

[kzak@redhat.com: - fix "warning: too many arguments for format" errors]

Signed-off-by: Davidlohr Bueso <dave@gnu.org>
Signed-off-by: Karel Zak <kzak@redhat.com>

fdisk: improve error message for missing extended partition

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

fdisk: fix "invalid partition number for type" error message

You can't input a partition number here.

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

docs: add Matej to AUTHORS

The original idea to have something like fallocate(1) is from Matej
Cepl -- his mkfile(1) prototype has been originally discussed on
coreutil mailing list. Thanks!

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

docs: agetty manual update

Tell users how to see what issue escape will print by using other
commands, such as uname.

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

agetty.c: move issue path to pathnames.h

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

agetty.c: further scrubbing

In reply to two requests.

http://permalink.gmane.org/gmane.linux.utilities.util-linux-ng/3990
http://permalink.gmane.org/gmane.linux.utilities.util-linux-ng/3999

The scubbing consists;

o Old school introduction back to the top of the source file.
o Indentation changes.
o Removal of void casting.
o Removal of braces from single statements.
o Unified format for comments.
o Comparison with W. Venema's code from 1993 to clean few comment
  mysteries.
o Web referrals added to further explanation to near the code or
  comments which some could find difficult to understand why.
o Clean up of comments.
o FIXME items added where attention is needed.
o Make functions static.
o Functions with same argument types use same argument order.
o Get rid of classic K&R style function declarations.

Please notice that this patch _should not_ affect how program
works e.g. this has no actual code changes.

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

findmnt: minor coding style changes

 * remove __<MACRONAME> from code
 * use 'static' for global variables

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

findmnt: filter filesystems and actions for --poll

The patch allows to filter by:

 * actions name:
$ findmnt --poll=umount

 * target, source, fstype, options:
$ findmnt --poll /mnt/test
$ findmnt --poll -O ro
$ findmnt --poll -t ext4

 * --first-only option to wait only for the first matching change:

$ findmnt --poll=umount --first-only /mnt/test

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

findmnt: print OLD-* columns only when necessary

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

findmnt: add --poll and --timeout to the man page

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

findmnt: add --timeout option

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

findmnt: add --poll option

The --poll functionality is based on poll(/proc/self/mountinfo) and
mnt_tabdiff_* functions from libmount.

 $ findmnt --poll
 ACTION     TARGET    SOURCE    FSTYPE OPTIONS
 mount      /mnt/test /dev/sda1 ext3   rw,relatime,errors=continue,user_xattr,acl,barrier=0,data=ordered

... after "mount /dev/sda1 /mnt/test".

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

libmount: minor fix to mnt_tabdiff_*

 - tabdiff has to support NULL fs->source
 - disable some debug messages

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

lib: [tt.c] support fixed width and multiple tt_print_table() calls

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

libmount: add mnt_tabdiff_* functions

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

libmount: add mnt_reset_table()

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

libmount: support NULL source path for mnt_table_find_* functions

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

build-sys: non-linux fixup

Signed-off-by: Samuel Thibault <samuel.thibault@ens-lyon.org>

libmount: add support for x-* mount comments

The current way how add any comment or extra information to /etc/fstab
is to use comment="" mount option. This is not too elegant and readable
if you want to add more than one comment.

It seems better to add any generic prefix that will be used for all
3rd-party userspace mount options.

Expected syntax is:

  x-<application>-<options>[=<data>]

For example:

  x-systemd-bar=managed

All x-* options (as well as comment=) are ignored by libmount and not
stored to the mtab file.

Requested-by: Lennart Poettering <lennart@poettering.net>
Signed-off-by: Karel Zak <kzak@redhat.com>

tests: don't run mount/regfile test on old kernel

The /sys/.../loop/backing_file is available since 2.6.37.

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

tests: add test for RLIMIT_FSIZE issue

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

tests: use libmount tab update tests for UID=0 only

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

getopt: fix gcc warning

getopt.c:80:13: warning: ‘set_initial_shell’ declared ‘static’ but
never defined

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

mesg: add \n to usage() output

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

mesg: man page clean wqup

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

mesg: new --verbose option

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

mesg: add long options to the man page

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

mesg: use long options, also --help and --version added

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

agetty: coding style - fix identation

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

agetty: coding style - fix comments, remove tailing whitespace

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

agetty: getopt case segment reordering

The case clause after getopts is in alphabetical order, just like
option string definition and manual page. The patch also convers
spaces to tabs in this code block.

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

docs: agetty long options

Long options to manual page with the new --version and --help.
The manual page option order is also set to be the same as it is
in code. For the persons who don't see the order it is short
option case in sensitive alphabetical order.

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

agetty: symbolic standard file descriptors

I find it easier to read code in which standard file descriptors
are symbolic strings rather than numbers.

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

agetty: #endif comments

The longer #ifdef ... #endif markups have now at the end section
comment where they begun, which should improve preprosessor
directive readability.

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

agetty: error message about required arguments

This patch also fixes only remaining magical number to EXIT_*
markup.

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

agetty: use long options

This patch will introducing --help and --version options, which
does not have short version. Reason to this is a that the -h was
already assigned.

[kzak@redhat.com: - fix one usage call,
                  - use %1$s in usage printf format
                  - remove "static const" from enum]

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

agetty: use program_invocation_short_name

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

agetty: use xalloc.h

Unifies and fixes malloc and strdup calls.

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

libmount: use libmnt_lock for utab flock

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

libmount: merge mtab and utab locking code

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

libmount: block signals when update utab

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

libmount: more robust mtab and utab update (CVE-2011-1676, CVE-2011-1677)

http://thread.gmane.org/gmane.comp.security.oss.general/4374

Changes:

 - always use temporary file

 - use fflush() for the temporary file

 - check fprintf() return value

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

mount: use lock from mnt_update_table()

The mnt_update_table() is able to create a mtab lock now.

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

umount: block signals when umounting and updating mtab (CVE-2011-1676, CVE-2011-1677)

http://thread.gmane.org/gmane.comp.security.oss.general/4374

Based on CVE-2011-1089.

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

mount: use fflush() and temporary file for mtab updates (CVE-2011-1089)

http://thread.gmane.org/gmane.comp.security.oss.general/4374

Changes:

  - force mount(8) to use /etc/mtab.tmp file every time. The original
    code used the tmp file for remount/move operations only.

  - call and check fflush() return code for the tmp file

Note mount(8) blocks all signals when writing to mtab, so it's not
affected by SIGXFSZ and the mtab lock file is always removed.

This patch does not fix the same issue in umount(8) and libmount.

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

libblkid: remove test for non-zero head count in FAT superblock probe.

This change enables the auto-detection of the internal storage on a VTech
Kidizoom camera, which has a zero head count value in the superblock.

Microsoft documentation, "FAT: General Overview of On-Disk Format", states
that the sectors and heads fields are only relevant for media that have
a geometry.

The Linux kernel also does not require a non-zero head count, and can
manually mount the filesystem.

Signed-off-by: Nick Holloway <Nick.Holloway@pyrites.org.uk>

script: optional timing output file argument added

And update to manual page accordingly.

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

docs: scriptreplay add note about new options

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

scriptreplay: allow arguments as command line switches

The command line switches override old style arguments

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

docs: scriptreplay mention basic long options

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

scriptreplay: support long options

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

scriptreplay: end printing with new line

Depending on opinion this is either modest bug fix or tiny
improvement.

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

libmount: avoid redundant declaration of mnt_context_do_umount

Signed-off-by: Jan Engelhardt <jengelh@medozas.de>

mkswap: add alignment check

This patch produces the program to warn if the given device is misaligned.

[kzak@redhat.com: - call the check for block devices only]

Signed-off-by: Davidlohr Bueso <dave@gnu.org>
Signed-off-by: Karel Zak <kzak@redhat.com>

blkdev: add blkdev_is_misaligned()

This functions returns the status of the device's alignment. It will
be 0 when aligned, otherwise return the offset.

[kzak@redhat.com: - returns 0 if the ioctl failed]

Signed-off-by: Davidlohr Bueso <dave@gnu.org>
Signed-off-by: Karel Zak <kzak@redhat.com>

hexdump: don't include err.h directly

script: don't include err.h directly

iosize: fix gcc warning

isosize.c: In function ‘main’:
isosize.c:168:8: warning: unused variable ‘p’

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

elvtune, isosize: print usage text in case of invalid option

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

isosize: use program_invocation_short_name

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

isosize: simplify some error messages

perror() and fprintf() are replaced with err() calls

Signed-off-by: Francesco Cosoleto <cosoleto@gmail.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

elvtune, isosize: remove redundant message in case of invalid option

This leaves getopt() only to print a similar error message on invalid
options.

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