fsadm cleanups & release prep

autoreconf  (using fedora 10 for this now)

pe_start can be overwritten in VG metadata, so it is not PV label-only field.

Try to avoid full rescan if label scan is enough.

Use pkgconfig to obtain corosync library details during configuration. (kabi)

Fix error returns in clvmd-corosync interface to DLM.

Thanks to Xinwei Hu for spotting this.

Update vgchange and vgmknodes man pages to include --refresh.

Author: Dave Wysochanski <dwysocha@redhat.com>

Fixed bug where lvresize option -t was not properly passed to fsadm.
Using argv[] list in exec_cmd() to allow more params for external commands.
Fsadm does not allow checking mounted filesystem.
Fsadm no longer accepts 'any other key' as 'no' answer to y/n.
Fsadm improved handling of command line options.

Fix include/.symlinks for lvm2.h to lvm.h renaming

Author: Thomas Woerner <twoerner@redhat.com>

Added files lib/lvm.h and lib/lvm_base.c:
New structure lvm (used as an alias to cmd_context), new type definition lvm_t
for the lvm handle. Added functions lvm_create, lvm_destroy and
lvm_reload_config using the new handle.

Modified test/api/test.c:
Use new lvm.h header file and lvm_t handle.

Removed lib/lvm2.h

Author: Thomas Woerner <twoerner@redhat.com>

Fix validation of dataalignment value introduced in previous commit.

Move tools/version.h to lib/misc/lvm-version.h.
Split LVM_VERSION into MAJOR, MINOR, PATCHLEVEL, RELEASE and RELEASE_DATE.

Add system_dir parameter to create_toolcontext() and call it system_dir
everywhere for consistency.

Comment out pvcreate pagesize alignment test for now.

Add --dataalignment to pvcreate to specify alignment of data area.  (mbroz)

This patch is not fully tested and leaves some related bugs unfixed.

Intended behaviour of the code now:

  pe_start in the lvm2 format PV label header is set only by pvcreate (or
vgconvert -M2) and then preserved in *all* operations thereafter.

  In some specialist cases, after the PV is added to a VG, the pe_start
field in the VG metadata may hold a different value and if so, it
overrides the other one for as long as the PV is in such a VG.

  Currently, the field storing the size of the data area in the PV label
header always holds 0.  As it only has meaning in the context of a
volume group, it is calculated whenever the PV is added to a VG (and can
be derived from extent_size and pe_count in the VG metadata).

Fix interrupt unblocking after vgcreate, for example: drop_cached_metadata()
previously left _vg_lock_count incremented.
Other locks are always held during drop_cached_metadata() so there's no
need to increment+decrement it.

Provide da and mda locations in debug message when writing text format label.

Mention the restriction on file descriptors at invocation on the lvm man page.

In testsuite's not.c, print a notice when program dies of a fatal signal.

Fix test output collection in harness.c.

Index cached vgmetadata by vgid not vgname to cope with duplicate vgnames.  (dwyso)

Fix lvm.static build dependence (to properly propagate changes in libdevmapper).

Only fail when tests have failed, but not when they have been just skipped.

If kernel supports only one dm major number, use in place of any supplied.
No longer require kernel and metadata major numbers to match.

Re-implement the test harness in C. This lets us pass through signals and
trigger proper test teardown upon harness interrupt or termination. I also
tweaked the output somewhat while I was at it...

Add a fully-functional get_cluster_name() to clvmd corosync interface.

Remove duplicate cpg_initialize from clvmd startup.

Add option to /etc/sysconfig/cluster to select cluster type for clvmd.

Allow clvmd to start up if its lockspace already exists.

Separate PV label attributes which do not need parse metadata when reporting.

When reporting explicitly label attributes (pv_uuid for example), we do not
need to read metadata.

This patch separate the label fileds and removes scan_vgs_for_pvs
in process_each_pv() if not needed.

(There should be no user visible change in output.)

Remove external dependency on the 'cut' command in fsadm

Fix pvs segfault when pv mda attributes requested for not available PV.

add support for ext4 resize in fsadm

Move locking_type reading inside init_locking().

No functional change.

Rename get_vgs() to get_vgnames() and clarify related error messages.

get_vgs() really returns a list of vgnames.  In the future we will use
get_vgs() to return a list of vg structures, similar to get_pvs().

Allow clvmd to be built with all cluster managers & select one on cmdline.

Mention --with-clvmd=corosync in ./configure

Add as-yet-unused vg_read_error() and vg_might_exist().  (mornfall)

Introduce as-yet-unused replacement vg_read() and vg_read_for_update()
functions.  (mornfall)

 Replace internal vg_check_status() implementation.  (mornfall)

Properly enforce cluster locking in as-yet-unused _vg_lock_and_read. (mornfall)

Introduce as-yet-unused _vg_lock_and_read() and associated header file
definitions.

Rename vg_read() to vg_read_internal(). (mornfall)

post-release

and another one missing

Add stuff people missed.

pre-release

Add a corosync/DLM cluster service to clvmd.

It's not integrated in the configure system yet though.

Use tools.h in for lvm-static.

Add --unquoted to pvs, vgs, lvs man pages.

Add missing --segments to pvs man pg.

Add --nameprefixes to pvs, vgs, lvs man pages.

Fix problems with static build

 - compiler warning (missing header)

 - configure should set static flag early to be able
   use STATIC_LINK flag during configure script

Add --rows option to pvs, vgs, lvs man pages.

Rename _parse_options() to _parse_fields() for naming consistency.

In libdm, we only ever use 'fields', while the tools use 'options' and
'fields' interchangeably.

Ideally it would be good to use 'fields' consistently everywhere.
However, 'options' most likely comes from the tool commandline '-o' and
'--options' which cannot be changed.

Add skeleton of fsadm nightly test but skip (doesn't work yet).

Fix fsadm lvresize for filesystem block sizes != 1024.

Fixes rhbz 480022.

A C implementation of "not" that handles fatal signals rather more
intelligently than the shell implementation. C code by Jaroslav Stava.

I have done a rudimentary review and checked that tests still pass.

...and a few more uninitialised dummy fields.

More fields can cause segfaults with orphans.
Fix these by populating the dummy VG struct more completely.

Fix pvs segfault when run with orphan PV and vg_mda_size or vg_mda_free fields

We display '0' for these fields now in this case.  Ideally these values are
undefined for an orphan PV but today there is no way to specify undefined
with display functions such as _size64_disp().

Fix some checks in lvm-utils.sh. Note that "$(test ...)" is always empty, since
"test" never prints anything. Therefore, "return $(test ...)" is equivalent to
just "return;" which means success in sh (same as return 0). We can however,
thanks to set -e, use "test foo = bar" as an assertion.

PS: test a == b is invalid syntax. It is either = or -eq: = is textual and -eq
is numeric comparison.

Add <report_type>_all to help text.

Add an "all" field which expands to all fields of the report type.

For example in LVM2, "pv_all" gives all PV fields.
"seg_all" gives all LV segment fields.

"all" gives all fields of the final report type.  I think this is more
useful than just adding the current prefix.

So "lvs -o seg_all" gives all the LV segment fields, whilst
"lvs --segments -o all" adds in LV and VG fields too.

"lvs -o all -O vg_name" has report type LVS+VGS so includes all LV and all
VG fields.

Display a 'dev_size' of zero for missing devices in reports.

Add pv_mda_size to pvs and vg_mda_size to vgs.

Add pv_mda_size to 'pvs' and vg_mda_size to 'vgs'.

Reports the size of the smallest metadata area in a PV or a VG.
Useful to confirm pvcreate --metadatasize or pvmetadatasize setting in
/etc/lvm/lvm.conf file.

NOTE: Actual value in these fields will most always differ from that
given in pvcreate options due to rounding and alignment effects.

Document current state of inconsistent metadata behaviour of a few commands in
a test. Should make changes to the !consistent bits of code easier. To be
expanded.

Initial import of the cluster log daemon from the 'cluster' repository.

There is a rudimentary make file in place so people can build by hand
from 'LVM2/daemons/clogd'.  It is not hooked into the main build system
yet.  I am checking this in to provide people better access to the
source code.

There is still work to be done to make better use of existing code in
the LVM repository.  (list.h could be removed in favor of existing list
implementations, for example.  Logging might also be removed in favor
of what is already in the tree.)

I will probably defer updating WHATS_NEW_DM until this code is linked
into the main build system (unless otherwise instructed).

Add checks for device names in dmsetup and show proper error messages.

Checks added for DM device names to allow only names < DM_NAME_LEN,
otherwise a part of lengthy name would be silently ignored and could
cause confusion while using dmsetup. Also, the name should not contain
'/' character, if it is used in context of creating a new device
or renaming the existing one (because we do not consider full path
to devices, they do not exist in filesystem yet) and appropriate error
messages are shown.

Fix lvmdump /sys listing to include virtual devices directory.

Fix "Calculate mirror log size" commit, the le_count should be always set.

Add "--refresh" functionality to vgchange and vgmknodes.

Add "--refresh" functionality to vgchange and vgmknodes.

lvm2cmdline.h:31: warning: declaration of `is_static' shadows a global declaration

Do not issue write behind lv size.

pvcreate $DEV
vgcreate -s 1k vg_test $DEV
lvcreate -l 1 -n lv1 vg_test
..
/dev/vg_test/lv1: write failed after 1024 of 4096 at 0: No space left on device

Just check for maximum write size in set_lv.

Calculate mirror log size instead of hardcoding 1 extent size.

It fails for 1k PE now.

Patch adds log_region_size into allocation habdle struct
and use it in _alloc_parallel_area() for proper log size calculation
instead of hardcoded 1 extent - which can fail.

Reproducer for incorrect log size calculation:
        DEV=/dev/sd[bcd]

        pvcreate $DEV
        vgcreate -s 1k vg_test $DEV
        lvcreate -m1 -L 12M -n mirr vg_test

https://bugzilla.redhat.com/show_bug.cgi?id=477040

The log size calculation is mostly copied from kernel code.

Fail add tree node when requested major/minor is used.

Check for major/minor collision is added in _add_dev_to_dtree()
where we already read info by uuid,
so in the case of requesting major/minor it queries device-mapper
by major/minor for device availability.

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=204992

Fix incorrect return value in help function.

Fix vgrename using UUID in case there are VGs with the same name.

Create global is_static() to eliminate from the library init function.

Very simple / crude method of removing 'is_static' from initialization.
Why should we require an application tell us whether it is linked
statically or dynamically to libLVM?  If the application is linked
statically, but libraries exist and dlopen() calls succeed, why
do we care if it's statically linked?

Remove struct arg * from struct cmd_context and create_toolcontext().

This allows us to remove one argument from create_toolcontext() and
moves it closer to a generic library init function.

In the arg_*() functions, we just use _the_args() directly.
For now we leave the first parameter to these
arg_*() functions (struct cmd_context *) because
of the number of files involved in removing the
parameter.

Move arg_* functions from toollib.c to lvmcmdline.c.

In preparation for removing cmd->args.
IMO, it makes more sense to put these accessor functions
in the same location as the static array _the_args.
Next patch will update arg_* functions to use _the_args[]
directly and remove cmd->args.

Remove status=noxfer from test, not all dd versions support this (RHEL4 for example).

Rename 'cmd' to 'clvmd_cmd' to remove ambiguity.

Signed-off-by: Dave Wysochanski <dwysocha@redhat.com>
Acked-by: James Cameron <james.cameron@hp.com>
Acked-by: Alasdair G Kergon <agk@redhat.com>

Fix segfault when invalid field given in reporting commands.

Problem is dm_report_init() may return NULL and subsequent call to
dm_report_set_output_field_name_prefix() doesn't handle NULL value.

Example:
pvs --nameprefixes --rows --unquoted --noheadings -opv_name,fred
  Logical Volume Fields
  ---------------------
    lv_uuid              - Unique identifier
    lv_name              - Name.  LVs created for internal use are enclosed in brackets.
 ...

  Physical Volume Segment Fields
  ------------------------------
    pvseg_start          - Physical Extent number of start of segment.
    pvseg_size           - Number of extents in segment.

  Unrecognised field: fred
Segmentation fault

Use dm_snprintf to avoid duplicating the snprintf compatibility code.

Create _init_globals() and call from bottom of create_toolcontext().

Move init_full_scan_done(0) and init_mirror_in_sync(0) from init_lvm()
after call to create_toolcontext() to _init_globals(), called from bottom
of create_toolcontext().  No functional change.

Author: Dave Wysochanski <dwysocha@redhat.com>
Acked-by: James Cameron <james.cameron@hp.com>
Acked-by: Alasdair G Kergon <agk@redhat.com>

Replace _dm_snprintf with EMIT_PARAMS macro for creating target lines

*** empty log message ***

Move initialization of cmd->fmt into init_formats().

init_formats() sets up the command formats, and currently sets cmd->fmt_backup
but does not set cmd->fmt to a default value.  This seems incorrect so we
set it to cmd->default_settings.fmt before returning.

The call we remove here may set cmd->fmt based on a command line setting.
But it is safe to remove this, because the only caller of init_lvm() that
cares about the cmdline override is the cmdline tools (clvmd does not care),
called from lvm2_main().  After lvm2_main() calls init_lvm(), it later calls
lvm_run_command().  In lvm_run_command(), we have a call to _apply_settings(),
which has the identical assignment of cmd->fmt that this patch removes.

Remove redundant init_msg_prefix() and init_cmd_name().

This is very obvious - _init_logging() makes the identical init_msg_prefix()
and init_cmd_name() calls with cmd->default_settings so these calls are
clearly redundant after calling create_toolcontext().

Remove redundant set_activation() call after create_toolcontext() calls.

Very similar argument to removal of init_debug() and other calls.

create_toolcontext() calls _process_config() which sets
cmd->default_settings.activation, then calls
set_activation(cmd->default_settings.activation).  Later, create_toolcontext()
sets cmd->current_settings = cmd->default_settings.  So these calls
set_activation(cmd->current_settings.activation) are redundant.

Remove backup_enable() calls after create_toolcontext() calls.

Identical argument to previous patch which removed archive_enable() calls.
We add a new parameter to backup_init() which sets the enable value based
on the cmd->default_settings.backup value.  This value was used to set
cmd->current_settings.backup, used in the removed backup_enable() call.

Remove archive_enable() calls after create_toolcontext() calls.

_init_backup() calls archive_init(), which originally set 'enabled' to
a hardcoded '1' value.  This seems incorrect based on my read of other
areas of the code so here we add a 'enabled' paramter to archive_init().
We pass in cmd->default_settings.archive, which is obtained from the
config tree.  Later in create_toolcontext, cmd->current_settings is
set to cmd->default_settings.  The archive_enable() call we remove
here was using cmd->current_settings to set the 'archive' enable
value.  The final value of cmd->archive_params->enabled should thus
be equivalent to the original code.

Move init_test() from _apply_settings into _init_logging().

This one we actually need to move.  _init_logging() is called from
    create_toolcontext(), which makes this call:
        /* Test mode */
        cmd->default_settings.test =
            find_config_tree_int(cmd, "global/test", 0);

But it does not call init_test().  So we need an init_test() somewhere.
The most logical place is to put it inside _init_logging(), since this
is where the config value is read and default_settings are set.  Placing
the init_test() call here matches what is done with other variables and
seems to make sense.

Remove handles_missing_pvs assignment after call to create_toolcontext().

This variable is set at the top of create_toolcontext() to 0.
Nothing later in create_toolcontext() changes the value.
In init_lvm(), nothing between create_toolcontext() call and this assignment
changes the value.  Thus, the assignment is redundant.

Remove init_verbose() calls immediately after create_toolcontext() calls.

The rationale for removing init_verbose() call is very similar to removing
init_debug() call.  create_toolcontext() calls _init_logging() which
makes these calls:
        /* Verbose level for tty output */
        cmd->default_settings.verbose =
            find_config_tree_int(cmd, "log/verbose", DEFAULT_VERBOSE);
        init_verbose(cmd->default_settings.verbose + VERBOSE_BASE_LEVEL);

And being that create_toolcontext() copies default_settings into
current_settings at the bottom, the init_verbose() call we are removing:
        init_verbose(cmd->current_settings.verbose + VERBOSE_BASE_LEVEL);

is redundant.

Remove init_debug() calls immediately after create_toolcontext() call.

We can safely remove because create_toolcontext() calls _init_logging(),
which makes these calls:
        /* Debug level for log file output */
        cmd->default_settings.debug =
            find_config_tree_int(cmd, "log/level", DEFAULT_LOGLEVEL);
        init_debug(cmd->default_settings.debug);

Then at the bottom of create_toolcontext() we do this:
        cmd->current_settings = cmd->default_settings;

So the call we are removing from init_lvm() functions (clvmd and lvmcmdline):
        init_debug(cmd->current_settings.debug);

Just sets the value of debug based on 'cmd->current_settings.debug'.
Since cmd->current_settings is equivalent to cmd->default_settings, and
init_debug() was called with cmd->default_settings, the call we remove is
redundant.

Replace _apply_settings() after create_toolcontext() with equivalent inline.

Subsequent patches will refactor / remove each of these lines, as many of
them are redundant when called immediately after create_toolcontext().