shaba/lvm2
1
0
Fork 0
mirror of git://sourceware.org/git/lvm2.git synced 2024-12-21 13:34:40 +03:00
Code Releases Activity

cleanup: drop default implementation

Browse Source 
Now we reference segment name via  lvseg_name() and
we can drop default implementation and leave its
function pointer to be NULL.

Default give us 'return seg->segtype->name'.
This commit is contained in:
Zdenek Kabelac 2014-10-20 20:09:42 +02:00
parent 9411c19b31
commit d4dab0aa34
10 changed files with 2 additions and 70 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
lib/cache_segtype/cache.c
View File

@ -31,11 +31,6 @@
dm_config_parent_name(sn), seg->lv->name), 0;
static const char *_name(const struct lv_segment *seg)
{
return seg->segtype->name;
}
static int _cache_pool_text_import(struct lv_segment *seg,
const struct dm_config_node *sn,
struct dm_hash_table *pv_hash __attribute__((unused)))
@ -262,7 +257,6 @@ static int _modules_needed(struct dm_pool *mem,
#endif /* DEVMAPPER_SUPPORT */
static struct segtype_handler _cache_pool_ops = {
.name = _name,
.text_import = _cache_pool_text_import,
.text_import_area_count = _cache_pool_text_import_area_count,
.text_export = _cache_pool_text_export,
@ -365,7 +359,6 @@ static int _cache_add_target_line(struct dev_manager *dm,
#endif /* DEVMAPPER_SUPPORT */
static struct segtype_handler _cache_ops = {
.name = _name,
.text_import = _cache_text_import,
.text_import_area_count = _cache_text_import_area_count,
.text_export = _cache_text_export,

6
lib/error/errseg.c
View File

@ -21,11 +21,6 @@
#include "activate.h"
#include "str_list.h"
static const char *_errseg_name(const struct lv_segment *seg)
{
return seg->segtype->name;
}
static int _errseg_merge_segments(struct lv_segment *seg1, struct lv_segment *seg2)
{
seg1->len += seg2->len;
@ -83,7 +78,6 @@ static void _errseg_destroy(struct segment_type *segtype)
}
static struct segtype_handler _error_ops = {
.name = _errseg_name,
.merge_segments = _errseg_merge_segments,
#ifdef DEVMAPPER_SUPPORT
.add_target_line = _errseg_add_target_line,

6
lib/freeseg/freeseg.c
View File

@ -16,18 +16,12 @@
#include "toolcontext.h"
#include "segtype.h"
static const char *_freeseg_name(const struct lv_segment *seg)
{
return seg->segtype->name;
}
static void _freeseg_destroy(struct segment_type *segtype)
{
dm_free(segtype);
}
static struct segtype_handler _freeseg_ops = {
.name = _freeseg_name,
.destroy = _freeseg_destroy,
};

6
lib/mirror/mirrored.c
View File

@ -38,11 +38,6 @@ struct mirror_state {
uint32_t default_region_size;
};
static const char *_mirrored_name(const struct lv_segment *seg)
{
return seg->segtype->name;
}
static void _mirrored_display(const struct lv_segment *seg)
{
const char *size;
@ -592,7 +587,6 @@ static void _mirrored_destroy(struct segment_type *segtype)
}
static struct segtype_handler _mirrored_ops = {
.name = _mirrored_name,
.display = _mirrored_display,
.text_import_area_count = _mirrored_text_import_area_count,
.text_import = _mirrored_text_import,

8
lib/raid/raid.c
View File

@ -25,11 +25,6 @@
#include "metadata.h"
#include "lv_alloc.h"
static const char *_raid_name(const struct lv_segment *seg)
{
return seg->segtype->name;
}
static void _raid_display(const struct lv_segment *seg)
{
unsigned s;
@ -243,7 +238,7 @@ static int _raid_add_target_line(struct dev_manager *dm __attribute__((unused)),
if (mirror_in_sync())
flags = DM_NOSYNC;
params.raid_type = _raid_name(seg);
params.raid_type = lvseg_name(seg);
if (seg->segtype->parity_devs) {
/* RAID 4/5/6 */
params.mirrors = 1;
@ -418,7 +413,6 @@ static int _raid_target_unmonitor_events(struct lv_segment *seg, int events)
#endif /* DEVMAPPER_SUPPORT */
static struct segtype_handler _raid_ops = {
.name = _raid_name,
.display = _raid_display,
.text_import_area_count = _raid_text_import_area_count,
.text_import = _raid_text_import,

6
lib/replicator/replicator.c
View File

@ -39,10 +39,6 @@
/*
* Replicator target
*/
static const char *_replicator_name(const struct lv_segment *seg)
{
return seg->segtype->name;
}
/* FIXME: missing implementation */
static void _replicator_display(const struct lv_segment *seg)
@ -410,7 +406,6 @@ static void _replicator_destroy(struct segment_type *segtype)
}
static struct segtype_handler _replicator_ops = {
.name = _replicator_name,
.display = _replicator_display,
.text_import = _replicator_text_import,
.text_export = _replicator_text_export,
@ -739,7 +734,6 @@ static int _replicator_dev_target_present(struct cmd_context *cmd,
#endif
static struct segtype_handler _replicator_dev_ops = {
.name = _replicator_name,
.display = _replicator_dev_display,
.text_import = _replicator_dev_text_import,
.text_export = _replicator_dev_text_export,

8
lib/snapshot/snapshot.c
View File

@ -25,18 +25,13 @@
log_error(t " segment %s of logical volume %s.", ## p, \
dm_config_parent_name(sn), seg->lv->name), 0;
static const char *_snap_name(const struct lv_segment *seg)
{
return seg->segtype->name;
}
static const char *_snap_target_name(const struct lv_segment *seg,
const struct lv_activate_opts *laopts)
{
if (!laopts->no_merging && (seg->status & MERGING))
return "snapshot-merge";
return _snap_name(seg);
return lvseg_name(seg);
}
static int _snap_text_import(struct lv_segment *seg, const struct dm_config_node *sn,
@ -234,7 +229,6 @@ static void _snap_destroy(struct segment_type *segtype)
}
static struct segtype_handler _snapshot_ops = {
.name = _snap_name,
.target_name = _snap_target_name,
.text_import = _snap_text_import,
.text_export = _snap_text_export,

12
lib/thin/thin.c
View File

@ -36,11 +36,6 @@ static const char _thin_module[] = "thin";
/* TODO: using static field here, maybe should be a part of segment_type */
static unsigned _feature_mask;
static const char *_thin_pool_name(const struct lv_segment *seg)
{
return seg->segtype->name;
}
static void _thin_pool_display(const struct lv_segment *seg)
{
log_print(" Chunk size\t\t%s",
@ -449,11 +444,6 @@ static int _target_unregister_events(struct lv_segment *seg,
# endif /* DMEVENTD */
#endif /* DEVMAPPER_SUPPORT */
static const char *_thin_name(const struct lv_segment *seg)
{
return seg->segtype->name;
}
static void _thin_display(const struct lv_segment *seg)
{
log_print(" Device ID\t\t%u", seg->device_id);
@ -720,7 +710,6 @@ static void _thin_destroy(struct segment_type *segtype)
}
static struct segtype_handler _thin_pool_ops = {
.name = _thin_pool_name,
.display = _thin_pool_display,
.text_import = _thin_pool_text_import,
.text_import_area_count = _thin_pool_text_import_area_count,
@ -740,7 +729,6 @@ static struct segtype_handler _thin_pool_ops = {
};
static struct segtype_handler _thin_ops = {
.name = _thin_name,
.display = _thin_display,
.text_import = _thin_text_import,
.text_export = _thin_text_export,

7
lib/unknown/unknown.c
View File

@ -18,12 +18,6 @@
#include "text_export.h"
#include "config.h"
static const char *_unknown_name(const struct lv_segment *seg)
{
return seg->segtype->name;
}
static int _unknown_text_import(struct lv_segment *seg, const struct dm_config_node *sn,
struct dm_hash_table *pv_hash)
{
@ -59,7 +53,6 @@ static void _unknown_destroy(struct segment_type *segtype)
}
static struct segtype_handler _unknown_ops = {
.name = _unknown_name,
.text_import = _unknown_text_import,
.text_export = _unknown_text_export,
.destroy = _unknown_destroy,

6
lib/zero/zero.c
View File

@ -19,11 +19,6 @@
#include "str_list.h"
#include "activate.h"
static const char *_zero_name(const struct lv_segment *seg)
{
return seg->segtype->name;
}
static int _zero_merge_segments(struct lv_segment *seg1, struct lv_segment *seg2)
{
seg1->len += seg2->len;
@ -79,7 +74,6 @@ static void _zero_destroy(struct segment_type *segtype)
}
static struct segtype_handler _zero_ops = {
.name = _zero_name,
.merge_segments = _zero_merge_segments,
#ifdef DEVMAPPER_SUPPORT
.add_target_line = _zero_add_target_line,