blkid: use "char **" rather than "unsigned char **"

Use "char" to be consistent with the old high level API.

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

mount: fix gcc warning (variable used uninitialized)

fstab.c: In function ‘getfs_by_spec’:
fstab.c:400  warning: ‘name’ may be used uninitialized in this
function
fstab.c:402  warning: ‘value’ may be used uninitialized in this
function

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

simmpleinit: fix gcc warning (buffer size in read())

simpleinit.c:846:
/usr/include/bits/unistd.h:43  warning: call to ‘__read_chk_warn’
declared with attribute warning: read called with bigger length than
size of the destination buffer

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

swapon: add -f/--fixpgsz option

The patch:

commit 3399a218f4eff4016a22044e7c416521bc37c53c
Author: Matthias Koenig <mkoenig@suse.de>
Date:   Thu Nov 27 12:32:56 2008 +0100
swapon: add swap format detection and pagesize check

introduced a new feature. This feature should be optional (disabled by
default) to keep happy people who use swap-space bad blocks or
nonstandard swap-space size.

CC: Hugh Dickins <hugh@veritas.com>
CC: Olaf Hering <olh@suse.de>
CC: Matthias Koenig <mkoenig@suse.de>
Signed-off-by: Karel Zak <kzak@redhat.com>

swapon: cleanup man page

The patch also fix stupid bug in swapoff long options array.

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

swapon: rewrite SWSUSPEND signature rather than exec mkswap

> On Thu, Jan 22, 2009 at 04:46:57PM +0000, Hugh Dickins wrote:
> Though your swapon may mkswap for suspend reasons: hmm, wouldn't it
> do very much better just to rewrite the swap signature, than exec
> mkswap - maybe nobody ever uses the badpages list, but it really
> should be respected.

CC: Hugh Dickins <hugh@veritas.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

swapon: do_swapon() refactoring (split into two functions)

The patch moves all checks to the separate swapon_checks() function.

This is a little more aggressive, but currently the do_swapon() is
too long.

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

swapon: use err.h stuff

The code is much more simple with warn()/err() that with fprintf() and
strerror(), ...

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

swapon: simplify spec to devname conversion

We needn't to use spec_to_devname() and check for pseudo filesystems.
The swap{on,off} is always checking for the 'swap' fstype.

This patch also removes the dependence on xmalloc.c.

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

swapon: add generic swap_get_header()

We need to proper swap header detection for swsuspend data and for
swap PAGE_SIZE checks. It's better to reuse the code in both cases.

The patch removes duplicate stat() call too.

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

swapon: do_swapon() refactoring (move stat() checks)

It's better to do stat() based checks immediately after the stat()
call.

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

mkswap: handle 2^32 pages

mkswap (when built 32-bit) could not quite support the maximum size
of swap area (2^32 pages): change unsigned long pagecounts to unsigned
long long pagecounts, and fix maxpages for the case when more is asked.

Signed-off-by: Hugh Dickins <hugh@veritas.com>

docs: add a note about kpartx to TODO

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

partx: use ioctls from lib/blkdev.c

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

fdisk: (and partx) remove BLKGETLASTSECT

This odd ioctl is unsupported in the current 2.4 and 2.6 mainline.

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

partx: don't duplicate lib/blkdev.c code

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

partx: convert hard sector size to 512-byte sectors

The msdos PT depends on a sector size (BLKSSZGET), but partx(8) counts
internally with 512-byte sectors only. The dos.c has to convert start
and size to 512-byte sectors.

sysfs (kernel uses 512-byte sectors only):

  # cat /sys/block/sdb/sdb1/{start,size}
  256
  16128

(note that 16128 * 512 = 8257536; 8Mb)

old version:

  # partx /dev/sdb
  1:        32-     2047 (     2016 sectors,      1 MB)
                                                  ^^^^
start, end and sectors are correct, but in 4KiB sectors
The size in MB is completely wrong.

new version:

  # partx -l /dev/sdb
  1:       256-    16383 (    16128 sectors,      8 MB)

start, end and sectors are converted to 512-byte sectors. The size in
MB is correct now.

Note that this change is important, because "partx -a" counts the size
of a new partition in 512-byte sectors for all PT formats (sun, gpt, ...).

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

addpart: 512-byte sectors in code, bytes in man-page

The addpart.c code uses 512-byte sectors for partition start and
length, but in the addpart.8 man page is "in bytes". The code (and
this code is pretty old) is always right... the man page has to be
fixed.

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

blockdev: add note that the StartSec is in 512-byte sectors

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

fdisk: use real sector size in verify() and warn_cylinders()

4KiB-sector disk:

Note: sector size is 4096 (not 512)

Disk /dev/sdb: 8 MB, 8388608 bytes
8 heads, 32 sectors/track, 8 cylinders, total 2048 sectors
Units = sectors of 1 * 4096 = 4096 bytes
Disk identifier: 0x6aad54da

   Device Boot      Start         End      Blocks   Id  System
/dev/sdb1              32        2047        8064   83  Linux

old version (mix 512 and real sectors):

Command (m for help): v
14367 unallocated sectors

new version:

Command (m for help): v
31 unallocated 4096-byte sectors

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

fdisk: doesn't handle large (4KiB) sectors properly

fdisk (at least with the -u option) does not handle sector sizes other
than 512.

Address-Red-Hat-Bugzilla: #487227
Signed-off-by: Eric Sandeen <sandeen@redhat.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

mount: remove useless if-before-free tests

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

getopt: remove useless if-before-free tests

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

fsck: remove useless if-before-free tests

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

blkid: fix a syntax nit

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

blkid: remove useless if-before-free tests

[kzak@redhat.com: - port the original e2fsprogs patch to util-linux-ng]

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

blkid: don't dereference NULL upon slashless module dependency line

* probers/ext.c (check_for_modules): Skip current line if it has no slash.

[kzak@redhat.com: - port the original e2fsprogs patch to util-linux-ng]

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

build-sys: add missing files to include/Makefile.am

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

fsck: remove \007 from warning message

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

tests: remove obsolete stuff from Makefile.am

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

losetup: cleanup man page

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

losetup: detach more devices by "-d <loop> [<loop> ..]"

This patch allows to detach more devices by "losetup -d",
for example:

  # losetup -a
  /dev/loop0: [0804]:9955739 (/home/images/ary0.img)
  /dev/loop1: [0804]:9955740 (/home/images/ary1.img)
  /dev/loop2: [0804]:9955745 (/home/images/ary2.img)

  # losetup -d /dev/loop0 /dev/loop1 /dev/loop2

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

tests: refresh expected mount(8) outputs

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

tests: disable suid mount test

Unfortunately, libtool wrapper cannot be suid...

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

mount: inform about UID and eUID when verbose > 2

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

blkid: blkid_evaluate_spec() shouldn't ignore $BLKID_FILE

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

tests: refresh ipcs expected outputs

Unfortunately, the last rebase in the topic/blkid branch has removed
tailing white spaces. Grrr...

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

tests: refresh cal(1) expected outputs

Unfortunately, the last rebase in the topic/blkid branch has removed
tailing white spaces. Grrr...

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

tests: we needn't blkid.sh

after "libtoolization" we needn't extra blkid.sh wrapper

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

tests: chmod -x ts/lscpu/mk-input.sh

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

tests: cleanup ts/md5

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

tests: cleanup ts/minix

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

build-sys: add fsck binary to .gitignore

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

tests: cleanup lscpu reg.tests

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

tests: cleanup ts/cramfs/mkfs-endianness

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

tests: cleanup ts/cramfs/fsck-endianness

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

tests: cleanup ts/bitops

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

tests: add fsck:ismounted reg.test

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

lib: add test_ismounted for regression test

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

po: update list of .c files

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

fsck: cosmetic changes (NLS, paths, ...)

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

fsck: link with generic fsprobe wrapper

This is a way how link fsck with libblkid (e2sprogs or util-linxu-ng
version) and libvolume_id.

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

lib: make open_device() optional in fsprobe.c

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

fsck: move fsck from e2fsprogs to util-linux-ng

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

blkid: fix low-probe mode return codes

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

blkid: add TODO hint about blkid_parse_tag_string()

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

blkid: add cmdline interface for blkid_probe_filter_usage()

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

blkid: check calloc() return value

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

blkid: add TODO hint about DM devnames in sysfs

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

build-sys: use pkg-config for blkid and volume_id

It seems that blkid.pc from e2fsprogs has been fixed:

   $ pkg-config --libs blkid
   -lblkid

and the pkg-config does not return any other extra libraries (such
-luuid or -ldevmapper).

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

mount: generic blkid/volume_id wrapper, use blkid_evaluate_*

 * moves the generic libblkid/libvolume_id wrapper (fsprobe.c) from
   mount/ to lib/. We'll use the wrapper in cfdisk and fsck.

   The wrapper supports:

    - obsolete volume_id (udev)
    - obsolete libblkid (e2fsprogs)
    - libblkid (util-linux-ng)

 * mount, umount and swapon when linked against the new libblkid use

    - low-level probing code to read LABEL, UUID or FSTYPE from a device
    - high-level blkid_evaluate_spec() to convert LABEL/UUID to devname

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

blkid: add Christoph's note about libdisk to TODO

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

blkid: refresh TODO file

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

build-sys: add temporary libtool *.m4 stuff

We require libtool-2 (because old 1.5.x is crap). Unfortunately,
libtool-2 is still not available in many Linux distributions  -- now I
see it in unstable distributions only. The painless way how to resolve
this problem is to __temporary__ commit generated libtool-2 stuff to
our repository.

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

build-sys: libtoolize mount/Makefile.am

 * cleanup mount/Makefile.am
 * add {VOLUMEID,BLKID}_CFLAGS -- necessary for pkg-config
 * add support for linking with in-tree libblkid

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

build-sys: libtoolize by libtool-2

We need an infrastructure for stared libraries. The latest libtool-2
seems useful and it's definitely better than the old 1.5 crap.

You need to install libtool-2 when you want to run ./autogen.sh script
after checkout from git repository.

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

blkid: start to use ABI versioning

The library ABI and API is backwardly compatible, so it does not
make sense to change the library SONAME. This patch adds a symbol
versioning, naming paradigm is:

BLKID_<maj>.<min>

The original libblkid from e2fsprogs uses "1.0" as a .so version and
"libblkid.1" as a SONAME for all time (at least according to stuff in
/lib/libblkid*)

And the original library is without symbols versioning. It means that
many private functions are exported to applications ;-(

Note that the original blkid_get_library_version() returns
E2FSPROGS_VERSION. The version in util-linux-ng returns BLKID_VERSION
which is <maj>.<min>.<rel>. The <maj>.<min> is the same version as we
use for ABI. This concept seems less confusing than mix a library
version and package version.

Summary:

OLD (e2fsprogs):
  ABI versioning:              -none-
  SONAME:                      libblkid.1
  .so version:                 libblkid.so.1.0
  blkid_get_library_version(): @E2FSPROGS_VERSION@ (e.g. 1.41.1)

NEW (util-linux-ng):
  ABI versioning:              BLKID_<maj>.<min>
  SONAME:                      libblkid.1
  .so version:                 libblkid.so.<maj.<min> (e.g. 1.41)
  blkid_get_library_version(): @BLKID_VERSION@ (e.g. 1.41.1)

  (BLKID_VERSION = <maj>.<min>.<rel>)

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

build-sys: add --with=fsprobe=builtin

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

blkid: add findfs(8)

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

blkid: refresh TODO file

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

blkid: clean up man pages

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

blkid: add -L -U options (evaluation API)

Signed-off-by: root <root@nb.net.home>

blkid: add blkid_evaluate_spec()

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

mount: move realpath.c code to lib/

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

blkid: fix typo (syntax error)

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

blkid: compile TEST_PROGRAMs

Well, the src/Makefile.am is not prefect. Maybe it's time to switch to
libtools ....

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

blkid: add support for /etc/blkid.conf file

We need a config file for the library. Now the library supports
config options:

EVALUATION=<method>[,<method>]

SEND_UEVENT=<yes|not>

CACHE_FILE=<path>

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

blkid: add new options to blkid.8 and help output

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

blkid: support via raid version 2

[kzak@redhat.com: patch from udev volume_id]

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

blkid: refresh TODO file

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

tests: fix typo in low-probe test

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

blkid: cleanup _LOGPROBE debug messages

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

blkid: add blkid_do_safeprobe()

The function blkid_do_probe() is able to detect more filesystems on
the device

while(blkid_do_probe(pr) == 0)
...

but in many cases we need only one exact answer, and we also need to
be sure that there is not any other FS on the device.

For example it's possible to create valid LUKS (or vfat, ...) header
and valid linux swap header on the same device -- in such case the
device can be interpreted (by mount/swapon) in two completely
different ways. An ambivalent result is always error -- the library
never returns such result.

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

blkid: add lvm2 reg.test

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

blkid: add version support to LVM2

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

blkid: add GFS2 reg. test

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

blkid: add uuid and version support to gfs2

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

blkid: add reg.tests for HFS and HFS+

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

blkid: add vol_id call to blkid regression test

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

blkid: add  to reiser

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

blkid: add ocfs2 version

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

blkid: fix ..._strncpy_uuid

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

blkid: add ddf raid regression test

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

blkid: fix blkid_do_probe()

Fix small non-sense from from a recent commit.

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

blkid: remove unnecessary debug message

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

blkid: use blkid_probe_strcpy_uuid() for luks

LUKS stores UUID as a string...

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

blkid: improve ddf detection

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

blkid: set size for non-blkdevs, add blkid_probe_strcpy_uuid()

- fix blkid_probe_set_device() to work for non-blkdevs (e.g. file images),
  in such case we need to set the size of data from stat->st_size

- add blkid_probe_strcpy_uuid() for filesystems where UUID is stored as a string
  (e.g. DDF raid, luks, ...)

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

blkid: add netware regression test

Thanks to Kay Sievers for the image and information.

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

blkid: netware SB has to be packed

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