Gitweb: http://git.fedorahosted.org/git/?p=lvm2.git;a=commitdiff;h=562c678ee23e76b67... Commit: 562c678ee23e76b675a8f4682bd6d2447d1d0de7 Parent: 7061b74ec6850651e5610a8da7ff0a16b1f6d155 Author: Jonathan Brassow jbrassow@redhat.com AuthorDate: Fri May 31 11:25:52 2013 -0500 Committer: Jonathan Brassow jbrassow@redhat.com CommitterDate: Fri May 31 11:25:52 2013 -0500
DM RAID: Add ability to throttle sync operations for RAID LVs.
This patch adds the ability to set the minimum and maximum I/O rate for sync operations in RAID LVs. The options are available for 'lvcreate' and 'lvchange' and are as follows: --minrecoveryrate <Rate> [bBsSkKmMgG] --maxrecoveryrate <Rate> [bBsSkKmMgG] The rate is specified in size/sec/device. If a suffix is not given, kiB/sec/device is assumed. Setting the rate to 0 removes the preference. --- lib/metadata/lv_manip.c | 3 ++ lib/metadata/metadata-exported.h | 5 +++ lib/raid/raid.c | 24 +++++++++++++++ lib/report/columns.h | 2 + lib/report/properties.c | 12 +++++++ lib/report/report.c | 34 +++++++++++++++++++++ libdm/libdm-deptree.c | 24 +++++++++++++-- man/lvchange.8.in | 16 ++++++++++ man/lvcreate.8.in | 16 ++++++++++ tools/args.h | 2 + tools/commands.h | 12 +++++-- tools/lvchange.c | 61 ++++++++++++++++++++++++++++++++++++++ tools/lvcreate.c | 19 ++++++++++++ 13 files changed, 224 insertions(+), 6 deletions(-)
diff --git a/lib/metadata/lv_manip.c b/lib/metadata/lv_manip.c index 48a1515..8f2f2ca 100644 --- a/lib/metadata/lv_manip.c +++ b/lib/metadata/lv_manip.c @@ -4721,6 +4721,9 @@ static struct logical_volume *_lv_create_an_lv(struct volume_group *vg, struct l stack; goto revert_new_lv; } + } else if (seg_is_raid(lp)) { + first_seg(lv)->min_recovery_rate = lp->min_recovery_rate; + first_seg(lv)->max_recovery_rate = lp->max_recovery_rate; }
/* FIXME Log allocation and attachment should have happened inside lv_extend. */ diff --git a/lib/metadata/metadata-exported.h b/lib/metadata/metadata-exported.h index 7ef4220..cfa5ad8 100644 --- a/lib/metadata/metadata-exported.h +++ b/lib/metadata/metadata-exported.h @@ -337,6 +337,8 @@ struct lv_segment { /* FIXME Fields depend on segment type */ uint32_t stripe_size; /* For stripe and RAID - in sectors */ uint32_t writebehind; /* For RAID (RAID1 only) */ + uint32_t min_recovery_rate; /* For RAID */ + uint32_t max_recovery_rate; /* For RAID */ uint32_t area_count; uint32_t area_len; uint32_t chunk_size; /* For snapshots/thin_pool. In sectors. */ @@ -631,6 +633,9 @@ struct lvcreate_params {
uint32_t mirrors; /* mirror */
+ uint32_t min_recovery_rate; /* RAID */ + uint32_t max_recovery_rate; /* RAID */ + const struct segment_type *segtype; /* all */ unsigned target_attr; /* all */
diff --git a/lib/raid/raid.c b/lib/raid/raid.c index 7f453f9..bc00691 100644 --- a/lib/raid/raid.c +++ b/lib/raid/raid.c @@ -129,6 +129,24 @@ static int _raid_text_import(struct lv_segment *seg, return 0; } } + if (dm_config_has_node(sn, "min_recovery_rate")) { + if (!dm_config_get_uint32(sn, "min_recovery_rate", + &seg->min_recovery_rate)) { + log_error("Couldn't read 'min_recovery_rate' for " + "segment %s of logical volume %s.", + dm_config_parent_name(sn), seg->lv->name); + return 0; + } + } + if (dm_config_has_node(sn, "max_recovery_rate")) { + if (!dm_config_get_uint32(sn, "max_recovery_rate", + &seg->max_recovery_rate)) { + log_error("Couldn't read 'max_recovery_rate' for " + "segment %s of logical volume %s.", + dm_config_parent_name(sn), seg->lv->name); + return 0; + } + } if (!dm_config_get_list(sn, "raids", &cv)) { log_error("Couldn't find RAID array for " "segment %s of logical volume %s.", @@ -155,6 +173,10 @@ static int _raid_text_export(const struct lv_segment *seg, struct formatter *f) outf(f, "stripe_size = %" PRIu32, seg->stripe_size); if (seg->writebehind) outf(f, "writebehind = %" PRIu32, seg->writebehind); + if (seg->min_recovery_rate) + outf(f, "min_recovery_rate = %" PRIu32, seg->min_recovery_rate); + if (seg->max_recovery_rate) + outf(f, "max_recovery_rate = %" PRIu32, seg->max_recovery_rate);
return out_areas(f, seg, "raid"); } @@ -227,6 +249,8 @@ static int _raid_add_target_line(struct dev_manager *dm __attribute__((unused)), params.stripe_size = seg->stripe_size; params.rebuilds = rebuilds; params.writemostly = writemostly; + params.min_recovery_rate = seg->min_recovery_rate; + params.max_recovery_rate = seg->max_recovery_rate; params.flags = flags;
if (!dm_tree_node_add_raid_target_with_params(node, len, ¶ms)) diff --git a/lib/report/columns.h b/lib/report/columns.h index 6754e6e..c55a685 100644 --- a/lib/report/columns.h +++ b/lib/report/columns.h @@ -83,6 +83,8 @@ FIELD(LVS, lv, NUM, "Cpy%Sync", lvid, 8, copypercent, sync_percent, "For RAID, m FIELD(LVS, lv, NUM, "Mismatches", lvid, 10, mismatch_count, mismatches, "For RAID, number of mismatches found or repaired.", 0) FIELD(LVS, lv, STR, "SyncAction", lvid, 10, sync_action, syncaction, "For RAID, the current synchronization action being performed.", 0) FIELD(LVS, lv, NUM, "WBehind", lvid, 7, write_behind, writebehind, "For RAID1, the number of outstanding writes allowed to writemostly devices.", 0) +FIELD(LVS, lv, NUM, "MinSync", lvid, 7, min_recovery_rate, minrecoveryrate, "For RAID1, the minimum recovery I/O load in kiB/sec/disk.", 0) +FIELD(LVS, lv, NUM, "MaxSync", lvid, 7, max_recovery_rate, maxrecoveryrate, "For RAID1, the maximum recovery I/O load in kiB/sec/disk.", 0) FIELD(LVS, lv, STR, "Move", lvid, 4, movepv, move_pv, "For pvmove, Source PV of temporary LV created by pvmove.", 0) FIELD(LVS, lv, STR, "Convert", lvid, 7, convertlv, convert_lv, "For lvconvert, Name of temporary LV created by lvconvert.", 0) FIELD(LVS, lv, STR, "Log", lvid, 3, loglv, mirror_log, "For mirrors, the LV holding the synchronisation log.", 0) diff --git a/lib/report/properties.c b/lib/report/properties.c index 3073d2a..884df2a 100644 --- a/lib/report/properties.c +++ b/lib/report/properties.c @@ -113,6 +113,14 @@ static uint32_t _writebehind(const struct logical_volume *lv) { return first_seg(lv)->writebehind; }
+static uint32_t _minrecoveryrate(const struct logical_volume *lv) { + return first_seg(lv)->min_recovery_rate; +} + +static uint32_t _maxrecoveryrate(const struct logical_volume *lv) { + return first_seg(lv)->max_recovery_rate; +} + static percent_t _snap_percent(const struct logical_volume *lv) { percent_t perc;
@@ -219,6 +227,10 @@ GET_LV_NUM_PROPERTY_FN(mismatches, _mismatches(lv)) #define _mismatches_set _not_implemented_set GET_LV_NUM_PROPERTY_FN(writebehind, _writebehind(lv)) #define _writebehind_set _not_implemented_set +GET_LV_NUM_PROPERTY_FN(minrecoveryrate, _minrecoveryrate(lv)) +#define _minrecoveryrate_set _not_implemented_set +GET_LV_NUM_PROPERTY_FN(maxrecoveryrate, _maxrecoveryrate(lv)) +#define _maxrecoveryrate_set _not_implemented_set GET_LV_STR_PROPERTY_FN(syncaction, _sync_action(lv)) #define _syncaction_set _not_implemented_set GET_LV_STR_PROPERTY_FN(move_pv, lv_move_pv_dup(lv->vg->vgmem, lv)) diff --git a/lib/report/report.c b/lib/report/report.c index d661596..2459ad0 100644 --- a/lib/report/report.c +++ b/lib/report/report.c @@ -1002,6 +1002,40 @@ static int _write_behind_disp(struct dm_report *rh __attribute__((unused)), return dm_report_field_uint32(rh, field, &first_seg(lv)->writebehind); }
+static int _min_recovery_rate_disp(struct dm_report *rh __attribute__((unused)), + struct dm_pool *mem, + struct dm_report_field *field, + const void *data, + void *private __attribute__((unused))) +{ + const struct logical_volume *lv = (const struct logical_volume *) data; + + if (!lv_is_raid_type(lv) || !first_seg(lv)->min_recovery_rate) { + dm_report_field_set_value(field, "", NULL); + return 1; + } + + return dm_report_field_uint32(rh, field, + &first_seg(lv)->min_recovery_rate); +} + +static int _max_recovery_rate_disp(struct dm_report *rh __attribute__((unused)), + struct dm_pool *mem, + struct dm_report_field *field, + const void *data, + void *private __attribute__((unused))) +{ + const struct logical_volume *lv = (const struct logical_volume *) data; + + if (!lv_is_raid_type(lv) || !first_seg(lv)->max_recovery_rate) { + dm_report_field_set_value(field, "", NULL); + return 1; + } + + return dm_report_field_uint32(rh, field, + &first_seg(lv)->max_recovery_rate); +} + static int _dtpercent_disp(int metadata, struct dm_report *rh, struct dm_pool *mem, struct dm_report_field *field, diff --git a/libdm/libdm-deptree.c b/libdm/libdm-deptree.c index 4482296..f8c0d7a 100644 --- a/libdm/libdm-deptree.c +++ b/libdm/libdm-deptree.c @@ -183,9 +183,11 @@ struct load_segment { struct dm_tree_node *replicator;/* Replicator-dev */ uint64_t rdevice_index; /* Replicator-dev */
- uint64_t rebuilds; /* raid */ - uint64_t writemostly; /* raid */ - uint32_t writebehind; /* raid */ + uint64_t rebuilds; /* raid */ + uint64_t writemostly; /* raid */ + uint32_t writebehind; /* raid */ + uint32_t max_recovery_rate; /* raid kB/sec/disk */ + uint32_t min_recovery_rate; /* raid kB/sec/disk */
struct dm_tree_node *metadata; /* Thin_pool */ struct dm_tree_node *pool; /* Thin_pool, Thin */ @@ -2133,6 +2135,12 @@ static int _raid_emit_segment_line(struct dm_task *dmt, uint32_t major, if (seg->writebehind) param_count += 2;
+ if (seg->min_recovery_rate) + param_count += 2; + + if (seg->max_recovery_rate) + param_count += 2; + /* rebuilds is 64-bit */ param_count += 2 * hweight32(seg->rebuilds & 0xFFFFFFFF); param_count += 2 * hweight32(seg->rebuilds >> 32); @@ -2166,6 +2174,14 @@ static int _raid_emit_segment_line(struct dm_task *dmt, uint32_t major, if (seg->writebehind) EMIT_PARAMS(pos, " writebehind %u", seg->writebehind);
+ if (seg->min_recovery_rate) + EMIT_PARAMS(pos, " min_recovery_rate %u", + seg->min_recovery_rate); + + if (seg->max_recovery_rate) + EMIT_PARAMS(pos, " max_recovery_rate %u", + seg->max_recovery_rate); + /* Print number of metadata/data device pairs */ EMIT_PARAMS(pos, " %u", seg->area_count/2);
@@ -2901,6 +2917,8 @@ int dm_tree_node_add_raid_target_with_params(struct dm_tree_node *node, seg->rebuilds = p->rebuilds; seg->writemostly = p->writemostly; seg->writebehind = p->writebehind; + seg->min_recovery_rate = p->min_recovery_rate; + seg->max_recovery_rate = p->max_recovery_rate; seg->flags = p->flags;
return 1; diff --git a/man/lvchange.8.in b/man/lvchange.8.in index 80747d3..2271d4c 100644 --- a/man/lvchange.8.in +++ b/man/lvchange.8.in @@ -26,6 +26,10 @@ lvchange - change attributes of a logical volume .RI { y | n }] .RB [ --poll .RI { y | n }] +.RB [ --maxrecoveryrate +.IR Rate ] +.RB [ --minrecoveryrate +.IR Rate ] .RB [ --syncaction .RI { check | repair }] .RB [ --sysinit ] @@ -111,6 +115,18 @@ process from its last checkpoint. However, it may not be appropriate to immediately poll a logical volume when it is activated, use \fB--poll n\fP to defer and then \fB--poll y\fP to restart the process. .TP +.IR \fB--maxrecoveryrate " " \fIRate [ bBsSkKmMgG ] +Sets the maximum recovery rate for a RAID logical volume. \fIRate\fP +is specified as a size/sec/device. If no suffix is given, then +kiB/sec/device is assumed. Setting the recovery rate to 0 means +it will be unbounded. +.TP +.IR \fB--minrecoveryrate " " \fIRate [ bBsSkKmMgG ] +Sets the minimum recovery rate for a RAID logical volume. \fIRate\fP +is specified as a size/sec/device. If no suffix is given, then +kiB/sec/device is assumed. Setting the recovery rate to 0 means +it will be unbounded. +.TP .BR --syncaction " {" \fIcheck | \fIrepair } This argument is used to initiate various RAID synchronization operations. The \fIcheck\fP and \fIrepair\fP options provide a way to check the diff --git a/man/lvcreate.8.in b/man/lvcreate.8.in index 92cf1e7..3d675ae 100644 --- a/man/lvcreate.8.in +++ b/man/lvcreate.8.in @@ -19,6 +19,10 @@ lvcreate - create a logical volume in an existing volume group .RB [ --ignoremonitoring ] .RB [ --monitor .RI { y | n }] +.RB [ --maxrecoveryrate +.IR Rate ] +.RB [ --minrecoveryrate +.IR Rate ] .RB [ -i | --stripes .IR Stripes .RB [ -I | --stripesize @@ -243,6 +247,18 @@ Sets the name for the new logical volume. Without this option a default name of "lvol#" will be generated where # is the LVM internal number of the logical volume. .TP +.IR \fB--maxrecoveryrate " " \fIRate [ bBsSkKmMgG ] +Sets the maximum recovery rate for a RAID logical volume. \fIRate\fP +is specified as a size/sec/device. If no suffix is given, then +kiB/sec/device is assumed. Setting the recovery rate to 0 means +it will be unbounded. +.TP +.IR \fB--minrecoveryrate " " \fIRate [ bBsSkKmMgG ] +Sets the minimum recovery rate for a RAID logical volume. \fIRate\fP +is specified as a size/sec/device. If no suffix is given, then +kiB/sec/device is assumed. Setting the recovery rate to 0 means +it will be unbounded. +.TP .B --noudevsync Disables udev synchronisation. The process will not wait for notification from udev. diff --git a/tools/args.h b/tools/args.h index cdc4b3b..2fed97e 100644 --- a/tools/args.h +++ b/tools/args.h @@ -89,6 +89,8 @@ arg(validate_ARG, '\0', "validate", NULL, 0) arg(syncaction_ARG, '\0', "syncaction", string_arg, 0) arg(writemostly_ARG, '\0', "writemostly", string_arg, ARG_GROUPABLE) arg(writebehind_ARG, '\0', "writebehind", int_arg, 0) +arg(minrecoveryrate_ARG, '\0', "minrecoveryrate", size_kb_arg, 0) +arg(maxrecoveryrate_ARG, '\0', "maxrecoveryrate", size_kb_arg, 0)
/* Allow some variations */ arg(resizable_ARG, '\0', "resizable", yes_no_arg, 0) diff --git a/tools/commands.h b/tools/commands.h index 24f6512..64d6d0d 100644 --- a/tools/commands.h +++ b/tools/commands.h @@ -84,6 +84,8 @@ xx(lvchange, "\t[--monitor {y|n}]\n" "\t[--poll {y|n}]\n" "\t[--noudevsync]\n" + "\t[--minrecoveryrate Rate]\n" + "\t[--maxrecoveryrate Rate]\n" "\t[-M|--persistent y|n] [--major major] [--minor minor]\n" "\t[-P|--partial] " "\n" "\t[-p|--permission r|rw]\n" @@ -95,7 +97,7 @@ xx(lvchange, "\t[-t|--test]\n" "\t[-v|--verbose]\n" "\t[--version]\n" - "\t[--writebehind BehindCount\n" + "\t[--writebehind BehindCount]\n" "\t[--writemostly PhysicalVolume]\n" "\t[-y|--yes]\n" "\t[-Z|--zero {y|n}]\n" @@ -103,7 +105,8 @@ xx(lvchange,
alloc_ARG, autobackup_ARG, activate_ARG, available_ARG, contiguous_ARG, discards_ARG, force_ARG, ignorelockingfailure_ARG, ignoremonitoring_ARG, - major_ARG, minor_ARG, monitor_ARG, noudevsync_ARG, partial_ARG, + major_ARG, minor_ARG, monitor_ARG, minrecoveryrate_ARG, maxrecoveryrate_ARG, + noudevsync_ARG, partial_ARG, permission_ARG, persistent_ARG, poll_ARG, readahead_ARG, resync_ARG, refresh_ARG, addtag_ARG, deltag_ARG, syncaction_ARG, sysinit_ARG, test_ARG, writebehind_ARG, writemostly_ARG, zero_ARG) @@ -194,6 +197,8 @@ xx(lvcreate, "\t -L|--size LogicalVolumeSize[bBsSkKmMgGtTpPeE]}\n" "\t[-M|--persistent {y|n}] [--major major] [--minor minor]\n" "\t[-m|--mirrors Mirrors [--nosync] [{--mirrorlog {disk|core|mirrored}|--corelog}]]\n" + "\t[--minrecoveryrate Rate]\n" + "\t[--maxrecoveryrate Rate]\n" "\t[-n|--name LogicalVolumeName]\n" "\t[--noudevsync]\n" "\t[-p|--permission {r|rw}]\n" @@ -244,7 +249,8 @@ xx(lvcreate, addtag_ARG, alloc_ARG, autobackup_ARG, activate_ARG, available_ARG, chunksize_ARG, contiguous_ARG, corelog_ARG, discards_ARG, extents_ARG, ignoremonitoring_ARG, major_ARG, minor_ARG, mirrorlog_ARG, mirrors_ARG, - monitor_ARG, name_ARG, nosync_ARG, noudevsync_ARG, permission_ARG, + monitor_ARG, minrecoveryrate_ARG, maxrecoveryrate_ARG, name_ARG, nosync_ARG, + noudevsync_ARG, permission_ARG, persistent_ARG, readahead_ARG, regionsize_ARG, size_ARG, snapshot_ARG, stripes_ARG, stripesize_ARG, test_ARG, thin_ARG, thinpool_ARG, type_ARG, virtualoriginsize_ARG, poolmetadatasize_ARG, virtualsize_ARG, zero_ARG) diff --git a/tools/lvchange.c b/tools/lvchange.c index ed0a35a..6e5fd25 100644 --- a/tools/lvchange.c +++ b/tools/lvchange.c @@ -810,6 +810,55 @@ static int lvchange_writemostly(struct logical_volume *lv) return 1; }
+static int lvchange_recovery_rate(struct logical_volume *lv) +{ + struct cmd_context *cmd = lv->vg->cmd; + struct lv_segment *raid_seg = first_seg(lv); + + if (strcmp(raid_seg->segtype->name, "raid1")) { + log_error("Unable to change the recovery rate of non-RAID" + " logical volume."); + return 0; + } + + if (arg_count(cmd, minrecoveryrate_ARG)) + raid_seg->min_recovery_rate = + arg_uint_value(cmd, minrecoveryrate_ARG, 0); + if (arg_count(cmd, maxrecoveryrate_ARG)) + raid_seg->max_recovery_rate = + arg_uint_value(cmd, maxrecoveryrate_ARG, 0); + + if (raid_seg->max_recovery_rate && + (raid_seg->max_recovery_rate < raid_seg->min_recovery_rate)) { + log_error("Minumum recovery rate cannot" + " be higher than maximum."); + return 0; + } + + if (!vg_write(lv->vg)) + return_0; + + if (!suspend_lv(cmd, lv)) { + vg_revert(lv->vg); + return_0; + } + + if (!vg_commit(lv->vg)) { + if (!resume_lv(cmd, lv)) + stack; + return_0; + } + + log_very_verbose("Updating recovery rate for "%s" in kernel", + lv->name); + if (!resume_lv(cmd, lv)) { + log_error("Problem reactivating %s", lv->name); + return 0; + } + + return 1; +} + static int lvchange_single(struct cmd_context *cmd, struct logical_volume *lv, void *handle __attribute__((unused))) { @@ -992,6 +1041,18 @@ static int lvchange_single(struct cmd_context *cmd, struct logical_volume *lv, docmds++; }
+ /* change [min|max]_recovery_rate */ + if (arg_count(cmd, minrecoveryrate_ARG) || + arg_count(cmd, maxrecoveryrate_ARG)) { + if (!archived && !archive(lv->vg)) { + stack; + return ECMD_FAILED; + } + archived = 1; + doit += lvchange_recovery_rate(lv); + docmds++; + } + if (doit) log_print_unless_silent("Logical volume "%s" changed", lv->name);
diff --git a/tools/lvcreate.c b/tools/lvcreate.c index 3f2f17b..f56cdbd 100644 --- a/tools/lvcreate.c +++ b/tools/lvcreate.c @@ -576,6 +576,25 @@ static int _read_raid_params(struct lvcreate_params *lp, return 0; }
+ if (arg_count(cmd, minrecoveryrate_ARG)) + lp->min_recovery_rate = arg_uint_value(cmd, + minrecoveryrate_ARG, 0); + if (arg_count(cmd, maxrecoveryrate_ARG)) + lp->max_recovery_rate = arg_uint_value(cmd, + maxrecoveryrate_ARG, 0); + + /* Rates are recorded in kiB/sec/disk, not sectors/sec/disk */ + lp->min_recovery_rate /= 2; + lp->max_recovery_rate /= 2; + log_error("min = %u, max = %u", + lp->min_recovery_rate, + lp->max_recovery_rate); + + if (lp->max_recovery_rate && + (lp->max_recovery_rate < lp->min_recovery_rate)) { + log_error("Minumum recovery rate cannot be higher than maximum."); + return 0; + } return 1; }
lvm2-commits@lists.fedorahosted.org