]> sourceware.org Git - lvm2.git/log
lvm2.git
7 years agoman pvscan: add description for --activate
David Teigland [Wed, 3 May 2017 20:37:48 +0000 (15:37 -0500)]
man pvscan: add description for --activate

7 years agoman vgimport: add description for --all
David Teigland [Wed, 3 May 2017 20:19:19 +0000 (15:19 -0500)]
man vgimport: add description for --all

7 years agoman pvchange: mention one option is required
David Teigland [Wed, 3 May 2017 20:10:10 +0000 (15:10 -0500)]
man pvchange: mention one option is required

7 years agoman pvchange: add description for --all
David Teigland [Wed, 3 May 2017 20:03:14 +0000 (15:03 -0500)]
man pvchange: add description for --all

7 years agocommands: check for memory failures
David Teigland [Wed, 3 May 2017 19:46:07 +0000 (14:46 -0500)]
commands: check for memory failures

just return for now

7 years agoman lvmlockd: minor wording updates
David Teigland [Wed, 3 May 2017 19:45:33 +0000 (14:45 -0500)]
man lvmlockd: minor wording updates

7 years agopost-release
Alasdair G Kergon [Wed, 3 May 2017 10:26:58 +0000 (11:26 +0100)]
post-release

7 years agopre-release v2_02_171
Alasdair G Kergon [Wed, 3 May 2017 10:23:31 +0000 (11:23 +0100)]
pre-release

7 years agoman: generate
Alasdair G Kergon [Wed, 3 May 2017 10:23:15 +0000 (11:23 +0100)]
man: generate

7 years agocommand: avoid compiler warning
Alasdair G Kergon [Wed, 3 May 2017 10:19:43 +0000 (11:19 +0100)]
command: avoid compiler warning

man-generator.c:3243: warning: declaration of ‘stat’ shadows a global
declaration

7 years agoman: references to lvm entities
David Teigland [Tue, 2 May 2017 21:47:02 +0000 (16:47 -0500)]
man: references to lvm entities

Try to reference lvm(8) at the start of topical
man pages, and spell out acronyms early in the
text, descriptions of which can be found in lvm(8).

7 years agoman lvm: define LVM and its terms
David Teigland [Tue, 2 May 2017 20:47:02 +0000 (15:47 -0500)]
man lvm: define LVM and its terms

The lvm(8) man page never said what LVM is,
it never defined what the acronym LVM stands for,
it never spelled out other common acronyms VG, PV, LV,
and never described what they are.

This adds a very minimal definition which at least defines
the acronyms and entities, but it could obviously be
expanded on, either here or elsewhere.

7 years agoman lvmsystemid: general improvements
David Teigland [Tue, 2 May 2017 19:55:50 +0000 (14:55 -0500)]
man lvmsystemid: general improvements

Mainly improve the readability/wording.
Use "system ID" instead of system_id.
A couple small additions.

7 years agodbus: Add --yes when --setphysicalvolumesize is used
Marian Csontos [Fri, 28 Apr 2017 14:47:58 +0000 (16:47 +0200)]
dbus: Add --yes when --setphysicalvolumesize is used

7 years agocommands: fix memory debug for cmd defs
David Teigland [Fri, 28 Apr 2017 21:27:19 +0000 (16:27 -0500)]
commands: fix memory debug for cmd defs

Clean up the handling of memory used for cmd defs
so it doesn't trip up memory debugging.

Allocate memory for commands[] from libmem.

Free temporary memory used by define_commands()
at the end of the function.

Clear all the command def state in in lvm_fin().

7 years agofix running commands from a script file
David Teigland [Fri, 28 Apr 2017 21:43:42 +0000 (16:43 -0500)]
fix running commands from a script file

Using any arg with a command name in a script file
would cause the command to fail.

The name of the script file being executed was being passed
to lvm_register_commands() and define_commands(), which
prevented command defs from being defined (simple commands
were still being defined only by name which was enough for those
to still work when run trivially with no args).

7 years agoWHATS_NEW: pvcreate prompt
David Teigland [Fri, 28 Apr 2017 14:06:41 +0000 (09:06 -0500)]
WHATS_NEW: pvcreate prompt

7 years agoWHATS_NEW: pvcreate --setphysicalvolumesize
Alasdair G Kergon [Fri, 28 Apr 2017 13:34:26 +0000 (14:34 +0100)]
WHATS_NEW: pvcreate --setphysicalvolumesize

7 years agocommands: improve syntax suggestion when no command is found
David Teigland [Thu, 27 Apr 2017 19:21:01 +0000 (14:21 -0500)]
commands: improve syntax suggestion when no command is found

The logic for suggesting the nearest valid command syntax
was missing the simplest case.  If a command has only one
valid syntax, that is the one we should suggest.  (We were
suggesting nothing in this case.)

7 years agopvcreate: add prompt when setting dev size
David Teigland [Thu, 27 Apr 2017 17:13:09 +0000 (12:13 -0500)]
pvcreate: add prompt when setting dev size

If the device size does not match the size requested
by --setphysicalvolumesize, then prompt the user.

Make the pvcreate checking/prompting code handle
multiple prompts for the same device, since the
new prompt can be in addition to the existing
prompt when the PV is in a VG.

7 years agodbus: log_debug needs qualifier
Marian Csontos [Thu, 27 Apr 2017 16:12:20 +0000 (18:12 +0200)]
dbus: log_debug needs qualifier

Adding qualifier makes the only unqualified log_debug occurence
consistent with other uses in the same file.

Other possible ways to fix this:

- using `from .utils import log_debug`
- moving the line below `from . import utils` line

7 years agopvresize: Missing a message on error path.
Alasdair G Kergon [Thu, 27 Apr 2017 13:59:40 +0000 (14:59 +0100)]
pvresize: Missing a message on error path.

7 years agotest: add -y to raid1 up conversions
Heinz Mauelshagen [Thu, 27 Apr 2017 13:56:58 +0000 (15:56 +0200)]
test: add -y to raid1 up conversions

7 years agolvconvert: preserve region size on raid1 image count changes (v2)
Heinz Mauelshagen [Thu, 27 Apr 2017 13:52:25 +0000 (15:52 +0200)]
lvconvert: preserve region size on raid1 image count changes (v2)

Unless a change of the regionsize is requested via "lvconvert -R N ...",
keep the region size when the number of images changes in a raid1 LV.

Related: rhbz1443705

7 years agotest: Update pattern to match code
Marian Csontos [Thu, 27 Apr 2017 08:22:11 +0000 (10:22 +0200)]
test: Update pattern to match code

Fix commit a3fdc966b5

7 years agoRevert "lvconvert: preserve region size on raid1 image count changes"
Heinz Mauelshagen [Thu, 27 Apr 2017 13:29:03 +0000 (15:29 +0200)]
Revert "lvconvert: preserve region size on raid1 image count changes"

This reverts commit 8333d5a969042a321f00656271bd55a37c1ea179.

7 years agotest: Add -y to pvresize --setphysicalvolumesize
Alasdair G Kergon [Thu, 27 Apr 2017 01:57:59 +0000 (02:57 +0100)]
test: Add -y to pvresize --setphysicalvolumesize

7 years agopvresize: Prompt when non-default size supplied.
Alasdair G Kergon [Thu, 27 Apr 2017 01:36:34 +0000 (02:36 +0100)]
pvresize: Prompt when non-default size supplied.

Seek confirmation before changing the PV size to one that differs
from the underlying block device.

7 years agotypo: s/extends/extents/ in lvmthin man page
Jonathan Brassow [Wed, 26 Apr 2017 20:49:28 +0000 (15:49 -0500)]
typo: s/extends/extents/ in lvmthin man page

bug 1439905

7 years agolvmdbusd: Improve error msg to include PV
Tony Asleson [Sat, 22 Apr 2017 05:07:52 +0000 (00:07 -0500)]
lvmdbusd: Improve error msg to include PV

Include PV device path when we believe it already exists.

7 years agolvmdbustest.py: Add nested testing
Tony Asleson [Sat, 22 Apr 2017 05:06:59 +0000 (00:06 -0500)]
lvmdbustest.py: Add nested testing

Make sure when a LV is used as a PV the dbus service works correctly.

Signed-off-by: Tony Asleson <tasleson@redhat.com>
7 years agolvmdbustest.py: Handle nested setUp & tearDown
Tony Asleson [Sat, 22 Apr 2017 05:06:16 +0000 (00:06 -0500)]
lvmdbustest.py: Handle nested setUp & tearDown

Handle cleaning up correctly if a LV is used as a PV.

7 years agolvmdbusd: Make sure we don't hang on lvcreate
Tony Asleson [Sat, 22 Apr 2017 05:03:46 +0000 (00:03 -0500)]
lvmdbusd: Make sure we don't hang on lvcreate

If we happen to create a lv that has a previous signature we hang on y/n
prompt, add '--yes'.

7 years agocommands: improve error messages for rules
David Teigland [Tue, 25 Apr 2017 17:19:11 +0000 (12:19 -0500)]
commands: improve error messages for rules

Make the error messages more consistent,
and use less code-centric wording.

7 years agolvconvert: FIXME
Heinz Mauelshagen [Mon, 24 Apr 2017 16:56:36 +0000 (18:56 +0200)]
lvconvert: FIXME

Add FIXME to move error path processing out of tool into library.

Related: rhbz1437653

7 years agolvconvert: fix inactive mirror up converting regression
Heinz Mauelshagen [Mon, 24 Apr 2017 15:44:54 +0000 (17:44 +0200)]
lvconvert: fix inactive mirror up converting regression

Up converting an inactive mirror with insufficient
devs results in an over concerned warning.

Resolves: rhbz1437653

7 years agolvconvert: preserve region size on raid1 image count changes
Heinz Mauelshagen [Sat, 22 Apr 2017 00:04:49 +0000 (02:04 +0200)]
lvconvert: preserve region size on raid1 image count changes

Unless a change of the regionsize is requested via "lvconvert -R N ...",
keep the region size when the number of images changes in a raid1 LV.

Resolves: rhbz1443705

7 years agoraid: handle insufficent PVs on takeover to/from raid4
Heinz Mauelshagen [Fri, 21 Apr 2017 23:19:44 +0000 (01:19 +0200)]
raid: handle insufficent PVs on takeover to/from raid4

Commit 7bc85177b08c7a6f4f0c3384c8d05b9089267c14
felt short relative to striped/raid0* -> raid4
and raid4 -> raid6.

Related: rhbz1438013

7 years agoraid: avoid superfluous variable
Heinz Mauelshagen [Fri, 21 Apr 2017 22:50:36 +0000 (00:50 +0200)]
raid: avoid superfluous variable

7 years agotest: add raid4 to upconvert allocation failure tests
Heinz Mauelshagen [Fri, 21 Apr 2017 22:43:31 +0000 (00:43 +0200)]
test: add raid4 to upconvert allocation failure tests

7 years agotest: consider changed default
Heinz Mauelshagen [Fri, 21 Apr 2017 22:42:09 +0000 (00:42 +0200)]
test: consider changed default

commit b81b4aad24807d2aa4968aa9fc7d970a65623620
raised the region size so demand the sizes the
test checks for.

7 years agotest: add upconvert allocation failure tests
Heinz Mauelshagen [Fri, 21 Apr 2017 18:57:31 +0000 (20:57 +0200)]
test: add upconvert allocation failure tests

7 years agotest: also prevent lvconvert-raid-reshape.sh on single core
Heinz Mauelshagen [Fri, 21 Apr 2017 00:17:55 +0000 (02:17 +0200)]
test: also prevent lvconvert-raid-reshape.sh on single core

7 years agotest: Adjust previous commit
Heinz Mauelshagen [Thu, 20 Apr 2017 23:21:24 +0000 (01:21 +0200)]
test: Adjust previous commit

Change have_single_core to  have_multi_core and go back to || logic in related test scripts.

7 years agotest: Fix skip some reshape tests that hang on single core machines
Heinz Mauelshagen [Thu, 20 Apr 2017 19:45:56 +0000 (21:45 +0200)]
test: Fix skip some reshape tests that hang on single core machines

Fix commit c7fb0cb861104e99c11e8b4e031d981dab7891a1.

7 years agoraid: remove double minimum area check on takeover
Heinz Mauelshagen [Thu, 20 Apr 2017 19:35:06 +0000 (21:35 +0200)]
raid: remove double minimum area check on takeover

7 years agoraid: define return value on error paths
Heinz Mauelshagen [Thu, 20 Apr 2017 19:32:40 +0000 (21:32 +0200)]
raid: define return value on error paths

7 years agoraid: avoid superfluous reload on takeover
Heinz Mauelshagen [Thu, 20 Apr 2017 19:17:08 +0000 (21:17 +0200)]
raid: avoid superfluous reload on takeover

Allow any reset rebuild flags to trigger the second update on takeover.
Use descriptive callback names.
Fix typo and add comments.

7 years agotest: Skip some reshape tests that hang on single core machines
Alasdair G Kergon [Thu, 20 Apr 2017 19:05:07 +0000 (20:05 +0100)]
test: Skip some reshape tests that hang on single core machines

Skip hanging raid reshape tests until https://bugzilla.redhat.com/1443999
is fixed

7 years agomirror/raid: display adjusted region size with units
Heinz Mauelshagen [Thu, 20 Apr 2017 18:42:21 +0000 (20:42 +0200)]
mirror/raid: display adjusted region size with units

Display adjusted region size in units (e.g. "4.00 MiB") rather than sectors.

7 years agoman: quote the word no
David Teigland [Tue, 18 Apr 2017 20:55:57 +0000 (15:55 -0500)]
man: quote the word no

7 years agoconfigure: Fix notify-dbus and dmfilemapd options.
Alasdair G Kergon [Tue, 18 Apr 2017 19:37:53 +0000 (20:37 +0100)]
configure: Fix notify-dbus and dmfilemapd options.

7 years agoWHATS_NEW: configure option enable-lvmlockd
David Teigland [Tue, 18 Apr 2017 17:58:18 +0000 (12:58 -0500)]
WHATS_NEW: configure option enable-lvmlockd

7 years agospec: rename lockd to lvmlockd
David Teigland [Tue, 18 Apr 2017 16:22:07 +0000 (11:22 -0500)]
spec: rename lockd to lvmlockd

7 years agoconfigure: rename lockd to lvmlockd
David Teigland [Tue, 18 Apr 2017 16:02:49 +0000 (11:02 -0500)]
configure: rename lockd to lvmlockd

7 years agotest: correcting binary usage
Zdenek Kabelac [Thu, 13 Apr 2017 23:03:18 +0000 (01:03 +0200)]
test: correcting binary usage

Ensure 'test suite' run uses fsadm and dmeventd from compiled dir,
while for 'rpm' installed test use binaries installed in system.

7 years agolvconvert: typo in message
Heinz Mauelshagen [Thu, 13 Apr 2017 20:19:29 +0000 (22:19 +0200)]
lvconvert: typo in message

7 years agotests: use raid_region_size 512
David Teigland [Thu, 13 Apr 2017 19:21:34 +0000 (14:21 -0500)]
tests: use raid_region_size 512

given the new default from 5ae7a016b8e5796d36cf491345b1cf8e43ec9ea5

7 years agoWHATS_NEW: raise region_size default
Heinz Mauelshagen [Thu, 13 Apr 2017 15:49:58 +0000 (17:49 +0200)]
WHATS_NEW: raise region_size default

7 years agoconf: Remove exec_prefix from confdir.
Alasdair G Kergon [Thu, 13 Apr 2017 15:19:33 +0000 (16:19 +0100)]
conf: Remove exec_prefix from confdir.

Some distros now want to put /etc outside prefix/exec_prefix.

7 years agolvcreate: raise default raid regionsize to 2MiB
Heinz Mauelshagen [Thu, 13 Apr 2017 14:10:49 +0000 (16:10 +0200)]
lvcreate: raise default raid regionsize to 2MiB

Related: rhbz1392947.

7 years agoconf: Add missing prefix to installation directory.
Alasdair G Kergon [Thu, 13 Apr 2017 01:17:27 +0000 (02:17 +0100)]
conf: Add missing prefix to installation directory.

Our Makefiles also support systems that set the installation prefix
at configuration time - the use of DESTDIR is not required.

7 years agopost-release
Alasdair G Kergon [Thu, 13 Apr 2017 00:47:47 +0000 (01:47 +0100)]
post-release

7 years agopre-release v2_02_170
Alasdair G Kergon [Wed, 12 Apr 2017 23:43:31 +0000 (00:43 +0100)]
pre-release

7 years agoautoconf
Alasdair G Kergon [Wed, 12 Apr 2017 23:38:46 +0000 (00:38 +0100)]
autoconf

7 years agomake generate
Alasdair G Kergon [Wed, 12 Apr 2017 23:30:05 +0000 (00:30 +0100)]
make generate

7 years agotests: use egrep to check for message
Zdenek Kabelac [Wed, 12 Apr 2017 20:52:56 +0000 (22:52 +0200)]
tests: use egrep to check for message

Seems we may get different '' around option on different nixes.

7 years agotests: aux defines fsadm path
Zdenek Kabelac [Wed, 12 Apr 2017 21:03:08 +0000 (23:03 +0200)]
tests: aux defines fsadm path

7 years agoraid: use log_error
Zdenek Kabelac [Wed, 12 Apr 2017 21:05:09 +0000 (23:05 +0200)]
raid: use log_error

Turn log_print into log_error for error path.

7 years agolvconvert: return error without conversion
Heinz Mauelshagen [Wed, 12 Apr 2017 20:11:30 +0000 (22:11 +0200)]
lvconvert: return error without conversion

lvconvert parameters not causing a conversion (i.e. no type,
number of stripes, stripesize or regionsize changes) will
remove any allocated reshape space in which case the command
returns success.  If reshape space does not exist though,
return error.

7 years agotests: test for new supported commands
Zdenek Kabelac [Wed, 12 Apr 2017 13:01:12 +0000 (15:01 +0200)]
tests: test for new supported commands

New tests for:

lvconvert --regionsize | --swapmetadata | --startpoll |
          --mergesnapshot | mergethin

7 years agotests: restore some test changes
Zdenek Kabelac [Mon, 3 Apr 2017 13:57:32 +0000 (15:57 +0200)]
tests: restore some test changes

Restore existing tests and separate new tests
for new options later.

Keep coverage for existing syntax

Reverts bits from commits:

a801b92b2c81213dddccc9f9a09794208f92bfc9.
d71aaca07b2c14049130ab368221510c65236af7
fa2a728a395639a32707bb75e6bb4197b02769de
1e2420bca85da9a37570871cd70192e9ae831786
46abc28a480069b3e8b2c5c7c3448ffa25ec17ba
0e3e611a137a264a343335d18d500fce7af8e0f7

7 years agofsadm: support configurable full path
Zdenek Kabelac [Sat, 8 Apr 2017 17:43:20 +0000 (19:43 +0200)]
fsadm: support configurable full path

Just like with other tools lvm2 is using allow to define
fully configurable path.

Default is selected by $PREFIX/sbin/fsadm

7 years agothin: properly check for status for max sizes metadata
Zdenek Kabelac [Sat, 8 Apr 2017 17:38:17 +0000 (19:38 +0200)]
thin: properly check for status for max sizes metadata

When metadata LV size was over DM_THIN_MAX_METADATA_SIZE sectors,
the info() routine was incorrectly trying to match bigger size,
while we do never pass any bigger device.

Fixing a case, where lvs should be displaying status for metadata
LV with 16GB size.

7 years agocmdline: ensure command struct is defined
Zdenek Kabelac [Sat, 1 Apr 2017 10:17:16 +0000 (12:17 +0200)]
cmdline: ensure command struct is defined

Make it clear to Coverity cmd will be defined.

7 years agolibdm: set min,maj
Zdenek Kabelac [Sat, 1 Apr 2017 10:13:37 +0000 (12:13 +0200)]
libdm: set min,maj

Reset derefered pointers instead of pointers.

7 years agodmfilemapd: free path
Zdenek Kabelac [Sat, 1 Apr 2017 10:11:41 +0000 (12:11 +0200)]
dmfilemapd: free path

Cleanup resources before exit.

7 years agodmfilemapd: keep 1 byte for \0
Zdenek Kabelac [Sat, 1 Apr 2017 10:05:49 +0000 (12:05 +0200)]
dmfilemapd: keep 1 byte for \0

Coverity noticed the buffer should have 1 byte for \0.

7 years agocommands: improve error message for option rules
David Teigland [Wed, 12 Apr 2017 19:05:36 +0000 (14:05 -0500)]
commands: improve error message for option rules

When a cmd def RULE fails because of a disallowed
combination of options, improve the error message
to show the option combination, not just the options
that broke the rule.

7 years agoWHATS_NEW: add items since last release
David Teigland [Wed, 12 Apr 2017 18:45:37 +0000 (13:45 -0500)]
WHATS_NEW: add items since last release

7 years agocommands: add rule for lvconvert commands that create pools
David Teigland [Wed, 12 Apr 2017 18:33:40 +0000 (13:33 -0500)]
commands: add rule for lvconvert commands that create pools

For an lvconvert command that creates a thin pool or cache pool,
--poolmetadata cannot be used with --readahead, --stripes or --stripesize.

7 years agoWHATS_NEW: add recent RAID entries
Heinz Mauelshagen [Wed, 12 Apr 2017 17:41:55 +0000 (19:41 +0200)]
WHATS_NEW: add recent RAID entries

7 years agolvconvert: better message on --regionsize
Heinz Mauelshagen [Wed, 12 Apr 2017 17:34:18 +0000 (19:34 +0200)]
lvconvert: better message on --regionsize

Enhance message on "lvconvert --regionsize size RaidLV".
in case the regionsize does not change and return error.

7 years agoWHATS_NEW_DM: add recent dmsetup and dmfilemapd entries
Bryn M. Reeves [Wed, 12 Apr 2017 16:41:29 +0000 (17:41 +0100)]
WHATS_NEW_DM: add recent dmsetup and dmfilemapd entries

7 years agoTypo: change loose to lose working
Jonathan Brassow [Wed, 12 Apr 2017 15:28:19 +0000 (10:28 -0500)]
Typo: change loose to lose

loose (v): set free; release
lose (v) : be deprived of or cease to have or retain

We 'lose' redundancy or 'lose' meaning.

7 years agolvconvert: fix failing valid regionsize change
Heinz Mauelshagen [Wed, 12 Apr 2017 12:38:49 +0000 (14:38 +0200)]
lvconvert: fix failing valid regionsize change

Reshape check failed when regionsize changed and current raid type
was provided with no other change requested (stripes or stripesize).

E.g. "lvconvert --type raid6 --regionsize 256K" on a raid6 LV
with != 256K regionsize.

Enable --type in test script.

7 years agolvconvert: add segment type raid10_near
Heinz Mauelshagen [Tue, 11 Apr 2017 23:28:22 +0000 (01:28 +0200)]
lvconvert: add segment type raid10_near

Introducing this alias for "raid10", avoid allocating
reshape space when converting between them.

Resolves: rhbz1441347

7 years agoraid: handle insufficent PVs on takeover from striped/raid0
Heinz Mauelshagen [Tue, 11 Apr 2017 22:25:23 +0000 (00:25 +0200)]
raid: handle insufficent PVs on takeover from striped/raid0

Remove any newly allocated sub LV (pair) remnants in case
allocation fails due to lag of (parallel) free PV space
and keep initial raid type.

Resolves: rhbz1438013

7 years agoman lvmraid: wrap long lines
David Teigland [Tue, 11 Apr 2017 20:23:48 +0000 (15:23 -0500)]
man lvmraid: wrap long lines

Break lines in examples to keep line length under 80.

7 years agoraid_manip: fix typo warning message
David Teigland [Tue, 11 Apr 2017 19:18:57 +0000 (14:18 -0500)]
raid_manip: fix typo warning message

7 years agoman lvmraid: fix line wrapping in examples
David Teigland [Tue, 11 Apr 2017 19:17:08 +0000 (14:17 -0500)]
man lvmraid: fix line wrapping in examples

7 years agoman lvmraid: use same style as generated pages
David Teigland [Tue, 11 Apr 2017 18:13:49 +0000 (13:13 -0500)]
man lvmraid: use same style as generated pages

7 years agolvconvert: require different LV for pool
David Teigland [Tue, 11 Apr 2017 17:15:10 +0000 (12:15 -0500)]
lvconvert: require different LV for pool

When convering an LV to a cache or thin LV, require
that an LV being converted to cache pool or thin pool
at the same time be a different LV.

7 years agolvconvert: prevent duplicate use of metadata lv
David Teigland [Tue, 11 Apr 2017 17:07:14 +0000 (12:07 -0500)]
lvconvert: prevent duplicate use of metadata lv

The LV being processed cannot be used as the
pool metadata LV.

7 years agoman lvmraid: improve text layout
David Teigland [Tue, 11 Apr 2017 15:55:02 +0000 (10:55 -0500)]
man lvmraid: improve text layout

7 years agolvconvert: validate vg name prefix for pool metadata name
David Teigland [Tue, 11 Apr 2017 15:40:08 +0000 (10:40 -0500)]
lvconvert: validate vg name prefix for pool metadata name

7 years agoman lvcreate/lvconvert: minor clean up rewording
David Teigland [Mon, 10 Apr 2017 21:29:40 +0000 (16:29 -0500)]
man lvcreate/lvconvert: minor clean up rewording

7 years agocommands: check required option value when matching command
David Teigland [Mon, 10 Apr 2017 18:41:47 +0000 (13:41 -0500)]
commands: check required option value when matching command

A command def can include a specific constant option value,
but the value was not being checked for optional opts.

e.g. this is an incorrect command and does not match any
command defs:

lvconvert --type cache --cachepool vg/lv

However, it was mistakely being matched to this cmd def,
where the required options match, but the optional options
do not:

lonvert --cachepool LV_linear_striped_raid_cachepool
OO: --type cache-pool, ...

The optional options were mistakely considered matching
because 'cache' and 'cache-pool' were not being compared.

7 years agosignals: fix comparison flaw
Heinz Mauelshagen [Mon, 10 Apr 2017 16:29:26 +0000 (18:29 +0200)]
signals: fix comparison flaw

commit 9a689fb8f0a6b1675a4e819f4837bf2e0269832f
fixed SIGINT blocking but introduced a comparision
flaw to sigint_restore().

Related: rhbz1440766

7 years agosignals: fix SIGINT blocking flaw causing inconsistent metadata
Heinz Mauelshagen [Mon, 10 Apr 2017 16:16:09 +0000 (18:16 +0200)]
signals: fix SIGINT blocking flaw causing inconsistent metadata

SIGINT isn't blocked properly after a sigint_allow(),
sigint_restore() cycle leading to illicit interruptable
metadata updates.  These can leave corrupted metadata behind.

Issues addressed in this commit:

sigint_allow() fails to set _oldmasked[] members properly due
to an offset by one bug on indexing the members of the array.

It bails out prematurely comparing to MAX_SIGINTS causing nesting
depths to be one less than MAX_SIGINTS.  Fix the comparision.

Correct the related comparison flaw in sigint_restore().

Initialize all sig_atomic_t variables consequently.

Resolves: rhbz1440766

This page took 0.065389 seconds and 5 git commands to generate.